hf mamba failing with generate tasks

Here's an error i get. I was able to resolve by just removing "attention_mask" from generate_kwargs here: https://github.com/EleutherAI/lm-evaluation-harness/blob/main/lm_eval/models/huggingface.py#L857
Although I'm not sure how this fix will affect other models. I am using the 0.4.4 version of the eval harness, but it doesn't look like it's fixed in the latest version either?
(perplexity-correlations) tthrush@jagupard19:/juice5/scr5/tthrush/perplexity-correlations/examples/get_error_and_bpb$ lm_eval --model hf --model_args pretrained=state-spaces/mamba-130m-hf --tasks coqa --limit 5000 --device cuda:0
2024-11-14:17:21:24,640 INFO     [__main__.py:279] Verbosity set to INFO
2024-11-14:17:21:24,728 INFO     [__init__.py:491] `group` and `group_alias` keys in TaskConfigs are deprecated and will be removed in v0.4.5 of lm_eval. The new `tag` field will be used to allow for a shortcut to a group of tasks one does not wish to aggregate metrics across. `group`s which aggregate across subtasks must be only defined in a separate group config file, which will be the official way to create groups that support cross-task aggregation as in `mmlu`. Please see the v0.4.4 patch notes and our documentation: https://github.com/EleutherAI/lm-evaluation-harness/blob/main/docs/new_task_guide.md#advanced-group-configs for more information.
2024-11-14:17:21:32,920 WARNING  [__main__.py:312]  --limit SHOULD ONLY BE USED FOR TESTING.REAL METRICS SHOULD NOT BE COMPUTED USING LIMIT.
2024-11-14:17:21:32,921 INFO     [__main__.py:376] Selected Tasks: ['coqa']
2024-11-14:17:21:32,925 INFO     [evaluator.py:161] Setting random seed to 0 | Setting numpy seed to 1234 | Setting torch manual seed to 1234
2024-11-14:17:21:32,925 INFO     [evaluator.py:198] Initializing hf model, with arguments: {'pretrained': 'state-spaces/mamba-130m-hf'}
2024-11-14:17:21:33,010 WARNING  [other.py:349] Detected kernel version 5.4.0, which is below the recommended minimum of 5.5.0; this can cause the process to hang. It is recommended to upgrade the kernel to the minimum version or higher.
2024-11-14:17:21:33,010 INFO     [huggingface.py:130] Using device 'cuda:0'
2024-11-14:17:21:33,306 INFO     [huggingface.py:366] Model parallel was set to False, max memory was not set, and device map was set to {'': 'cuda:0'}
/nlp/scr/tthrush/miniconda3/envs/perplexity-correlations/lib/python3.12/site-packages/mamba_ssm/ops/selective_scan_interface.py:163: FutureWarning: `torch.cuda.amp.custom_fwd(args...)` is deprecated. Please use `torch.amp.custom_fwd(args..., device_type='cuda')` instead.
  @custom_fwd
/nlp/scr/tthrush/miniconda3/envs/perplexity-correlations/lib/python3.12/site-packages/mamba_ssm/ops/selective_scan_interface.py:239: FutureWarning: `torch.cuda.amp.custom_bwd(args...)` is deprecated. Please use `torch.amp.custom_bwd(args..., device_type='cuda')` instead.
  @custom_bwd
/nlp/scr/tthrush/miniconda3/envs/perplexity-correlations/lib/python3.12/site-packages/mamba_ssm/ops/triton/layer_norm.py:985: FutureWarning: `torch.cuda.amp.custom_fwd(args...)` is deprecated. Please use `torch.amp.custom_fwd(args..., device_type='cuda')` instead.
  @custom_fwd
/nlp/scr/tthrush/miniconda3/envs/perplexity-correlations/lib/python3.12/site-packages/mamba_ssm/ops/triton/layer_norm.py:1044: FutureWarning: `torch.cuda.amp.custom_bwd(args...)` is deprecated. Please use `torch.amp.custom_bwd(args..., device_type='cuda')` instead.
  @custom_bwd
/nlp/scr/tthrush/miniconda3/envs/perplexity-correlations/lib/python3.12/site-packages/mamba_ssm/distributed/tensor_parallel.py:25: FutureWarning: `torch.cuda.amp.custom_fwd(args...)` is deprecated. Please use `torch.amp.custom_fwd(args..., device_type='cuda')` instead.
  @custom_fwd
/nlp/scr/tthrush/miniconda3/envs/perplexity-correlations/lib/python3.12/site-packages/mamba_ssm/distributed/tensor_parallel.py:61: FutureWarning: `torch.cuda.amp.custom_bwd(args...)` is deprecated. Please use `torch.amp.custom_bwd(args..., device_type='cuda')` instead.
  @custom_bwd
/nlp/scr/tthrush/miniconda3/envs/perplexity-correlations/lib/python3.12/site-packages/mamba_ssm/ops/triton/ssd_combined.py:757: FutureWarning: `torch.cuda.amp.custom_fwd(args...)` is deprecated. Please use `torch.amp.custom_fwd(args..., device_type='cuda')` instead.
  @custom_fwd
/nlp/scr/tthrush/miniconda3/envs/perplexity-correlations/lib/python3.12/site-packages/mamba_ssm/ops/triton/ssd_combined.py:835: FutureWarning: `torch.cuda.amp.custom_bwd(args...)` is deprecated. Please use `torch.amp.custom_bwd(args..., device_type='cuda')` instead.
  @custom_bwd
2024-11-14:17:21:38,847 INFO     [evaluator.py:279] Setting fewshot random generator seed to 1234
2024-11-14:17:21:38,847 WARNING  [model.py:422] model.chat_template was called with the chat_template set to False or None. Therefore no chat template will be applied. Make sure this is an intended behavior.
2024-11-14:17:21:38,864 INFO     [task.py:423] Building contexts for coqa on rank 0...
100%|██████████████████████████████████████████████████████████████████████████████████████████████████████████| 500/500 [00:00<00:00, 56175.72it/s]
2024-11-14:17:21:39,110 INFO     [evaluator.py:465] Running generate_until requests
Running generate_until requests:   0%|                                                                                | 0/500 [00:00<?, ?it/s]Traceback (most recent call last):
  File "/nlp/scr/tthrush/miniconda3/envs/perplexity-correlations/bin/lm_eval", line 8, in <module>
    sys.exit(cli_evaluate())
             ^^^^^^^^^^^^^^
  File "/nlp/scr/tthrush/miniconda3/envs/perplexity-correlations/lib/python3.12/site-packages/lm_eval/__main__.py", line 382, in cli_evaluate
    results = evaluator.simple_evaluate(
              ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/nlp/scr/tthrush/miniconda3/envs/perplexity-correlations/lib/python3.12/site-packages/lm_eval/utils.py", line 397, in _wrapper
    return fn(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^
  File "/nlp/scr/tthrush/miniconda3/envs/perplexity-correlations/lib/python3.12/site-packages/lm_eval/evaluator.py", line 301, in simple_evaluate
    results = evaluate(
              ^^^^^^^^^
  File "/nlp/scr/tthrush/miniconda3/envs/perplexity-correlations/lib/python3.12/site-packages/lm_eval/utils.py", line 397, in _wrapper
    return fn(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^
  File "/nlp/scr/tthrush/miniconda3/envs/perplexity-correlations/lib/python3.12/site-packages/lm_eval/evaluator.py", line 476, in evaluate
    resps = getattr(lm, reqtype)(cloned_reqs)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/nlp/scr/tthrush/miniconda3/envs/perplexity-correlations/lib/python3.12/site-packages/lm_eval/models/huggingface.py", line 1279, in generate_until
    cont = self._model_generate(
           ^^^^^^^^^^^^^^^^^^^^^
  File "/nlp/scr/tthrush/miniconda3/envs/perplexity-correlations/lib/python3.12/site-packages/lm_eval/models/huggingface.py", line 842, in _model_generate
    return self.model.generate(
           ^^^^^^^^^^^^^^^^^^^^
  File "/nlp/scr/tthrush/miniconda3/envs/perplexity-correlations/lib/python3.12/site-packages/torch/utils/_contextlib.py", line 116, in decorate_context
    return func(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^
  File "/nlp/scr/tthrush/miniconda3/envs/perplexity-correlations/lib/python3.12/site-packages/transformers/generation/utils.py", line 1689, in generate
    self._validate_model_kwargs(model_kwargs.copy())
  File "/nlp/scr/tthrush/miniconda3/envs/perplexity-correlations/lib/python3.12/site-packages/transformers/generation/utils.py", line 1243, in _validate_model_kwargs
    raise ValueError(
ValueError: The following `model_kwargs` are not used by the model: ['attention_mask'] (note: typos in the generate arguments will also show up in this list)

Hi! Thanks for reporting the issue. mamba_ssm model type should be able to handle the HF versions as well.


Do not copy Function#length

Fixes #13. Previous PR at #18.
This prevents copying Function#length. That property should reflect the actual parameters length, not the one of the function that is being copied.
For example:
const original = value => value
const wrapper = () => original(true)
mimicFn(wrapper, original)
// This should be 0 not 1
console.log(wrapper.length)

Fixed.

Do you intend to handle the other issues you opened soon or should I do a new release now?

I intend to fix them now. This might be one at a time (to avoid conflicts) depending on the issue though.


Backlog middleware security bug fix

I use HttpRequestMessage.Headers.Authorization instead of httpClient.DefaultRequestHeaders.

Make sure to rebase and squash your PR to avoid merging these ugly "merge" commits :smile:

@PinpointTownes Out of curiosity - do you not think it holds value to know who merged the PR? Rebasing does not provide that information. Or am I missing something?

By "merge" commits, I meant these ones...

... not the "merge pull request" commits that are automatically added when you press the magical green "merge this PR" button :smile:

But yeah, ideally, you'd avoid both ones, as they add unnecessary noise in the git history (specially the first one). Of course, knowing the identity of the person who merged the PR is important, but there's a better way to do that using cherry picking:

https://github.com/aspnet/Security/commit/9bb8b6114697f14a42e747acd1d1a41e6eed5af1
Cherry picking allows you to keep the identity of both the author and the committer (basically, you :smile:). It's rather easy to use, as you just have to run git fetch [fork-address] + git cherry-pick [commit-hash] in your git console to "merge" a remote fork into your local repository ` :smile:

Aah, good info to have, thanks :)

I fixed valiable names and my ugly remote commit log.So, I close this PR and will open new one.

Note that re-opening a PR is usually not needed. You could have removed this commits by simply rebasing :smile:


Add batch synthesis sample code

Purpose

Add C# sample code for TTS batch synthesis API.

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[x] Other... Please describe: Add sample code

@Ling-Cao @dargilco - Should the folder be named "batchsynthesis" or "batch-synthesis" instead of "batchSynthesis"?

@Ling-Cao @dargilco - Should the folder be named "batchsynthesis" or "batch-synthesis" instead of "batchSynthesis"? @yulin-li created batch-synthesis which seems good.

@Ling-Cao @dargilco - Should the folder be named "batchsynthesis" or "batch-synthesis" instead of "batchSynthesis"? @yulin-li created batch-synthesis which seems good.

I refined the folder name and structure to align with Python sample code which looks good for me.

close, see #1730


the style in my htmp wont work

Environment (please complete the following information):

OS (e.g., Windows 10):
Extension version (e.g., 1.2.3):

Describe the bug:

Repro steps:

Expected behavior:

Additional context:

Closing as incomplete. Please feel free to add more information if you have any and we can re-open the issue.


max-inflight tuner

This is an MVP implementation of a basic controller that tunes max-inflight values based on the following trigger:

double the max-inflight values if the number of nodes exceeds a  given threshold
double the max-inflight values if the number of pods exceeds a given threshold

TODO:

The Pod and Node threshold are hard coded today, making it configurable is left as a todo for the future if it proves to be useful.

Need to reset a few hard coded value before it can get in.
/hold

actually never mind. i understand now how scale down works. Please ignore my previous question.

/retest

/retest

/retest

/retest

/retest

/hold cancel

/retest

/retest


Remove test 3

Issue #:
https://github.com/bcgov/met-public/issues/
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the met license (Apache 2.0).

Codecov Report

Merging #8 (6db6126) into main (92f7e2c) will decrease coverage by 0.41%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main       #8      +/-   ##
==========================================
- Coverage   63.11%   62.69%   -0.42%
==========================================
  Files         118      118
  Lines        3131     3131
  Branches      218      218
==========================================
- Hits         1976     1963      -13
- Misses       1127     1140      +13
  Partials       28       28

Flag
Coverage Δ

metapi
62.79% <ø> (-0.74%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

met-api/src/met_api/schemas/engagement.py
68.18% <0.00%> (-9.10%)
:arrow_down:

met-api/src/met_api/services/engagement_service.py
46.15% <0.00%> (-7.70%)
:arrow_down:

...api/src/met_api/services/object_storage_service.py
46.42% <0.00%> (-3.58%)
:arrow_down:

met-api/src/met_api/models/engagement.py
55.43% <0.00%> (-2.18%)
:arrow_down:


[webview_flutter] Android Keyboard automatically closed in landscape mode

In Android landscape mode, input field cannot be edited because the keyboard is automatically closed as soon as it shows up.
Here's the result when I run the example project in my Xperia device (Android 7.1.1)

flutter doctor:
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel beta, v1.17.0, on Mac OS X 10.15.4 19E287, locale en-US)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.0-rc2)
[✓] Xcode - develop for iOS and macOS (Xcode 11.4)
[✓] Chrome - develop for the web
[✓] Android Studio (version 3.6)
[✓] VS Code (version 1.44.2)
[✓] Connected device (3 available)

I can temporarily workaround it by set resizeToAvoidBottomInset=false but it leads to other problem that the keyboard overlapping the input fields

The input field is larger than the available space. Can you try making a demo html page that has an input field that is smaller and see if it replicates

I don't think it's the matter of input field size. As a further investigation, I can keep the keyboard on the screen (with resizeToAvoidBottomInset=true) if I place a bottom margin to the webview. In my device, a bottom margin value at least 80.0 (the height of the keyboard, in this case, is 200.0) will keep the keyboard on the screen, otherwise, the keyboard will be automatically closed.

I am able to replicate this issue on dev channel (1.20.0-0.0.pre). Used below code sample to replicate and tested on Emulator Pixel 3XL, api 27.
In landscape mode, when tapped on input field, the keyboard opens and closes again.

Code Sample
import 'dart:convert';
import 'package:flutter/material.dart';
import 'package:webview_flutter/webview_flutter.dart';

void main() {
  runApp(MyApp());
}

const String kNavigationExamplePage = '''
<!DOCTYPE html>
<title>Navigation Delegate Example</title>

The navigation delegate is set to block navigation to the youtube website.

<input type='search'  placeholder="search"/>
<input type='password'  placeholder="password" />
<input type ='month'  />
<input type = 'file' />

''';

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
        visualDensity: VisualDensity.adaptivePlatformDensity,
      ),
      home: MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key key, this.title}) : super(key: key);
  final String title;

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  WebViewController _webViewController;
  String filePath = "data/user_protocol.html";

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text("html"),
      ),
      body: Container(
        child: WebView(
          initialUrl: "",
          onWebViewCreated: ((WebViewController vc) {
            _webViewController = vc;
            _loadlocalData();
          }),
        ),
      ),
    );
  }

  _loadlocalData() async {
    final String contentBase64 =
        base64Encode(const Utf8Encoder().convert(kNavigationExamplePage));
    _webViewController.loadUrl('data:text/html;base64,$contentBase64');
  }
}

flutter doctor -v
[✓] Flutter (Channel dev, 1.20.0-0.0.pre, on Mac OS X 10.15.2 19C57, locale en-US)
    • Flutter version 1.20.0-0.0.pre at /Users/deeptibelsare/Documents/Fluttersdk/flutter
    • Framework revision d9653445f4 (8 days ago), 2020-06-09 18:43:03 -0400
    • Engine revision e8c13aa012
    • Dart version 2.9.0 (build 2.9.0-14.0.dev 5c1376615e)

[!] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/deeptibelsare/Library/Android/sdk
    • Platform android-29, build-tools 29.0.3
    • ANDROID_HOME = /Users/deeptibelsare/Library/Android/sdk
    • ANDROID_SDK_ROOT = /Users/deeptibelsare/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
    ✗ Android license status unknown.
      Try re-installing or updating your Android SDK Manager.
      See https://developer.android.com/studio/#downloads or visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed instructions.

[✓] Xcode - develop for iOS and macOS (Xcode 11.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.3, Build version 11C29
    • CocoaPods version 1.9.3

[✓] Android Studio (version 3.6)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 45.1.1
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)

[✓] Connected device (1 available)
    • Android SDK built for x86 • emulator-5554 • android-x86 • Android 8.1.0 (API 27) (emulator)

! Doctor found issues in 1 category.

This has been fixed in Flutter 1.22/ WebView 1.0 - use hybrid composition in the Flutter WebView plugin.
https://github.com/flutter/plugins/tree/master/packages/webview_flutter#using-hybrid-composition


feat: Add support for FileSetSpec

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ x ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[x ] Ensure the tests and linter pass
[ x] Code coverage does not decrease (if any source code was changed)
[ x ] Appropriate docs were updated (if necessary)

Fixes #2830 ☕

@shollyman is there someone else I should tag for review?

merge-on-green bot doesn't seem to work, merging manually


Docuemnt and handle throttling errors from MS API

By default there is a retry handler which is added as a middleware here. While it does not throttle at exactly 1200 requests, it seems like around that is what we can consistently hit it with every minute as mentioned here(at least for 0-1k users). As of now, we have a timeout outside the 3 retry that the middware is doing. Not sure if we should remove it, but maybe tweak the number of parallel requests as well as limit to 1200 requests max in a minute.


Cameras don't work when put into a spatial node

Godot version
3.4.4 stable
System information
Windows 10 GLES3
Issue description
making a simple point and click game, the player gets into an area and signals to a camera to get current. This happens until I organize the project, moving all cameras into a simple spatial node. After that cameras no longer work. Deleting the spatial node makes no difference as the problem remains.
Steps to reproduce
Minimal reproduction project

@sanyi12 Did you resolve the problem? If so, please comment here and document the solution :slightly_smiling_face:


How to filter outgoing Heavy Messages?

So using the syntax presented here, I have been able to send data back into Unity without any problems. Now, if I want to have two or more different messages in my patch - [s #messagename @hv_param] - how could I go about specifying which one to read in my C# script? Would I do something like this?
 void OnFloatMessage(Hv_sine_AudioLib.FloatMessage message)
 {
         if (message.receiverName == "justThisMessage")
         {
               do something
         }
 }

or is there a better way?

Yes exactly,
Say you have two output parameters [s thisOutput @hv_param] and [s thatOutput @hv_param]
In your delegate method to filter messages sent to these parameters you can do the following:
 void OnFloatMessage(Hv_myPatch_AudioLib.FloatMessage message)  {
         if (message.receiverName == "thisOutput") {
             // do this
         }
         else if (message.receiverName == "thatOutput") {
               // do that
         }
 }

Understood. Thanks, @diplojocus


[MNT] binder install option does not install optional dependencies used in notebooks

The binder install option should install these so that all notebooks can be run. I'm not even sure if all notebooks run in their current state, so this would require a bit of a review.

@MatthewMiddlehurst  hey i am interested in solving this issue, if its not yet assigned or worked upon

Hey @RohitP2005, I don't think anyone is working on this currently, as this is an old issue, we should first check its status.
A first step would be to try to run the notebooks on binders to see if you get any failures due to missing optional dependencies.

like this :
@aeon-actions-bot assign @RohitP2005

Thanks @baraline


Make sure plugin can be packaged with java 8

1.3.1 is going to be packaged on the java 8 plugin release.

Works great, first try. Now waiting on fiji to release the jogamp java3d version.


fix(SDK): regeneration after content review

PR summary

PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

Current vs new behavior

Does this PR introduce a breaking change?

[x] Yes
[ ] No

Other information

:tada: This PR is included in version 0.0.15 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


n is not a function on Demo > Navigation drawers page

In demo page if I open the Navigation drawers page with browser console opened I have the below error if I click on "Show drawer" checkbox:
Uncaught TypeError: n is not a function
    at jl.update (stores.a1170375.js:1)
    at index.6914141d.js:1
    at Array.forEach (<anonymous>)
    at index.6914141d.js:1
    at HTMLDivElement.check (index.ce2c7e03.js:1)

Thanks! Fixed in 0.1.11, looks like some stores API has changed.


Drop engine argument from scene

@pathunstrom @nicoo I wanted to run this through a PR because it does disable a use case, as evidenced by the test_event_extension test case. Namely: A scene registering itself as an event extension.
My take is that this is ok because this actually has surprising effects, namely that the the event extension is not limited to just when that scene is active (unlike everything else that happens under scenes).
But I just want to run this by you, first.

First thought is we don't have to remove the engine argument completely, just move it to a keyword argument, since **kwargs is loosely the signature for Scene.__init__  folks can still do the weird registration tricks if they want, but I'm also pretty cool with cutting that off, since it's. . . somewhat magical and will probably not do what they expect.

My take is "Just write a system, or use make_engine() and register a top-level"


Footer abnormal movement in empty groups

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
If the user not in any groups/ own any groups the footer persisting an abnormal movement in the groups page

Steps To Reproduce
->profile dropdown
->my groups
->see bug(make sure you have no groups/ your in any group)
Expected Behavior
Irrespective of the groups the footer need to be in bottom of the page
Screenshots
No response
Device Information [optional]
- OS:
- Browser:
- version:

Additional context
No response
Are you working on this issue?
No

I'd like to work on this issue.

Hi,
I think the issue is already resolved as I'm not able to replicate the same in my environment.
Please lmk if the issue still persists.

May be Your seeing in development.
I am found in the production.
Let's wait for sometime if it is duplicated will close.

In production i saw to reproduce the issue in multiple screen but i thing there is no issue with that bottom page


数据集中有数据被误封了，见#925，请求帮忙解封。注意不是恢复数据，是解封数据中某个音频

Thanks for your error report and we appreciate it a lot.
之前的回答未解决问题，详情请见 #925
Checklist

I have searched the tutorial on modelscope  doc-site
I have searched related issues but cannot get the expected help.
The bug has not been fixed in the latest version.

Describe the bug
A clear and concise description of what the bug is.
To Reproduce

What command or script did you run?

A placeholder for the command.

Did you make any modifications on the code or config? Did you understand what you have modified?
What dataset did you use?

Your Environments (required)

OS: uname -a
CPU: lscpu
Commit id (e.g. a3ffc7d8)
You may add addition that may be helpful for locating the problem, such as

How you installed PyTorch [e.g., pip, conda, source]
Other environment variables that may be related (such as $PATH, $LD_LIBRARY_PATH, $PYTHONPATH, etc.)

Please @ corresponding people according to your problem:
Model related: @wenmengzhou @tastelikefeet
Model hub related: @liuyhwangyh
Dataset releated: @wangxingjun778
Finetune related: @tastelikefeet  @Jintao-Huang
Pipeline related: @Firmament-cyou @wenmengzhou
Contribute your model: @zzclynn

数据集卡片上数据集文件那里看起来正常的

数据集卡片上数据集文件那里看起来正常的

假的，API调用的时候打印出来只有202个数据，实际上压缩包里有203个音频；貌似你们只是屏蔽掉了这个提示，但没有治本
请尽快帮忙解除对 https://www.modelscope.cn/datasets/ccmusic-database/bel_canto/files 这个数据集的限制

整个数据集将文件名全部重命名规范化后重做重传了一遍，目前是203，暂时修好了


Structured logging in 4.5-beta4 feedback

Type (choose one):

Feature request (essentially)

NLog version: 4.5-beta4
Platform: all
Hi! I've just updated Seq.Targets.NLog to the new 4.5-beta04 🎉
A few things seem to be missing or are unclear:

MessageTemplateParameter.Format is null in a case like "The number is {0:000}" - I would expect the format to be "000" in this case
In a positional template, there doesn't seem to be a way to get the position of a placeholder except by parsing the parameter's Name
By including @ and $ directives in MessageTemplateParameter.Format, it's ambiguous whether they constitute capture types or .NET format strings (like $ for currency)

With my limited understanding of the feature at this point, it appears these could be resolved cleanly by:

Exposing CaptureType on MessageTemplateParameter, and not duplicating the capture type in Format
Always using Format for the text-formatting information
Providing int or int? Position on MessageTemplateParameter, with a value provided when IsPositional is true

Can you please let me know if I'm on the right track, or misunderstanding how NLog should work, here?
Congratulations on shipping the new beta! :-)

Yes I have not given the MessageTemplateParameters so much love when IsPositional. Will try and improve this.
Did not think about $ being used as actual format-string. Thought I could use it as reserved word.

Thanks for the reply. (I just found #2332 - I can close this and post over there, if you'd prefer - sorry about that!)

Thanks for the reply. (I just found #2332 - I can close this and post over there, if you'd prefer - sorry about that!)

NP! Thanks for the fast feedback!

@nblumhardt or .NET format strings (like $ for currency)

I have looked at the format strings, and I can see one can use "C" to output as localized currency (Ex $), but I cannot see that one can use $ and @ in the format-string.
https://docs.microsoft.com/en-us/dotnet/standard/base-types/standard-numeric-format-strings
If we add the CaptureType to the MessageTemplateParameter, then must also be added as input parameter to the IValueSerializer. What do you think @304NotModified  ?

Providing int or int? Position on MessageTemplateParameter, with a value provided when

Should be easy to implement a method that quickly converts single-digit string-values into integer values.  Should I implement a new getter on MessageTemplateParameter, that does the conversion. What do you think @304NotModified  ?

I have looked at the format strings, and I can see one can use "C" to output as localized currency (Ex $), but I cannot see that one can use $ and @ in the format-string.

@snakefoot thanks for taking a look! I was just giving an arbitrary example; format strings in .NET are extensible, so it's possible for custom types to use @ and $ in their format strings, even if there aren't any readily-located cases of it in the BCL. Seems like it would be better to protect against this during the limited-distribution beta, rather than face a breaking change if/when a conflict is found in the wild.

If we add the CaptureType to the MessageTemplateParameter, then must also be added as input parameter to the IValueSerializer. What do you think @304NotModified ?

I think that's a bit odd, maybe multiple ValueSerializers?

@nblumhardt Have fixed the problem with the parsing of IsPositional-messages, now it performs the full parsing when requested. Just need to have #2340 approved and merged, and new BETA released.

new beta is in the build queue! Thanks @snakefoot !

@nblumhardt Exposing CaptureType on MessageTemplateParameter, and not duplicating the capture type in Format

Providing int or int? Position on MessageTemplateParameter, with a value provided when IsPositional is true

Both features are now implemented with #2348. Guess it is time for a new BETA :)

yes! beta6 is the build queue.
@nblumhardt feel free to open a new issue if there is new feedback. :)

it's on nuget: https://www.nuget.org/packages/NLog/4.5.0-beta06

Thank you! I'll try it out later today 👍


Consider adding CORS support

A welcome feature would be allowing CORS preflight OPTIONS responses. An example use case: running a local web application that points directly at duty.
While avenues can be used to achieve this, it may be worth the implementation cost in this repo for simpler setup for end users.

New image pushed with CORS response implemented


Update CircleCI to use TestEngine v3.4.0

Summary or solution description
Updating CircleCI workflow to run the tests using the updated version of the TestEngine

Task linked: CU-2x0bfc2 Update TestEngine to Neo 3.4.0


windows build fix

please do not merge until both linux and windows builds finish successfully.
The changes include

remove the --all-features from the juniper tests
skip juniper rocket for beta/stable rust channel.
both appveyor and travis will invoke the exact same flow.

Codecov Report

Merging #117 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #117      +/-   ##
==========================================
- Coverage   88.93%   88.92%   -0.01%
==========================================
  Files          95       95
  Lines       13335    13336       +1
==========================================
  Hits        11859    11859
- Misses       1476     1477       +1

Impacted Files
Coverage Δ

juniper/src/lib.rs
88.63% <0%> (-2.07%)
:arrow_down:

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 ca9d5c1...ffb13da. Read the comment docs.

I'm hesitant to merge this as the big problem with cargo-make on Windows was that it didn't fail builds with failing tests. You can see e.g. https://github.com/graphql-rust/juniper/pull/109 - the build log is unfortunately no longer available apparently - but Travis failed (correctly) due to failing tests, while the Appveyor build passed despite having failed tests.

I'll try to reproduce on my fork to see if it is still an issue.
I do know of 2 issues on windows which I fixed yesterday (the PR here came only after I did some fixes for windows).

Ok, found the issue (too bad wasn't reported earlier) and now it is resolved.
You can see example build:
https://ci.appveyor.com/project/sagiegurari/juniper/build/1.0.14/job/40wxjpbbuc068kg6

Interesting but specific platform failed to compile ring 0.11.0 in juniper_rocket.
Seems in your build script you set it up only for msvc platform. I'll do the needed changes.

I see I'll have to work a bit more on it. closing it for now.


[chip-test/sival] Add EDN auto mode test

This PR adds the EDN auto mode test for chip level sival.
For further details, see the commit messages.
This PR resolves #20048.

I ran the test on Verilator and it completes after around 6400 seconds.
We may play around with the parameters OTBN_ROUNDS, OTBN_RANDOMNESS_ITERATIONS, and TEST_PROCEDURE_REPETITIONS to shorten this time, but I don't know how much we could gain there (probably not a lot).
Also please only consider the last few commits when reviewing. The others already have been covered in  #20022.
FYI: @andreaskurth @vogelpi

I managed to get the duration of the test down to around 50 minutes worst case with DVSim. The feedback should also be implemented along with some small additional changes.

Note that CI currently fails with this message:
ERROR: /home/vsts/work/1/s/sw/device/lib/testing/BUILD:44:11: Compiling sw/device/lib/testing/aes_testutils.c failed: (Exit 1): clang failed: error executing command (from target //sw/device/lib/testing:aes_testutils) external/crt/toolchains/lowrisc_rv32imcb/wrappers/clang -MD -MF bazel-out/k8-fastbuild-ST-2cc462681f62/bin/sw/device/lib/testing/_objs/aes_testutils/aes_testutils.d ... (remaining 53 arguments skipped)

Use --sandbox_debug to see verbose messages from the sandbox and retain the sandbox build root for debugging
sw/device/lib/testing/aes_testutils.c:233:33: error: use of undeclared identifier 'kAesTestutilsTimeout'
                                kAesTestutilsTimeout);
                                ^
sw/device/lib/testing/aes_testutils.c:242:33: error: use of undeclared identifier 'kAesTestutilsTimeout'
                                kAesTestutilsTimeout);
                                ^
sw/device/lib/testing/aes_testutils.c:244:33: error: use of undeclared identifier 'kAesTestutilsTimeout'
                                kAesTestutilsTimeout);
                                ^
sw/device/lib/testing/aes_testutils.c:267:33: error: use of undeclared identifier 'kAesTestutilsTimeout'
                                kAesTestutilsTimeout);
                                ^
sw/device/lib/testing/aes_testutils.c:272:33: error: use of undeclared identifier 'kAesTestutilsTimeout'
                                kAesTestutilsTimeout);
                                ^

It seems that on English Breakfast kAesTestutilsTimeout isn't available. It's probably easiest if we have a look at this together on THU.

Note that CI currently fails with this message:
ERROR: /home/vsts/work/1/s/sw/device/lib/testing/BUILD:44:11: Compiling sw/device/lib/testing/aes_testutils.c failed: (Exit 1): clang failed: error executing command (from target //sw/device/lib/testing:aes_testutils) external/crt/toolchains/lowrisc_rv32imcb/wrappers/clang -MD -MF bazel-out/k8-fastbuild-ST-2cc462681f62/bin/sw/device/lib/testing/_objs/aes_testutils/aes_testutils.d ... (remaining 53 arguments skipped)

Use --sandbox_debug to see verbose messages from the sandbox and retain the sandbox build root for debugging
sw/device/lib/testing/aes_testutils.c:233:33: error: use of undeclared identifier 'kAesTestutilsTimeout'
                                kAesTestutilsTimeout);
                                ^
sw/device/lib/testing/aes_testutils.c:242:33: error: use of undeclared identifier 'kAesTestutilsTimeout'
                                kAesTestutilsTimeout);
                                ^
sw/device/lib/testing/aes_testutils.c:244:33: error: use of undeclared identifier 'kAesTestutilsTimeout'
                                kAesTestutilsTimeout);
                                ^
sw/device/lib/testing/aes_testutils.c:267:33: error: use of undeclared identifier 'kAesTestutilsTimeout'
                                kAesTestutilsTimeout);
                                ^
sw/device/lib/testing/aes_testutils.c:272:33: error: use of undeclared identifier 'kAesTestutilsTimeout'
                                kAesTestutilsTimeout);
                                ^

It seems that on English Breakfast kAesTestutilsTimeout isn't available. It's probably easiest if we have a look at this together on THU.

Thanks for noticing, I fixed it. It was an ifdef that I missed on accident.


Make the performance reporting update frequency customizable

The default update frequency has been changed from 1000ms to 250ms. This makes FPS counter and monitor updates smoother and more reactive. I couldn't notice any significant performance decreases by lowering the default reporting rate to 250ms, even on a nearly empty project which ran at over 4,000 FPS on my machine.
I had to redesign the FPS counting algorithm after this one so that FPS counting no longer relies on an update frequency of 1000ms.
This implements https://github.com/godotengine/godot/issues/2794.

Would have liked @reduz to review this one before merging, because Main::iteration() is very critical for performance.

I reverted this, as this is not the right approach to solve the problem, and all it may cause is game stalls (which are happening already with the remote scene tree).
We learned from the remote scene tree than a different way to send this information over from the debugger is needed, probably from a thread. This needs more research before any of this should be touched.


Support for Requestor field in lifecycle event log

Requestor field is missing in lifecycle event log after adding it to EventLifecycle struct.

This pull request didn't trigger Jenkins as its author isn't in the whitelist.
An organization member must perform one of the following:

To have this branch tested by Jenkins, use the "ok to test" command.
To have a one time test done, use the "test this please" command.

Those commands are simple Github comments of the format: "jenkins: COMMAND"

jenkins: test this please

Testsuite passed


Implement ast_builder for is_empty function

resolves #1336

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

10 of 13 (76.92%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.08%) to 98.604%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

core/src/ast_builder/expr/function.rs
10
13
76.92%

Totals

Change from base Build<PHONE_NUMBER>:
0.08%

Covered Lines:
45570

Relevant Lines:
46215

💛  - Coveralls


improve logs

INFO:reporter.reporter:Notification successfully processed

list entity ids, tenant, and servicePaths

INFO:geocoding.factory:Env variable USE_GEOCODING set to 'true', using this value.
INFO:geocoding.factory:Env variable REDIS_HOST set to 'redis-redis-ha', using this value.
INFO:geocoding.factory:Env variable USE_GEOCODING set to 'true', using this value.
INFO:geocoding.factory:Env variable REDIS_HOST set to 'redis-redis-ha', using this value.
INFO:geocoding.factory:Geo Cache env variables set, building a cache.
INFO:geocoding.factory:Env variable REDIS_HOST set to 'redis-redis-ha', using this value.
INFO:geocoding.factory:Env variable REDIS_PORT not set, using default value of: 6379
INFO:reporter.reporter:Env variable LOGLEVEL not set, using default value of: INFO
INFO:translators.factory:Env variable QL_DEFAULT_DB not set, using default value of: crate
INFO:translators.factory:Backend selected for tenant 'ekz' is: crate

move to debug

WARNING:translators.crate:'string' is not a supported NGSI type. Please use any of the following: Array, Boolean, ISO8601, DateTime, Integer, geo:json, geo:point, Number, Text, StructuredValue, timeindex. Falling back to Text.

List entityId and attribute

Hi @chicco785 , Please assign this issue to me if no one is working on it.
Also i want to confirm my understanding on this:

Need to  include list entity ids, tenant, and servicePaths with Backend selected for tenant 'ekz' is: crate.
INFO needs to be moved to DEBUG
Need to include List entityId and attribute  with WARNING:translators.crate:'string' is not a supported NGSI type. Please use any of the following: Array, Boolean, ISO8601, DateTime, Integer, geo:json, geo:point, Number, Text, StructuredValue, timeindex. Falling back to Text.
WARNING needs to be moved to DEBUG.

Please confirm the understanding. Thank you

Hi @chicco785 , Please assign this issue to me if no one is working on it.
Also i want to confirm my understanding on this:

Need to  include list entity ids, tenant, and servicePaths with INFO:reporter.reporter:Notification successfully processed

correct

INFO needs to be moved to DEBUG

geocoding.factory:Env
geocoding.factory:Geo
translators.factory:Backend
should be DEBUG

Need to include List entityId and attribute  with WARNING:translators.crate:'string' is not a supported NGSI type. Please use any of the following: Array, Boolean, ISO8601, DateTime, Integer, geo:json, geo:point, Number, Text, StructuredValue, timeindex. Falling back to Text.

This warning tells that one of the attributes provided is not in the proper format,
the warning needs to include the entityId where this occurred and the attribute that caused the error.

WARNING needs to be moved to DEBUG.

No, it's ok to stay WARNING

Please confirm the understanding. Thank you

closed by #355


EF Core command failed in Linux

Describe the bug
When I try to run any of the Entity Framework Core UI tools I get the same error. "EF Core command failed". The error shows the command that I tried however if I were to copy the command into the terminal in Rider the command will execute.
To Reproduce
Steps to reproduce the behavior:

Create a new ASP.NET Core Web Application. SDK: Net 6, Web App (MVC), Individual Authentication
Try to use the tool

Expected behavior
This is the error I received:
Command: dotnet ef database update --project /home/<username>/<path-to-project>/<projectname>.csproj --startup-project /home/<username>/<path-to-project>/<projectname>.csproj --context <projectname>.Data.ApplicationDbContext --configuration Debug<PHONE_NUMBER>0000_CreateIdentitySchema Output: Unable to retrieve project metadata. Ensure it's an SDK-style project. If you're using a custom BaseIntermediateOutputPath or MSBuildProjectExtensionsPath values, Use the --msbuildprojectextensionspath option. Error: Could not execute because the application was not found or a compatible .NET SDK is not installed. Possible reasons for this include: * You intended to execute a .NET program: The application 'msbuild' does not exist. * You intended to execute a .NET SDK command: It was not possible to find any installed .NET SDKs. Install a .NET SDK from: https://aka.ms/dotnet-download Exit code: 1
Environment (please complete the following information):

OS: Ubuntu 20.04.4 x86_64
.NET SDK info (installed ones and version used within Rider): (installed) 3.1, 5; (using) Net 6.0.301
Plugin version: 1.4.0
Rider version: 2022.1.2

Additional context
I have manually installed the .Net SDKs that I used because snap is annoying. I don't know if that affects this bug.

Hi
Which terminal do you use for manual command execution? Is it the same as the default one in Ubuntu? Seems like the default one's (which is used by GeneralCommandLine we are using from Intellij SDK) environment is used when executing commands via the plugin.

The terminal I use is the default in Rider's environment so /bin/bash.

Are you able to execute command manually from any location, not only from the solution folder?

No I execute the command in the project folder or the solution folder using the terminal window in Rider. If I were to run the command from, for example, my home directory then I would get that error. I think the command from the EF Core tool is not being run from the solution or project directory.

That's the reason. In the current implementation, we are running commands not from solution directory, but using absolute paths to startup and migrations projects.
I'm working on huge refactoring right now that contains necessary change (to also support the local dotnet ef tool) so in the next major update, the issue should be fixed. But it will be available starting from 2022.2 version of Rider.

Ok thank you for your hard work on this tool.

Closing this as a fix will be available in the next update.


Using any instantSearch theme other than algolia throws an error

Version
@nuxtjs/algolia: ^0.9.1
nuxt: 3.0.0-rc.3
Steps to reproduce
algolia: {
    apiKey: "82913680b786acdb395702cbbe8ad805",
    applicationId: "JQ8RQBS5H4",
    instantSearch: {
      theme: "reset",
    },
  },

What is Expected?
As per the docs https://algolia.nuxtjs.org/advanced/
"You can choose a theme from satelite, reset, and algolia. Check out setup docs here"
What is actually happening?
ERROR  [@nuxtjs/algolia] Invalid theme: reset
ERROR  [@nuxtjs/algolia] Invalid theme: satelite
Setup docs also goes to a broken link
https://algolia.nuxtjs.org/advanced/'/setup#instantSearch.theme'

Hi @andyjamesn
Thanks for reporting that issue. I will take a look at it tomorrow and probably release a new version with a quick fix as it should not be something complicated 🙂

@andyjamesn
I did some testing and in my case the algolia and satellite (check if you typed double t in the name, there is a typo in the docs), however it does not work for some reason with 'reset'.
I mean, it is working because the styles are being reset but the error appears in the console for some reason. The error should appear if a selected theme is out of the scope of supported styling.
          if (InstantSearchThemes[theme]) {
            nuxt.options.css.push(`instantsearch.css/themes/${theme}.css`)
          } else {
            console.error('`[@nuxtjs/algolia]` Invalid theme:', theme)
          }

While InstantSearchThemes is:
enum InstantSearchThemes {
  'reset',
  'algolia',
  'satellite',
}

So your 'reset'  should be in the scope of the themes.
Maybe @yassilah you will have some clue why this is like that.
In the meantime I will make some corrections to the docs for vue instantsearch

Hey! Yes, I see why, enum uses the index as value and vice-versa, so InstantSearchThemes.reset is '0' which becomes false in the if statement. I'll fix that in your PR.

@andyjamesn
The module should now work as expected. Thanks for rising this issue and thanks @yassilah for resolving the issue :)

Excellent. Thanks guys! Time to add the reset theme so I can add my own styling with TailwindCSS :D


Resolve Face eagerly when constructing StyledString

This is step 1 to eliminating our type-piracy struggles
StyledStrings needs to put some type that it owns into its AnnotatedStrings so that we have a right to hook into the display logic (and the display logic can know which copy of StyledStrings to delegate to).
It is also a semantic change to how constructing StyledStrings behaves, but overall I think it's a lot more intuitive to have a StyledString actually compute its style information at construction rather than defer it to be looked up at display time (let know if you agree @tecosaur)
Resolves https://github.com/JuliaLang/StyledStrings.jl/issues/87
Made on top of https://github.com/JuliaLang/StyledStrings.jl/pull/95, since that will probably be landed first.

The tests will need adjusting for this - I haven't done that yet sorry
They currently expect that most Faces are unresolved in the AnnotatedStrings we produce

Great to see this work take shape! I'm yet to go through all of your recent flurry of PRs, but from this and a few others, I've developed two linked concerns that I think are worth discussing.
Both concerns stem from the more eager use of getface. The obvious (and seemingly intended) consequence of this is that details are resolved when the StyledString is constructed. There are two implications of this I am least wary of.
Baking in different values, unexpectedly
Firstly, if a string is constructed at compile time/before customisations are loaded, a later use of the same face can look different. Here's an example to illustrate:
# in some package Foo

@define_faces!(:my_foo => Face(foreground=:red, underline=:yellow)) # Assuming future work

const reused_string_or_similar = styled"... {my_foo:stuff} ..."

function something_a(io::IO)
    ...
    println(io, reused_string_or_similar)
    ...
end

function something_b(io::IO)
    ...
    println(io, styled"... {my_foo:other stuff} ...")
    ...
end

In this scenario, should my_foo be customised then something_a differ from something_b in output style, in what I'd consider a surprising way.
Inability to customise downstream
One of the features of the current design is to remap faces on the fly. There are particular use-cases I see this enabling. For instance, say JuliaSyntaxHighlighting is used to bake in static highlighting (an idea that's of some interest to the Documenter crowd). To do this, and produce output for all the various themes you currently could do something like:
const DOCUMENTER_THEME_X = Dict(:julia_number => Face(...), ...)
const DOCUMENTER_THEME_Y = Dict(:julia_number => Face(...), ...)

# ...

function html_highlight_julia(code::String, themes::Vector{Dict{Symbol, Face}})
    hl_code = highlight(code)
    map(themes) do theme
        withfaces(theme) do
            sprint(show, MIME("text/html"), hl_code)
        end
    end
end

Here the code is highlighted once, and then styled with multiple different themes. From what I can tell, this pattern wouldn't be possible with this PR. In this particular example, you could accept the overhead to re-parse the code and apply the faces for each theme before display, but are also other cases where you might want to pass some content across functions/packages this would be a more major impediment.
Closing thoughts
I see how this gets us closer to avoiding type piracy by avoiding a Base-owned type as the value, making it possible to call the annotation owner's specialised printing function.
However, I'm not quite onboard with the specific tradeoffs made here, and wonder if there's a better balance that could be struck. Considering this together with our discussions on effectively implementing custom face namespaces, I wonder if a FaceRef type or similar could be introduced to allow us to avoid the problems with Symbol, without the consequences I've outlined above.

Firstly, if a string is constructed at compile time/before customisations are loaded, a later use of the same face can look different. Here's an example to illustrate:

I think @define_faces! should update the FACES dictionary even for top-level code.
For me, it's just as surprising that this doesn't work:
@define_faces!(:my_foo => Face(foreground=:red, underline=:yellow)) # Assuming future work
const face = getface(:my_foo) # my_foo... wasn't defined yet?
const str = make_annotated("Test string", face)

It's fine to have @define_faces! update the FACES dictionary at pre-compile time - it just also needs to add an __init__ hook since otherwise the entry will be lost at runtime

In this particular example, you could accept the overhead to re-parse the code and apply the faces for each theme before display, but are also other cases where you might want to pass some content across functions/packages this would be a more major impediment.

I think that use case can be handled just by a LazyStyledString equivalent of some kind? I think the lazy approach is OK, but it should be specifically opt-in and a FaceRef would be a useful type to implement it internally.
There are other design problems to make the fully-lazy approach behave correctly though..
The merge(...) we have right now partially resolves inheritance as part of its implementation, but that's not the right behavior since it means you stop seeing future updates to inherited Faces. To make these fully lazy, you need to defer the resolution of inheritance as well, which adds a lot of additional complexity / inefficiency (or surprising merges)
The eager solution lets you perform the merge up-front once and optimize on that concrete style info - that is a good thing.

In this particular example, you could accept the overhead to re-parse the code and apply the faces for each theme before display, but are also other cases where you might want to pass some content across functions/packages this would be a more major impediment.

For that use case, you can use an AnnotatedString (which doesn't know how to display itself yet), and provide a styled(::AnnotatedString) method that applies a set of styling / resolves Faces before you display it?
That way, you still have the same functionality effectively, but it's obvious to the user that something important is happening when styled(...) is called (it is resolving Face definitions and baking them into the string)
That seems better to me than resolving implicitly at every display (which is also a lot of repeated, unnecessary computation)

I think @define_faces! should update the FACES dictionary even for top-level code.

Indeed, but that doesn't prevent the behaviour I describe.

For me, it's just as surprising that this doesn't work:

I'm not quite sure what you're referring to here, if we presume @define_faces! acts like a better version of the addface! of today, getface(:my_foo) works.

I think that use case can be handled just by a LazyStyledString equivalent of some kind? I think the lazy approach is OK, but it should be specifically opt-in and a FaceRef would be a useful type to implement it internally.

We could branch out into lazy and non-lazy versions, but if we can reasonably get away with just one that seems preferable to me.

The merge(...) we have right now partially resolves inheritance as part of its implementation, but that's not the right behavior since it means you stop seeing future updates to inherited Faces and freeze their definition (but only sometimes). To make these fully lazy, you need to defer the resolution of inheritance as well, which adds a lot of additional complexity / inefficiency.

I don't quite see the respect in which inheritance is non-lazy currently, could you elaborate?

The eager solution lets you perform the merge up-front once and optimize on that concrete style info - that is a good thing IMO.

I'm tempted to see how far we can get by having a single lazy form, but offering a freeze/bake/etc. function that fully resolves all faces.

I don't quite see the respect in which inheritance is non-lazy currently, could you elaborate?

Yeah I'm referring to this code:
https://github.com/JuliaLang/StyledStrings.jl/blob/729f56cb7c7064687a19ed02990a532606d83e4c/src/faces.jl#L527-L528

I'm tempted to see how far we can get by having a single lazy form, but offering a freeze/bake/etc. function that fully resolves all faces.

Yeah - We're on the same page about the two "types" of strings at least (and the reify operation)
I just have a very hard time that it's called a "StyledString" and it's not... actually styled yet - it's just marked-up / tagged
We can have those lazy semantics, but if you want the string to know how to resolve its style automatically, we need to add a styler field (and/or type parameter) to AnnotatedString so that it has something to ask to resolve its tags. That's required by the Base / StyledStrings split unfortunately.
It might mean giving up on some composability and making the Base interfaces a bit more complicated (it's actually what I tried first before going this label-based direction)


Bug report - Cannot call type 'array' as a function

What happened?
Dataview: Every row during final data extraction failed with an error; first 3:

            - Cannot call type 'array' as a function
- Cannot call type 'array' as a function

DQL
TABLE
rows.file.aliases AS Aliase,

> rows.link(file.link, file.aliases) AS Link

GROUP BY file.folder

JS
No response
Dataview Version
0.4.26
Obsidian Version
0.14.2
OS
Windows

It's just link:
TABLE row.file.aliases AS "Aliases",
link(file.link, file.aliases) AS Link,
GROUP BY file.folder

Can you show us what the output of this query is? Full output won't be necessary and feel free to strip out sensitive info.
```dataview
table
  file.folder
```

I will explain what I want to do. I need a list (rows) where the name of the link is replaced (by another value) and it can be grouped.

table without id
  link(rows.file.path, rows.file.aliases) as Link,
  rows.file.name
from "Folder"
group by file.aliases

Note that all files without aliases will be grouped together.

Is there an opportunity to make grouping on folders (first column) and change the name of the link to the file with its pseudular (second column)?
| Folders Group | Link (file.link as aliases) |
| ------------- | --------------------------- |
| /folder/      | - 1                         |
|               | - 2                         |
|               | - 3                         |
|               | - 4                         |
|               | - 5                         |
| ---           | ---                         |
| /folder2/     | - 6                         |
|               | - 7                         |
|               | - 8                         |
|               | - 9                         |
|               | - 10                        |

For the next query, two prior conditions:

always a defined alias;
only one alias.

TABLE rows.LK as Link
FLATTEN link(file.link, file.aliases) AS LK
GROUP BY file.folder

Please explain the meaning of the "Flatten" operator

Separates a result having multiple results in a cell (as a list) into separate entries. https://blacksmithgu.github.io/obsidian-dataview/query/queries/#flatten

Hi, is this still an issue?


[DEV] Error code standardization

[x] Error code standardization on ERRCode.md
[x] Modify unwrap panicking
[ ] Testing with Error code

Issue : I think we can only change unwrap-panic to unwrap in the surface layer of our backend application. For example, pool calls equation and equation calls math. Replacing unwrap is possible on pool, but not on equation or math.
This is because unwrap-panic throws runtime error and exits the execution, while unwrap or unwrap-err! just returns an error.
During the runtime , until the public function returns an error and revoke the execution, calculation using the err value happens which internally invokes weird execution and throw totally different error.
To be more detail, best is to check whether changing unwrap-panic to unwrap!-err pair and work. If it does not work, we can just change the things which doest make any trouble.

pool : swapXforY :  if dx=0 it returns true, I inserted assertion but it doesn't work.


Use poetry as the packaging tool

Use the poetry tool for providing package management. This is a natural step towards providing an automated build of the tabatkins.html file.

What are the advantages of poetry over pip?
On Sat, 28 May 2022, 10:50 Stephen Leach, @.***> wrote:

@sfkleach https://github.com/sfkleach requested your review on: #15
https://github.com/GetPoplog/Grammar/pull/15 Use poetry as the
packaging tool.
—
Reply to this email directly, view it on GitHub
https://github.com/GetPoplog/Grammar/pull/15#event-6696869207, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHL4PJJNS7NRKSCTSQYHSTVMHT7BANCNFSM5XGM44RA
.
You are receiving this because your review was requested.Message ID:
@.***>

What are the advantages of poetry over pip?

Roughly speaking ('cos I don't know the exact truth) poetry sits on top of pip and venv to provide a smoother dev experience. In doing so it provides a completely standardised getting-started process. Basically you clone the project and run poetry install. That standardised process was the main reason I switched from using requirements.txt. I also prefer using the poetry command rather than having to activate my virtual environment (or fuss about with direnv to make it automatic) but I guess that is subjective.
It handles dependencies in the pyproject.toml, distinguishing dev dependencies from runtime dependencies, and sets the versions in poetry.lock. The lock file should be checked in order to manage upgrades explicitly. It handles searching for packages, adding them, listing the installed ones - in a reasonably intuitive way. A very nice-to-have feature is the ability to publish to PyPI with a simple command - irrelevant for this tiny script but I figured any answer to your question should include this. I have that working for my TeamLocker project.
The next step will be to automate the build of the tabatkins.html file. Using any python-oriented container it becomes a trivial script: clone, poetry install, make tabatkins.html, save artefact. The only bit I am not 100% sure of is where to save it.

Thanks for the explanation. Sounds good to me. LGTM


3.1.5 has dependency to Azure.Identity containing CVE-2023-36414

Describe the bug
Similar to what was fixed in 5.1.4, the latest release on the 3.1 branch has a transitive dependency on Azure.Identity which has an CVE-2023-36414.
Can this also be fixed on the 3.1 branch ?
If not possible, could there be a comment on how / if consumers of SqlClient 3.1.x are exposed to this CVE. From my own analyses it seems to be only relevant if SqlClient is used with Azure specific credential models, consumers connecting to on premises MS-SQL should not be affected, is this correct ?

@simdanne It think it is not possible, see https://github.com/dotnet/SqlClient/issues/2291#issuecomment-1885683821
What is preventing you form using 5.1.4 / why do you need to stay on 3.1 ?

@ErikEJ we tried that and got into a lot of dependency issues / conflicts.
Not sure on the latest blocker but the dependencies on System.Diagnostics.DiagnosticSource >= 6.0.0 lead to some other packages around EF needed upgrades which then ended up beeing .Net 6+ only (no .net standard version) which is deal break for us since we right now have a dual Model of .Net 6 / 4.8 codebase sharing same DB layer.

@simdanne EF Core 3.1 is out of support https://dotnet.microsoft.com/en-us/platform/support/policy/dotnet-core

We know and are working on migrating away from it. However Data.SqlClient 3.1.x still seems to be in support according to: https://learn.microsoft.com/en-us/sql/connect/ado-net/sqlclient-driver-support-lifecycle?view=sql-server-ver16
So it would great if we would be able to get a fix or as mentioned in the first comment an assessment on how the CVE can actually be triggered through Data.SqlClient.

@simdanne My uderstanding (I cannot locate the exact comment now) is that M.D.S. does not expose the vulnerable path in Azure.Identity

@kf-gonzalez i assume releasing a fix only Azure.identity based on same stable version currently referenced here, then depending on that is not possible ?

i assume releasing a fix only Azure.identity based on same stable version currently referenced here, then depending on that is not possible ?

That's more of a question for the Azure.Identity team.

@David-Engel okay. Since both components are Microsoft owned i would have expected this project to reach out to Azure.Identity team to avoid shipping vulnerable dependencies in a supported version...
Since that doesnt seem to happen i created an issue in their project: https://github.com/Azure/azure-sdk-for-net/issues/41448


I created an API server + Client for tempest/highlight

Because I couldn't run tempest/highlight on PHP 8.2, I created a micro API server for this project. So now it can be used by any project of any language, including solely front-end JavaScript:
API Server: https://highlight.phpexperts.pro/
Git Repo: https://github.com/PHPExpertsInc/highlight.phpexperts.pro
Pure HTML5 + JS client:
<link rel="stylesheet" href="https://highlight.phpexperts.pro/css/highlight-tempest.css" />

This is the actual code from this page to run your own syntax highlighting in pure HTML5.

console.log("Hello, World!");

Install via composer (for self-hosting):
composer create-project phpexperts/tempest-highlight-api
docker compose up -d

It uses phpexperts/dockerize, so it just works out of the box with the necessary PHP 8.3 and nginx already configured.
Detailed instructions for installing on nginx are in the README on GitHub.
PHP client (v7.2-8.3+):
  use PHPExperts\RESTSpeaker\NoAuth;
  use PHPExperts\RESTSpeaker\RESTSpeaker;

  $url = 'https://highlight.phpexperts.pro';
  $highlighter = new class(new NoAuth(), $url) extends RESTSpeaker {
      public function highlight(string $language, string $text): string
      {
          $result = $this->post('/highlight', [
              'lang' => $language,
              'text' => $text
          ]);

          if ($this->getLastStatusCode() === 400) {
              throw new \RuntimeException($result);
          }

          return $result;
      }
  };

  echo $highlighter->highlight('PHP', 'echo "Hello, World!\n";');

Hey, @hopeseekr! I'm glad you were able to find a solution for your use case. As the issue board is here to help track actionable tasks, I'm not sure this is the best place for this, hence why I am closing this specific issue.


There is no way to customize UI, I want Camera Preview within certain height or width in container

There is no proper way to adjust Camera Preview as it takes full screen but I want to combine it with other widgets on Screen and want to give a desired width and height but there is no way to do that, I would request please make it possible  and do a little bit change to the code so that we can customize our UI according to our own width and height as oppose to full screen camera preview

@AmolGangadhare, maybe this is not the best place to put this, but as @majidflutter asked, I would like something like qr_mobile_vision where you can put the camera wherever you like. Maybe even inside a floating_overlay


Use status code integers instead of the actual number

Use the StatusCodes from here - https://github.com/aspnet/HttpAbstractions/blob/dev/src/Microsoft.AspNet.WebUtilities/StatusCodes.cs

ac6a1a6a808ca4d88231db65cf3e5d0b54a1b92f


[stable/kube2iam] Add in imagePullSecrets configuration

Signed-off-by: Benjamin Farley<EMAIL_ADDRESS>
Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it:
In the Kube2IAM chart, it currently doesn't provide a user with the means of pulling the image from a private registry. This PR provides this functionality.
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

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

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @mattfarina

/ok-to-test

/lgtm


Access Denied to TMPE.log

I am running Skylines in Mac Catalina without Steam and currently cannot run the mod due to this issue:
System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation. ---> System.TypeInitializationException: An exception was thrown by the type initializer for TrafficManager.State.GlobalConfig ---> System.UnauthorizedAccessException: Access to the path "/private/var/folders/wr/r9rjlbbd0qjdg71y5p6j6bth0000gn/T/AppTranslocation/1581B0BD-2580-4181-83DD-A50DC144E330/d/Cities.app/Contents/TMPE.log"
Seems that since Sierra version there are some problems writing in these temp folders.
As this seems to be only a log file, I suppose I don't need it. Is there any way I can prevent this file to be written?
Regards

Hi run game with administrator privileges and should be good.
There is no option to turn off logging

Tagging: #379

Sudoing did the trick, thanks


ci: updates workflow for v9 of semantic release

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[X] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[X] All new and existing tests passed.
[X] All commits are signed-off.

Summary
Updates semantic release logic to match trestle. This has been testing to a point with act locally, but not with GitHub Actions in a fork.
Fixes https://github.com/oscal-compass/compliance-trestle-fedramp/issues/37
Related to https://github.com/oscal-compass/compliance-trestle/pull/1564 and will require the same pypi setup
Key links:

Sonar coverage

@vikas-agarwal76 I believe the changes in pypi from https://github.com/oscal-compass/compliance-trestle/pull/1564 would also need to be done for the fedramp plugin. This could possibly change based on the outcome of https://github.com/oscal-compass/community/issues/52.


Heatstorm Heater Error Temperature Display Units

Hello,
I am getting this error on my Heatstorm Heating device.  It appears to be working but in HomeKit it is showing me a tempature of 156 degrees F, which is obviously not accurate.
[11/01/2022, 20:28:49] [homebridge-tuya-platform] This plugin threw an error from the characteristic 'Temperature Display Units': Unhandled error thrown inside read handler for characteristic: This callback function has already been called by someone else; it can only be called one time.. See https://git.io/JtMGR for more info.
[11/01/2022, 20:28:49] [homebridge-tuya-platform] Error: This callback function has already been called by someone else; it can only be called one time.
at /usr/local/lib/node_modules/homebridge/node_modules/hap-nodejs/src/lib/util/once.ts:6:13
at TemperatureDisplayUnits. (/usr/local/lib/node_modules/homebridge-tuya-platform/lib/heater_accessory.js:114:11)
at TemperatureDisplayUnits.emit (node:events:402:35)
at /usr/local/lib/node_modules/homebridge/node_modules/hap-nodejs/src/lib/Characteristic.ts:1544:14
at new Promise ()
at TemperatureDisplayUnits. (/usr/local/lib/node_modules/homebridge/node_modules/hap-nodejs/src/lib/Characteristic.ts:1542:12)
at step (/usr/local/lib/node_modules/homebridge/node_modules/tslib/tslib.js:143:27)
at Object.next (/usr/local/lib/node_modules/homebridge/node_modules/tslib/tslib.js:124:57)
at /usr/local/lib/node_modules/homebridge/node_modules/tslib/tslib.js:117:75
at new Promise ()

I'm pretty sure your heater is locked into celsius.  I have an atomi heater set up through this plug-in and it shows 153 degrees, but if you go into the settings and look at the "unit" option, it shows it's displaying celsius.  It looks like it is going to let you switch it to fahrenheit, but then you can't actually click it.  Hope this helps in resolving the issue.

I’m glad you mentioned Atomi. I was wondering if their products would work with Tuya or not.
Sent from my iPhone

On Jan 16, 2022, at 6:52 PM, sdaltons1 @.***> wrote:

I'm pretty sure your heater is locked into celsius. I have an atomi heater set up through this plug-in and it shows 153 degrees, but if you go into the settings and look at the "unit" option, it shows it's displaying celsius. It looks like it is going to let you switch it to fahrenheit, but then you can't actually click it. Hope this helps in resolving the issue.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.

Atomi does work and I was pretty pumped when I figured it out lol.
Really wish there was a database somewhere, especially since so many different smart apps look and feel very similar to Tuya/Smart Life. Just a list of brands that work with Tuya would be huge.

This looks really similar https://github.com/home-assistant/homebridge-homeassistant/issues/222

I think the enum types need to start at 1 instead of 0 and the error should go away

I'm pretty sure your heater is locked into celsius. I have an atomi heater set up through this plug-in and it shows 153 degrees, but if you go into the settings and look at the "unit" option, it shows it's displaying celsius. It looks like it is going to let you switch it to fahrenheit, but then you can't actually click it. Hope this helps in resolving the issue.

Yeah, I've got three Atomi heaters and have noticed the same thing.  Also noticed when I click on Fahrenheit, it won't select and the check mark stays on Celsius but then the heater gives a "No Response" back.  Clicking on Celsius again and the heater is back online.

I'm also having the same issue with 2 Heat Storm devices. Any updates?

Any updates on this issue? I'm still seeing the same issue mentioned here on an Atomi tower heater.

I have the same issue with the Heat Storm wall heater. The heater itself can be configured to show either C or F, but in HomeKit, it only shows C.

Still an issue. Is there any hope for this?


httpcheck package for beeping

Move all Response's structs in httpcheck package.
Also move CheckSSL to httpcheck namespace.

Thanks !


Question: play-json compatibility

Hello,
I apologize in advance for asking without trying it out first. But when I generate my Json with play-json, will your generated typescript classes be compatible with play-json?
I mostly generate it with (the XYRows are generated by scala-slick or are hand created case-classes):
  implicit val encoder: ai.x.play.json.NameEncoder = ai.x.play.json.CamelToSnakeNameEncoder()
  implicit val samplesFormat = Jsonx.formatCaseClass[SamplesRow]
  implicit val readmodesFormat = Json.format[ReadmodesRow]

I could also switch to another scala-json library that you suggest if I can then use your generator.
thank you very much,
ido

scala-tsi will use field names identical to your case class field names. This is the same behavior as play-json and  xdotai:play-json-extensions.
Because you change camel case to snake name in fields that won't work (CamelToSnakeNameEncoder()). Remove that and it will do the right thing.
Here is how you could do that if you still want to: #107


Need help: Scratch installation causes Error in02tlib.vim and fugitive.vim

Since yesterday I receive an error  when firing up vim.
I don't actually understand it quite well, so maybe someone could support me.
It doesn't seem to restrict the usage. Vim works like usual, it only shows this message on startup
$ vim
Error detected while processing /home/tn/.vim_runtime/sources_non_forked/tlib/plugin/02tlib.vim:
line   77: E1208: -complete used without -nargsError detected while processing /home/tn/.vim_runtime/sources_non_forked/vim-fugitive/plugin/fugitive.vim:
line  431: E1208: -complete used without -nargsline  432: E1208: -complete used without -nargsline  436: E1208: -complete used without -nargsline  440: E1208: -complete used without -nargsPress ENTER or type command to continue

I removed all .vim* dotfiles/folders and could start vim without any flaws (as expected).
I reinstalled this repo.
git clone --depth=1 https://github.com/amix/vimrc.git ~/.vim_runtime
sh ~/.vim_runtime/install_awesome_vimrc.sh

After that I received the exact same error message (unexpected).
So I guess it might have to do something with this config.
Fedora 34 / zsh (oh-my-zsh)
I couldn't find the error message online, only a fix  from 2015 regarding a different error and a note one would not be using vim.

This looks to be a duplicate of https://github.com/amix/vimrc/issues/645. A manual workaround is presented: https://github.com/amix/vimrc/issues/645#issuecomment-880528000 (i.e. delete the -complete clause wherever the error is thrown).

Thanks @arewm I'm gonna close this now and try the workaround on monday.
I wonder why I couldn't find the issue via complete Error description.


[ADD]: Weeman Tool

AIM
HTTP server for phishing in python.
Features
This tool seems to involve testing compatibility with Python versions and platforms, handling command-line options, and launching different functionalities based on user input.
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)
[X] I am a GSSoC'23 contributor

/assign


Property Suggestion: Monotonically normal

Property Suggestion
A space is said to be monotonically normal if it is $T_1$ and there is a function $G$ which assigns to each ordered pair $(H, K)$ of disjoint closed subsets of $X$ an open set $G(H, K)$ such that,

$H \subset G(H, K) \subset \overline{G(H, K)} \subset X \setminus K$;
if $(H', K')$ is a pair of disjoint closed sets having $H \subset H'$ and $K \supset K'$, then $G(H, K) \subset G(H', K')$.

Such a function $G$ is called a monotone normality operator for the space.
Rationale
This property appears in the papers 1, 2, 3, and Chapter 17 of 4. See also its Wikipedia page. All metrizable spaces (see Theorem 6.3 in 1), linearly ordered spaces (see Theorem 5.3 in 1), as well as the Sorgenfrey line (as it is a subspace of the double arrow space, which is linearly ordered), are monotonically normal. Monotonically normal spaces are $T_4$ (basically by definition) and not Dowker (see Theorem 2.3 in Chapter 17 of 4). Equivalently, they are $T_4$ and countably paracompact. This property is also hereditary (see Remarks 2.3(c) in 1), so in particular monotonically normal spaces are hereditarily normal. This provides a general framework for showing a large class of spaces are hereditarily normal and countably paracompact. In particularly, all ordinal spaces are linearly ordered (T489), so they are monotonically normal, thus countably paracompact, thus not Dowker (T158), which is the proposed theorem in issue #916. This issue is inspired by issue #916 and its linked Math.StackExchange answer.
Relationship to other properties
Every metrizable space is monotonically normal.
Every LOTS is monotonically normal.
More generally, every GO-space is monotonically normal, as GO-spaces are homeomorphic to subspaces of LOTS and being monotonically normal is hereditary.
As an example, the Sorgenfrey line is monotonically normal.
Every monotonically normal space is $T_4$.
Every monotonically normal space is countably paracompact.
Every monotonically normal space is not Dowker.
Note: The final two theorems are equivalent to each other, per T158 and T159.

I've been wanting to add that one for a long time.  So I'll try to get this going, as least some part of it.

Closed by #1012 and #1073


Enhancement Request - Ability to retrieve Values from JSONOBJECT using Key

I've been playing with the JSON utility, and one thing I would find very useful is to have the ability to retrieve a value by key.
Say I had the following JSON:
{
"customerId": 1,
"customerName": "Customer A",
"customerAddress": { "street": "1 Line One Street",
"city": "Nowhereville",
"state": "BestState",
"postCode" : null
},
"customerPurchases": [ 	{ "purchaseId" : 1,
"productId" : 231,
"productName" : "A JSON Book",
"purchasePrice" : 24.99
},
{ "purchaseId" : 2,
"productId" : 365,
"productName" : "A JavaScript Book",
"purchasePrice" : 49.95} ],
"isLoyalCustomer": false
}
With the above JSON I would like to be able to retrieve any value using its key. The return data type could be a string, an array, an integer, a float, a null or a JSON depending on what the returned value actually is.
For example, if we had an overloaded method "getValueByKey", then I would like to do the following:

Return the customerId as an integer (1). For example, something like:
var customerId = JSONOBJECT_1.getValueByKey("customerId");

Return the customer's city ("Nowhereville"). For example, something like:

var customerCity = JSONOBJECT_1.getValueByKey("customerAddress").getValueByKey("city");
This example makes the assumption that can use dot notation to return a JSONOBJECT component, then return a string from that JSONOBJECT. I am not sure if that is possible considering that JSONOBJECT is a component, so perhaps some like this is more feasible (though less elegant):
JSONOBJECT_2.createFromJSON(JSONOBJECT_1.getValueByKey("customerAddress"));
var customerCity = JSONOBJECT_2.getValueByKey("city");

Return the productName and purchasePrice of the customerPurchase with purchaseId 2 ("A JavaScript Book", 49.95)

I am not sure how to construct an example for this, mainly because I imagine it would involve looping and I am not so sure of Design Studio's/BIAL's support for looping on JSON objects. That being said, if you guys have a way to achieve this it would be truly useful.
Is it possible to add method(s) to the JSONOBJECT component to allow the above?

Hi Patrick,
please try the release of today. I have placed 2 new functions:
getValueAsJsonObject
getValueAsString
with those, following code should help you. only what you need is to have second JSON object which you cna use as temporary storage.
Karol
JSONOBJECT_1.createFromValue("{"customerId": 1, "customerName": "Customer A","customerAddress": { "street": "1 Line One Street","city": "Nowhereville","state": "BestState","postCode" : null},"customerPurchases": [ { "purchaseId" : 1, "productId" : 231, "productName" : "A JSON Book", "purchasePrice" : 24.99},{ "purchaseId" : 2, "productId" : 365, "productName" : "A JavaScript Book", "purchasePrice" : 49.95} ],"isLoyalCustomer": false}");
// put the output into seconf JSON object
JSONOBJECT_2.createFromJSON(JSONOBJECT_1.getValueAsJsonObject("customerAddress"));
// print full JSON
TEXTAREA_1.setValue(JSONOBJECT_1.getAsJSON());
// read city from customer adress
TEXTAREA_1.setValue(JSONOBJECT_2.getValueAsString("city") + "\r\n" + TEXTAREA_1.getValue());
// read customer id as string
TEXTAREA_1.setValue(JSONOBJECT_1.getValueAsString("customerId") + "\r\n" + TEXTAREA_1.getValue());

Hi Karol,
I'm back working with DS after a long period working on other projects. Just wanted to let you know that I am using this component and it is working perfectly for me. Thank you for adding in these methods.
Cheers, Pat


Disabling ui-router for a single page

How would I go about disabling ui-router for a single page of my app, while not changing its behavior anywhere else?

In case it helps others, I had the exact same question and here is how I solved it (note that the wasn't a similarly titled StackOverflow question)
I basically called ```$urlRouterProvider.deferIntercept();`` which seemed to do the trick
extends index.jade
append head
  //base(href='/angular/component/')
  script.
    angular.module('TM_App')      //disable routing
           .config(
              function($stateProvider, $urlRouterProvider, $locationProvider)
                {
                  $urlRouterProvider.deferIntercept();
                });

append body
  login_form


Save session.json and settings.json

Populate ClientHandshakePacket and ClientConnectionDesc
Refactoring

If ALVR.exe does not update it's content after changes, delete "Temporary Internet Files" with Disk Cleanup. I didn't find a way to refresh the page content without this procedure.

If ALVR.exe does not update it's content after changes, delete "Temporary Internet Files" with Disk Cleanup. I didn't find a way to refresh the page content without this procedure.

You should set the header to non caching to prevent this. We don't need to cache the ui on the client anyway

I was sure there were an easy fix. I'll let you do it.


[BUG] replace the old tooltip package

Description
The current tooltip package which we are using is 2 years old.
We should create a new tooltip which is latest and feature rich
Screenshots
No response

Yup, good point!

might help you https://flowbite.com/docs/components/tooltips/


Mobile national pie map expanding text functionality

Have the text and expanding and collapsing functionality for the national pie map mobile text.
A few things to note:
I really didn't want to be loading in a tiny new JS file so did a tricky rename of the dropdown_selector.js to mobile_ui.js to allow this bit of functionality to make sense in the file.
The current functionality has no transition or fanciness.  We may want to consider using jQuery on the next viz if we want easy to implement transitions and effects as they are built in and jQuery generally cuts the amount of code to write JS down by half.  I don't think we should add it to this one as we are pretty bloated as is.

I forgot that Lindsay is out of town all week. @mwernimont , could you just shorten the Read More and Hide text and then I'll merge? @lindsaycarr - please add comments or new issues later if needed.

@aappling-usgs there ya go!


Cannot Open Jupyter Notebook Files (file.ipynb) in WSL - Remote mode

VS Code Version: 1.54.0-insider (user setup)
OS Version: OS: Windows_NT x64 10.0.19042
Installed Extensions: Python, Remote - WSL, Jupyter Extension for Visual Studio Code.

Steps to Reproduce:

Open VScode Insider
Open folder in WSL2
Click on an file.ipynb to open it
Error Message: "Unable to open '*file*.ipynb': Cannot read property 'Code' of undefined."

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

/extJupyter


Question: How to have a OR in a query?

First I love this plugin, amazing how fast one can find files.
The issue I have not been able to overcome: I have this keymap for searching all lines for the phrase breakpoint:
vim.keymap.set(
    "n",
    "<leader><S-DEL>",
    function()
        require("fzf-lua").lines({query="breakpoint"})
    end,
    {noremap = true, silent = true}
)

However I often work with JavaScript and Python, so I would like to able to match debugger OR breakpoint, I tried the following query terms and none of them worked:
"debugger breakpoint"
"debugger|breakpoint"
"debugger\\|breakpoint"
"\\(debugger\\|breakpoint\\)"

Is it possible?

From man fzf:
OR operator

A single bar character term acts as an OR operator.
For example, the following query matches entries that start with core and end with either go, rb, or py.

e.g. ^core go$ | rb$ | py$

Alternatively you can use an actual regex with lgrep_curbuf:
-- we use no_esc to avoid regex escaping the special char |
require("fzf-lua").lgrep_curbuf({search="debugger|breakpoint",no_esc=true})
-- if you prefer to fuzzy match the regex results use `grep_curbuf` instead
require("fzf-lua").grep_curbuf({search="debugger|breakpoint",no_esc=true})
-- you can also pre enter the fuzzy match query while still having a regex as your base result set
require("fzf-lua").grep_curbuf({search="debugger|breakpoint",no_esc=true,query="foo})

Closing, feel free to reopen if this doesn’t answer your question.

Thanks so much for the reply, I really appreciate the detailed info!
Maybe it's worth noting for others it important to include the spaces around the pipes:
vim.keymap.set("n", "<leader><S-DEL>",
    function()
        require("fzf-lua").lines({query="debugger | breakpoint", no_esc=true})
    end,
    {noremap = true, silent = true}
)


Development

added nav bar

added users list

added users list and working on css improvements

added users list and working on css improvements


Improve getting started experience

What
Currently out best "Getting started guide" is to go through the OCM walkthrough. While this works, it is a lot of work.
Proposal
create a page similar to this:
https://open-cluster-management.io/getting-started/quick-start/
that has pre-reqs and a script based setup. This page should also explain what we are going to setup and perhaps link to the more detailed walkthrough doc
Prereqs:

Kind installed

**Create a new script ** that is for locally trying the system
that prompts for the root domain, and AWS creds if not present
installs OCM and Istio (which we already do)
sets up the ground work in the OCM walkthough (IE two kind clusters, Istio installed, kuadrant (MGC and Data plane) installed via latest images and hooked up, clusterset set up and bound, placement created that targets the clusterset)
Essentially once it is done, they get to create a gateway, create a HTTPRoute and see it work. We should then have follow ons for using the RateLimit policy and creating / updating the DNS Policy

[ ] new script to get you to the point you can create a gateway
[ ] update the getting started page for kuadrant.io and github readme
[ ] Link to the follow ons ie (rate limiting)

@maleck13 Is this different than #280, which is part of the Kuadrant.io Refresh

yeah I would say this is a quick start whereas the install guide should prob be more focused on actual installation rather than setting up placements and clusters for you

As part of this task i'm starting with stepping through https://github.com/Kuadrant/multicluster-gateway-controller/blob/main/docs/how-to/ocm-control-plane-walkthrough.md
I've made a copy of that document here https://docs.google.com/document/d/1cLtLNT9HqGqfQCntpEUIqaX22LMUrHQ8VrRuqIqHswc/edit where myself and @eoinfennessy are making notes about potential improvements as well as discussion about a script.

Hey @maleck13 or anyone. i've been looking at the existing script local-setup.sh to see what can be done for a quick start one line setup. There's a lot that that existing one relies on locally. configuration files and helm charts kept in the config directory and helper functions. before starting into it I want to double check on an approach. i'm new to helm so wondering if those resources can be gotten dynamically somehow? the alternative feels like we'll end up with a massive script which duplicates some of the existing work. did you have any other approach in mind as a preference for this?


Breaking linting setup

Just  installed unity-builder repo on a new pc. I found that it's installing 4 additional hooks and tests are running both through lint staged (only the relevant ones)  and as part of the git hook (all of them).
Subsequently

Lint-staged no longer fails
The catch all test that's now in the pre-commit hook has a bunch of tests that ended up failing (on windows) too
The time to run pre-commit is now way beyond a few seconds, which breaks developer flow.

Result:
It looks like the repository is currently not very developer friendly.
I'm happy with the upgrades to the latest husky and lint-staged on all repos but now we have to make sure that:

we only lint staged files on pre-commit (i.e. using lint staged) to minimize work inside pre-commit hook
it should work on Linux, MacOS and Windows
a fresh clone should require no additional steps other than running yarn and having IDE plugins installed
no new tracked files should be generated right after cloning or running yarn.

I think the changes we've made for unity-builder seem to be working well, so is the next step simply to apply the same changes to the other repos?

Yes. One more thing we should also do is upgrade the requirements to node >=16, as it allows for use of fs/promises and resolves some other weird bugs.


rc-dukes project reuse for device independence

Scenario: rc-dukes project reuse
Situation
The https://github.com/rc-dukes/dukes project exists, created by Tim (@timve) of jpoint.nl
and Bert Jan (@bertjan) of OpenValue
Action
The software is modified to support a spiderbot (e.g. Quadruple).
Expected Result
The existing functionality:

start light detection
lane detection
stop zone detection
engine & steering control

is still available but abstracted.
Comment
The control commands should be different and direct to a different device than with the current software but the resulting movement should be "the same". It might be necessary to scale the movement in time and distance e.g. from 9 m in 10 secs to 1.8m in 15secs.

First concept


[Bug]: Datasource listing page: Design is required

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Datasource listing page: Design is required
Steps To Reproduce
Refer to the screenshot

Public Sample App
No response
Issue video log
No response
Version
1.9.6

@AnandShahs123 @ayushpahwa please find the designs for the active data sources list here.
I've also uploaded the updated designs on the zeplin handoff file.


Enable GPU controls for inference

As a customer using GPU for inference, I want to control if model can use GPU. If a model trained with GPU, but no GPU is available at the inference time, then the models should fall-back to CPU inference without issues.

A simple solution is to set CUDA_VISIBLE_DEVICES in the environmental variable.

Hi! Any updates here? I'm trying to transition a TabularPredictor trained on GPU to CPU for inference (by setting no CUDA_VISIBLE_DEVICES) and am getting the following error

@sxjscience What is the status for CPU inference on GPU trained models for MultimodalPredictor? Refer to the above comment by @jturner116

@Innixma @jturner116 We support training the model with GPU and running inference with CPU.

@sxjscience The MultimodalPredictor (thanks @Innixma , quite right) I was trying to use had a TextPredictor component. That's why it failed when taking over to CPU? Could I skip loading that model manually when loading on CPU?

@sxjscience Yes, I was loading a saved TabularPredictor with these models

The model loaded on CPU but errored on inference.

@jturner116 can you try training with the latest mainline / pre-release? A lot has changed since v0.6.2 regarding multimodal support in Tabular.

@sxjscience Can we open a PR to add a section in quick-start tutorial on how to force CPU inference?

@Innixma @jturner116 I mean, if the instance has only CPUs, we will be using CPU inference by default and there is nothing else you need to do.
This is the most common scenarios we have right now.

@sxjscience @Innixma This model was trained with v0.5.2, I will retrain on GPU with v0.6.2, I'm sure that will fix it.

@jturner116 can you instead try pre-release?
pip install --pre autogluon

As I mentioned, a lot has changed in mainline in preparation for v0.7, so it would be most useful to check that things work in the pre-release.

I am going to mark this as resolved for now, as I suspect it should work without issue in latest mainline / upcoming v0.7 release. Please re-open if you continue having issues.


Is the hack for iOS still valuable?

In SpriteRenderer.js , there are some code for iOS  performance :
https://github.com/pixijs/pixi.js/blob/dev/src/core/sprites/webgl/SpriteRenderer.js#L386
( about canUploadSameBuffer(). )
In the last iOS ( 11.x ) , do we still need them ?

Did you find an answer to this @finscn ? Personally, I never had any issues with ios 11 performance wise.

I don't know any thing about this issue. maybe @ivanpopelyshev could help us

@GoodBoyDigital  might help with it. He's developing new renderer for v5 we can ask him whether this optimization is present there.


Using porthos playground in docker-compose

@alexandrevicenzi @skrater this is how we can use the playground in our porthos' projects. There are lots of room for improvements, but now we can experiment our services with a bit flexibility.


add type (or kind) of textfield

It would be nice to have the ability to control input fields a bit, kind of like secure for password fields, but more general, for these types (sort of like html5 field types).

arbitrary strings (default)
password (the current secure inside style)
email (brings up the email keyboard)
integer (brings up the numeric keyboard)

perhaps one day down the road you could have touch id and other kinds.
not a high priority, since there is a decent alternative. just a nice to have.

Amen.


Modify sync

Changes look good apart from the small comment! Can you also update the documentation page to make sure we describe the new @sync(n); syntax? Merge once done!