Feature/crossvalidation

Implementation of cross validation (which will be used to calculate the overall AUC)
Updated load function to transfer categorical data into dummy variables (so that sklearn random forest classifier and logistic regression classifier can fit the data)
Fixed small errors.

In general I'm not the biggest fan of the term "scorer" because we just set "scores" aka a "score". Might just be me though :)
But besides that just a few minor comments again.

Is scoring better? So because I did have to take the scores and turn them into scorings/scorers so that they take estimators and y values instead of only comparing predicted and true values.

In general I'm not the biggest fan of the term "scorer" because we just set "scores" aka a "score". Might just be me though :)
But besides that just a few minor comments again.

Is scoring better? So because I did have to take the scores and turn them into scorings/scorers so that they take estimators and y values instead of only comparing predicted and true values.

I see. Yeah no worries, you can keep it :)


Fix StereoSound

Its in bad shape.

Does this plugin still work? Or is it entirely broken?

when will stereo be fixed?

any plans on updating the plugin?

I cannot find what is breaking the plugin. I do not know how to fix it, sorry.


Misplaced  in saved MEI after edit

Sometimes after saving an edited MEI file, the output will have a misplaced </layer> before the content is finished. For example
...
<layer xml:id="layer-0000001762477840" n="1">
    <syllable xml:id="syllable-0000001718454274">
        <syl xml:id="syl-0000001225983262" n="initial">
            <rend xml:id="rend-0000001644850468" color="red">O</rend>
        </syl>
        <neume xml:id="neume-0000000787547370">
            <nc xml:id="nc-0000000177574793" oct="3" pname="g" />
            <nc xml:id="nc-0000000213781314" oct="3" pname="f" intm="d" />
            <nc xml:id="nc-0000001799941834" oct="3" pname="g" intm="u" />
        </neume>
    </syllable>
</layer>
<syllable xml:id="syllable-0000000944613709">
    <syl xml:id="syl-0000000867234019">splen_</syl>
    <neume xml:id="neume-0000001065376939">
        <nc xml:id="nc-0000000755714094" oct="4" pname="d" />
        <nc xml:id="nc-0000001352791161" oct="3" pname="b" intm="d" />
    </neume>
    <neume xml:id="neume-0000000065042260">
        <nc xml:id="nc-0000000179101831" oct="3" pname="a" />
        <nc xml:id="nc-0000001633605519" oct="3" pname="b" intm="u" />
    </neume>
</syllable>
...

Any LayerElements outside of the <layer>...</layer> are not rendered and verovio reports this as an error. The rendered neume also is not rendered right.

Issue appears to occur regardless of editor actions. The internal structure of the MEI seems fine (all the Syllables are children of what seems to be the same Layer) so I think it's happening with how verovio writes the MEI.

Nope the MEI doesn't have the same issue using the CLI. So it might be happening in the editor toolkit in which case this will be horrible to debug.

Went through the debugger and the structure is right. Every syllable is a child of the same layer.

Just saving we're getting different output from the CLI and from the JS toolkit. We get a perfectly fine MEI that matches the input by content using ./verovio -o cli.mei -t mei mei4.mei -a while just saving and reloading the MEI in Neon3 misplaces the </layer>.
mei4.mei is the input for both, while cli.mei and neon3.mei are the outputs of the CLI verovio and Neon3 after saving respectively.

cli.mei
neon3.mei
mei4.mei

This issue appears to be due to the JS optimization step in emscripten. Disabling it while keeping LLVM optimizations in place seems to solve the problem.


disable selinux for debian

Issues Resolved
Continuity of PR #122

@peterzhuamazon Can you check this one?

Hi @saravanan30erd ,

#164.
Thanks.


[FLINK-13263] [python] Supports explain DAG plan in flink-python

What is the purpose of the change
Supports explain DAG plan in flink-python
Brief change log

extends existing explain method to supports explain DAG plan
add TableException

Verifying this change

Added test that validates the behavior of extended explain method

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

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

Documentation

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

cc @dianfu @sunjincheng121

@godfreyhe  Thanks a lot for the PR. LGTM. +1 from my side.


Youtube translation issue

+1

PyTube seems to be failing. Another issue is filed here. Will dig deeper and try to fix it in the next couple of days.

Should be fixed with this PR. Closing this for now. Please reopen if it persists.


Fix TypeError: tuple indices must be integers or slices, not str issue with collate_fn

Context
What is the purpose of this PR? Is it to

[ ] add a new feature
[x] fix a bug
[ ] update tests and/or documentation
[ ] other (please add here)

Please link to any issues this PR addresses.
https://github.com/pytorch/torchtune/issues/1010
Changelog
What are the changes made in this PR?
Change indexing
Test plan
Please make sure to do each of the following if applicable to your PR. (If you're not sure about any one of these just ask and we will happily help.)

[ ] run pre-commit hooks and linters (make sure you've first installed via pre-commit install)
[ ] add unit tests for any new functionality
[ ] update docstrings for any new or updated methods or classes
[ ] run unit tests via pytest tests
[ ] run recipe tests via pytest tests -m integration_test
[ ] manually run any new or modified recipes with sufficient proof of correctness

[ ] include relevant commands and any other artifacts in this summary (pastes of loss curves, eval results, etc.)

Appreciate you patching this. Actually we should go the other way - we are standardizing on outputting dictionaries for each sample now instead of a tuple of list[int]. TextCompletionDataset should be updated to return a dictionary.
Since you're already here you can do this or I can make the change, let me know how you want to proceed :)

I can go ahead and amend this PR - give me a few minutes

you'll need to update test_text_completion_dataset.py, test_wikitext_dataset.py, and test_cnn_dailymail_dataset.py as well. You can run these tests locally with pytest tests/torchtune/datasets.

Confirmed it runs on my end with the changes by running one of the built-in configs with wikitext


Add float8_e4m3

This PR adds f8E4M3 type.
f8E4M3 type  follows IEEE 754 convention
f8E4M3 (IEEE 754)
- Exponent bias: 7
- Maximum stored exponent value: 14 (binary 1110)
- Maximum unbiased exponent value: 14 - 7 = 7
- Minimum stored exponent value: 1 (binary 0001)
- Minimum unbiased exponent value: 1 − 7 = −6
- Precision specifies the total number of bits used for the significand (mantisa),
    including implicit leading integer bit = 3 + 1 = 4
- Follows IEEE 754 conventions for representation of special values
- Has Positive and Negative zero
- Has Positive and Negative infinity
- Has NaNs

Additional details:
- Max exp (unbiased): 7
- Min exp (unbiased): -6
- Infinities (+/-): S.1111.000
- Zeros (+/-): S.0000.000
- NaNs: S.1111.{001, 010, 011, 100, 101, 110, 111}
- Max normal number: S.1110.111 = +/-2^(7) x (1 + 0.875) = +/-240
- Min normal number: S.0001.000 = +/-2^(-6)
- Max subnormal number: S.0000.111 = +/-2^(-6) x 0.875 = +/-2^(-9) x 7
- Min subnormal number: S.0000.001 = +/-2^(-6) x 0.125 = +/-2^(-9)

Related LLVM PRs:

LLVM PR-97179 [APFloat] Add support for f8E4M3 IEEE 754 type (Merged)
LLVM PR-97118 [MLIR] Add f8E4M3 IEEE 754 type (Merged)

RFCs:

StableHLO PR-2486 [RFC] Add f8E4M3 and f8E3M4 types support

Related ml_dtypes PRs:

PR-57 Add float8_e4m3fnuz and float8_e5m2fnuz

C++ Testing
Tested as described in PR-123 Add CMakeLists for C++ tests
cmake --build build -- all test

100% tests passed, 0 tests failed out of 250

Total Test time (real) =  12.50 sec

@apivovarov I'm on vacation this week, but I'll take a look next week. Sorry for the delay...

@apivovarov I'm on vacation this week, but I'll take a look next week. Sorry for the delay...

Hi Peter, it seems this PR got overlooked. @hawkinsp
BTW, I've also attached a link to the StableHLO [RFC] Add f8E4M3 and f8E3M4 types support https://github.com/openxla/stablehlo/pull/2486
Not sure what change should be merged first StableHLO or ml_dtypes.

The linter is also sad, please fix.

And it doesn't matter whether the stablehlo change or the ml_dtypes change is merged first, really, they aren't directly coupled.

Linter is still sad.

Linter is still sad.

Fixed. pre-commit run --all-files  - all green now


Add react kawaii project

I added React Kawaii made by @miukimiu
I wasn't sure in what category I should put that, so I created Web category. I accept better suggestions :)

Categories are not yet so well defined, I believe it is better to be dynamic and add and remove categories while we add new projects.


CEO-420 Fix issue with creating parent answer

Purpose
Small bug with parent question answers not showing.
Testing

Create a new project.
Create a question with two answers (I used a button - text component)
Go to create a new question and select "Parent Question" to be the question you just created.
Click the Parent Answer drop down.

This fixes the issue of not having any options in the Parent Answer drop down, but I discovered a new bug with this. Steps to replicate:

Create a new project.
Create a question of type dropdown text with at least two answers.
Create a new question (still of type dropdown text) and select "Parent Question" to be the question you just created. Add at least one answer.
In the survey question tree on the right hand side of the page, try to select any of the dropdown answers from your first question.
The page crashes and the following error is thrown:

Uncaught TypeError: Cannot read properties of undefined (reading 'parentQuestionId')
    at SurveyCollection.componentDidUpdate (SurveyCollection.js?34e0:35:1)


Is there a support for Bower?

Hi,
is it possible to install your script via bower?

@lauriro is there any reason why you don't tag and release? Using a short-sha seems impractical to me.

No reason, I have not needed them.
Now there are the first tag and there are no problem to add them in the future


Mute simulator using the simulator toolbar

Split the sim toolbar into two parts (as you can see in the screenshot)

Looks great. When receiving a mute message, should you call stop() in case the AudioManager is already ringing a tone?


(test) Add test for more than one forwarding hop

Description

closes: #6682

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against the correct branch (see CONTRIBUTING.md).
[ ] Linked to GitHub issue with discussion and accepted design, OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/).
[ ] Added relevant godoc comments.
[ ] Provide a conventional commit message to follow the repository standards.
[ ] Include a descriptive changelog entry when appropriate. This may be left to the discretion of the PR reviewers. (e.g. chores should be omitted from changelog)
[ ] Re-reviewed Files changed in the GitHub PR explorer.
[ ] Review SonarCloud Report in the comment section below once CI passes.

I think we can also turn this into a table test at some point, but I'm totally good to move forward with this


remove duplicate and error code.

fix typo.

Thanks for this @johncming, but this behavior is desired. it's using the check-lock-check paradigm to make sure that a connection for a given address gets set-up once.
It first checks if the connection is already there. if it is, it returns it and exits.
if its not, it locks. 2 or more requests might be waiting, but only one should go in and create the connection. once the lock is freed, the remaining requests will again check if the connection is there, and will exit if it is.
As such, I will close this PR, but thanks again for your willingness to contribute to Dapr, this is highly appreciated.

@yaron2
But question is that he first check is not concurrency safe in go env which leads to an error.
How to resolve the unsafe case.
Thanks

@yaron2
But question is that the first check is not concurrency safe in go env which leads to an error.
How to resolve the unsafe case.
Thanks

So that's a different issue. The solution here would be to turn the lock into an RWMutex, and RLock/RUnlock for the first check.

@yaron2
But question is that the first check is not concurrency safe in go env which leads to an error.
How to resolve the unsafe case.
Thanks

So that's a different issue. The solution here would be to turn the lock into an RWMutex, and RLock/RUnlock for the first check.

OK. If it needs to be modified, I can do it.

Yes, if you can please open a new PR for that. Thanks!

Yes, if you can please open a new PR for that. Thanks!

@yaron2
Read-write locks may not be required, because of the field to protect is connectionPool
type Manager struct {
	AppClient      *grpc.ClientConn
	lock           *sync.Mutex
	connectionPool map[string]*grpc.ClientConn
	auth           security.Authenticator
}

The longer operation is that create a new connection if not exists
	if !skipTLS && g.auth != nil {
		signedCert := g.auth.GetCurrentSignedCert()
		cert, err := tls.X509KeyPair(signedCert.WorkloadCert, signedCert.PrivateKeyPem)
		if err != nil {
			return nil, fmt.Errorf("error generating x509 Key Pair: %s", err)
		}

		ta := credentials.NewTLS(&tls.Config{
			ServerName:   id,
			Certificates: []tls.Certificate{cert},
			RootCAs:      signedCert.TrustChain,
		})
		opts = append(opts, grpc.WithTransportCredentials(ta))
	} else {
		opts = append(opts, grpc.WithInsecure())
	}

So make the following changes
func (g *Manager) GetGRPCConnection(address, id string, skipTLS, recreateIfExists bool) (*grpc.ClientConn, error) {
	g.lock.Lock()
	if val, ok := g.connectionPool[address]; ok && !recreateIfExists {
		g.lock.Unlock()
		return val, nil
	}

	opts := []grpc.DialOption{
		grpc.WithBlock(),
	}
	if !skipTLS && g.auth != nil {
		signedCert := g.auth.GetCurrentSignedCert()
		cert, err := tls.X509KeyPair(signedCert.WorkloadCert, signedCert.PrivateKeyPem)
		if err != nil {
			return nil, fmt.Errorf("error generating x509 Key Pair: %s", err)
		}

		ta := credentials.NewTLS(&tls.Config{
			ServerName:   id,
			Certificates: []tls.Certificate{cert},
			RootCAs:      signedCert.TrustChain,
		})
		opts = append(opts, grpc.WithTransportCredentials(ta))
	} else {
		opts = append(opts, grpc.WithInsecure())
	}

	conn, err := grpc.Dial(address, opts...)
	if err != nil {
		return nil, err
	}

         g.lock.Lock()
	g.connectionPool[address] = conn
	g.lock.Unlock()

	return conn, nil
}


Add tags to all Amplify resources

Is your feature request related to a problem? Please describe.
It can be difficult to get an overview of all resources provisioned by Amplify. This would be greatly helped if all the resources had a common tag.
Describe the solution you'd like
All resources tagged with an "amplify-project" tag.
Ability to add other custom tags.

Yes, sorry for my very late reply.
What I am trying to do is to get a quick view of all resources provisioned by Amplify for a given project. I know that I can look through all the stack instances and see what was deployed, but it would be even better if I can find them via resource groups.
Another use case would be for companies who might be required to tag resources with certain compliance tags like environment or cost centre.

It would also be useful to tag resources with their env. That way any custom IAM polices can reference these resources based on tags, rather than resource names or arns (which may get recreated)

This is important for AWS Amplify to be managed by enterprise projects where teams share AWS accounts. bump

We're running into this as well to run into a better understanding of where AWS costs are going, is this on the roadmap anywhere?

Same need here, my use case is to configure a backup plan on AWS Backup service.
A default tag like this would be great: ${projectName}-${env}-${category} (so I can easily define different backup rules for storage and API).

Hi folks. I'm closing this issue as we've launched resource tagging support. See #365


Add documentation for Microsoft Robotics Studio

No details provided.

⚠ Idea migrated from UserVoice
Created By: Vincent Nwonah
Created On: 2017/12/09 00:14:40 +0000
Votes at Migration: 1
Supporters at Migration: 1

#log-suggestion

🚀 ATTENTION: Internal request logged.

Closing as this will be part of the migration.


Hiding floatingActionButton on new page also hides it on the previous page

Steps to Reproduce
I am running into an issue where we have a floating action button on two consecutive screens.  There is a floating action button on the first screen, and when pressed takes you to a new screen with a different floating action button. After some action is done on this second screen, we hide the floating action button.  Now, when popping to the previous screen, its floating action button is also hidden which seems like an unintended bug.
First Screen
import 'package:flutter/material.dart';
import 'SecondScreen.dart';

class FirstScreen extends StatefulWidget {
  FirstScreen();

  @override
  State<StatefulWidget> createState() {
    return new FirstScreenState();
  }
}

class FirstScreenState extends State<FirstScreen>{

  loadSecondScreen() {
    SecondScreen secondScreen = new SecondScreen();
    Navigator.of(context).push(new MaterialPageRoute(builder: (context) => secondScreen));
  }

  @override
  Widget build(BuildContext context) {
    return new Scaffold(
      floatingActionButton: new FloatingActionButton(
        onPressed: () {loadSecondScreen();},
        tooltip: "Submit",
        child: new Icon(Icons.add),
      ),
      appBar: new AppBar(
        title: new Text("First Screen"),
        elevation: 0.0,
        centerTitle: true,
      ),
      body: new Container(),
    );
  }
}

Second Screen
import 'package:flutter/material.dart';

class SecondScreen extends StatefulWidget {
  SecondScreen();

  @override
  State<StatefulWidget> createState() {
    return new SecondScreenState();
  }
}

class SecondScreenState extends State<SecondScreen>{
  bool shouldShowFloatingActionButton = true;

  hideFloatingActionButton() {
    setState(() {
      shouldShowFloatingActionButton = false;
    });
  }

  @override
  Widget build(BuildContext context) {
    return new Scaffold(
      floatingActionButton: shouldShowFloatingActionButton ? new FloatingActionButton(
        onPressed: () {},
        tooltip: "Submit",
        child: new Icon(Icons.add),
      ) : null,
      appBar: new AppBar(
        title: new Text("First Screen"),
        elevation: 0.0,
        centerTitle: true,
      ),
      body: new Container(
        child: new Center(
          child: new RaisedButton(
            child: Text('Tap to hide floating action button'),
            onPressed: () {
              hideFloatingActionButton();
            },
          ),
        ),
      ),
    );
  }
}

❯ flutter analyze

   info • Dead code • lib/Controller/DashboardController.dart:151:5 • dead_code
   info • Unused import: 'dart:math' • lib/Controller/WorkOrderCreatorController.dart:2:8 • unused_import
   info • The value of the field '_employeeText' isn't used • lib/Controller/WorkOrderCreatorController.dart:1642:10 • unused_field
   info • The value of the local variable 'jsonResponse' isn't used • lib/calls.dart:3494:7 • unused_local_variable

4 issues found. (ran in 7.2s)

❯ flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel dev, v1.9.6, on Mac OS X 10.14.6 18G95, locale en-US)

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 10.3)
[✓] Android Studio (version 3.3)
[✓] VS Code (version 1.37.1)
[✓] Connected device (1 available)

• No issues found!

I just tried it but could not reproduce it. I used master V1.10.3-pre.39.
Could you try if it also happens if you switch to the latest master?

Switching to the newest master commit worked for me.  Thanks!

Could you please check if it also works on the latest stable?

I just tried it on the stable branch and the issue occurs there as well.

Thanks

@HansMuller do we have a label for problems that are fixed in coming versions?

Hi @LampeMW
Switching to latest master, solved the issue for you comment reference
if you're still experiencing this problem, please create a new issue and provide the information
required in the template.
Thank you


FAQ Section

As a user I want to have quick answers to common issues so that I can speed up my development.
Suggestions:

Provide a FAQ section
Provide a Slack or Discord channel for quick answers (especially for the tool competition)

Thanks for the suggestion, I enabled the discussion section to facilitate this.
Hopefully, it will grow to include all common QAs.


BootWifi.boot() failing

BootWifi.boot() hangs (or crashes) in esp_wifi_init_internal().  I must be doing something stupid, but I can't think what it is. My main.cpp is below. The esp-idf-template runs OK when using the same sdkconfig.
#include "freertos/FreeRTOS.h"
#include "esp_system.h"
#include "driver/gpio.h"
#include "BootWiFi.h"

#define LED_PIN   GPIO_NUM_2

BootWiFi boot;

extern "C" void app_main(void)
{
    boot.boot();

    gpio_set_direction(LED_PIN, GPIO_MODE_OUTPUT);
    int level = 0;
    while (true) {
        gpio_set_level(LED_PIN, level);
        level = !level;
        vTaskDelay(300 / portTICK_PERIOD_MS);
    }
}

Howdy my friend.  GREAT find.  I was able to recreate form your sample just fine.  MANY thanks for that.  The problem is interesting.   If we look at your code, you have a global/static instance called boot.  What I mean by this is that you have created an instance of "boot" that is global to this file and static.  What that means is that the instance of the called (BootWiFi) is constructed when the C++ runtime is initialized.  This occurs before the FreeRTOS runtime is initialized.  The BootWiFi class requires the FreeRTOS runtime to be available because we create semaphores and other items during class instance construction.  If we change your code to:
...
extern "C" void app_main(void)
{
   BootWiFi boot;
   boot.boot();
   ...

all appears to work.  If need a global, then we can change the code to:
BootWifi *pBoot;
extern "C" void app_main(void)
{
   pBoot = new BootWifi();
   boot->boot();
   ...

Can you review this post and see if it makes sense and test it in your environment.  Is this new usage pattern satisfactory for you?  If not, it would be possible for me to reworks the internals of BootWiFi to accommodate late creation of FreeRTOS data on demand.

Hi. Thank you for the clear explanation and the change to a variable instantiated inside app_main() works fine. I thought I must be doing something simple but stupid! No need to make changes to the code, a comment in the README will suffice to stop others making the same mistake.
BTW, many thanks for the excellent collection of code.  My plans include some of the enhancements you list in the BootWiFi README, I will send the changes if any of it looks sensible.


HullIntegrity NRE

Reported in RT ticket-2719 and ticket-2712
2020-01-12T16:41:35 FYLS [ERROR] CRITICAL ERROR, PLEASE REPORT:
Delegate OnAttackSequenceEnd - Standard for message type OnAttackSequenceEnd failed with exception
Object reference not set to an instance of an object
  at CBTBehaviorsEnhanced.HullIntegrity.AttackDirector_OnAttackSequenceEnd.Postfix (BattleTech.AttackDirector __instance, MessageCenterMessage message) [0x00027] in <185c52a92410424f8ed81cd62b2574c1>:0
  at (wrapper dynamic-method) BattleTech.AttackDirector.OnAttackSequenceEnd_Patch1(object,MessageCenterMessage)
  at MessageCenter.SendMessagesForType (MessageCenterMessageType messageType, MessageCenterMessage message) [0x00037] in <3a3b490f618743a4be67f60469483e7a>:0

Fixed in 0.3.8


Use container element ref in place of id

This PR changes the container element from having an id attribute to using an element ref. The configuration is now passed the container in place of divId.

 Get rid of id on chart container in seatsio-vue

👍
Do we still need to set width and height of 100% on the container div, like we do in React? If not I assume the container div will not adapt to the dimensions of its parent.

👍
Do we still need to set width and height of 100% on the container div, like we do in React? If not I assume the container div will not adapt to the dimensions of its parent.

Hmm, will need to do something there apparently. I see the designer is very narrow currently. Checking

Seems to be fixed now 👍


Error when evaluating SSR module /src/routes/__layout.svelte

Describe the bug
New error showing up 99% of the time now, and no idea what causes it to work once and a while.
package.json is there....
marcgodard@pop-os:~/Documents/GitHub/gs-web$ npm run dev

><EMAIL_ADDRESS>dev
> svelte-kit dev

Pre-bundling dependencies:
  svelte
  svelte/store
  svelte-intl-precompile
  svelte/transition
  @feathersjs/client
  (...and 15 more)
(this will be run only when your dependencies or config have changed)

  SvelteKit v1.0.0-next.260

  local:   http://localhost:3000
  network: not exposed

  Use --host to expose server to other devices on this network

4:31:00 PM [vite] Error when evaluating SSR module /src/routes/__layout.svelte:
Error: Cannot find module '/home/marcgodard/Documents/GitHub/gs-web/package.json' imported from '/home/marcgodard/Documents/GitHub/gs-web/node_modules/vite/dist/node/chunks/dep-f5552faa.js'
    at viteResolve (/home/marcgodard/Documents/GitHub/gs-web/node_modules/vite/dist/node/chunks/dep-f5552faa.js:60126:25)
    at Function._resolveFilename (/home/marcgodard/Documents/GitHub/gs-web/node_modules/vite/dist/node/chunks/dep-f5552faa.js:60140:28)
    at Function.Module._load (node:internal/modules/cjs/loader:778:27)
    at Module.require (node:internal/modules/cjs/loader:1005:19)
    at require (node:internal/modules/cjs/helpers:102:18)
    at Object.search (/home/marcgodard/Documents/GitHub/gs-web/node_modules/vite/dist/node/chunks/dep-f5552faa.js:12705:33)
    at async resolvePostcssConfig (/home/marcgodard/Documents/GitHub/gs-web/node_modules/vite/dist/node/chunks/dep-f5552faa.js:20506:22)
    at async compileCSS (/home/marcgodard/Documents/GitHub/gs-web/node_modules/vite/dist/node/chunks/dep-f5552faa.js:20329:27)
    at async TransformContext.transform (/home/marcgodard/Documents/GitHub/gs-web/node_modules/vite/dist/node/chunks/dep-f5552faa.js:20015:50)
    at async Object.transform (/home/marcgodard/Documents/GitHub/gs-web/node_modules/vite/dist/node/chunks/dep-f5552faa.js:36985:30)
Cannot find module '/home/marcgodard/Documents/GitHub/gs-web/package.json' imported from '/home/marcgodard/Documents/GitHub/gs-web/node_modules/vite/dist/node/chunks/dep-f5552faa.js'
Error: Cannot find module '/home/marcgodard/Documents/GitHub/gs-web/package.json' imported from '/home/marcgodard/Documents/GitHub/gs-web/node_modules/vite/dist/node/chunks/dep-f5552faa.js'
    at viteResolve (/home/marcgodard/Documents/GitHub/gs-web/node_modules/vite/dist/node/chunks/dep-f5552faa.js:60126:25)
    at Function._resolveFilename (/home/marcgodard/Documents/GitHub/gs-web/node_modules/vite/dist/node/chunks/dep-f5552faa.js:60140:28)
    at Function.Module._load (node:internal/modules/cjs/loader:778:27)
    at Module.require (node:internal/modules/cjs/loader:1005:19)
    at require (node:internal/modules/cjs/helpers:102:18)
    at Object.search (/home/marcgodard/Documents/GitHub/gs-web/node_modules/vite/dist/node/chunks/dep-f5552faa.js:12705:33)
    at async resolvePostcssConfig (/home/marcgodard/Documents/GitHub/gs-web/node_modules/vite/dist/node/chunks/dep-f5552faa.js:20506:22)
    at async compileCSS (/home/marcgodard/Documents/GitHub/gs-web/node_modules/vite/dist/node/chunks/dep-f5552faa.js:20329:27)
    at async TransformContext.transform (/home/marcgodard/Documents/GitHub/gs-web/node_modules/vite/dist/node/chunks/dep-f5552faa.js:20015:50)
    at async Object.transform (/home/marcgodard/Documents/GitHub/gs-web/node_modules/vite/dist/node/chunks/dep-f5552faa.js:36985:30)
[svelte-preprocess] PostCSS configuration was not passed or is invalid. If you expect to load it from a file make sure to install "postcss-load-config" and try again.

Error: Cannot find module '/home/marcgodard/Documents/GitHub/gs-web/package.json' imported from '/home/marcgodard/Documents/GitHub/gs-web/node_modules/lilconfig/dist/index.js'

Reproduction
Unknown.
Logs
No response
System Info
System:
    OS: Linux 5.15 Pop!_OS 21.10
    CPU: (16) x64 11th Gen Intel(R) Core(TM) i7-11800H @ 2.30GHz
    Memory: 3.94 GB / 15.36 GB
    Container: Yes
    Shell: 5.1.8 - /bin/bash
  Binaries:
    Node: 16.13.1 - ~/.nvm/versions/node/v16.13.1/bin/node
    npm: 8.1.2 - ~/.nvm/versions/node/v16.13.1/bin/npm
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Firefox: 96.0
  npmPackages:
    @sveltejs/kit: ^1.0.0-next.196 => 1.0.0-next.260
    svelte: ^3.44.2 => 3.46.4

Severity
blocking all usage of SvelteKit
Additional Information
No response

Vite 2.8 is coming out next week. Hopefully it will fix it. You could try a beta in the meantime

How can i use the beta as i am unable to run my server ?

Any update on solution @Conduitry

It is little urgent as my production is down.

I just noticed that when I turn on my graphics card, I don't get this error.

Please don't ping specific maintainers
You can try the beta with pnpm overrides
There's no way graphics card has anything to do with this. Rather it just happens sometimes and that's a spurious correlation

See https://github.com/sveltejs/kit/issues/3118#issuecomment-1016595911 to use overrides to fix this. The issue is fixed in Vite 2.8 beta, a stable 2.8 version will be released soon this week.

See #3118 (comment) to use overrides to fix this. The issue is fixed in Vite 2.8 beta, a stable 2.8 version will be released soon this week.

How would you do this in your package.json?

Please don't ping specific maintainers
You can try the beta with pnpm overrides
There's no way graphics card has anything to do with this. Rather it just happens sometimes and that's a spurious correlation

How would you do this in your package.json?

I just think it is a race condition and having my graphic card on makes the system run faster. I did a package update and it has completely disappeared now.


Change twitter package maintainer

This patch changes the twitter package maintainer. @kubo39 is the original developer, but has decided to archive the project, so no further pull request can be made.
The forked version 0.2.1 fixes the build issues due to changes in latest nim.

Thank you for taking over the maintenance of this!


trash-cli: fix build

Motivation for this change
More fallout from enabling strictDeps in #55757. buildInputs are not
accessible from the check phase.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Backported for 19.03 (#56826)
[release-19.03 42db1326556] trash-cli: fix build (#56818)
Date: Mon Mar 4 14:52:33 2019 +0100
1 file changed, 4 insertions(+), 2 deletions(-)


Add description of timeout values

WHAT
I was in doubt if the timeout values were in miliseconds or seconds, I confirmed checking one issue which mentioned the default value was in seconds https://github.com/ashkan18/graphlient/issues/51#issuecomment-487369055
I think is a good option to make this information explicit

Thank you, merged.


Different colors for different projects

Is it possible to have something like this?
The year overview could still be monochromatic, but when showing specific projects like in week and day overview, it would be nice if they could have different colors. Maybe doing something like below..
"details": [{
    "name": "Project 1",
    "date": "2016-01-01 12:30:45",
    "value": 9192,
    "color": '#ff4500'
  }]

Absolutely! you could easily add this to the week or day overviews.
I'm not sure how much interest there is from others for this feature to be part of the library (and to be honest I don't want to update all versions of the heatmap like react etc). You can fork this repo and customize it for your project though.


Too many tools in one pull-down

I DON’T LIKE:

having all the tools in one dropdown box.  Having the markup tools in one dropdown is fine, but I most often use the Pan, ID, and Selection tools, so it would be nice to see all three without the dropdown, even though I realize that would make the map slightly less clean.

From Stephanie Weyer at KZLA

By now supporting of tablets and mobile devices, GPV screen real estate is at a premium. Storing all GPV tools in one drop-down a straight forward way to save screen space and store the many GPV tools in one place.


user-local-time shows empty time for people with no commits

Maybe a nicer message will help.

We do already show this on hover:

In my system, it comes only when I hover over that time area.

Right, it's a title attribute

Will that be enough? How about not adding time part for such users?

Yeah ideally we wouldn't show the icon at all, but sometimes it might take several seconds until it appears, so we have a "loading" in place. The loading is then replaced by the time or -. Perhaps we could:

show Not found, instead
check the cache first, if it's there and it's false, we shouldn't add the icon at all.

how Not found, instead

👍

I believe that it would be better if no icon nor message was displayed for people with no commits,
i.e. as @fregante described as a different approach:

check the cache first, if it's there and it's undefined, we shouldn't add the icon at all.

I agree with the OP that having a plain clock icon (followed by a - or not) doesn't help,
but I also find the current Not found message distracting, and, at first glance, it always gives me the wrong impression that the relevant account itself no longer exists/has been deleted, instead that's just a RG feature message. e.g.:
.

What if we do Timezone not found?

I think it will be a long, distracting string (even if it's in smaller font)
which doesn't offer any info to the user apart from RG debugging
(i.e. that RG is indeed enabled and that it couldn't find the user timezone).
Here is a mockup:

I prefer it in its initial form:

If we show the clock icon while loading and then hide it without a reason, it might look like a bug.
If we don't show the clock icon while loading, the user doesn't know whether some information is coming… or if it tried and failed.
"Timezone unknown" would be the more appropriate response.
There could be an additional optimization: if the time is already cached as "missing", then don't show the icon at all.

That's what I also had in mind. Just to summarize:

at the initial mouse hover, show the clock icon while loading. Then:

if the information has been found, append the found timezone
if the information has not been found, append "Timezone unknown" instead

at next mouse hover:

if there's information stored in cache, show the clock icon and the retrieved timezone
if the time is already cached as "missing", then don't show anything at all, not the icon not any text message.

Code-wise, this could be done with a timer instead of manually checking the cache, etc.
Example to be inserted on line 104
const timeout = delay(300);
const date = getLastCommitDate();
// This will delay adding the icon for 300ms, enough to check if the date is store as "not found"
const race = await Promise.race([timeout, date]);
if (race === false) {
	// `date` was cached as "not found", so don't add the icon at all
	return;
}


fix update core config bug

Codecov Report
Base: 62.36% // Head: 62.39% // Increases project coverage by +0.03% :tada:

Coverage data is based on head (62b6213) compared to base (babf377).
Patch coverage: 50.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #534      +/-   ##
==========================================
+ Coverage   62.36%   62.39%   +0.03%
==========================================
  Files          94       94
  Lines        7787     7788       +1
==========================================
+ Hits         4856     4859       +3
+ Misses       2151     2149       -2
  Partials      780      780

Impacted Files
Coverage Δ

api/node.go
56.39% <50.00%> (+0.05%)
:arrow_up:

server/mis_server.go
72.13% <0.00%> (+3.27%)
:arrow_up:

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

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


Install new nuget for protos. Update references.

Uncoupled the Campaigns module from the proto files and installed the new nuget package.
Closes #261

@xenry @mayapeneva Can you please check this PR and merge it whenever you're available if nothing else needs to be added


Update README.md

Update README with information the library currently does not function correctly on API 23+ devices - As per information given in #14

A message had already been added in the dev branch. I cherry-picked the commit and pushed it to master in stead.


Fixes odata from output for planner bucket get

Closes #3825

Thank you @huseyinsaatci
We'll review it ASAP!


Upgrade hclfmt (+alpha: see description)

This addresses https://github.com/gruntwork-io/terragrunt/issues/1092, where the hcl2 library was too old, and thus it did not include the fix for handling heredoc syntax correctly.
However, in order to update to the latest hcl2 version, we needed to switch to hashicorp/hcl/v2 as the hcl2 repo is now archived and merged into the mainstream hcl repo. The problem was that this did not work too well with dep, and so I also took this chance to update to using go modules.
Note that in the process, all the packages were updated to the latest version that supports all the imports used throughout this repo (this is done automatically when you do go mod tidy without explicit version locking). This includes:

terraform: 0.12.9 => 0.12.24
aws-sdk-go: 1.25.4 => 1.29.34
go-getter: 1.3.0 => 1.4.2

Thanks for review!

BTW, please check all our docs for mention of dep and update those docs accordingly!

Ah good catch! Just updated: 949b1b8
Will merge and release!


Data Source/Data Component Generator integration

Adds Data Source/Data Component integration to the generator scripts.
Closes #36

This looks good to me. Sorry it took a while to get to it. Can you address the CHANGELOG.md conflict, then I'll merge it in. Thanks!


Contract Verification is Broken

It appears that a change on Sourcify's side has broken contract verification.
Current requests for https://sourcify.dev/server/files/contracts/40 are being met with a 404 and:
"error": "Cannot fetch more than 200 contracts (1075 full matches, 152 partial matches), please use /contracts/{full|any}/40 with pagination", "message": "Cannot fetch more than 200 contracts (1075 full matches, 152 partial matches), please use /contracts/{full|any}/40 with pagination"
I am going to try modifying the script to use https://sourcify.dev/server/files/contracts/full/40, which appears to work and does not require pagination (yet).

Committed changes to dev branch here: https://github.com/telosnetwork/api-telos-net/blob/develop/src/libs/sourcify.mjs


Could we get a working Vanilla Installation for 1.14.2?

Because of the launcher redesign, the current Vanilla 1.14.2 installation doesn't work anymore. I still play on 1.14.2 because of some bugs with 14.3, and this mod would be super helpful. Could I get a working 1.14.2 installation, or at least instructions on how to fix the issue myself?

Keep an eye out on the curseforge page:- https://www.curseforge.com/minecraft/mc-mods/bounding-box-outline-reloaded/files/all


Chuckles reviewing last payments

    oh probably handling check balance before sending would be nice to preserve close to reality state

Originally posted by @yangwao in https://github.com/kodadot/nft-gallery/issues/4488#issuecomment-1356795611
If someone would help takeover on this till I get to it would be amazing to mitigate this kind of situations tho
Reviewing failed transactions now
Ref

#2116

quick forensic science

https://github.com/kodadot/nft-gallery/pull/4488
https://github.com/kodadot/nft-gallery/pull/4503
https://github.com/kodadot/nft-gallery/pull/4508
https://github.com/kodadot/nft-gallery/pull/4473

oh probably handling check balance before sending would be nice to preserve close to reality state

@yangwao shall i open an issue to check balance before sending?
ill be happy to tackle that


Availability of Tag for v3.15.2

When will the Tag for v3.15.2 be available?

Thank you for your feedback. The missing tag is created.


Gateway API TLS conformance

Problem Statement
Our ControlPlane implemented the TLSRoute but is not conformant with the TLS profile. An effort to be conformant with such a profile has been done unsuccessfully, because of the impossibility to dynamically setup the Kong Gateway Stream Ports from KIC: https://github.com/Kong/kubernetes-ingress-controller/pull/3797.
In KGO we do not have such a limitation, hence we should be able to pass all the TLS conformance tests
Proposed Solution
Enable the TLS feature and successfully run the following conformance tests:
### Core features
- [ ] https://github.com/Kong/gateway-operator/issues/55
- [ ] https://github.com/Kong/gateway-operator/issues/54
- [ ] https://github.com/Kong/gateway-operator/issues/66
- [ ] https://github.com/Kong/gateway-operator/issues/67
- [ ] https://github.com/Kong/gateway-operator/issues/68

Blocked by https://github.com/Kong/gateway-operator/issues/64


Flairs of "MBAM Client not Supported" Should be Added

There is no information about how to start encryption on server SKU, customers trying to install MBAM Client to server OS.
Please add guide to activate bitlocker after enabling features and add "not supported" notice for MBAM/ SCCM client on Server SKU.

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

ID: 71d85a90-230f-84b0-a024-88a8da36825d
Version Independent ID: bf2db39e-16b3-f5a8-af07-91ed301a5ca7
Content: BitLocker How to deploy on Windows Server 2012 and later
Content Source: windows/security/information-protection/bitlocker/bitlocker-how-to-deploy-on-windows-server.md
Product: windows-client
Technology: itpro-security
GitHub Login: @frankroj
Microsoft Alias: frankroj

I am not sure what you are ask is. This is not MBAM or Configuration Manager documentation. This is BitLocker documentation. If you want supportability statements regarding MBAM and Configuration Manager, please consult their respective documentation. In general, we do not document what is not supported - that list is infinite. Instead, we document what is supported. If you do not specifically find a statement that something is supported, then you can assume it is not supported.
With that said, both the MBAM and Configuration Manager BitLocker Management (BLM) do call out that they do not support server OSes, at least not for the management of BitLocker piece. But again, that is in their documentation where it belongs.


Vectra C - communication every 45 seconds?

Szia :-)
I can see we have the same car (I have Vectra C 2.0 2002, motor Y 20 DTH). While I am waiting for the parts to build the device I'd like to ask you if you are by any chance aware of some CAN communication on Vectra C which takes place every 45 seconds?
I am asking because when I start the engine then RPM and speed indicators drop to zero for a moment exactly every 45 seconds and when that happens 4 times then the RPM and speed indicators stay at zero and some warning lights (ABS, alternator, ...) turn on and I can't use signal lights, horn, full beam headlights and the radio turns off.
After turning the engine off and back on everything repeats again and for the next 4 x 45 seconds everything works well. I removed the CIM and cleaned all the connectors but still the same. When the warning lights are on the error reported by OPCOM is "No communication with CIM".
However, after the engine is on for half an hour or maybe an hour (depends on outside temperature and humidity) then everything start working well.
I removed and cleaned UEC, all fuses and some other connectors and added another battery negative to chassis wire but still couldn't find what is causing the problem. The same problem on many Vectra B 1.6i cars could be solved by just removing the cabin light bulb.
Seems as CIM or ECU is trying to communicate with something every 45 seconds and if there is an error then after 4 retries the CIM goes into safe mode. When the CIM is in safe mode hazard lights are still working but there isn't sound of the relay like when everything works well.
In the meantime I tested your GUI program, I still can't test the communication but I can see it's made very well. There is just a small problem, despite the program works well there is a warning:

QWidget::setLayout: Attempting to set QLayout "" on ModernWindow "", which already has a layout

Are you getting the same warning or the library changed? Seems as this could be the solution.
Since you recomended Python 3.8 I am using Python 3.8.10 in virtualenv + Visual Studio Code.

https://google.com

Hello.
Unfortunately, I'm not a car mechanic, so I cannot add anything to the CIM issue.
I have not seeing that issue, but it can be a library version mismatch somewhere. Let me know if you've gottent further with that problem or not.


The Time should be 00:00 why 12:00?

Have set the 24 Hour Mode.
see the official demo
DateTime Picker: 24 Hour Mode
<input class=flatpickr data-enabletime=true data-time_24hr=true data-timeFormat="H:i">

view result
2016-06-05 12:00:00

model result
2016-06-05 00:00:00

and can't set the time to 00:00,it always to be 12:00 PM

@axetroy There is an option available at the bottom for setting appropriate time.

Looks ok here

@chmln thanks for the reply
I mean,why set the default time to 12:00 not 00:00
beginning time of a day should be 00:00, isn't it?

beginning time of a day should be 00:00, isn't it?

Yes you're right, but I chose 12 because its halfway through the day.
That way any hour is at most 12 apart, and its closer to typical business hours

@chmln
in some cases, we need filter time between from date and to date.
I didn't set the time,and pick 2016-06-29 to 2016-06-30.
it should filter 2016-06-29 00:00:00 to 2016-06-30 00:00:00 like expect.
but actually the date pick output 2016-06-29 12:00:00 to 2016-06-30 12:00:00 and out of expect.
it make me set the time to zero every time.
I think this is unfriendly

I didn't set the time,and pick 2016-06-29 to 2016-06-30.

It sounds like time picker isn't really necessary in your case, why not use just date pickers ?
Datetime pickers are for selecting a date and a time.
Most default the time to user's current time. flatpickr defaults to a sensible time - middle of the day.
Going to keep things as they are now, but might consider resetting to 00:00 in the future.
Thanks for contributing 👍

Most default the time to user's current time. flatpickr defaults to a sensible time - middle of the day.

Strongly disagree. Middle of the day is not intuitive. You don't expect the default time to be midday, you expect midnight, just the same as you expect default date to be the first day of the year (1st of January) and not some date in the middle of the year (1st of July ?).

Just in case someone comes here looking for how to do it: check the defaultHour and defaultMinute config options.

flatpickr defaults to a sensible time - middle of the day.

In which galactic? 8-D Sensible is selecting 00:00 for start date and 23:59 for end date.

What about this issue.?

I can't set the time to 00:00,it always to be 12:00 PM


account.address_transactions(address_id) broken

When I call account#address_transactions(address_id) like it says to do in the do the API. I get a
NoMethodError: undefined method `address_transactions' for #<Coinbase::Wallet::Account:0x00000006552188>

either the docs need to be updated to say:
client.address_transactions(client.primary_account['id'], address['id'])
or we need to add support for that method.

Good issue.
There pull-request to add this method: https://github.com/coinbase/coinbase-ruby/pull/99

Implemented in #99.


If a bound network interface is removed/down it renders Music assistant unusable.

What version of Music Assistant has the issue?
2.0.0.b114
What version of the Home Assistant Integration have you got installed?
2024.3.1
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
If the streamserver is bound to a specific network interface in HassOS but then that interface is unavailable it breaks Music assistant, This in itself is not that unexpected, but the problem is you cannot then get to the settings to unbind the interface.

How to reproduce
Bind MusicAssistant to an interface in core > streamserver
Music Providers
N/A
Player Providers
N/A
Full log output
2024-03-18 09:05:38.128 INFO (MainThread) [music_assistant] Starting Music Assistant Server (404ead21ce104b419123afe485ac3ddc) version 2.0.0b114 - uvloop: False 2024-03-18 09:05:38.132 INFO (MainThread) [music_assistant.cache] Initializing cache controller... 2024-03-18 09:05:38.143 INFO (MainThread) [music_assistant.webserver] Starting server on  <IP_ADDRESS>:8095 - base url: http://<IP_ADDRESS>:8095 2024-03-18 09:05:38.170 INFO (MainThread) [music_assistant.music] Using a sync interval of 180 minutes. 2024-03-18 09:05:38.230 INFO (MainThread) [music_assistant.streams] Detected ffmpeg version 5.1.4 with libsoxr support 2024-03-18 09:05:38.230 INFO (MainThread) [music_assistant.streams] Starting server on  <IP_ADDRESS>:8096 - base url: http://<IP_ADDRESS>:8096 2024-03-18 09:05:38.232 ERROR (MainThread) [music_assistant] Error doing task: Task exception was never retrieved Traceback (most recent call last):   File "/usr/local/lib/python3.12/site-packages/aiorun.py", line 219, in new_coro     await coro   File "/usr/local/lib/python3.12/site-packages/music_assistant/__main__.py", line 211, in start_mass     await mass.start()   File "/usr/local/lib/python3.12/site-packages/music_assistant/server/server.py", line 150, in start     await self.streams.setup(await self.config.get_core_config("streams"))   File "/usr/local/lib/python3.12/site-packages/music_assistant/server/controllers/streams.py", line 335, in setup     await self._server.setup(   File "/usr/local/lib/python3.12/site-packages/music_assistant/server/helpers/webserver.py", line 74, in setup     await self._tcp_site.start()   File "/usr/local/lib/python3.12/site-packages/aiohttp/web_runner.py", line 119, in start     self._server = await loop.create_server(                    ^^^^^^^^^^^^^^^^^^^^^^^^^   File "/usr/local/lib/python3.12/asyncio/base_events.py", line 1573, in create_server     raise OSError('could not bind on any address out of %r' OSError: could not bind on any address out of [('<IP_ADDRESS>', 8096)] 2024-03-18 09:05:40.810 WARNING (MainThread) [music_assistant.webserver] Invalid command: players/all 2024-03-18 09:05:40.943 WARNING (MainThread) [music_assistant.webserver] Invalid command: music/recently_played_items 2024-03-18 09:05:40.943 WARNING (MainThread) [music_assistant.webserver] Invalid command: music/browse 2024-03-18 09:05:43.813 WARNING (MainThread) [music_assistant.webserver] Invalid command: config/providers 2024-03-18 09:05:46.623 WARNING (MainThread) [music_assistant.webserver] Invalid command: config/core 2024-03-18 09:07:38.518 WARNING (MainThread) [music_assistant.webserver] Invalid command: players/all 2024-03-18 09:07:38.665 WARNING (MainThread) [music_assistant.webserver] Invalid command: music/recently_played_items 2024-03-18 09:07:38.666 WARNING (MainThread) [music_assistant.webserver] Invalid command: music/browse 2024-03-18 09:07:41.512 WARNING (MainThread) [music_assistant.webserver] Invalid command: config/providers 2024-03-18 09:07:42.903 WARNING (MainThread) [music_assistant.webserver] Invalid command: config/core 2024-03-18 09:10:26.890 WARNING (MainThread) [music_assistant.webserver] Invalid command: config/providers
Additional information
No response
What version of Home Assistant Core are your running
2024.3.1
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Generic x86-64 (e.g. Intel NUC)

Maybe IP binding is something that could be passed in via the Add-ON configuration (Via a docker env variable) then you'd be able to easily unset/reset it if MA cannot start.

I just fixed this one, available from b146


ci: Switch iroha2 default image to debian-based

Solution

Use Debian based image by default for iroha2 image.
Set the default retention policy for executor.wasm artifact as we need it to be able to be downloaded.
Enable checkout branch/commit option for all custom images build & push job.

Close #5100
Checklist

[ ] I've read CONTRIBUTING.md.
[ ] (optional) I've written unit tests for the code changes.
[ ] All review comments have been resolved.
[ ] All CI checks pass.

@nxsaken please update iroha_swarm to use /bin/bash here, here and here.


Allow custom user domains

Allow users to specify a domain for example like GitHub does with a CNAME file.

It seems the vhost module can be used along with connect to achieve all kinds of things along these lines. The express repo provides an example

I was about to request this too, does this mean that this feature will be out with 0.2 of this?

+1 Definitely agree.


Fix list-pipeline if output repo is missing

Fixes panic when output repo is nil.
$ pachctl list-pipeline
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x0 pc=0x15f5d5]

goroutine 1 [running]:
github.com/pachyderm/pachyderm/src/pps/pretty.PrintPipelineInfo(0x2043028, 0xc82010e800, 0xc82017b400)
	/Users/toni/code/gopath/src/github.com/pachyderm/pachyderm/src/pps/pretty/pretty.go:38 +0x435
github.com/pachyderm/pachyderm/src/pps/cmds.Cmds.func6(0x10f5cd0, 0x0, 0x0, 0x0, 0x0)
	/Users/toni/code/gopath/src/github.com/pachyderm/pachyderm/src/pps/cmds/cmds.go:226 +0x3ea
github.com/pachyderm/pachyderm/vendor/go.pedge.io/pkg/cobra.RunBoundedArgs.func1(0xc820249000, 0x10f5cd0, 0x0, 0x0)
	/Users/toni/code/gopath/src/github.com/pachyderm/pachyderm/vendor/go.pedge.io/pkg/cobra/pkgcobra.go:25 +0xb0
github.com/pachyderm/pachyderm/vendor/github.com/spf13/cobra.(*Command).execute(0xc820249000, 0x10f5cd0, 0x0, 0x0, 0x0, 0x0)
	/Users/toni/code/gopath/src/github.com/pachyderm/pachyderm/vendor/github.com/spf13/cobra/command.go:572 +0x869
github.com/pachyderm/pachyderm/vendor/github.com/spf13/cobra.(*Command).ExecuteC(0xc820126400, 0xc820249000, 0x0, 0x0)
	/Users/toni/code/gopath/src/github.com/pachyderm/pachyderm/vendor/github.com/spf13/cobra/command.go:662 +0x54e
github.com/pachyderm/pachyderm/vendor/github.com/spf13/cobra.(*Command).Execute(0xc820126400, 0x0, 0x0)
	/Users/toni/code/gopath/src/github.com/pachyderm/pachyderm/vendor/github.com/spf13/cobra/command.go:618 +0x2d
main.do(0x7aa3a0, 0xc82015c5a0, 0x0, 0x0)
	/Users/toni/code/gopath/src/github.com/pachyderm/pachyderm/src/cmd/pachctl/main.go:118 +0x7a7
github.com/pachyderm/pachyderm/vendor/go.pedge.io/env.Main(0xd42750, 0x7aa3a0, 0xc82015c5a0, 0x0, 0x0, 0x0)
	/Users/toni/code/gopath/src/github.com/pachyderm/pachyderm/vendor/go.pedge.io/env/env.go:46 +0x1ff
main.main()
	/Users/toni/code/gopath/src/github.com/pachyderm/pachyderm/src/cmd/pachctl/main.go:37 +0x74

lgtm


Flutter doctor instructions for SDK update incorrect

While running flutter upgrade and flutter doctor, the doctor tells me to update sdkmanager. When updating sdkmanager, I see an exception:
Exception in thread "main" java.lang.NoClassDefFoundError: javax/xml/bind/annotation/XmlSchema
	at com.android.repository.api.SchemaModule$SchemaModuleVersion.<init>

Running flutter doctor...
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel master, v1.2.2-pre.24, on Mac OS X 10.13.6 17G3025, locale en-US)
[!] Android toolchain - develop for Android devices (Android SDK version 25.0.2)
    ✗ Flutter requires Android SDK 28 and the Android BuildTools 28.0.3
      To update using sdkmanager, run:
        /Users/rischpater/Library/Android/sdk/tools/bin/sdkmanager "platforms;android-28" "build-tools;28.0.3"
      or visit https://flutter.io/setup/#android-setup for detailed instructions.
    ! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses
[!] iOS toolchain - develop for iOS devices (Xcode 10.0)
    ✗ Verify that all connected devices have been paired with this computer in Xcode.
      If all devices have been paired, libimobiledevice and ideviceinstaller may require updating.
      To update with Brew, run:
        brew update
        brew uninstall --ignore-dependencies libimobiledevice
        brew uninstall --ignore-dependencies usbmuxd
        brew install --HEAD usbmuxd
        brew unlink usbmuxd
        brew link usbmuxd
        brew install --HEAD libimobiledevice
        brew install ideviceinstaller
[✓] Android Studio (version 3.1)
[✓] Connected device (1 available)

! Doctor found issues in 2 categories.

rischpater-macbookair:~/Downloads/bookmarker
$ ls
README.md	bookmarker.iml	lib		pubspec.yaml
android		ios		pubspec.lock
rischpater-macbookair:~/Downloads/bookmarker
$ /Users/rischpater/Library/Android/sdk/tools/bin/sdkmanager "platforms;android-28" "build-tools;28.0.3"
Exception in thread "main" java.lang.NoClassDefFoundError: javax/xml/bind/annotation/XmlSchema
	at com.android.repository.api.SchemaModule$SchemaModuleVersion.<init>(SchemaModule.java:156)
	at com.android.repository.api.SchemaModule.<init>(SchemaModule.java:75)
	at com.android.sdklib.repository.AndroidSdkHandler.<clinit>(AndroidSdkHandler.java:81)
	at com.android.sdklib.tool.sdkmanager.SdkManagerCli.main(SdkManagerCli.java:73)
	at com.android.sdklib.tool.sdkmanager.SdkManagerCli.main(SdkManagerCli.java:48)
Caused by: java.lang.ClassNotFoundException: javax.xml.bind.annotation.XmlSchema
	at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:582)
	at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:190)
	at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:499)
	... 5 more
rischpater-macbookair:~/Downloads/bookmarker

Might be rather a Java setup issue than a Flutter issue

https://stackoverflow.com/questions/46402772/failed-to-install-android-sdk-java-lang-noclassdeffounderror-javax-xml-bind-a
https://github.com/flutter/flutter/issues/16025#issuecomment-377545368
https://github.com/flutter/flutter/issues/16452#issuecomment-380580196
https://github.com/NativeScript/nativescript-cli/issues/3139#issuecomment-358832687

Same issue:
Windows 10 : flutter getting started installation / demo
Intelij idea 2019  - Java 12 - Android sdk 28

@EArminjon Your java version betrays you.
This might be of help:
https://stackoverflow.com/questions/46402772/failed-to-install-android-sdk-java-lang-noclassdeffounderror-javax-xml-bind-a

Yes i fortgot to going back here, i took java 8

Android Studio -> Preferences ->System Setting ; Add the requires Android SDK 28 and the Android BuildTools 28.0.3

1- Android SDK version 28
Android Studio -> SDK Manager -> SDK Platforms tab -> Select Android 9.0 Pie and press ok to download it
2- Android BuildTools version 28.0.3
Android Studio -> SDK Manager -> SDK Tools tab -> Select Show Package Details -> Under Android SDK Build-Tools section select 28.0.3 and press ok to download it
then restart cmd then test fluter doctor again

1- Android SDK version 28
Android Studio -> SDK Manager -> SDK Platforms tab -> Select Android 9.0 Pie and press ok to download it
2- Android BuildTools version 28.0.3
Android Studio -> SDK Manager -> SDK Tools tab -> Select Show Package Details -> Under Android SDK Build-Tools section select 28.0.3 and press ok to download it
then restart cmd then test fluter doctor again

i'm failed ,why ?

I had a similar problem this morning (trying to build for Android using Unity3D). I ended up uninstalling JDK9 and installing Java SE Development Kit 8u144. Hope this helps.
Find more at Here https://icetutor.com/question/failed-to-install-android-sdk-java-lang-noclassdeffounderror-javax-xml-bind-annotation-xmlschema

The problem is unrelated to flutter even though I encountered it while trying to get flutter to work.
The problem is that the package corresponding with javax/xml/bind/annotation/XmlSchema was removed from the latest versions of Java run times.  The fix is to run SDK Manager in Java 8 or 1.8(1.8 and 8 are meaningfully the same as Java versions).
In other words, be sure that running "java -version" outputs a number that is less than "1.9".  If it outputs 9, ...13, install JRE or JDK 8 and point your PATH environment variable at it.  Remember also to reopen your terminal windows so the environment variable changes take effect.

Closing this, since the workaround is included below and the solution is outside of Flutter.

1- Android SDK version 28
Android Studio -> SDK Manager -> SDK Platforms tab -> Select Android 9.0 Pie and press ok to download it
2- Android BuildTools version 28.0.3
Android Studio -> SDK Manager -> SDK Tools tab -> Select Show Package Details -> Under Android SDK Build-Tools section select 28.0.3 and press ok to download it
then restart cmd then test fluter doctor again

I have tried above too but still
flutter doctor --android-licenses
Exception in thread "main" java.lang.NoClassDefFoundError: javax/xml/bind/annotation/XmlSchema
at com.android.repository.api.SchemaModule$SchemaModuleVersion.(SchemaModule.java:156)
at com.android.repository.api.SchemaModule.(SchemaModule.java:75)
at com.android.sdklib.repository.AndroidSdkHandler.(AndroidSdkHandler.java:81)
at com.android.sdklib.tool.sdkmanager.SdkManagerCli.main(SdkManagerCli.java:73)
at com.android.sdklib.tool.sdkmanager.SdkManagerCli.main(SdkManagerCli.java:48)
Caused by: java.lang.ClassNotFoundException: javax.xml.bind.annotation.XmlSchema
at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:583)
at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:178)
at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:521)
... 5 more

Installing Android SDK Command-line Tools solved this problem for some people. https://github.com/flutter/flutter/issues/57017#issuecomment-696559427

La instalación Android SDK Command-line Toolsresolvió este problema para algunas personas (incluido yo). # 57017 (comentario)

Muchas gracias, me ayudaste demasiado!.
Problema solucionado.

https://www.edoardovignati.it/solved-flutter-android-licenses-exception/

https://www.edoardovignati.it/solved-flutter-android-licenses-exception/

@sajjadG Thank you sir

Running
$ flutter doctor --android-licenses

Worked for me

Just install java1.8 SDK It'll work

had this error, and this fix worked for me:
(mac)
https://stackoverflow.com/a/64389804/8007293

had this error, and this fix worked for me:
(mac)
https://stackoverflow.com/a/64389804/8007293

this work for me

All Possible Working Solution Added Here [Solved] Exception in thread “main” java.lang.NoClassDefFoundError in flutter

Installing Android SDK Command-line Tools solved this problem for some people (me included). #57017 (comment)

This works. Thank you very much.


Add dapr control plane status for Kubernetes

Closes #312
This PR allows a user to see the Dapr control plane status when running on Kubernetes.
Command: dapr status -k or dapr status --kubernetes.
Output example: (2 services running, 2 failing):
  NAME                   NAMESPACE    HEALTHY  STATUS                      VERSION  AGE  CREATED
  dapr-operator          dapr-system  False    Waiting (CrashLoopBackOff)  0.6.0    2d   2020-04-04 11:19.39
  dapr-placement         dapr-system  True     Running                     0.6.0    2d   2020-04-04 11:19.39
  dapr-sentry            dapr-system  False    Terminated                  0.6.0    2d   2020-04-04 11:19.39
  dapr-sidecar-injector  dapr-system  True     Running                     0.6.0    2d   2020-04-04 11:19.39

/cc @msfussell

Updated text to system services (control plane). Coordinated and signed off by @msfussell


New version: Amazon.WorkspacesClient version 5.2.1

- [X] Have you signed the Contributor License Agreement?

[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.2 schema?

Resolves #70727

Microsoft Reviewers: Open in CodeFlow

@wingetbot run

New version should be: <IP_ADDRESS>23

The marketed version is 5.2.1, the DisplayVersion is <IP_ADDRESS>23, which is shown in the AppsAndFeaturesEntries. If you view the release notes on their website, they always refer to only the first three digits. Therefore, 5.2.1 is correct in this specific case

New version should be: <IP_ADDRESS>23

The marketed version is 5.2.1, the DisplayVersion is <IP_ADDRESS>23, which is shown in the AppsAndFeaturesEntries. If you view the release notes on their website, they always refer to only the first three digits. Therefore, 5.2.1 is correct in this specific case

New version should be: <IP_ADDRESS>23

The marketed version is 5.2.1, the DisplayVersion is <IP_ADDRESS>23, which is shown in the AppsAndFeaturesEntries. If you view the release notes on their website, they always refer to only the first three digits. Therefore, 5.2.1 is correct in this specific case

I disagree as all previous versions have been a full version, as in the picture above. This could potentially lead to issues if a new build is released.

I disagree as all previous versions have been a full version, as in the picture above. This could potentially lead to issues if a new build is released.

When they release a new build, they remove the old build, which is why there is only one version in the previous version list. The reason the full version was always used before was because there was not the ability for the client to use the AppsAndFeaturesEntries to determine the version. However, the 1.3 client does have that ability.
The version you showed in your screenshot is actually referred to by AWS as 5.2.0 in their documentation - https://docs.aws.amazon.com/workspaces/latest/userguide/amazon-workspaces-windows-client.html#windows-release-notes
@officialesco @jedieaston @ItzLevvie - Any thoughts or feedback here?

When they release a new build, they remove the old build, which is why there is only one version in the previous version list. The reason the full version was always used before was because there was not the ability for the client to use the AppsAndFeaturesEntries to determine the version. However, the 1.3 client does have that ability.
The version you showed in your screenshot is actually referred to by AWS as 5.2.0 in their documentation - https://docs.aws.amazon.com/workspaces/latest/userguide/amazon-workspaces-windows-client.html#windows-release-notes
@OfficialEsco @jedieaston @ItzLevvie - Any thoughts or feedback here?

Yes, I understand what you are saying, but I believe using a full version of the files & app would be the correct course of action and appears to be standard across all of the MS apps, and all Amazon CLI and previous versions on this client

I agree with @Trenly, when you ask Amazon for support they’re going to ask for the version number in the release notes, not the ARP version. Previous versions should be able to upgrade fine since winget will rematch them.


Toolbar navigation, page:beforein and page:beforeout fire instantly for the destination page

Well, this isn't common in the examples, but I wanted to have both - side panel navigation and a toolbar at the bottom, to navigate between pages.
My home page listens to both page:beforein and page:beforeout to manipulate DOM and change some styling (different statusbar color etc.)
This worked OK with the side panel which resides outside f7-view, but started to fail when using the Toolbar.
Below my toolbar definition, it is present on every page:
<f7-toolbar labels>
        <f7-link href="/home">

            Home
        </f7-link>
        <f7-link href="/documents">

            Other page
        </f7-link>
        //...
</f7-toolbar>

When I go from Home to the Other Page and from that I again click on "Home", then I get page:beforein and page:beforeout for the Home page fired right after each other which ruins my styling :(.
Any ideas why it behaves that way, or suggestions how to get the result I'm after? This is a really handy navigation style I would like to keep.

There is something wrong. Would be good to see JSFiddle or live example with the issue


Binary search in a 2-d sorted array

I would like to add C++ code for searching an element in 2-d sorted array using binary search.

/assign

/assign


Sinkhorn Knopp NaN

I was running a debug job on 2 processes. I noticed that Sinkhorn Knopp produces NaN valeus for teacher randomly(the previous loss & data in current batch all look normal). Upon further inspection, I noticed that the all_reduce function in sinkhorn_knopp doesn't wait for all other processes, though I'm not quite sure if that's the reason.
Here's the code:
def sinkhorn_knopp_teacher(self, teacher_output, teacher_temp, n_iterations=3):
import os
print("teacher sinkhorn on RANK", os.environ["RANK"])
if os.environ["RANK"] == 0:
teacher_output = torch.load("tmp0.pth")
else:
teacher_output = torch.load("tmp.pth")
teacher_temp = torch.load("temperature.pth")
teacher_output = teacher_output.float()
print("starting sinkhorn iteration on RANK", os.environ["RANK"])
world_size = dist.get_world_size() if dist.is_initialized() else 1
print("world size on RANK", os.environ["RANK"], world_size)
Q = torch.exp(teacher_output / teacher_temp).t()  # Q is K-by-B for consistency with notations from our paper
B = Q.shape[1] * world_size  # number of samples to assign
K = Q.shape[0]  # how many prototypes
    # make the matrix sums to 1
    sum_Q = torch.sum(Q)
    if dist.is_initialized():
        print("reducing sum_Q on RANK", os.environ["RANK"])
        dist.all_reduce(sum_Q)
    Q /= sum_Q

    for it in range(n_iterations):
        # normalize each row: total weight per prototype must be 1/K
        sum_of_rows = torch.sum(Q, dim=1, keepdim=True)
        if dist.is_initialized():
            dist.all_reduce(sum_of_rows)
        Q /= sum_of_rows
        Q /= K

        # normalize each column: total weight per sample must be 1/B
        Q /= torch.sum(Q, dim=0, keepdim=True)
        Q /= B

    Q *= B  # the columns must sum to 1 so that Q is an assignment
    print("finished sinkhorn iteration on RANK", os.environ["RANK"])
    return Q.t()

where the loaded .pth files are tensors saved from previous tries that produced NaN.
(I tried simulating all_reduce by loading both tensors and doing sinkhorn_knopp within a single process and manually adding the sum_Q and sum_of_rows variable before normalizing, no NaN values are produced.)
Here's the printed output:
forward on RANK 1
forward on RANK 0
teacher sinkhorn on RANK 1
teacher sinkhorn on RANK 0
starting sinkhorn iteration on RANK 1
world size on RANK 1 2
reducing sum_Q on RANK 1
finished sinkhorn iteration on RANK 1
starting sinkhorn iteration on RANK 0
world size on RANK 0 2
reducing sum_Q on RANK 0
I'm running this on a single node with 2 L4 GPUs(same issue arises when I run on 4 nodes each with 8 L4 GPUs)
Is there a missing call to a barrier?

On another note, shouldn't Koleo loss use all_gather instead of all_reduce?


fix(net): reset backoff counter on graceful disconnect

if we were able to successfully establish an active session and disconnected it gracefully, we should reset the backoff counter

Would like us to add a test for this edge case, no rush


Problem with vanilla HTTP server

Despite having a very simple server set up, I receive a script error when the dispatcher is called.
/path/node_modules/dispatch/lib/dispatch.js:77
            next();
            ^
TypeError: undefined is not a function
    at Server.<anonymous> (/path/node_modules/dispatch/lib/dispatch.js:77:13)
    at Server.emit (events.js:98:17)
    at HTTPParser.parser.onIncoming (http.js:2108:12)
    at HTTPParser.parserOnHeadersComplete [as onHeadersComplete] (http.js:121:23)
    at Socket.socket.ondata (http.js:1966:22)
    at TCP.onread (net.js:527:27)

Here is the script I'm using:
var os = require('os'),
    http = require('http'),
    dispatch = require('dispatch'),
    server,
    port;

server = http.createServer(
    dispatch({
        '/': function (request, response) {
            response.end();
        }
    })
);

port = process.argv[2];

console.log('Listening for requests at http://' + os.hostname() + ':' + port);
server.listen(port);

I've triggered the error several times simply by visiting the web address of the server. Do you know what's wrong?

I've fixed the error with #7. I hope to get merged soon.

I'm seeing this too. It seems like it was never tested with node's http server since the next argument is specific to the connect module. To fix it it's enough to just check that next is non-null before calling it.

@emanuelpalm, it's been two months now... are you able to merge it?

@Asuza Unfortunately, I don't have any particular repo privileges.

@emanuelpalm, I apologize. I mistakenly thought you were the repo owner.

I believe this issue is fixed thanks to #7.


[#4508]feat(iceberg) support load custom catalog as backend

What changes were proposed in this pull request?
support load custom catalog as backend
Why are the changes needed?
improvement: #4508
Does this PR introduce any user-facing change?

add a property key

How was this patch tested?

add UT
manual test

@jerryshao @FANNG1 please help to review, Thanks.

Do you think is it necessary to support a custom catalog for the Gravitino Iceberg catalog not only for the Iceberg REST service?

Do you think is it necessary to support a custom catalog for the Gravitino Iceberg catalog not only for the Iceberg REST service?

I want to keep a PR as simple as possible. Maybe that part putting in another PR is better？
of course, if it's confusing, I will put them together.

Do you think is it necessary to support a custom catalog for the Gravitino Iceberg catalog not only for the Iceberg REST service?

I planned to do it in a separate PR to keep this one simple, but to avoid confusion, I'll combine them.

Do you think is it necessary to support a custom catalog for the Gravitino Iceberg catalog not only for the Iceberg REST service?

I planned to do it in a separate PR to keep this one simple, but to avoid confusion, I'll combine them.

I prefer to split the PR to make it more clear, could you create corresponding issues to add custom catalog backend for Graviitno server?

LGTM, except one comment

Do you think is it necessary to support a custom catalog for the Gravitino Iceberg catalog not only for the Iceberg REST service?

I planned to do it in a separate PR to keep this one simple, but to avoid confusion, I'll combine them.

I prefer to split the PR to make it more clear, could you create corresponding issues to add custom catalog backend for Graviitno server?

👌，the corresponding issue is here: #4584

@theoryxu , merged to main, thanks for your contribution


Update all platform stuff, make it work on current flutter/dart2 (android only)

iOS not checked

Closing, handled by another PR.


PL-releasenotes-782xx

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Preview environment: https://ci.pr.harness.io/pr-367

Preview environment: https://ci.pr.harness.io/pr-367


Is OnBackgroundHandler supported?

The library does not display any notification by itself. You can do whatever you want with the message received, and handled by onMessage

I'm closing this issue since the library already allow to achieve what you want to do. This can be reopened if needed


Add /ru translation

Will add Russian translation. Please help with adding a flag. How am I supposed to do it? thanks!

Hey @kalpovskii, thanks for opening this PR. For translations we use a separate platform called Localazy, which is easier to use than just updating the JSON files. I recently tried to ask someone else to help with Russian translations actually. Let me see how that is going first. If they are doing it, then it would be easiest to let them finish it. But if they aren't finishing it, then it would be amazing if you could help out. Let me get back to you later this week!

Hey @kalpovskii, could you send me a message on Telegram or Twitter (RoscoKalis) so I can invite you to the Localazy project where you can help with the translations?

Thank you again @kalpovskii. The Japanese translations are also ready now, so I'll release both of them at the same time.


line numbers do not appear in all files in vs2019

we are using microfocus visual cobol and editing dbl files.  All other file types have line numbers except for the cbl

Do you know if the CBL editor uses the normal line number column?
I assume line numbers are turned on?
Were the line numbers removed due to installing this extension?

Thank you!, they had them off by default in their own configuration settings. :)


AuthorityValueService getAuthorityValueType() should match on field name, not type name

Describe the bug
AuthorityValueService.getAuthorityValueType(metadataString) should - I think! - match the supplied string against the field of each configured value (eg "dc_contributor_author"), not the type name (eg. "Orcidv2AuthorityValue") before returning.
Expected behaviour
The method should treat the supplied metadata string as a field name, not a type name
To reproduce
Inspect the output of getAuthorityValueType(...) when passed a field name that is definitely configured for authority control in spring XML.
Notes
Method is here:
https://github.com/DSpace/DSpace/blob/737d3e677ef0c5762a6d70da7298d88fcc3cad80/dspace-api/src/main/java/org/dspace/authority/AuthorityValueServiceImpl.java#L197
This isn't actually used anywhere to my knowledge so is pretty low priority, but I noticed it when doing some authority customisation work that required me to find the class configured for a field in advance.
It is possible that I'm misunderstanding the original intent of this method - it is not commented.

@kshepherd : I'm not sure I fully understand...it looks, to me, like that getAuthorityValueType() method is looking for a match against the output of AuthorityValue.getAuthorityType() here: https://github.com/DSpace/DSpace/blob/main/dspace-api/src/main/java/org/dspace/authority/AuthorityValueServiceImpl.java#L200
That getAuthorityType() method returns the "type" of AuthorityValue we are looking at...see this README: https://github.com/DSpace/DSpace/blob/main/dspace-api/src/main/java/org/dspace/authority/README.md#extend-orgdspaceauthorityauthorityvalue-add-the-fields-and-implement-the-methods  So, for example, PersonAuthorityValue.getAuthorityType() returns "person"
So, I think this code is correct.  It looks like it is used by the CSV import process in a few areas...but I agree it's not well documented, so I'm not entirely sure if I understand it's behavior either.
For now, I'm going to put this in the "backlog" until determined it's a bug.  I'm not yet confident it is, but I could be wrong here.

@tdonohue I see what you're saying, perhaps I just misunderstood the usage. So the idea is that I can call authorityValueService.getAuthorityValueType("person") and it will return a PersonAuthorityValue?
For some reason I thought a field name would be passed to the method, not the type name. Perhaps it's just the 'metadataString' parameter name that confused me.

@kshepherd : Yes, based on my reading of the code, it really looks like that String metadataString param should be renamed to be String type.  In the Impl code, it's very obviously not looking up anything by a metadata string...instead, it is expecting you to call authorityValueService.getAuthorityValueType("person") to retrieve PersonAuthorityValue.
As previously mentioned though, the AuthorityValueService classes are all undocumented and uncommented.  So, I'm only answering based on what the code seems to currently do.  However, I honestly will admit, I don't know whether it's current behavior is correct. It's very possible you'll pointing out that it should be doing one thing, while it's obviously doing something different.  I'm not an expert in this area of the code...and I honestly haven't had a chance to dig deeply into how/where this getAuthorityValueType() method is being used.
At the very least, it seems like this area of the code could use much better code comments / JavaDocs.  So, if you have the time or interest to figure this stuff out, please do submit a PR to improve the JavaDocs, as I think it'll help us avoid this same sort of confusion in the long run.


Description doesn't get saved during registration process

When registering and a description is set, when viewing profile immediately after log in, it shows empty.

It is similar to #58  Please check other issues before opening one.


Update dcount-aggfunction.md

A function trading performance for accuracy indicates that you are giving up performance in exchange for accuracy.  This is not the case here, as the function is giving up accuracy for better performance

Docs Build status updates of commit 63e4c31:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

data-explorer/kusto/query/dcount-aggfunction.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@shsagir
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Hi @briandelmsft thanks for noticing that and contributing to our docs.
Approved by ADX content team
#sign-off


fix: responses occasionally corrupted

Fixes issue where responses can have their bodies corrupted. This is caused by us giving JS the body's bytes as a reference to a byte slice, which is then dropped causing a use after free. Most of the time this is fine, but occasionally the allocator will allocate something that overlaps with the response body.
To fix this, we instead give the Response constructor an owned Uint8Array that doesn't have any references into the wasm memory.
Probably fixes #64 or at least parts of it.

Hi!
Thanks for the fix.
It works perfectly fine for my case.


Use pydot again instead of pydotplus

According to the HISTORY the switch from pydot to pydotplus was done approx. 2 year ago. I assume this was done for a lack of activity and missing Python 3 support. Some time back the development of pydot was stale and it was not clear which of the many forks to use.
Now the situation with pydot seems to have recovered and developments seems to take place and converging at https://github.com/erocarrera/pydot. pydot now claims to support python 3. See also https://pypi.python.org/pypi/pydot/1.2.3
Wouldn't it make sense to switch back to pydot again?

Many thanks @petrrr for checking this. I looked at pydot build and see that it is now tested against Python 3 (https://travis-ci.org/erocarrera/pydot).
You're right that we switched to pydotplus because it supports Python 3. Having said that, I'm not sure whether both libraries are drop-in equivalent or not.
Is there any compelling reason to go for pydot?

@trungdong: I intend to packages prov in Macports, a package manager for the OSX.
The point here is that dealing with multiple fork of the same/equivalent packages gets quite tricky/cumbersome in such environments, because they are conflicting. If it is possible to avoid this upstream that would help a lot.
To my understanding pydotplus was an intermediate fork which has not seen activity since 2014. The principal contributer to pydotplus (@carlos-jenkins) now contributes to pydot.

@petrrr,
Thank you for the. I did a quick test with the latest pydot version and it seems to work fine.
Will commit the changes soon.

+1 @petrrr, same rationales regarding the Debian packaging. A new release with the latest changes would be nice.


[SCB-942]Choose the highest priority from all implementation classes

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Format the pull request title like [SCB-XXX] Fixes bug in ApproximateQuantiles, where you replace SCB-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run mvn clean install to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage decreased (-0.004%) to 86.199% when pulling a69294e1ba7cf0e54132111f8076cc1f2f36736c on laijianbin:bbbb into 6aadae0de82b3b3940f0c163b815d6675aedc2bc on apache:master.

closed


Cleanup namespace validation

Currently, namespace name is only validated in certain circumstances (eg: if we are going to access a container statedir). This may lead to inconsistencies, where we would allow forbidden namespaces in certain scenarios but properly restrict them in other scenarios.
Also, the current set of rules for validation are inadequate, and fail to cover a range of conditions that will fail on the filesystem.
So, suggesting:

we always validate namespace, once, during argument parsing
we use the known good validator we have in place already in the store (ValidatePathComponent) instead of the current validator

This is introducing changes. Specifically, it will restrict previously allowed namespaces:

namespaces that are longer than 255 characters
namespaces containing protected OS/filesystem patterns (specifically: on windows, con[0-9], etc)

And on the other hand, relax restrictions on certain patterns that were previously forbidden for no apparent reason (tilde, etc).
Most importantly though, note that containerd will anyhow identifier-validate the namespace name whenever it touches it... with the much more restrictive^[A-Za-z0-9]+(?:[._-](?:[A-Za-z0-9]+))*$ - so, this is merely about cleaning things up on our side.
Let me know what you think.

Thanks @djdongjin
Latest push addresses your comment, and has been rebased to account for newly added unit tests.

TestComposeUp failure seems unrelated.


Ananya's Views

Name of the PR
:recycle: Current situation & Problem
Ananya Vasireddy is not part of the class app.
:bulb: Proposed solution
Add Ananya's subview.
:gear: Release Notes
Ananya's views.
:heavy_plus_sign: Additional Information
N/A
Related PRs
N/A
Testing
Tested different buttons/links.
Reviewer Nudging
@vishnuravi @PSchmiedmayer I'd love feedback on my UI tests!
Code of Conduct & Contributing Guidelines
By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

[x] I agree to follow the Code of Conduct and Contributing Guidelines.

@ananya-vasireddy A short reminder to improve your Assignment 1 submission. We would like to merge it in the next few days so you can concentrate on your project assignment.
Feel free to tag me if you have any questions.
Please also tag me if you want me to take an other look at the PR.
I am also happy to help you with any merge conflicts that you might have to speed up the merge so that you can focus on the content of the exercise!
Feel free also to approach me after class today so we can take a look at the PR together and merge it directly if you have incorporated my feedback by then 🚀


Android -CosmosDB : AzureData.getDocument() API call fails to return document

I've successfully configured (AzureData.Configure()) and also created a database.
However, I am unable to access the document from the existing collection using the API AzureData.getDocument().
This API is returning the error " {"code": "NotFound", "message": "Entity with the specified id does not exist in the system}" Please help me with suggestions in this regard

That result is standard from the REST api if the document with that Id can't be found in the collection/db specified.
Can you share the pertinent bits of your code (without private data like keys, etc., of course)?
Can you also verify:

You're using the latest version of the library (0.3.0)
Are you using a master key to connect to Cosmos DB, or a permission provider?

If using a key, you may want to generate a new key to rule out anything to do with that

The Db and collection exist in your cosmos account, and you can find the document in the Azure portal via the data explorer (a screen cap of this would help)

Can you get the Db and collection using this library? If not, this points to a setup issue or something amiss with the connection to your Cosmos DB environment
Does the collection have a partition key? (If you created it via the Azure portal recently, it should). This library is designed to work with partitioned collections, though there are some deprecated API members that don't include it. If the collection has a partition key, then you must also use the partition key when retrieving the document. (See this test to see the call with/without partition key)

It may help to review the example apps to make sure there's nothing missing or out of place within your code, setup-wise. (Though, note that they've not been updated to account for the partition key requirement noted above).

Hi Nate,
Not really sure what I was doing wrong before, now I've fetched the documents and the API call AzureData.getdocuments() returned all the doc ids.
I am using the latest 0.3.0 library and the cosmosDB read-only key in Azure.Configure() method.
CosmosDB collection has a partitionkey and called the AzureData.getdocument() API version that takes the partition key variable.
This issue cannot be reproduced.
Please close the issue and Thank you for your support.

Great to hear it's working :)


[Bug] Authorization header not included when calling specific endpoints

When OIDC is enabled on the OHIF viewer and I try calling endpoints such as searchImageSets or getImageSetMetadata Authorization header is not included. However, when I try to open an image and getImageFrame is called then the Authorization header is included.
I would like to always have Authorization header present so I could enable authorization flow on my proxy.
I suspect the behaviour between endpoints is different as getImageFrame is using xhr while other two endpoints are using fetch.
Could you please confirm if this is actually the case?

Hey @hgrgic thaks for your report, it may be a case that we have not faced yet ... in our case, we don't have yet any using OIDC with health imaging.
I double-checked the code in the file src/imageLoader/loadImageSets.ts:49 we are not checking for the OIDC  (AuthService headers), which would be an interesting improvement for sure.
I will keep this issue open to implement that in the next releases. The implementation would be around the loadImageSets and check the auth service header before adding the headers from the aws signature.
PRs are welcome too


adding a new rebus

Associated Issue: #121
Summary of Changes

Added a new rebus, rebus 179

Pull Request Test Coverage Report for Build 577

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 87.143%

Totals

Change from base Build 514:
0.0%

Covered Lines:
158

Relevant Lines:
180

💛  - Coveralls


Fixed: IntelliJ DeltaSqlBaseBaseVisitor type not found

Problem
A common error I have seen when developing Delta Lake using IntelliJ is the following (below). These errors prevent IntelliJ from running unit tests (though, running unit tests via build/sbt core/test still works fine).
Error:(46, 28) object DeltaSqlBaseParser is not a member of package io.delta.sql.parser
import io.delta.sql.parser.DeltaSqlBaseParser._
Error:(143, 34) not found: type DeltaSqlBaseBaseVisitor
class DeltaSqlAstBuilder extends DeltaSqlBaseBaseVisitor[AnyRef] {
Error:(84, 53) not found: type DeltaSqlBaseParser
  protected def parse[T](command: String)(toResult: DeltaSqlBaseParser => T): T = {
Error:(71, 13) value visit is not a member of io.delta.sql.parser.DeltaSqlAstBuilder
    builder.visit(parser.singleStatement()) match {
Error:(85, 21) not found: type DeltaSqlBaseLexer
    val lexer = new DeltaSqlBaseLexer(
Error:(91, 22) not found: type DeltaSqlBaseParser
    val parser = new DeltaSqlBaseParser(tokenStream)

Solution
Project Settings > modules > delta-core > source folders > remove any of the target folders. should only be src/main/scala
Credits to Ah Lag for bringing this to my attention.

I think we should add a How to set up IDE somewhere in our doc.

+1 This should be in our docs.

@scottsand-db
Just wondering if the documentation is finished. If not, I can help out if you are okay with it.

@ahlag this will be going into our delta-io web

@scottsand-db I think we can add a section Setup IDE after https://github.com/delta-io/delta#building


Clicking a SpaceView bar immediately triggers drag-n-drop

When you click on a spaceview bar, it triggers a drap-n-drop motion to a random place -- it's particularly annoying when you misclick the bar while you meant to click the name to access the properties.
https://github.com/rerun-io/rerun/assets/2910679/e849a584-4afd-4484-9ae4-42896c6bf050

We should fix this in egui_tiles. A drag shouldn't start until the mouse moves a bit

egui_tile issue: https://github.com/rerun-io/egui_tiles/issues/79