Request for Quotation or Draft Purchase Order

What I'm trying to achieve
I'm trying that the client could create a quotation instead of a purchase order.
The quotation should be similar to a purchase order, but the difference is that must not be payed.
Once the quotation was created, the store should create the purchase order with final amount and send a link to the customer to perform the payment.
This is useful in cases where:

there is no way to assure the published unit price for requested products.
and when there is no guarantee of stock for the requested products.

Describe a proposed solution
I do not have a proposed solution.
Other solutions I've tried and won't work
I did not found other solutions.
Screenshots or mockups
NA

This is actually a very important step in a e-commerce business and should be given way more importance and attention.

It seems that it became an important part of the saleor enterprise, not available for the common folk tho.

We are going to add this feature soon, you can track it here https://github.com/saleor/saleor/issues/5447


package_info is retrieving different  build number in apk from debugging Application ? any idea for this issue?

in pubspac.yml ---->version: 1.0.19+81
but when I print the build number in this mode debug apk or release like this
debug--build number -->81
apk--build number ->1081
release--build number ->81
why apk print show the 1081 build number ??

Hi @hemalivekariya Thanks for filing the issue. Please share the output of flutter doctor -v.

apk--build number ->1081

Could you please explain how you got apk-build number 1081 ?
I verified on the latest stable 2.5.1 using the package_info: ^2.0.2 on Android and I could see the correct build number, verified in debug, profile, and release mode.
Also please share the plugin version you are having issues with

code sample
import 'package:flutter/material.dart';
import 'package:package_info/package_info.dart';

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

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({Key? key, required this.title}) : super(key: key);

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _counter = 0;

  Future<void> _incrementCounter() async {
    PackageInfo packageInfo = await PackageInfo.fromPlatform();

    String appName = packageInfo.appName;
    String packageName = packageInfo.packageName;
    String version = packageInfo.version;
    String buildNumber = packageInfo.buildNumber;
    print('''
      appName: $appName,
      packageName: $packageName,
      version: $version,
      buildNumber: $buildNumber
    ''');
    setState(() {
      _counter++;
    });
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            const Text(
              'You have pushed the button this many times:',
            ),
            Text(
              '$_counter',
              style: Theme.of(context).textTheme.headline4,
            ),
          ],
        ),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: _incrementCounter,
        tooltip: 'Increment',
        child: const Icon(Icons.add),
      ),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel stable, 2.5.1, on macOS 11.6 20G165 darwin-arm, locale en-GB)
    • Flutter version 2.5.1 at /Users/mahesh/Documents/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ffb2ecea52 (5 days ago), 2021-09-17 15:26:33 -0400
    • Engine revision b3af521a05
    • Dart version 2.14.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.5.1, Build version 12E507
    • CocoaPods version 1.10.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 60.1.4
    • Dart plugin version 212.5080.8

[✓] VS Code (version 1.59.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.26.0

[✓] Connected device (3 available)
    • Redmi K20 Pro (mobile) • d9dac45d • android-arm64  • Android 11 (API 30)
    • macOS (desktop)        • macos    • darwin-arm64   • macOS 11.6 20G165 darwin-arm
    • Chrome (web)           • chrome   • web-javascript • Google Chrome 94.0.4606.54

• No issues found!

logs
mahesh@Maheshs-MacBook-Air-M1 pigeon_example % flutter run --release
Launching lib/main.dart on Redmi K20 Pro in release mode...
Warning: Mapping new ns http://schemas.android.com/repository/android/common/02 to old ns http://schemas.android.com/repository/android/common/01
Warning: Mapping new ns http://schemas.android.com/repository/android/generic/02 to old ns http://schemas.android.com/repository/android/generic/01
Warning: Mapping new ns http://schemas.android.com/sdk/android/repo/addon2/02 to old ns http://schemas.android.com/sdk/android/repo/addon2/01
Warning: Mapping new ns http://schemas.android.com/sdk/android/repo/repository2/02 to old ns http://schemas.android.com/sdk/android/repo/repository2/01
Warning: Mapping new ns http://schemas.android.com/sdk/android/repo/sys-img2/02 to old ns http://schemas.android.com/sdk/android/repo/sys-img2/01
Running Gradle task 'assembleRelease'...                           53.2s
✓  Built build/app/outputs/flutter-apk/app-release.apk (6.0MB).
Installing build/app/outputs/flutter-apk/app.apk...              1,021ms

Flutter run key commands.
h List all available interactive commands.
c Clear the screen
q Quit (terminate the application on the device).
W/FlutterActivityAndFragmentDelegate(32575): A splash screen was provided to Flutter, but this is deprecated. See flutter.dev/go/android-splash-migration for migration steps.
I/flutter (32575):       appName: pigeon_example,
I/flutter (32575):       packageName: com.example.pigeon_example,
I/flutter (32575):       version: 1.0.19,
I/flutter (32575):       buildNumber: 81
I/flutter (32575):

Thanks.

Thanks for ans..i am using the package "package_info: ^2.0.0"
and
Code:--
getPackageInfo(){
PackageInfo packageInfo = await PackageInfo.fromPlatform();
int _appVersion = int.parse(packageInfo.buildNumber);
log("appVersion---> ${_appVersion}
}
but when i am make apk from this command :flutter build apk --release --split-per-abi --no-shrink
after that i m install that apk in my mobile and check the Logcat which i have print
and i get the buildNumber in logcat
Locat log:
apkMode--build number get like --->1081

@hemalivekariya Thanks for the info, This is an intended behavior, Since you are using split apks by adding
--split-per-abi to build command.
Please read this for more info on how versioning works for split apks.
On a side note: if you are building app for releasing to playstore

prefer building app bundle when releasing the app on play store as From August 2021, the Google Play Console will require all new apps to publish with the Android App Bundle format.
https://android-developers.googleblog.com/2020/11/new-android-app-bundle-and-target-api.html

Closing the issue since it is working as expected, If you disagree feel free to write in the comments and I will reopen it.
Thank you.


After the string length error in git commit, send button is disabled

VSCode Version: Code 1.10.1 (653f8733dd5a5c43d66d7168b4701f94d72b62e5, 2017-03-02T00:33:15.706Z)
OS Version: Windows_NT ia32 10.0.14393
Extensions:

Extension
Author
Version

vscode-file-peek
abierbaum
1.0.1

vscode-theme-onedark
akamud
1.3.1

one-monokai
azemoh
0.2.4

theme-onedark
azemoh
0.4.3

theme-firewatchneutral
bernardodsanderson
0.2.0

theme-desertex
casualjim
0.0.5

theme-itgdark
chriscarrick
0.2.0

theme-harmonic16-light
cotto
0.0.3

php-debug
felixfbecker
1.10.0

php-intellisense
felixfbecker
1.1.3

file-icons
file-icons
1.0.2

vscode-flatland-icon-theme
gebilaoxiong
0.0.6

Theme-Ocean-Dark-Soda
gerane
0.0.2

crane
HvyIndustries
0.2.2

hopscotch
idleberg
0.1.4

peachpie-vscode
iolevel
0.5.4

crayon-vscode
jansenfuller
0.1.3

theme-onedark-sublime
joshpeng
0.3.6

theme-material-theme
jprestidge
1.0.1

theme-glacier
Kal
0.0.1

format-indent
Kasik96
1.3.0

csharp
ms-vscode
1.7.0

Theme-MarkdownKit
ms-vscode
0.1.1

theme-railscasts
PaulOlteanu
2.0.0

material-icon-theme
PKief
1.3.4

vscode-icons
robertohuertasm
7.3.0

ayu
teabyii
0.2.1

Material-theme
zhuangtongfa
2.1.2

html-css-class-completion
Zignd
1.0.3

Steps to Reproduce:

What send button are you refering to? Can you please provide a picture?
Can you also reproduce this if you disable all your extensions?

Closing this issue as we havent heard back from the OP in a while


Add support for the time_intervals key; fix #215

Metrika's bytime response contains a list of corresponding time_intervals, but those were being omitted
nixsolutions/yandex-php-library#215

Coverage decreased (-0.6%) to 82.875% when pulling 40719dbe9cf06696443441fa5b986bb5cec7ec95 on mehov:fix/time-intervals into 9f1d6eee82810cccbdfbb8f3b66f7f6dc1d76c89 on nixsolutions:master.


Feature/fix #29

Start code re-ordering as per #29 and #31
Code that wasn't yet re-organized is in the legacy/ folder. It should be re-organised before merging

Merging this into a v0-1-0 development branch.


Double scrollbars showing (native and simplebar)

I have a Bootstrap's modal:

        ...

        ...

        ....

The modal-content has a max-height: 100%, and I want to scroll the modal-body when it is too large.
The problem is that when I set a max-height: 100% in modal-content, modal-body shows double scrollbars (the nativa and the simplebar)

This happend to me because of the box-sizing property, if you have a border you can delete that too that might be it


HESA reports shouldn't error when no degrees on applications

Context
As it is right now, the HESA reports errors when candidate doesn’t have degrees.
With the upcoming degree apprenticeship program will be possible to submit an application without degrees (only for courses within the program) so the report shouldn't error when an application doesn't have a single degree.
Changes proposed in this pull request
We leave the HESA degree fields 'no degree' when candidate doesn’t have degrees
Guidance to review

Does it error when an application doesn't have a degree?

Link to Trello card
https://trello.com/c/MfxfdxII/1458-tda-manage-update-hesa-reporting-as-they-require-a-degree-to-function

 TDA Manage - Update HESA reporting as they require a Degree to function


Restyling v1

To be improved.

Great job !
The combination of the horizontal and vertical navbars make navigation less heavy, but also harder to navigate (more clicks) :thinking:
Why renaming "Computing" -> "VMs" ?
I have a weird glitch when switching pages, it goes through a dark appearance that is very disturbing. Any idea what can be causing that ? (on Firefox)
The light grey footer at the bottom is not very readable.

I would like to wait with publishing this until we fine-tuned the theme to be fully aleph branded.

But the current one HURRRRTS MY EYEEEEEs

Great job !

Thanks! :)

The combination of the horizontal and vertical navbars make navigation less heavy, but also harder to navigate (more clicks) 🤔

I found the left navigation bar overwhelming and too spread out across topics. Over time, there will only be more pages on the various topics, which might require refactoring at some point.

Why renaming "Computing" -> "VMs" ?

It is a more concrete description of what this chapter entails. "Computing" sounds too generic.

I have a weird glitch when switching pages, it goes through a dark appearance that is very disturbing. Any idea what can be causing that ? (on Firefox)

Like a short flash before going back to normal?

The light grey footer at the bottom is not very readable.

I also noted this, will change :+1:

@claudioALEPH Also, what would it take to get there? Do we really have to wait until January?

@claudioALEPH Also, what would it take to get there? Do we really have to wait until January? Would it be fine to have this merged on a different branch, but without the CSS?

Yes, of course, that would be great. The problem I have is that we're introducing yet another theme here, which I want to avoid. We have already too many things that are off brand, and we've been working hard to get everything aligned.

@hoh @claudioALEPH Made another PR without the CSS changes here: https://github.com/aleph-im/aleph-docs/pull/39


[Feature]: Combined table with columns

Description
I would like to compare several runs table into a large table, but the current wandb only support combine them by rows. Let's say I have:
run1: 200x2
run2: 200x2
I want to have a 200x4 tables to compare their outputs, but now I can only have a 400x2 table.

Suggested Solution
Add a combining columns option
Alternatives
No response
Additional Context
No response

@fecet Hi! So you are interested in stacking tables on top of each other? Or how would you like to see that implemented?

Sorry for confusion, I would like to concat their columns, e.g. combine 2 200x2 tables into a 200x4 tables.

No worries! Could you send me a link to your workspace?

Uma Krishnaswamy commented:
Hi @fecet,
Just wanted to follow up with you regarding your  request as we have not heard back from you. Please let us know if we can be of further assistance or if your issue has been resolved.
Best,
Uma

So this is a demo workspace for this issue https://wandb.ai/facat/tlem-test/groups/<null>/workspace?workspace=user-facat

Hey @fecet
I let our eng team know about this request and will update you on progress.


Some text improvements in the new publish map modal window

See my proposal in the original ticket: https://github.com/CartoDB/cartodb/issues/3525

@viddo & @saleiva: can you review this, please?

+1

Frontend tests were OK :+1: (details)

:sparkles: :shipit:

retest this please

retest this please

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)


feature: soundtouch.js filter to preserve pitch when changing tempo (…

The same as #1157 but rebased on master.

Should be able to merge when #1233 is merged. (or cherry-pick the commit before)


UnicodeDecodeError: 'gbk' codec can't decode byte (problem with Chinese characters)

Test example:
example.zip
If I use
markdown-pp index.mdpp -o out.md
on these two files, markdown-pp throws this error:

UnicodeDecodeError: 'gbk' codec can't decode byte 0x80 in position 10: illegal multibyte sequence

append "-e latexrender" doesn't work for this case.

What version of Python are you using?  markdown-pp only supports unicode documents in Python 3.

What version of Python are you using? markdown-pp only supports unicode documents in Python 3.

Python 3.8.2, my example files are in UTF-8 without BOM.

The project tries to read files with the default encoding used by Python. If your system uses a locale that specifies encodings other than UTF-8, then it's going to fail on decoding the contents of a UTF-8 file. You can override the system locale by specifying the appropriate environment values, and you can test the default encoding with locale.getpreferredencoding().


[SAML] SAML document page outdated

What is the bug?
I followed the sample steps to run SAML, I cannot login SAML login page with Log in as admin with a password of admin
How can one reproduce the bug?

Download and unzip the example ZIP file.

Review the files:

docker-compose.yml defines two OpenSearch nodes, an OpenSearch Dashboards server, and a SAML server.
custom-opensearch_dashboards.yml add a few SAML settings to the default opensearch_dashboards.yml file.
config.yml configures SAML for authentication.

Access OpenSearch Dashboards at http://localhost:5601/. Note that OpenSearch Dashboards immediately redirects you to the SAML login page.
Log in as admin with a password of admin.  // I cannot login by username admin and password admin

What is the expected behavior?
I should successfully logging in.
What is your host/environment?
MacOS
Do you have any screenshots?

Do you have any additional context?
NO

@opensearch-project/security The custom-opensearch-dashboards.yml file contained in the downloaded zip file includes opensearch.username and opensearch.password properties that specify "kibanaserver" values. Is there a connection with the "admin" credentials users are told to use in documentation?

@Flyingliuhub Could you post the modifications to your configuration?
I tested the example with no modifications and it worked with admin:admin to get access, confirm by logging in and seeing the user information populated through the saml server

@cwillum The docker images created by OpenSearch contain a demo scripts installer that copies many default such as kibanaserver into the image.  The only connection to the admin credentials as mentioned is that are 'bootstrapped' with the same process by operationally they are unrelated.

@peternied Explanation appreciated. Thank you.

@Flyingliuhub Could you post the modifications to your configuration?
I tested the example with no modifications and it worked with admin:admin to get access, confirm by logging in and seeing the user information populated through the saml server

I deployed without modifications. did you successfully login with admin:admin in the following saml redirect from local saml server ?

Yes I did, could you check the log output from any of the docker image to see if there are any errors establishing the connections between the containers?

@peternied - Is there anything we can do from a documentation standpoint to clarify this, or should I close? Thanks!


Refactor forward transforms to remove buffer weirdness

"Split forward transform buffers into separate input and output buffers" is maybe more descriptive than buffer weirdness :)


path is missing from 'struct uri'

While testing websocket transport using Alfred's re patch, I noticed that path component is missing from struct uri:
struct uri {
	struct pl scheme;    /**< URI scheme e.g. "sip:" "sips:"    */
	struct pl user;      /**< Username                          */
	struct pl password;  /**< Optional password                 */
	struct pl host;      /**< Hostname or IP-address            */
	int af;              /**< Address family of host IP-address */
	uint16_t port;       /**< Port number                       */
	struct pl params;    /**< Optional URI-parameters           */
	struct pl headers;   /**< Optional URI-headers              */
};

Path is needed when websocket connection to SIP proxy is established via a normal web server acting as websocket proxy.  In Apache, for example, websocket proxy function can be configured like this:
<VirtualHost _default_:443>
...
	<LocationMatch /wss/>
	        ProxyPass ws://<IP_ADDRESS>:7998/
	</LocationMatch>
</VirtualHost>

In baresip, the corresponding outbound proxy would then be configured like this:
;outbound="sip:<IP_ADDRESS>:443/wss/;transport=wss"

Path /wss/ in the URI is needed for the location match.
As a starter, would it be OK to add path to struct uri:
struct pl path;    /**< Optional URI-path        */

and then modify sip_addr_decode to set its value?
baresip was able to register via web server proxy when I hard coded path /wss/ to ws_connect_send:
        /* TODO: how to select ports of outbound SIP/WS proxy ?
	 * TODO: http url path "test" is temp, add config
	 */

	/* Use port if specified, otherwise use default HTTP/HTTPS ports */
	if (sa_port(dst)) {
		if (re_snprintf(ws_uri, sizeof(ws_uri),
				"%s://%J/wss/", prefix, dst) < 0) {
			err = ENOMEM;
			goto out;
		}
	}

But that was just a test.  I would like to have real support for path in outbound URI.
Any suggestions?  What does the second TODO in the above mean?

Commits https://github.com/baresip/re/commit/20da56993227918459c9c0a45d8eaa6cf67d71f1 and https://github.com/baresip/re/commit/20da56993227918459c9c0a45d8eaa6cf67d71f1 added path field to struct uri as well as path support to uri_decode and uri_encode functions.
Alfred's re websocket patch creates websocket uri in ws_connect_send function:
static int ws_conn_send(struct sip_connqent **qentp, struct sip *sip,
			bool secure,
			const struct sa *dst, struct mbuf *mb,
			sip_transp_h *transph, void *arg)
...
	if (sa_port(dst)) {
		if (re_snprintf(ws_uri, sizeof(ws_uri),
				"%s://%J/", prefix, dst) < 0) {

In order to add path to the ws_uri, I would need access to struct uri, which is not included  in struct sa *dst argument.   It is included in the dialog that was allocated for the request, but I have not been able to figure out if the dialog would be available via  struct sip *sip argument.
Any suggestions?

I have made progress with this.
After adding path to struct uri, I added char *path to sip/request.c struct sip_request.  When request.c function sip_request() is called, path gets its value from const struct uri *route param.  I also added to request.c function sip_request_path() that provides access to the path from other *.c files in sip dir.
After setting path, sip_request() calls sip/ctrans.c function sip_ctrans_request() and passes sip_request object to it as value of void *arg argument.  In order to provide access to this object from other *.c files in sip dir, I added sip_ctrans_arg() function to ctrans.c.
With these preparations,  ws_conn_send() function is able to add path (defaults to "/")  to websocket uri:
static int ws_conn_send(...)
{
        ...
	struct sip_request *req;
	char *path;
        ...
	err = sip_ctrans_arg((struct sip_ctrans *)arg, (void **)&req);
	if (err)
		goto out;
        err = sip_request_path(req, &path);
	if (err)
		goto out;
	if (sa_port(dst)) {
		if (re_snprintf(ws_uri, sizeof(ws_uri),
				"%s://%J%s", prefix, dst,
				strlen(path) == 0 ? "/" : path) < 0) {
			err = ENOMEM;
			goto out;
		}
         ...

I have tested this and looks like it works OK.
Next I would like to add websocket branch to both re and baresip, based on Alfred's websocket patches plus the above described path enhancements.  I'm not sure if I have latest version of Alfred's patches and looks like creytiv.com has does not exist anymore.
Is adding of the branches OK?

I just added websocket branch to both re and baresip.  Currently they only include Alfred's sip web socket patches.  Sometime in the future, I would like to create pull requests from these branches to master, but would like to give people time to test if they find some issues.
I myself have found one:  according to RFC 7118, transport in Route header should always be ;transport=ws even when WSS transport is used and Via header specifies WSS.  Currently baresip in such case is using ;transport=wss.  I'll try to get that fixed.
I'm closing this issue now even when the path stuff is not implemented yet.

I myself have found one: according to RFC 7118, transport in Route header should always be ;transport=ws even when WSS transport is used and Via header specifies WSS. Currently baresip in such case is using ;transport=wss. I'll try to get that fixed.

I did some searching and perhaps it is OK to use ;transport=wss even when the RFC does not specify it.  The reason is that sips: scheme is more or less deprecated as a bad idea and it is better to always use sip: scheme and indicate with tls or wss transport value that transport is secure.


C# support?

I noticed that C# isn't on the list of supported languages. Could you please add C# support to this tool? Thanks!

Thanks!


BUG: time counter reports incorrect time to run all the tests

          Known issue: time counter reports incorrect time to run all the tests

Originally posted by @sloorush in https://github.com/Dyalog/ullu/issues/45#issuecomment-1865916900

False alarm: The counter does not report incorrect time but it reports the CPU time instead of the elapsed time. I am not sure why CPU time and elapsed time are so different. I think running the tests multiple times causes too much time spent on IO things?


Filter failure PRICE_FILTER

It works great, thank you 🥇
but it made an error:
Entering SELL order, buy_value: 10691, sell_value: 10749.96000000, order: 10749.95999999, quantity: 0.01363, profit : 0.005514919090823976
BTCUSDT depth cache update
asks { '10750.00000000': 0.534341, '10750.96000000': 0.020583 }
bids { '10742.32000000': 0.002575, '10742.31000000': 0.0036 }
best ask: 10750.00000000
best bid: 10742.32000000
BTCUSDT: ORDER WAITING ++2055
logD->2018-2-22 06:19:23 > Sell Order error:
{"code":-1013,"msg":"Filter failure: PRICE_FILTER"}

Thanks for your feedback. Error about fractional part on price, fixed I think ...


vent.template files need to support a 'swarm/kubernetes' section

added complexity without specific use case for it. closing.


Update contentscript.js

Update delete code to match spec so it works with the latest version of Chrome.

Thanks for the PR! The remove() method already exists behind a flag and will be enabled in M110. Since this is a bleeding edge extension for a bleeding edge feature, I prefer to leave the code simpler, that is, as is.

(M110 will go to stable on Feb 1, 2023, so not a long wait.)

Oh sweet. remove() will make things much easier for my other projects. Thanks for the info.

(This came up again in #4, but closing for the same reasons.)


Automatic testing of copier-template.

I just wrote a short bash script that can test generating a project from the template and runs a few of tox tests.
The CI test is failing because I broke the make_base.py script just before...!

Not sure what you changed now, but we cannot make the workflows jinja templates because then dependabot will not work.


Sync published with master

Deploy preview for docsdocker ready!
Built with commit 16bfdbd2a25ba46dff80caf2a4cf932f182cfc06
https://deploy-preview-8529--docsdocker.netlify.com


Feature Request: Web map

A leaflet powered web map would be nice.
I already made one, see https://lambdacraft.dev/craft/map/
The source https://github.com/LambdaCraft/LambdaMap
It's in react but can be statically compiled to deliverable html and js. The player positions are handled by a REST api in a fabric mod. The output of blockmap is passed through pngquant for small file delivery.
It would be nice however to have all this bundled into one server fabric mod, with updating player positions, refreshing tiles, etc.
Personally I would prefer if the server mod was lightweight as to not take up too much CPU in a client/server-worker model but perhaps if it hooked into the server instance to find the average TPS/MSPT and only do rendering when there is compute time available that would be great.

Thanks for your work on BlockMap! A feature like this is definitively planned, but it may take some time.
There already is some form of server support in BlockMap, even though it's a bit WIP and less documented. You seem to have implemented a few features that are already covered by BlockMap, so I'm very interested in your opinion.

The player positions are handled by a REST api in a fabric mod

When rendering a world in BlockMap, it already generates a lot of information about the world—including player names and positions—, saved into a compressed json file next to the images.

Personally I would prefer if the server mod was lightweight as to not take up too much CPU in a client/server-worker model but perhaps if it hooked into the server instance to find the average TPS/MSPT and only do rendering when there is compute time available that would be great.

How often would you like to update the map? Due to BlockMap only rendering regions modified since the last time, simply calling it in a systemd timer (or cron job) a few times an hour should not have a noticable impact on performance.

I didn't notice the new server render feature. I tried to use it but every dimension is failing for me on 1.16.1, I'll open a separate issue for that.
Really I'd like a dynmap-like map that is easy to drop into a server. Here's my take so far:

The rendered.json.gz is too large a payload for downloading every so often just for player positions, it's 330KB compared to 1KB payload of only sending POI and player positions through a REST API. Separating the data in separate endpoints, for ex. one /poi and one /players means that the two endpoints can be reverse proxied differently, with poi cached every 5 min or so, and /players cached every few seconds. This is important for me as a server owner because I don't want to be DDoS'd through the exposed TCP http ports on the MC server so I only let a reverse proxy server talk to it and cache the responses to distribute to many web clients.

I don't think the render-many function does zoom levels (full tiles shrunk to smaller and smaller scales), which is kind of important for a leaflet map. Showing the overview of a map should only take a few MB, but sending all the full zoom tiles is going to take 100MB+ for a 10km wide map. So if it were running on a cron job, and a chunk gets updated, that updated tile will have to be propagated up the zoom levels. I'm not sure what kind of algorithm that will take, perhaps it would be faster if the updated chunks could be cut, shrunk, and spliced into the existing higher zoom levels pngs.

I've done testing and pngquant is the most efficient compression for these type of maps. It has no artifacts like jpg while being about the same size a jpg quality 80. The positions of pixels are lossless, only the color space is slightly affected and hard to notice. pngquant is a C lib with JNI bindings so it would add a binary to distribution, and it needs to be included if blockmap is going to handle zoom levels and care about tile payload size.

Having the ability to hook into MC code as a server mod via a platform like fabric will allow rendering while server TPS is healthy, be able to announce on server when these renders are taking place, have in game commands, allow instant player placed POI pins on the map without doing a scan in the render phase, and any other features that requires MC server API.

These features may sound outside of scope for you as just a map tile generator, but it's all the things I have to worry about as a server operator who can't and won't use dynmap for performance or incompatibility reasons.

There is the (untested and undocumented) feature I added recently that gives control on which pins are included in the output and which are not. As for the size, the map pins may be the culprit, as they include the image of the map as it is shown in Minecraft. If you are ready to dive into the code, have a look at ServerSettings.PinSettings. All settings are optional and default to include if not set. The following may get you started (this disables everything except player positions, also it's untested ^^ ):
{
    …
    "show pins": {
        "players": "ALL",
        "maps": "false",
        "slime chunks": "false",
        "force-loaded chunks": "false",
        "barrier": "false",
        "world spawn": "false",
        "POIs": {},
        "structures": {},
    }
}

The finest granularity level in BlockMap is a region file, or 32x32 chunks
No, the BlockMap render engine does not do zoom levels, I stripped out this functionality for code simplicity and moved it into the GUI code. I can see that this may pose a problem for web views.
A full rendered map can take up quite some space yes, but as the BlockMap GUI caches this and only requests modified tiles after this, it wasn't a big problem for me. But again, this probably isn't really feasible for a web view.
The usual way I do MIP mapping is by creating multiple maps, each half the size of the previous. The tiles are still kept at 512x512 pixels, so 2x2 regions files are zoomed down to one of the next level. This forms a simple tree structure that can be invalidated from bottom to top (as a region file changes, all upper levels are updated as well)

I'm a bit reluctant to add this dependency to the source code

These features sound awesome and I'd really like native server support for BlockMap, but I don't own any plugin servers and thus don't really know how to achieve this. I'll try to help out as best as I can though.

These features may sound outside of scope for you as just a map tile generator

The tile renderer is the BlockMap-core sub project. The CLI and GUI are separate projects that build on top of this, though they are all tailored to work together and thus in a single repository. If there's something to the CLI that can be improved for your use case, feel free to request features/changes. But if you want to do something custom that really fits your needs, you may want to consider building your tooling directly on top of BlockMap-core itself instead of the CLI middle layer. The core library has a pretty stable and fairly well documented API, but no standalone releases (e.g. you won't find it on Maven Central) yet.


Attempt to decode JSON with unexpected mimetype: text/html

Rasa version:
rasa --version
Rasa 1.0.0
Rasa X version (if used & relevant):
rasa/rasa-x:stable from docker
Python version:
python3 --version
Python 3.6.8
Operating system (windows, osx, ...):
Linux 7e5eb7ff3f80 4.4.0-1083-aws #93-Ubuntu SMP Wed May 8 16:08:41 UTC 2019 x86_64 GNU/Linux
Issue:

Run rasa-x, login
select an model that has already been trained
click "talk to your bot" and enter text

ERROR: nothing happens.  Log of production container shows the following:
$ docker logs -f rasa_rasa-production_1
[...]
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/sanic/app.py", line 917, in handle_request
    response = await response
  File "/usr/local/lib/python3.6/site-packages/rasa/core/channels/channel.py", line 430, in receive
    sender_id = await self._extract_sender(request)
  File "/usr/local/lib/python3.6/site-packages/rasa/core/channels/rasa_chat.py", line 50, in _extract_sender
    user = await self._check_token(req.headers.get("Authorization"))
  File "/usr/local/lib/python3.6/site-packages/rasa/core/channels/rasa_chat.py", line 38, in _check_token
    return await resp.json()
  File "/usr/local/lib/python3.6/site-packages/aiohttp/client_reqrep.py", line 1027, in json
    headers=self.headers)
aiohttp.client_exceptions.ContentTypeError: 0, message='Attempt to decode JSON with unexpected mimetype: text/html'

Is this a known issue?  This is using the following versions of aiohttp
$ pip3 list | grep aio
aiofiles                0.4.0
aiohttp                 3.5.4

@ricwo i remember seeing this bug, not sure what caused it though, do you know?

@akelad This is a clean setup, using an amazon EC2 instance, and using the docker-based instructions in http://rasa.com/docs/rasa-x/deploy/ (making use of the install.sh script).  Is there a way I can test the API call to the server, to see if it is actually responding with the wrong mime type?

@ddombrowsky It's probably due to a wrong entry in credentials.yml - this will be fixed in the next release. in the meantime, can you change the contents of the file to
rasa:
  url: ${RASA_X_HOST}/api

and try again?

@ricwo
Thank you very much.  Updating credentials.yml seems to have brought the bot back to life.
I will check to see if https://github.com/RasaHQ/rasa/issues/3723 was caused by the same issue.

@ricwo
Out of curiosity, is the root cause the fact that the worker was hitting the wrong url, and the server was responding with some 404 HTML page, which the worker was then trying to interpret as JSON, and failing?  Is that the theory?

@ddombrowsky yes, that's correct 🙂

I got same error. Thanks @ricwo your solutions solved it

i'm gonna close this issue now, a fix will be part of an upcoming release


HMS Push notifications

I im using HMS Push Notification with react native ( react-native-hms-push 5.0.2-301). I tested it on Huawei Mate 10 lite and it is working fine. But when in production on P40 lite i can't get token. How to solve problem?

Hi @tomto1,
The problem cause is probably HMS Core version. Could you please try it with different version. If the problem is persist try to disable auto_init or please wait for new version of push plugin. Will be released on july.
Thank You.


Lift and name anonymous function

Lift a function expression from current use location, and create a function declaration using the parameters, body, and user supplied name
This is an action-type behavior
Key behaviors:

Accessible by the same means as other automated tools

Context menu (action submenu)
Action palette
No keybinding

Allows user to simply place cursor on function expression and perform the lift and name action

Must provide an opportunity for the user to choose the scope for declaration

Must provide user with an opportunity to name the function

Lifted arrow functions must be declared with a const, rather than being converted to a function declaration

Error conditions:

No function selected
No scope selected
No name provided

Shifted to function expression -- behavior complete.
https://github.com/cmstead/js-codeformer/releases/tag/v0.24.0


docs: add CAP_NET_RAW to the capability list

What this PR does / why we need it:
Update the capability list in the networking dev guide to reflect reality: #3338 added a new capability.
Release note:

NONE

/cc @EdDev

Thanks for catching this.


Multiple camera connection / What do I need to setup a multiple camera environment?

Hello, I bought a Realsense D435i and now consider to buy two more.
I want to setup a synchronized multicamera configuration.
I've alread read the guide in the following link,  but not sure how many connectors to buy.
[https://dev.intelrealsense.com/docs/multiple-depth-cameras-configuration]
Could you please give me a guide how many connectors and housing do I have to buy?
I am thinking of 3-camera system to take a 3 photos simultaneously.
Thank you in advance.

Hi @kickserver  There is information at the link below about working about how many sync cable components you need.
https://support.intelrealsense.com/hc/en-us/community/posts/360050894574/comments/360013337454

If you are able to use C++ code though then you might not need sync cabling for the photo capture application that you have in mind.  https://github.com/IntelRealSense/librealsense/issues/2219#issuecomment-412899468 has a C++ script for capturing an image simultaneously from all attached cameras.
The original script was designed to check if the camera model was a D415 and prevent capture if it was not, so you will need to edit this line of the code to change 'D415' to 'D435i' or remove the camera name check from the script.
if (strcmp(dev.get_info(RS2_CAMERA_INFO_NAME), "Intel RealSense D415") == 0) // Check for compatibility

Hi @kickserver  Do you require further assistance with this case, please?  Thanks!

Hi MarthG,
Two new D435i cameras just delivered, so I have 3 cameras now.
Thanks to your information, I am thinking of using C++ code to get the simultaneous images.
My concern is even I do not need sync cabling, but cameras should be attached.
What do I need to attach the cameras?
Thank you for your help.

Yes, a USB 3 hub would be fine.  I would recommend a USB 3.1 or 3.2 hub rather than USB 3.0.  There are USB 3,0 hubs available if there is a reason why you need USB 3.0 specifically, otherwise go for 3.1 or 3.2.
Amazon's Amazon Basics own-brand hubs are highly reliable with RealSense cameras.
You just plug the cameras into the hub and plug the hub into a USB port on the computer.  If you are using a C++ program that is designed for automatic detection of multiple cameras like the simultaneous capture one that I highlighted earlier then the cameras attached to the hub will be detected when the program is run.

Hi @MartyG-RealSense,
Thank you for your answer. I just tested the C++ code and got 2 RGB images.
I have few more questions.
In the code of #2219 , what does "9.json" file do? I erased this lines to run the code.
And, is there another code that I can get depth images together with RGB images?

It looks as though 9.json is simply the name of a custom camera configuration preset file that the script's creator was using to import and apply a list of values for camera settings.  It would not be required in order for the script to work successfully.
The SDK C++ example rs-save-to-disk demonstrates saving both depth and RGB to .PNG file.
https://github.com/IntelRealSense/librealsense/tree/master/examples/save-to-disk

@MartyG-RealSense
Thank you for your help!

You are very welcome, @kickserver - I'm pleased that I could help.  Thanks very much for the update!


"PASSED" step.result.status within JSON is being interpreted as Skipped by the report generation

Environment (please complete the following information):

multiple-cucumber-html-reporter: 1.21.4

Config of multiple-cucumber-html-reporter
Default configuration, freshly installed.
Describe the bug
Using Cypress with Cucumber, the JSON output after formatting from .ndjson into .json has steps.result.status value in capitals. This leads the multiple-cucumber-html-reporter to present them as Skipped due to case sensitivity.
EG:
"steps":[{"keyword":"Given ","line":8,"name":"the region and audience selection form is opened","result":{"status":"PASSED"

appears as Skipped.
The fix would be quite simple, within _parseSteps function the status can be changed toLowerCase() to work for any context.
Line 377 onwards:
			if (step.result.duration) {
				scenario.duration += step.result.duration;
				step.time = formatDuration(step.result.duration)
			}

			if (step.result.status.toLowerCase() === RESULT_STATUS.passed) {
				return scenario.passed++;
			}

			if (step.result.status.toLowerCase() === RESULT_STATUS.failed) {
				return scenario.failed++;
			}

			if (step.result.status.toLowerCase() === RESULT_STATUS.notDefined) {
				return scenario.notDefined++;
			}

			if (step.result.status.toLowerCase() === RESULT_STATUS.pending) {
				return scenario.pending++;
			}

			if (step.result.status.toLowerCase() === RESULT_STATUS.ambiguous) {
				return scenario.ambiguous++;
			}

			scenario.skipped++;

Please let me know if this is a good solution & if it could be added.
I have done it locally to have our reports appearing correctly but it would be great to have it in case of future versions :)

Actually there are more reporting issues under the FAILED condition than the one I have fixed, the capitals in the JSON must be impacting other areas of the report.

Closing this one as it will be fixed by this PR https://github.com/wswebcreation/multiple-cucumber-html-reporter/pull/207


DEV2-2972 add "ask chat" action

note: this action doesn't have a default shortcut, so users should define one in Settings -> Keymap -> "Ask Chat".

looks great! the only thing I would change is to make the icon aligned to the same line of the input :)

I'm using Messages.showInputDialog, and that's where they place the icon. To change the icon location I need to create a custom dialog, which I don't think is worth the effort for now


Chrome Issue: "Unresponsive Pages Warning - Kill or Wait?"

On the latest Chrome (58.0.3029.110 as of this writing), every tab with a page where the script is included will issue an "Unresponsive Pages Warning - Kill or Wait?" warning if at least one of the tabs is not in focus.  This occurs after a certain interval of inactivity.
Please help.

Does it happen on the demo page? http://mattstow.com/experiment/responsive-image-maps/rwd-image-maps.html
If not, then you'll have to provide a reduced test case that reproduces the issue.


Cannot create a directory that does not exist

Version
Microsoft Windows [Version 10.0.19043.2006]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Ubuntu 22.04
Other Software
No response
Repro Steps
I have not been able to find what exact steps caused the issue, but I had some files and directories created in docker (wsl) on a bind mounted directory and then removed them in windows powershell.
Expected Behavior
If a directory cannot be seem by dir it should not exist!
Actual Behavior
mkdir fails with an error that says "File exists" but neither dir in wsl shows the directory nor the directory can be seen in Windows File Explorer.
Diagnostic Logs
No response

Try unmounting the dir:
$ unmount -f /path/to/dir

-f
Force unmount (in case of an unreachable NFS system). (Requires kernel 2.1.116 or later.)

A restart fixed the problem. But, I still think this is a bug.

Can you provide concise repro steps? Or else this issue should be marked as solved.

Unfortunately, I can't.
If you let me know what diagnostic information I can provide, I'll do it next time if it happens again.

Yep, seeing this exact thing right now.
ls -lh .
ls: cannot access 'dir': No such file or directory
total 0
d????????? ? ?         ?        ?            ? dir

After deleting it from windows:
mkdir dir
mkdir: cannot create directory ‘dir’: File exists
rm -r dir
rm: cannot remove 'dir': No such file or directory


bruh

Go on with your V$ Code. Even though I use VS Code it still doesn't make sense to shut down Atom.

VSCode is:

built on the same tech (electron) as Atom
open source
has a binary version built on the open source without MS telemetry (https://github.com/VSCodium/vscodium/)
has many of the same features as atom, albeit slightly different
is just as configurable as atom
can be used more or less... the same ways as atom.

This isn't the place to bicker or complain. This is just kind of what happens when a company makes an open source project, and gets bought by another company with a similar/competing product. VSCode isn't bad. Its got good intent, and takes advantage of whatever resources a large corp wants to throw behind it. It is what it is. The people making VSCode are the same kinda devs who wanted to make something good with Atom, and the featureset reflects that if you take the time to get over how slightly different it looks.
The world moves on. Software gets abandoned. New projects emerge from the ashes of the old. Atom helped create Electron, which is an invaluable platform for developing desktop apps. Lets not bring toxic bickering into a f%$king github issue. Take your disappointment and anger somewhere else.

hey there, atom's source code is public, you can maintain it yourself, there's no reason to curse anyone and this isn't even a issue so please goto twitter and argue there.

 Github is owned by Microsoft :P
And like the others said, there is no reason to hate the people behind Atom. it is time consuming having to maintain a Hackable Editor, and be happy that the source code will still be there in 10 years, you can always make your own, that is why it is on Github. You can make your own text editor with the base of Atom if you truly want. Don;t hate on the people, sunsetting was foreseen already, since it didn't get a lot of updates anymore, and it slowly died over time. At least we got an anouncment

Yeah makes sense


Implement zip_with()

https://doc.rust-lang.org/std/option/enum.Option.html#method.zip_with
TODO

[ ] Implement zip_with() operator for Maybe
[ ] Implement zip_with() operator for Nullable
[ ] Implement zip_with() operator for Undefinable
[x] #1546

Done by #1800


Branch sync - Merge master with dev 1.19

Sync master with dev-1.19
cc: @zacharysarah @jimangel @kbarnard10

/LGTM

Hi @savitharaghunathan ,
The changes look good except for this change (that shows an X):
https://github.com/kubernetes/website/pull/21073/commits/f1929c87ea9026546e456eba5f4a465dd4aefd8d

I'm not sure why https://github.com/kubernetes/website/pull/21073#issuecomment-631179373 / https://app.netlify.com/sites/kubernetes-io-master-staging/deploys/5ec42918bba0b0000785a9a1 is a problem - that merge seems valid to me.
@kbhawkey do you have a theory about why GitHub shows that as a failed commit?

Hi. Nothing stands out in that pull request.
Look like the changes did not deploy. What happens if you "hit redeploy" in this pull request?
@savitharaghunathan , did you build your branch locally before sending in the PR?

Hi @kbhawkey, do you mean building it locally with hugo? If so, No I did not.

Hi @kbhawkey, do you mean building it locally with hugo? If so, No I did not.

Yes, did you build your branch locally with the correct version of Hugo (extended version).

Hi @kbhawkey, do you mean building it locally with hugo? If so, No I did not.

Yes, did you build your branch locally with the correct version of Hugo (extended version).

@kbhawkey No, I did not.
I followed the guide from docs role handbook -https://github.com/kubernetes/sig-release/tree/master/release-team/role-handbooks/docs#maintain-the-current-and-upcoming-dev-branch. If building with hugo locally is required, @jimangel or I could add this to the role handbook.
Do I need to build locally and fore push this PR?

The changes look good except for this change (that shows an X):
f1929c8
https://app.netlify.com/sites/kubernetes-io-master-staging/deploys/5ec42918bba0b0000785a9a1

Failed during stage 'deploying site': Failed to execute deploy: Error: deploy timed out while waiting to enter states: prepared,ready
I almost wonder if this was just a blip in Netlify rather than a specific failure to the PR. I saw some funky time outs on 5/14 that I added more details on slack.

did you build your branch locally with the correct version of Hugo (extended version)?

Since this PR is a merge from master to sync the dev-x.Xy branch (no new content) what would the purpose be of testing a local build?
If it fails, it would mean there's issues in master with no clear ownership for repairing and if the merge fails, we'll see that in the open PR.
I am open to including the step in the handbook, but I want to be sure I understand the value.

Hi. I was not asking to change the handbook or challenge the process
Just thought that I would check my branch locally before pushing (because I do).
Regarding,
I almost wonder if this was just a blip in Netlify rather than a specific failure to the PR. I saw some funky time outs on 5/14 that I added more details on slack.
I too think that this is probably a Netlify error. Can we just hit the redeploy button and see if this "blip" resolves itself? In the past, I think we have given the go ahead and pushed the changes.
I think it would be good practice to figure out how to get a green line (every time).

Does Netlify test every commit? I didn't think it did.

Can we just hit the redeploy button and see if this "blip" resolves itself?

I like this idea! Just kicked off again.

I think it would be good practice to figure out how to get a green line (every time).

Agreed.

It looks like redeploying builds from the latest state not that specific commit / PR. That might answer the question from @sftim ...

This looks good now

/approve


UTF-8 characters are unescaped even if they were escaped in response

I have a service that returns an xml containing, among others, value CALLE MARÍA LOZANA, but in prettyPrint() I see CALLE MARÍA LOZANA. And object I get from response via JAXB also has CALLE MARÍA LOZANA for this field. But every other http client (postman, soapUI, curl) show escaped symbol (Í, not Í). So, what restassured shows happens to be different from what there really is. The my service's behaviour is definitely  a bug, so I need to see such things and to be able to check for them, and that is impossible with restassured unescaping things that are escaped.
For those who might think that problem is in other clients: there is another service that returns values with diacritic signs, and for that service all http clients (postman, soapUI, curl) show that symbols are not escaped.

What version of rest assured are you using?

3.0.0

Hmm... It's strange that you also get the same problems if you use Jersey. Could it be a JVM setting? I always start my JVM with java -Dfile.encoding=UTF-8 .... Could this be the issue?

I usually run my tests from IDEA, and I don't know what settings it uses, but I specified that option in run properties, and I got the same as previously.
Just to be sure that we understand each other correctly: restassured is giving me nice human-readable symbols (including symbols with diacritics). The problem is that service I call via restassured actually responds with escaped symbols (like Í). And I need to know that it returned these ugly things, but restassured hides this, it only shows Í no matter what was in service response, Í or Í

Ah ok then I misunderstood you. Isn't the server also returning a charset? What does wireshark show you?

Just checked: server is returning charset UTF-8

Hmm ok, I'm not sure what to make of this. Maybe it's the JVM that does this automatically?!

Then it's strange that SoapUI shows it differently, as it is a Java application. Do you know how to check if it's JVMs fault?

Ah OK, I confused SoapUI with Postman. Then I'm not sure what could be the cause of this. Something with Apache HTTP Client perhaps?

Is there a way to check? I would post them a bug if it's their issue.

The only way I can think of to check this would be to try to make the same request using vanilla Apache Http Client.


Store workflow name and tower run id

 Add full mapping of nf-core pipelines to analysis types using CV


NeRFStudio + Volinga: Wrong ckpt format with NeRFStudio latest commit.

Describe the bug
I have just updated NeRFStudio to the latest commit, and since then it seems that the ckpt (trained using ns-train volinga...) do not work with the Volinga convertor. After a successful upload I get the error code 607 and message "Wrong ckpt format"
To Reproduce
Steps to reproduce the behavior:

Train a NeRF using ns-train volinga...
Upload the generated ckpt on Volinga's website
After few second that processing has started, it fails

Additional context
Even if there is a thread on Volinga's Discord server about this, I thought it could be useful to open an issue here too. It seems that the ckpt structure has changed.

Can you try from commit https://github.com/nerfstudio-project/nerfstudio/commit/d4b04376abd46d9bddf8c299d1687177fe027951 and see if that works?

Although v0.3.0 was a workaround, it was able to read the file without failure.

It should be solved now :)

I met the same error of "Wrong ckpt format" in Volinga.ai.
My environment is as follows:

Windows10 64bit
GeForce RTX3060
NVIDIA Driver Version: 537.58
no python installed

I took following steps:

installed anaconda3

create virtual env with python==3.10.13

installed pip packages

python -m pip install --upgrade pip
conda install -c "nvidia/label/cuda-11.8.0" cudatoolkit=11.8
conda install -c "nvidia/label/cuda-11.8.0" cuda-nvcc
python -m pip install torch==2.0.1+cu118 torchvision==0.15.2+cu118 torchaudio==2.0.2 --extra-index-url https://download.pytorch.org/whl/cu118

installed ninja and skipped tiny-cuda-nn
I gave up tiny-cuda-nn due to the installation error.

installed nerfstudio
python -m pip install nerfstudio

installed volinga-model

git clone https://github.com/Volinga/volinga-model
cd volinga-model
python -m pip install -e . --user

then, I checked "volinga" is in the console output after executing `ns-train -h`

created volinga ckpt file

ns-train volinga --data data/nerfstudio/poster --vis viewer

, and successfully created ckpt file.

uploaded ckpt file to https://volinga.ai/main

failed uploading
uploading procedure became 100%, but it failed.
the error information is displayed in volinga.ai as below:

Error code: 607
	Info: Wrong ckpt format

I would be appreciate if anybody could help me with the solution.

I think you may need to install a specific version of nerfstudio to train volinga?  I saw this from a quick look... https://github.com/Volinga/volinga-model#1-install-nerfstudio--v032

Thank you, machenmusik.
I re-installed all the packages, and specified the version of nerfstudio==0.3.2.
However, ns-train volinga --data data/nerfstudio/poster --vis viewer did not work properly.
I thought this was due to another reason (version mismatch? or source code problem?).
I would be grateful if you have any clue about solving this problem.
I paste error message after performing ns-train volinga --data data/nerfstudio/poster --vis viewer below:
[NOTE] Not running eval iterations since only viewer is enabled.
Use --vis {wandb, tensorboard, viewer+wandb, viewer+tensorboard} to run with eval.
No Nerfstudio checkpoint to load, so training from scratch.
Disabled tensorboard/wandb event writers
Printing profiling stats, from longest to shortest duration in seconds
Trainer.train_iteration: 4.0080
VanillaPipeline.get_train_loss_dict: 3.9890
Traceback (most recent call last):
  File "D:\anaconda3\envs\nerfstudio032_py310_pip\lib\runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "D:\anaconda3\envs\nerfstudio032_py310_pip\lib\runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "D:\anaconda3\envs\nerfstudio032_py310_pip\Scripts\ns-train.exe\__main__.py", line 7, in <module>
    sys.exit(entrypoint())
  File "D:\anaconda3\envs\nerfstudio032_py310_pip\lib\site-packages\nerfstudio\scripts\train.py", line 261, in entrypoint
    main(
  File "D:\anaconda3\envs\nerfstudio032_py310_pip\lib\site-packages\nerfstudio\scripts\train.py", line 246, in main
    launch(
  File "D:\anaconda3\envs\nerfstudio032_py310_pip\lib\site-packages\nerfstudio\scripts\train.py", line 189, in launch
    main_func(local_rank=0, world_size=world_size, config=config)
  File "D:\anaconda3\envs\nerfstudio032_py310_pip\lib\site-packages\nerfstudio\scripts\train.py", line 100, in train_loop
    trainer.train()
  File "D:\anaconda3\envs\nerfstudio032_py310_pip\lib\site-packages\nerfstudio\engine\trainer.py", line 255, in train
    loss, loss_dict, metrics_dict = self.train_iteration(step)
  File "D:\anaconda3\envs\nerfstudio032_py310_pip\lib\site-packages\nerfstudio\utils\profiler.py", line 127, in inner
    out = func(*args, **kwargs)
  File "D:\anaconda3\envs\nerfstudio032_py310_pip\lib\site-packages\nerfstudio\engine\trainer.py", line 468, in train_iteration
    _, loss_dict, metrics_dict = self.pipeline.get_train_loss_dict(step=step)
  File "D:\anaconda3\envs\nerfstudio032_py310_pip\lib\site-packages\nerfstudio\utils\profiler.py", line 127, in inner
    out = func(*args, **kwargs)
  File "D:\anaconda3\envs\nerfstudio032_py310_pip\lib\site-packages\nerfstudio\pipelines\base_pipeline.py", line 281, in get_train_loss_dict
    model_outputs = self._model(ray_bundle)  # train distributed data parallel model if world_size > 1
  File "D:\anaconda3\envs\nerfstudio032_py310_pip\lib\site-packages\torch\nn\modules\module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "D:\anaconda3\envs\nerfstudio032_py310_pip\lib\site-packages\nerfstudio\models\base_model.py", line 142, in forward
    return self.get_outputs(ray_bundle)
  File "D:\anaconda3\envs\nerfstudio032_py310_pip\lib\site-packages\nerfstudio\models\nerfacto.py", line 278, in get_outputs
    field_outputs = self.field.forward(ray_samples, compute_normals=self.config.predict_normals)
  File "D:\anaconda3\envs\nerfstudio032_py310_pip\lib\site-packages\nerfstudio\fields\base_field.py", line 124, in forward
    density, density_embedding = self.get_density(ray_samples)
  File "D:\anaconda3\envs\nerfstudio032_py310_pip\lib\site-packages\nerfstudio\fields\nerfacto_field.py", line 216, in get_density
    h = self.mlp_base(positions_flat).view(*ray_samples.frustums.shape, -1)
  File "D:\anaconda3\envs\nerfstudio032_py310_pip\lib\site-packages\torch\nn\modules\module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "D:\anaconda3\envs\nerfstudio032_py310_pip\lib\site-packages\torch\nn\modules\container.py", line 217, in forward
    input = module(input)
  File "D:\anaconda3\envs\nerfstudio032_py310_pip\lib\site-packages\torch\nn\modules\module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "D:\anaconda3\envs\nerfstudio032_py310_pip\lib\site-packages\nerfstudio\field_components\mlp.py", line 178, in forward
    return self.pytorch_fwd(in_tensor)
  File "D:\anaconda3\envs\nerfstudio032_py310_pip\lib\site-packages\nerfstudio\field_components\mlp.py", line 164, in pytorch_fwd
    for i, layer in enumerate(self.layers):
  File "D:\anaconda3\envs\nerfstudio032_py310_pip\lib\site-packages\torch\nn\modules\module.py", line 1614, in __getattr__
    raise AttributeError("'{}' object has no attribute '{}'".format(
AttributeError: 'MLP' object has no attribute 'layers'

Hello! @AFMagnon, If I understood correctly, you are not using tiny cuda to train your models right?

Thank you for your comment, Frivas97.
Thats right, I did not install tiny-cuda-nn.
After installing ninja, I failed installing tiny-cuda-nn with following error message:
 Built on Wed_Sep_21_10:41:10_Pacific_Daylight_Time_2022
      Cuda compilation tools, release 11.8, V11.8.89
      Build cuda_11.8.r11.8/compiler.31833905_0
      Detected CUDA version 11.8
      Targeting C++ standard 17
      running bdist_wheel
      D:\anaconda3\envs\nerfstudio_py310\lib\site-packages\torch\utils\cpp_extension.py:476: UserWarning: Attempted to use ninja as the BuildExtension backend but we could not find ninja.. Falling back to using the slow distutils backend.
        warnings.warn(msg.format('we could not find ninja.'))

Without this package, I succeeded to implement ns-train nerfacto --data data/nerfstudio/poster.
So, I thought this is not necessary...

If tiny cuda is not available, NeRFStudio will use vanilla Pytorch to train the model. The problem is that the way in which the model is stored in the ckpt varies depending if you use tiny cuda or not. At this moment, we only support the tiny cuda "format" on Volinga exporter. This is probably the reason for the failure.

Thank you!
To sum up, I should build the following environment, right?
・nerfstudio==0.3.2
・manage to install tiny-cuda-nn
So, I will focus on installation of tiny-cuda-nn with nerfstudio of 0.3.2.
I found the useful link for the installation. that reads:
git clone https://github.com/NVlabs/tiny-cuda-nn.git
cd tiny-cuda-nn
git submodule update --init --recursive
python -m pip install ./bindings/torch

but it failed...
I'll make an effort somehow...

That's right, that is the setup you need.

There have been various periods where the latest version of tiny-cuda-nn was broken and so new installs would fail.
(@Frivas97 if it's not already there, you may want to add the tiny-cuda-nn requirement to docs and maybe even implementation of your method, as it differels from others...)

(@AFMagnon do you have CUDA libraries and  Visual Studio installed?  CUDA version 11.8 is known to work well, and IIRC community versions of VS may suffice... not sure whether just windows-build-tools does)

@machenmusik ,
thank you for your suggestion.
I have tried to build environment that satisfy :

cuda-toolkit, cuda-runtime, cuda-visual-studio integration,  and nvidia graphics driver by cuda_11.8.0_522.06_windows.exe
The installation was checked by executing nvcc and nvidia-smi. They are all version 11.8.
visual c++
This was checked by cl command.
python version of 3.10.13

After building the environment, I re-tried the installation:
#install pip packages
python -m pip install --upgrade pip
python -m pip install torch==2.0.1+cu118 torchvision==0.15.2+cu118 torchaudio==2.0.2 --extra-index-url https://download.pytorch.org/whl/cu118
#tiny-cuda installation
git clone https://github.com/NVlabs/tiny-cuda-nn.git
cd tiny-cuda-nn
git submodule update --init --recursive
python -m pip install ./bindings/torch

But python -m pip install ./bindings/torch failed...
The error message is as follows:
Processing c:\users\UserName\tiny-cuda-nn\bindings\torch
  Installing build dependencies ... done
  Getting requirements to build wheel ... error
  error: subprocess-exited-with-error

  × Getting requirements to build wheel did not run successfully.
  │ exit code: 1
  ╰─> [18 lines of output]
      <string>:5: DeprecationWarning: pkg_resources is deprecated as an API. See https://setuptools.pypa.io/en/latest/pkg_resources.html
      Traceback (most recent call last):
        File "C:\Users\UserName\AppData\Local\Programs\Python\Python310\lib\site-packages\pip\_vendor\pyproject_hooks\_in_process\_in_process.py", line 353, in <module>
          main()
        File "C:\Users\UserName\AppData\Local\Programs\Python\Python310\lib\site-packages\pip\_vendor\pyproject_hooks\_in_p          json_out['return_val'] = hook(**hook_input['kwargs'])
        File "C:\Users\UserName\AppData\Local\Programs\Python\Python310\lib\site-packages\pip\_vendor\pyproject_hooks\_in_process\_in_process.py", line 118, in get_requires_for_build_wheel
          return hook(config_settings)
        File "C:\Users\UserName\AppData\Local\Temp\pip-build-env-0c0t_z5q\overlay\Lib\site-packages\setuptools\build_meta.py", line 355, in get_requires_for_build_wheel
          return self._get_build_requires(config_settings, requirements=['wheel'])
        File "C:\Users\UserName\AppData\Local\Temp\pip-build-env-0c0t_z5q\overlay\Lib\site-packages\setuptools\build_meta.py", line 325, in _get_build_requires
          self.run_setup()
        File "C:\Users\UserName\AppData\Local\Temp\pip-build-env-0c0t_z5q\overlay\Lib\site-packages\setuptools\build_meta.py", line 507, in run_setup
          super(_BuildMetaLegacyBackend, self).run_setup(setup_script=setup_script)
        File "C:\Users\UserName\AppData\Local\Temp\pip-build-env-0c0t_z5q\overlay\Lib\site-packages\setuptools\build_meta.py", line 341, in run_setup
          exec(code, locals())
        File "<string>", line 9, in <module>
      ModuleNotFoundError: No module named 'torch'
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> See above for output.

note: This error originates from a subprocess, and is likely not a problem with pip.

That says "ModuleNotFoundError: No module named 'torch'".
Torch installation I think was successful because I entered python interpreter mode and import torch was successful.

To install tiny-cuda-nn, I have generally used what is in the nerfstudio readme...
pip install ninja git+https://github.com/NVlabs/tiny-cuda-nn/#subdirectory=bindings/torch

@machenmusik ,
Yes.
when I install tiny-cuda-nn, I used both
pip install ninja git+https://github.com/NVlabs/tiny-cuda-nn/#subdirectory=bindings/torch
and
git clone https://github.com/NVlabs/tiny-cuda-nn.git
cd tiny-cuda-nn
git submodule update --init --recursive
python -m pip install ./bindings/torch

The former command would fail because fmt and cutlass folders were empty https://github.com/NVlabs/tiny-cuda-nn/issues/208 , and I also failed...
The latter command can install fmt and cutlass components, but I failed......
I also tried these two installation procedure with x64 native developer command prompt, but I failed.........
The failures are displayed with the same error message below
  × python setup.py bdist_wheel did not run successfully.
  │ exit code: 1
  ╰─> [32 lines of output]
      C:\Users\UserName\tiny-cuda-nn\bindings\torch\setup.py:5: DeprecationWarning: pkg_resources is deprecated as an API. See https://setuptools.pypa.io/en/latest/pkg_resources.html
        from pkg_resources import parse_version
      Building PyTorch extension for tiny-cuda-nn version 1.7
      Obtained compute capability 86 from PyTorch
      nvcc: NVIDIA (R) Cuda compiler driver
      Copyright (c) 2005-2022 NVIDIA Corporation
      Built on Wed_Sep_21_10:41:10_Pacific_Daylight_Time_2022
      Cuda compilation tools, release 11.8, V11.8.89
      Build cuda_11.8.r11.8/compiler.31833905_0
      Detected CUDA version 11.8
      Targeting C++ standard 17
      running bdist_wheel
      D:\anaconda3\envs\nerfstudio_py310\lib\site-packages\torch\utils\cpp_extension.py:476: UserWarning: Attempted to use ninja as the BuildExtension backend but we could not find ninja.. Falling back to using the slow distutils backend.
        warnings.warn(msg.format('we could not find ninja.'))
      running build
      running build_py
      creating build
      creating build\lib.win-amd64-cpython-310
      creating build\lib.win-amd64-cpython-310\tinycudann
      copying tinycudann\modules.py -> build\lib.win-amd64-cpython-310\tinycudann
      copying tinycudann\__init__.py -> build\lib.win-amd64-cpython-310\tinycudann
      running egg_info
      creating tinycudann.egg-info
      writing tinycudann.egg-info\PKG-INFO
      writing dependency_links to tinycudann.egg-info\dependency_links.txt
      writing top-level names to tinycudann.egg-info\top_level.txt
      writing manifest file 'tinycudann.egg-info\SOURCES.txt'
      reading manifest file 'tinycudann.egg-info\SOURCES.txt'
      writing manifest file 'tinycudann.egg-info\SOURCES.txt'
      copying tinycudann\bindings.cpp -> build\lib.win-amd64-cpython-310\tinycudann
      running build_ext
      error: [WinError 2] The system cannot find the file specified.
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
  ERROR: Failed building wheel for tinycudann
  Running setup.py clean for tinycudann
Failed to build tinycudann
ERROR: Could not build wheels for tinycudann, which is required to install pyproject.toml-based projects

Im so dipressed...

I tried with miniconda3 virtual-env of Python 3.9.18.
However, the installation of tiny-cuda-nn failed due to the same error above...
Could you teach me the version of tiny-cuda-nn or commit SHA-1?

Thank you!!
I re-installed OS and other software, then I succeeded to install tiny-cuda-nn!!
Successful conversion from ckpt file to nvol file!!
Maybe, software or library dependencies made things complex.


[SPARK-18256] Improve the performance of event log replay in HistoryServer

What changes were proposed in this pull request?
This patch significantly improves the performance of event log replay in the HistoryServer via two simple changes:

Don't use extractOpt: it turns out that json4s's extractOpt method uses exceptions for control flow, causing huge performance bottlenecks due to the overhead of initializing exceptions. To avoid this overhead, we can simply use our own Utils.jsonOption method. This patch replaces all uses of extractOpt with Utils.jsonOption and adds a style checker rule to ban the use of the slow extractOpt method.
Don't call Utils.getFormattedClassName for every event: the old code called Utils.getFormattedClassName dozens of times per replayed event in order to match up class names in events with SparkListener event names. By simply storing the results of these calls in constants rather than recomputing them, we're able to eliminate a huge performance hotspot by removing thousands of expensive Class.getSimpleName calls.

How was this patch tested?
Tested by profiling the replay of a long event log using YourKit. For an event log containing 1000+ jobs, each of which had thousands of tasks, the changes in this patch cut the replay time in half:

Prior to this patch's changes, the two slowest methods in log replay were internal exceptions thrown by Json4S and calls to Class.getSimpleName():

After this patch, these hotspots are completely eliminated.

Test build #68077 has started for PR 15756 at commit 2717f79.

LGTM. That's a massive amount of time spent in Class.getSimpleName!

Test build #68077 has finished for PR 15756 at commit 2717f79.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68077/
Test FAILed.

retest this please

Test build #68149 has started for PR 15756 at commit 2717f79.

Test build #68149 has finished for PR 15756 at commit 2717f79.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68149/
Test PASSed.

LGTM

Cool. Merging to master!


Fix cluster local label InferenceService

What this PR does / why we need it:
This PR fixes the advanced cluster local example for recent (recommended) Knative versions. The label to create a local InferenceService has changed to networking.knative.dev/visibility: cluster-local, see https://knative.dev/docs/serving/services/private-services/.
Type of changes

[X] Bug fix (non-breaking change which fixes an issue)

Feature/Issue validation/testing:
I created the cluster_local.yaml example with:

KServe: 0.7.0,
Istio 1.12.2,
Knative 0.23.1

Using the label in this will create a private service, the original not.

Thanks for the label fix!  Some Instructions here.

Sure, sign-off added. Happy to contribute!

Also find the deprecated annotation name here, probably should change this too.

Sure, sign-off added. Happy to contribute!

Also find the deprecated annotation name here, probably should change this too.

@TimKleinloog Are you able to verify if changing the label name on isvc works as I find there are a couple of places in the code using the old label name?

@yuzisun just changing the label on the isvc did change the URL for me from:
NAME            URL                                        READY   PREV   LATEST   PREVROLLEDOUTREVISION   LATESTREADYREVISION                     AGE
lightgbm-iris   http://lightgbm-iris.default.example.com   True           100                              lightgbm-iris-predictor-default-00002   18h

to
NAME            URL                                              READY   PREV   LATEST   PREVROLLEDOUTREVISION   LATESTREADYREVISION                     AGE
lightgbm-iris   http://lightgbm-iris.default.svc.cluster.local   True           100                              lightgbm-iris-predictor-default-00001   18h

I will dive a bit deeper if I experience any issues related to the other labels. Do you expect scenarios where this will be the case?
I think we need to change them anyway and could be logical to include them in this PR.

/lgtm
/approve


[test] Remove inline tests from the swiftpr category

The logic to apply decorators to these tests does not quite work
correctly. More tests than intended end up in the swiftpr category
when a decorator is added to a single inline test.
Related to r://37173570.

@swift-ci test

I'm torn about this one.
We have already relatively little coverage and this would just decrease it. What's the immediate problem we have with the current solution [there are some tests which are failing] ?

This is a temporary measure as I have no clean solution in mind.
Apart from TestSwiftReturns.py, which is failing on my machine (I believe you ran into this too?), there is no immediate problem. But it's a problem waiting to happen as the bots are running tests not marked swiftpr. IMO it's worth having reliable control over which tests run during PR testing -- the "full" lldb testing jobs can be responsible for running everything.
Some alternatives;

Add and support a categories kwarg to MakeInlineTest
Have the decorator maintain a list of tests which should run

I think having the decorator maintaining a list is probably a better approach than this one, but, I'd love to hear your opinions as well (or Adrian/Jonas as they worked on the testsuite relatively extensively recently).

IMO it'd be a bit of a leaky interface if the decorator maintained a list of tests (seems like something the test runner should be in exclusive control of?). I think @JDevlieghere had a clearer idea of how this might work.

I don't disagree with you, but I'm really afraid about losing even more coverage. Any chance we can postpone a decision until Jonas comments? Thanks!

Sounds good

I'm having a quick look at some alternatives. I'll report back here as soon as I know more!

Alright, so here's what I found out. The way we assign categories is by setting an attribute on the function or method. Before calling the test, we check what categories are set. The problem is that for inlineTests we just have different instances of a class with the same methods (e.g. we always have _InlineTest__test_with_dwarf, _InlineTest__test_with_gmodules, etc). Once the category is applied, it applies to every instance of the InlineTest, which are effectively different test. This is obviously not what we want...
So I wondered why the other decorators did work with the InlineTests. From looking at the source code for skipIf in unittest2 I found out that they wrap the test in another function. This is different from how categories work, which are checked before the function is called.
I think the proper solution is to follow the same approach for add_test_categories. Maybe it's just a matter of turning this into an implementation of skipIf that skips the test if the current categories are not defined. Alternatively we'd have to roll our own implementation that performs the necessary checks in the wrapper.

Nice! It seems that there's some logic in decorators.py to handle custom skip reasons. I'll try it out.

Good, thanks for looking at this, Jonas!

@JDevlieghere did you get a chance to look at this? Thanks!

Yes, Jonas and I have been iterating on this offline. There's a prototype for using a skipIf-style decorator to mark tests for swift-pr testing, but we're not sure whether it's running the correct set of tests yet.

@vedantk / @JDevlieghere Can this one be closed now that we are iterating on #342 ?


Removed assumption of 2 sensor arrays

Removes the assumption that there are 2 arrays connected to the Pi.
Closes #17

Superseded by #26


Rule proposal: enforce that the self-closing terminator is written as a single token

For some reason, / > is valid JSX, as is / followed by any kind of whitespace until the >. This rule would require that the self-closing terminator is always written as an atomic />.
Not having a check for this also allows for funny workarounds for jsx-closing-bracket-location, such as this (when using after-props):
<SomeComponent
  with='many'
  props='specified' /
>

I'm a strong +1 on this - there should never be a space between / and > and I'm horrified that someone would ever come up with that.

It also turns out that <Tag><   /Tag> is weird but valid JSX too, so I made the rule also check for this.

Also </ Tag> ideally.

I can implement that, but I think it should be a separate rule. This one concerns itself with space in the opener/closer itself, while the other rule would worry about whitespace between the opener and the component name (analogous to jsx-space-before-closing).

Per discussion with @ljharb on the babel slack:
Merge this proposed rule with jsx-space-before-closing and a rule for checking space after opening, with a default schema like:
{
  closingSlash: 'never', // internal space between the "</" and "/>" tokens, e.g. "< /" would warn
  beforeClosing: 'always', // same thing as the current jsx-space-before-closing option
  afterOpening: 'never', // same thing as jsx-space-before-closing, but right after the "<" or "</"
}

afterOpening would also have an 'allow-multiline' suboption. Adding an 'allow-multiline' suboption to beforeClosing (and using it as default instead) is possible, but a stricter beforeClosing: 'always' would conflict with jsx-closing-bracket-location.

Let's also rename this rule to jsx-tag-spacing - and then we can deprecate jsx-closing-bracket-location and copy the tests over.

Because jsx-space-before-closing only works on self-closing tags, I'll name the equivalent option beforeSelfClosing, and have beforeClosing apply to regular closing tags instead. The defaults would be beforeSelfClosing: 'always' and beforeClosing: 'never'.


[AIRFLOW-2465] Fix wrong module names in the doc

Make sure you have checked all steps below.
JIRA

[x] My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-2465
In case you are fixing a typo in the documentation you can prepend your commit with [AIRFLOW-XXX], code changes always need a JIRA issue.

Description

[x] Here are some details about my PR, including screenshots of any UI changes:

Fix wrong module names in the document
Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:

No additional test since it's just a documentation fix.
Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[x] In case of new functionality, my PR adds documentation that describes how to use it.

When adding new operators/hooks/sensors, the autoclass documentation generation needs to be added.

Code Quality

[x] Passes git diff upstream/master -u -- "*.py" | flake8 --diff

Codecov Report

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

@@            Coverage Diff             @@
##           master    #3359      +/-   ##
==========================================
+ Coverage   76.45%   76.45%   +<.01%
==========================================
  Files         203      203
  Lines       15007    15007
==========================================
+ Hits        11473    11474       +1
+ Misses       3534     3533       -1

Impacted Files
Coverage Δ

airflow/models.py
86.83% <0%> (+0.04%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8fa0bbd...c4770e6. Read the comment docs.


where is vocab file?

How many vocab size used here?

For the official LLaMA model you'll need to obtain the tokenizer and weights from Meta. For our OpenLLaMA model the vocab file can be directly downloaded from our released on HuggingFace hub. The vocab size for all LLaMA models is 32000.


[Bug] : Tests won't compile due to dfdx error

I encountered compilation errors when running tests of Luminal. The issue arises a bunch of compilation errors related to dfdx crate.
Steps to Reproduce:
cargo run test

Expected Behavior:
The tests should compile and run without any issues.
Actual Behavior:
The compilation fails with a bunch of errors, like the following:
error[E0310]: the parameter type `T` may not live long enough
  --> /Users/raphaeldoukhan/.cargo/registry/src/index.crates.io-6f17d22bba15001f/dfdx-0.13.0/src/nn/flatten.rs:27:9
   |
27 |         input.try_reshape()
   |         ^^^^^^^^^^^^^^^^^^^
   |         |
   |         the parameter type `T` must be valid for the static lifetime...
   |         ...so that the type `T` will meet its required lifetime bounds
   |
help: consider adding an explicit lifetime bound
   |
18 | impl<const C: usize, const H: usize, const W: usize, D: Device<E>, E: Dtype, T: Tape<E, D> + 'static>
   |                                                                                +++++++++

Luminal Version:
Latest commit: d311003

I'm not sure what cargo run test is doing there, just use cargo test. I just pulled main, cleaned, rm Cargo.lock, and ran cargo test and didn't see any issues.

Sorry, I made a typo when writing the issue. I actually ran cargo test. Hmm weird, probably an error on my system, I'll investigate. Thanks!

It was due to my rustc version. I was using a nightly version. With the last stable version it's working.


feat(logging): pretty print received and sent bidi server messages

Example:
Instead of having the output message string in a single line.

performance

Hmmm, maybe we could do something like "if (logging_on) {..." to skip the parsing when there is no logging.

https://www.npmjs.com/package/debug#checking-whether-a-debug-target-is-enabled

re: performance: updated, PTAL


Rework Fighting

Using Decision Tree for/and more dynamic cop reactions

[x] #20
[ ] #21

f2d004b8147d2bee91a816b97e071b5d7c233044

Done: f2d004b8147d2bee91a816b97e071b5d7c233044

Improving by learned lessons from #14    ->    8e7702466df16677870db176029b3cf21b325fa1

Improving by learned lessons from #14    ->    8e7702466df16677870db176029b3cf21b325fa1

40c1d1f56fc4330a684a7ff2cdf0b1e0addc7b27

Reopend due to futher childIssues -> improvements


uninterpolated placeholders in HTML report

Describe the bug
The generated HTML report contains uninterpolated placeholders:
k8s.io/kubernetes:1.13.0

The report is mainly about Go language modules.
The problem occurs using version 6.1.0 of the Maven plugin.

The fix will be included in the next release.

The fix will be included in the next release.


Fix CacheFileStore Increment Concurrent Writes

Fix #24496
Route::get('/init', function () {
    \Illuminate\Support\Facades\Cache::forever('num',0);
});

Route::get('/inc', function () {
    \Illuminate\Support\Facades\Cache::increment("num");
});

Route::get('/get', function () {
    $num = \Illuminate\Support\Facades\Cache::get("num");
    dd($num);
});

visit http://hostname/init
ab -c10 -n2000 http://hostname/inc/
visit http://hostname/get

You will find that the result is always less than 2000, This pr is solved through a mutex lock.

I suggest using a more robust cache storage layer if you need atomic counting operations... like Redis. Laravel offers Redis and Memcached drivers out of the box.

Hello, I know that redis can be used, but I think this is a bug that should be fixed. Since this method is provided, it should be able to be used normally, and the tested performance has not declined.


Replace jade with pug

Except the Express generator still actually puts Jade in package.json:
...
"jade": "~1.11.0",
...

And when you run npm install you get this warning:
npm WARN deprecated<EMAIL_ADDRESS>Jade has been renamed to pug, please install the latest version of pug instead of jade

So not sure it's entirely accurate to say it uses Pug...

@profiluefter, maybe you could submit a pr to update the [generator](https://github.com/expressjs/generator?  If we have that we could for sure update the docs.

@wesleytodd I'm not very experienced with node or express so I think it would be better if someone else would do this. In fact I was just getting started with node in general and was confused with the deprecated package on the page.

It is never to soon to start @profiluefter :)
In this case the change would be almost just a search and replace in this file.  Specifically it would be these lines:
https://github.com/expressjs/generator/blob/master/bin/express-cli.js#L463-L467
https://github.com/expressjs/generator/blob/master/bin/express-cli.js#L229-L230
https://github.com/expressjs/generator/blob/master/bin/express-cli.js#L301-L303
https://github.com/expressjs/generator/blob/master/bin/express-cli.js#L463-L467
As far as simple changes go, this is a great first contribution!


Update harness-platform-faqs.md

Updated 2 FAQ
Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: \

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66851a772b41ecc8c5db4985--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6685c1e8f895d400956ad5be--harness-developer.netlify.app


fix(variables): fixing proper default number type for repo_port

Issue
A recent commit to v0.2.0 introduced a breaking change where the type validation was enforced here: https://github.com/cyralinc/terraform-aws-repo-crawler/commit/1905e7f753667b22bf69115af6e4f3de0ffabf04#diff-05b5a57c136b6ff596500bcbfdcff145ef6cddea2a0e86d184d9daa9a65a288eR97
This fixes https://github.com/cyralinc/terraform-aws-repo-crawler/issues/6.
Error results
╷
│ Error: Invalid default value for variable
│
│   on .terraform/modules/example.cyral_repo_crawler/variables.tf line 97, in variable "repo_port":
│   97:   default = ""
│
│ This default value is not compatible with the variable's type constraint: a number is required.
╵

Thanks @bernadinm - apologies for breaking this. We will cut a release just to fix the regression however I unfortunately do not think that the crawler will work if the repo port is actually set to null, as the JSON rendered here will have a value of null, which will eventually get passed to the crawler as a command line argument -repo-port=<nil> and it will complain. I will probably need to refactor this to use the templatefile function instead of jsonencode. So for now, please continue to set the repo_port variable until I can confirm a fix.


publish python bindings?

hey all, hey @dmilkie! :wave:
I'm just getting acquainted here (saw this mentioned in a talk by @kevinyamauchi). i'd love to try to use/depend on this as a ground truth source for microsim ... haven't looked too deeply at dependencies and platform compatibilities yet.  But just curious to gauge your interest/openness to making it more distributable

This  looks quite promising as well.
TLDR: Run this command and it will make a 5 time point (-t 5) version of the data and put it into C:\tmp.  FIJI can open the image and label files.
docker run -it -v "C:\tmp:/data" ghcr.io/tp81/cytopacq:master -c /usr/local/config/granulocyte-debug.ini -f /data/im.ics -l /data/lb.ics -e /data/error.log -t 5

Dockerfile and instructions from the first author here: https://github.com/tp81/cytopacq


viber complains for old version of PulseAudio server

archlinux
% yay -Qs viber
local/viber <IP_ADDRESS>-1

$ apulse viber
PulseAudioService: failed to subscribe to context notifications
qml: CALL_WEB_VIEW_DEBUG: callWebEngineLoader active: false
# theres more crap about qml and qrc
...
Error: "PulseAudio server version is too old."

This error is shown when I try to play received video in the chat.
That's all I know.

I have the same problem. I tried to recompile apulse after changing apulse/3rdparty/pulseaudio-headers/pulse/version.h strings where pulse versions are mentioned. I changed them to the latest ones. apulse/src/apulse-volume.c had to be changed also. Yet I still had the same error.
Nevertheless, to my amazement, viber calls can be made without pulseaudio (or apulse of course)! Just start the app, you'll see the error in console, but you can make calls. Adjust audio levels, out and mic, using alsamixer. Also regarding received media files, videos, audio recordings etc, they open in external players, mpv in my case, so playback is fine for them also.
So in a nutshell, ALSA only viber: Calls ok, media files ok, no sound notifications.


[WIP] Fix SimpleHandwrittenPDE wpd

See https://docs.sciml.ai/SciMLBenchmarksOutput/dev/SimpleHandwrittenPDE/allen_cahn_fdm_wpd/, many of the simple PDE benchmarks fail. Where should I check the usage of LinSolveFactorize? It seems we no longer use this syntax.

LinSolveFactorize was replaced with linsolve and LinearSolve.jl, effectively all the stuff in here: https://docs.sciml.ai/DiffEqDocs/stable/tutorials/advanced_ode_example/

Is there a reason why you removed Radau as the reference solution? That's a good method for the refernence.

LinSolveFactorize was replaced with linsolve and LinearSolve.jl, effectively all the stuff in here: https://docs.sciml.ai/DiffEqDocs/stable/tutorials/advanced_ode_example/

I see, I will change to the linear solvers in LinearSolve.jl

Is there a reason why you removed Radau as the reference solution? That's a good method for the refernence.

Yeah, RadauIIA5 is a great method for stiff equations,  but I tested locally and it always errors with
Error: Non-concrete Jacobian not yet supported by RadauIIA5.

I see, I will change to the linear solvers in LinearSolve.jl

It can probably just be removed. The default LU is better these days than the one that chooses.

Yeah, RadauIIA5 is a great method for stiff equations, but I tested locally and it always errors with

Create a separate version of the problem which is not split. We should test non-split methods vs split methods using prob_choice.


feat(coral): Add view "Tenant" in "User information"

Design
Design in Angular app:

Todo

[x] #2047
[x] #2054

This has been resolved by #2047 and #2054


Fix month sort

January was being created as February due to the %12 + 1 happening in lib/archives_sidebar/lib/archives_sidebar.rb#L33 and then being used in lib/archives_sidebar/lib/archives_sidebar.rb#L36. lib/archives_sidebar/app/views/archives_sidebar/_content.html.erb#L8 was changed so that the link would go to the correct place.

Nice work! :+1:


Use dirname(@FILE) instead of Pkg.dir

this allows the package to be installed and loaded from elsewhere on LOAD_PATH

I guess the tests fail here because Travis builds from pull requests don't have access to secure environment variables?

Seems correct. I should modify travis to run only tests that don't requiere environment variables... I'll checkout the pull request locally and run the tests


Pagination Performance Improvement

Currently, the new pagination feature requests all results from PPAPI, which might be inefficient, as there are around 6000 records for courses. PPAPI endpoint does not support range queries for now. Therefore, the temporary solution, before the feature is implemented, will be to request only the first few pages, and then request more only when users reach the end of the list.

I think this might be more related to websoc-fuzzy-search, given that we call that first on the frontend, get the list of courseIds, and request information for those specific courses from PPAPI through our backend.

Potential improvement: only get websoc-fuzzy-search results for a window of 3-5 pages at a time.
e.g. the current page + the 2 pages after it; for page 1 this would be the current page + 4 after, for page 2 + 3 after.
We only need enough data to be able to render the 5 page number buttons (and prev, next) in the pagination component appropriately
Limitting results speeds up response time of websoc-fuzzy-search greatly. The problem is, we can not specify offset for websoc-fuzzy-search. This means we would have to load all the results up to the current page plus the 2 after.
Being able to specify an offset may allow for faster response times but I'm not entirely sure since I don't know the implementation details. It may be just as slow as loading everything up to that page plus the 2 pages after so we should try that first.

search overhauled in #505


maintain backwards compatibility with existing client class (SpeeechClient = Speech::Client)

PR #56 moved the client class to be at the service level (Google::Cloud::Speech::V1::Speech::Client) rather than at the package level (Google::Cloud::Speech::V1::SpeechClient). Do we want to move the client class back to the previous location? It is possible to alias the old client constant to the new client constant. Do we want to keep it where it is? Move it back? Or add an alias?

Decided not to support this.


The Unofficial builds of VSCode are not supported..

https://github.com/codercom/sshcode/blob/cb036d8b8c45a43ebc804e085202f4feff84e8d1/main.go#L307
Path to the config directory is hardcoded here ( $HOME/.config/Code/User/ ),
the unofficial builds doesn't use this path,
they use  $HOME/.config/Code - OSS/User/ instead.
(I'm not sure about Mac paths)
We should probably support them as well.
EDIT:
Also, VSCodium uses $HOME/.config/VSCodium/User/

@SscSPs
$HOME/.config/Code - OSS/User/ is the dev config folder for vscode on Linux. So this will not cause any problems.
However, how do Insiders version users will deal with these? @emerose

@jeasonstudio it causes issue while syncing,
2019-04-22 18:04:48 INFO        syncing settings
sending incremental file list
rsync: change_dir "/home/sahil/.config/Code/User" failed: No such file or directory (2)

sent 20 bytes  received 12 bytes  12.80 bytes/sec
total size is 0  speedup is 0.00
rsync error: some files/attrs were not transferred (see previous errors) (code 23) at main.c(1189) [sender=3.1.3]
2019-04-22 18:04:50 FATAL       failed to sync settings: failed to rsync '/home/sahil/.config/Code/User/' to 'myserver:.local/share/code-server/User/': exit status 23

thats because the path doesn't exist

Fixed in dcd29f0


java.net.BindException: Can't assign requested address (Bind failed)

Expected behavior
No response
Actual behavior

Steps to reproduce the problem
I configured an HTTP domain name that can be accessed, but when I started it, the request result reported an error. Why is this happening?

I accessed this HTTP interface address in the browser, although it returned a 404 error, I can see that the domain is accessible.

JMeter Version
5.5
Java Version
java version "1.8.0_371"
Java(TM) SE Runtime Environment (build 1.8.0_371-b11)
Java HotSpot(TM) 64-Bit Server VM (build 25.371-b11, mixed mode)
OS Version
macOS Monterey 12.6.1

这是来自QQ邮箱的假期自动回复邮件。

您好，来件已收到，我会尽快回复您。。。。

Hi @vlsi , the address "webgpt-spider-backend.vmic.xyz" is a domain name for a remote service, not a local address.
And this is my advanced configuration.

source address configures IP address that should be used for sending a request. It is useful when the load generator has several network interfaces or at least several IP addresses.
I suggest to clear Source address field.