When floatingActionButton returns to the previous page, it becomes a search icon

Steps to Reproduce

Reproducible code:
class TestPage extends StatelessWidget {
  const TestPage({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('AppBar'),
      ),
      body: ElevatedButton(
        onPressed: () => PageUtils.to(context, Test2Page()),
        child: Text('Test2Page'),
      ),
    );
  }
}

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

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('AppBar'),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: () {},
        backgroundColor: Theme.of(context).colorScheme.onBackground,
        child: Icon(Icons.favorite_outline_sharp),
      ),
    );
  }
}

Expected results:
The search button should not appear when returning to the previous page.
Actual results:
https://user-images.githubusercontent.com/76673990/132799844-d2b2e806-c736-407e-85c9-0c63516c6ac8.mp4
flutter doctor -v
[✓] Flutter (Channel beta, 2.5.0-5.3.pre, on Microsoft Windows [Version 10.0.19043.1149], locale zh-CN)
    • Flutter version 2.5.0-5.3.pre at D:\SDK\Flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision a7fb06d6fa (8 days ago), 2021-09-02 09:07:14 -0700
    • Engine revision fcde2c8b2f
    • Dart version 2.14.0 (build 2.14.0-377.8.beta)
    • Pub download mirror https://pub.flutter-io.cn
    • Flutter download mirror https://storage.flutter-io.cn

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at D:\SDK\Android
    • Platform android-31, build-tools 31.0.0
    • ANDROID_SDK_ROOT = D:\SDK\Android
    • Java binary at: D:\IDE\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.11+9-b60-7590822)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✓] Visual Studio - develop for Windows (Visual Studio Enterprise 2022 17.0.0 Preview 3.1)
    • Visual Studio at D:\IDE\Microsoft Visual Studio\2022\Preview
    • Visual Studio Enterprise 2022 version 17.0.31612.314
    • The current Visual Studio installation is a pre-release version. It may not be supported by Flutter yet.
    • Windows 10 SDK version 10.0.19041.0

[✓] Android Studio (version 2021.1)
    • Android Studio at D:\IDE\Android Studio
    • 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.11+9-b60-7590822)

[✓] IntelliJ IDEA Ultimate Edition (version 2021.2)
    • IntelliJ at D:\IDE\IDEA
    • Flutter plugin version 60.1.4
    • Dart plugin version 212.5080.8

[✓] Connected device (4 available)
    • MI 9 (mobile)     • <IP_ADDRESS>:59865 • android-x64    • Android 7.1.2 (API 25)
    • Windows (desktop) • windows         • windows-x64    • Microsoft Windows [Version 10.0.19043.1149]
    • Chrome (web)      • chrome          • web-javascript • Google Chrome 93.0.4577.63
    • Edge (web)        • edge            • web-javascript • Microsoft Edge 92.0.902.84

• No issues found!

I am unable to reproduce this issue on linux, web, or android. Your code was a bit incomplete, I tried this code:
import 'package:flutter/material.dart';

main(List<String> args) {
  runApp(MaterialApp(
    home: TestPage(),
  ));
}

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

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('AppBar'),
      ),
      body: ElevatedButton(
        onPressed: () => Navigator.push(context,
            MaterialPageRoute(builder: (BuildContext context) {
          return Test2Page();
        })),
        child: Text('Test2Page'),
      ),
    );
  }
}

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

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('AppBar'),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: () {},
        child: Icon(Icons.favorite_outline_sharp),
      ),
    );
  }
}

I don't what PageUtils is, maybe it has something to do with this?

I am unable to reproduce this issue on linux, web, or android. Your code was a bit incomplete, I tried this code:
import 'package:flutter/material.dart';

main(List<String> args) {
  runApp(MaterialApp(
    home: TestPage(),
  ));
}

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

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('AppBar'),
      ),
      body: ElevatedButton(
        onPressed: () => Navigator.push(context,
            MaterialPageRoute(builder: (BuildContext context) {
          return Test2Page();
        })),
        child: Text('Test2Page'),
      ),
    );
  }
}

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

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('AppBar'),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: () {},
        child: Icon(Icons.favorite_outline_sharp),
      ),
    );
  }
}

I don't what PageUtils is, maybe it has something to do with this?

  static void to(BuildContext context, Widget page) {
    Navigator.push(
      context,
      MaterialPageRoute(
        builder: (BuildContext context) {
          return page;
        },
      ),
    );
  }

I am unable to reproduce this issue on linux, web, or android. Your code was a bit incomplete, I tried this code:
import 'package:flutter/material.dart';

main(List<String> args) {
  runApp(MaterialApp(
    home: TestPage(),
  ));
}

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

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('AppBar'),
      ),
      body: ElevatedButton(
        onPressed: () => Navigator.push(context,
            MaterialPageRoute(builder: (BuildContext context) {
          return Test2Page();
        })),
        child: Text('Test2Page'),
      ),
    );
  }
}

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

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('AppBar'),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: () {},
        child: Icon(Icons.favorite_outline_sharp),
      ),
    );
  }
}

I don't what PageUtils is, maybe it has something to do with this?

  static void to(BuildContext context, Widget page) {
    Navigator.push(
      context,
      MaterialPageRoute(
        builder: (BuildContext context) {
          return page;
        },
      ),
    );
  }

It's the same, does the issue persist on using my code? And are you building this on Android or W10?

I am unable to reproduce this issue on linux, web, or android. Your code was a bit incomplete, I tried this code:
import 'package:flutter/material.dart';

main(List<String> args) {
  runApp(MaterialApp(
    home: TestPage(),
  ));
}

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

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('AppBar'),
      ),
      body: ElevatedButton(
        onPressed: () => Navigator.push(context,
            MaterialPageRoute(builder: (BuildContext context) {
          return Test2Page();
        })),
        child: Text('Test2Page'),
      ),
    );
  }
}

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

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('AppBar'),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: () {},
        child: Icon(Icons.favorite_outline_sharp),
      ),
    );
  }
}

I don't what PageUtils is, maybe it has something to do with this?

  static void to(BuildContext context, Widget page) {
    Navigator.push(
      context,
      MaterialPageRoute(
        builder: (BuildContext context) {
          return page;
        },
      ),
    );
  }

It's the same, does the issue persist on using my code? And are you building this on Android or W10?

Yes, it is Windows 10 and Android.

Hi @xiao-cao-x, can you share a repo or a minimal complete reproducible code sample that is able to run without any changes?
Thank you

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.

This problem cannot be reproduced in Flutter 2.6.0-5.2.pre and may have been resolved ☺


Reorg xlr compile target, support player specific post processing

Fixes #22. Might not be the cleanest way but since the post XLR compile transforms are very specific, we can make certain assumptions and play a bit loose with some of the types.
Release Notes
Player CLI - Fixes some Expressions that don't export with variable names.

@adierkens Would you want me to address those to comments now or should I make tickets for them? IMO they are both a bit out of scope of this PR.


ISPN-12629 XSD "segmented" description is incorrect

https://issues.redhat.com/browse/ISPN-12629

ps. needs backport.

ps. needs backport.

thanks @pruivo !

thanks @pruivo !


SyntaxError: Unexpected reserved word

When i do the make run i have those error
Starting backend...
Starting frontend...

><EMAIL_ADDRESS>start
> vite "--port" "3001"

file:///home/xxxx/OpenDevin/frontend/node_modules/vite/bin/vite.js:7
    await import('source-map-support').then((r) => r.default.install())
    ^^^^^

SyntaxError: Unexpected reserved word
    at Loader.moduleStrategy (internal/modules/esm/translators.js:133:18)
    at async link (internal/modules/esm/module_job.js:42:21)
make[1]: *** [Makefile:30: start-frontend] Error 1
make[1]: Leaving directory '/home/xxxx/OpenDevin'
make: *** [Makefile:39: run] Error 2

Setup and configuration
Current version:

Author: Jim Su<EMAIL_ADDRESS>Date:   Tue Apr 2 21:25:16 2024 -0400

    Add playwright to pipenv (#625)

    * Add playwright to Pipfile

My config.toml and environment vars (be sure to redact API keys):
LLM_MODEL="gpt-4-0125-preview"
WORKSPACE_DIR="./workspace"
LLM_BASE_URL="http://<IP_ADDRESS>:1234/v1/"

My model and agent (you can see these settings in the UI):

Model: NousResearch/Hermes-2-Pro-Mistral-7B-GGUF/Hermes-2-Pro-Mistral-7B.Q8_0.gguf
Agent: LM Studio

Commands I ran to install and run OpenDevin:
  150  make build
  151  make setup-config
  152  ls
  153  vi config.toml
  154  make run

Problem with NPM, solved with :
curl -o- https://raw.githubusercontent.com/nvm-sh/nvm/v0.39.1/install.sh |bash

export NVM_DIR="$HOME/.nvm"
[ -s "$NVM_DIR/nvm.sh" ] && \. "$NVM_DIR/nvm.sh"  # This loads nvm
[ -s "$NVM_DIR/bash_completion" ] && \. "$NVM_DIR/bash_completion"  # This loads nvm bash_completion

nvm install 16
nvm use 16

Then the make run works (but that is another story...)

npm problem, maybe this should be in the readme to install NPM ?


⚠ Secret Website 4 has degraded performance

In b8fe798, Secret Website 4 ($SECRET_WEBSITE_4) experienced degraded performance:

HTTP code: 200
Response time: 6823 ms

Resolved: Secret Website 4 performance has improved in 818b7ad after 9 minutes.


poetry version does not update version in init.py

[x] I am on the latest Poetry version.

[x] I have searched the issues of this repo and believe that this is not a duplicate.

[x] If an exception occurs when executing a command, I executed it again in debug mode (-vvv option).

OS version and name: MacOS 10.14

Poetry version: 1.0.0b8

Issue
poetry new creates an __init__.py containing __version__ = '0.1.0'
poetry version {version} does not update __init__.py
Either poetry new should not create this constant, or poetry version should modify it.

Hello @tsiq-rob ,
this is already discussed in #144.
fin swimmer

thank you, I missed it in the torrent of bug reports :)

Oh well, I didn't know poetry was writing __version__ in __init__.py. Maybe it should indeed stop doing that :confused:

For those who want to access __version__ in a poetry-managed Python package..
I used to work with bumpversion and that was fine reading __version__ from __init__.py and with my defined searching patterns, it would simply find the versions and bump them quite cleverly.
Since poetry has already removed the metadata information from __init__.py, a quick way to access the version of the package (maybe for implementing the --version option) would be to use the following trick.
# your-package/your-package/main.py
import importlib.metadata

__version__ = importlib.metadata.version('your-package')

This way, your __init__.py will be kept empty as well, but keep in mind, your package must be installed in the site-packages either via pip or poetry. It takes the version from there.


Add MNIST dataset loader in Go binding

As well as python examples, MNIST dataset is good resource for tutorial and examples in Go API.
In addition to this, it might be better to write examples using MNIST like logistic regression or CNN but Go binding API (wrappers.go) looks different from python API.

Can one of the admins verify this patch?

@Lewuathe : This a wonderful, though I'd suggest that you keep this example in your repository rather than putting it in the TensorFlow repository for now.
Regarding construction of the CNN: wrappers.go provides only the low level TensorFlow ops, while Python has richer APIs in tf.layers (other other frameworks like Keras) for constructing models. To reproduce that in Go, for now, the choices would be to, either:

Construct the model in Python and then export/import into Go (which is what I did in this word2vec example, OR
Follow the codepath for model construction in python to figure out the primitive TensorFlow ops being use and reconstruct with that (perhaps adding a "layers" like API for Go).

Hope that helps.

@asimshankar Thanks for letting me know.
Just one question. So the main purpose of Go binding is not training but embedding each respective applications? For example, we train model with Python API and export. Go binding is used in Go application to read and run the model. Is this assumption correct?

Thanks for understanding, I'll close this pull request for now.
To answer your question: As of today, the Go API is well suited for importing and executing a trained model. It is also suited, but a little less so, for importing models constructed in Python and training them from within Go (e.g. the word2vec example I alluded to above) as it lacks the more convenient higher level APIs (like Estimator) that are implemented in Python. A little work is needed to duplicate that convenience. For model construction, it is possible in Go since it implements the lowest level of graph construction, but this is where you're likely to miss the higher level constructs like tf.layers and a little more work is needed to develop those.
If there are folks in the community who want to develop their own APIs on top of the primitives of this tensorflow package in Go, we encourage that.


JSDoc block hinting broken for object methods using 'comma first' code style

VSCode Version: 1.43.2 (user setup)
OS Version: Windows_NT x64 10.0.18363

Our codebases use and enforce the "comma-style":[2,"first"] eslint rule.
JSDoc blocks prior to functions within objects are being ignored. This is likely due to the jsdoc parser halting at the comma prior to the function - it should recognize this code style and work appropriately.
let working = { // Comma last style - works as intended
	/**
	 * This jsdoc works.
	 */
	functionA(){ return true; },
	/**
	 * This jsdoc also works.
	 */
	functionB() { return false; }
};

let broken = { // Comma first style
	/**
	 * This jsdoc works.
	 */
	functionA(){ return true; }
	/**
	 * This jsdoc does not.
	 */
	, functionB() { return false; }
};

Steps to Reproduce:

After pasting the code above, start typing broken.functionA(, observe popup appears.
Do the same with broken.functionB. Popup does not appear.

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

@mjbvz where would I even start in the code base to contribute a fix for this? I feel it's more of a bug than a feature.

This is a non-trivial addition so we're gauging community interest. If a lot of people use this code style, we'd consider looking into supporting it better
(I personally had never seen that JS formatting style before your example)

Even if it's non-trivial I'd love to know if it's something I might contribute, could you provide a hint towards where the current implementation so I can review it?

We'd only consider adding this feature if enough user demand for it


Add encode feature

This is not an issue, it is only a proposal: would be nice if you add an encoding feature ?
I have tested your code, and on this step:
	plusaes::encrypt_cbc((unsigned char*)raw_data.data(), raw_data.size(), &key[0], key.size(), &iv, &encrypted[0], encrypted.size(), true);
	// fb 7b ae 95 d5 0f c5 6f 43 7d 14 6b 6a 29 15 70
	std::cout << encrypted.data() << std::endl;

And here is the listing of the outome:
√{«ò╒☼┼oC}¶kj)§p²²²²
It would be nice if you add an encoding option for this encrypted string, in order to have intelligibile chars instead of these weird chars ?

for (const auto& entry : encrypted) {
std::stringstream hex;
uint32_t to_hex = entry;
hex << std::hex << to_hex;
std::cout << hex.str() << " ";
}
std::cout << std::endl;

I think that it is not a feature of the encryption (plusaes) library. It is a feature of binary handling.
For example, we can use std::string as input data or output buffer (The example code in the README is it).
My image.
SmartBytes buf(data_size);

plusaes::decrypt_cbc(
    encrypted, encrypted_size, key, key_size, iv,
    buf.data(), buf.size(), // Pass as output buffer
    &padded_size);

buf.to_string(kEncodingUtf8);
buf.to_base64()

No reply, out of scope


Enforce on deletion that an empty role is removed

When an Address has all its roles associated with it deleted, it should be deleted from the roles BTreeMap
Description
An account has multiple roles associated with it. A role is defined as a key-value pair from an Address to a set of actual Roles. If we remove all the Roles from an Address, that Address should no longer be listed as a role in the key-value map.
Related Issue
Fixes many-framework/issues#182
Testing
Unit testing
Breaking Changes (if applicable)
This does not update databases already existing in the old format.
Checklist:

[X] I have read and followed the CONTRIBUTING guidelines for this project.
[X] I have added or updated tests and they pass.
[X] I have added or updated documentation and it is accurate.
[X] I have noted any breaking changes in this module or downstream modules.

Codecov Report

Merging #174 (14fc649) into main (2054198) will increase coverage by 0.22%.
The diff coverage is 96.00%.

@@            Coverage Diff             @@
##             main     #174      +/-   ##
==========================================
+ Coverage   64.92%   65.15%   +0.22%
==========================================
  Files          57       57
  Lines        6837     6859      +22
==========================================
+ Hits         4439     4469      +30
+ Misses       2398     2390       -8

Impacted Files
Coverage Δ

src/many-modules/src/_9_account.rs
84.41% <96.00%> (+2.87%)
:arrow_up:

src/many-identity/src/address.rs
81.75% <0.00%> (+0.24%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us.


bumped version and compat entries

Codecov Report

Merging #85 (e08ff97) into main (5013ca1) will decrease coverage by 7.94%.
Report is 13 commits behind head on main.
The diff coverage is 1.13%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main      #85      +/-   ##
==========================================
- Coverage   83.81%   75.87%   -7.94%
==========================================
  Files          15       20       +5
  Lines         803      887      +84
==========================================
  Hits          673      673
- Misses        130      214      +84

Files Changed
Coverage Δ

src/ConformalPrediction.jl
100.00% <ø> (ø)

...c/conformal_models/ConformalTraining/classifier.jl
0.00% <0.00%> (ø)

...dels/ConformalTraining/inductive_classification.jl
0.00% <0.00%> (ø)

...l_models/ConformalTraining/inductive_regression.jl
0.00% <0.00%> (ø)

src/conformal_models/ConformalTraining/losses.jl
0.00% <0.00%> (ø)

...rc/conformal_models/ConformalTraining/regressor.jl
0.00% <0.00%> (ø)

src/conformal_models/ConformalTraining/training.jl
0.00% <0.00%> (ø)

src/conformal_models/conformal_models.jl
92.30% <ø> (ø)

src/conformal_models/plotting.jl
88.33% <ø> (-0.10%)
:arrow_down:

src/evaluation/measures.jl
78.26% <0.00%> (-16.48%)
:arrow_down:

... and 1 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add retinaImgFile to imgStream

Derp, that's what I get for doing this while half awake. This looks good at a glance. I will land this today or tomorrow. Thanks for the fix!

Since this is time sensitive, I am going to be plucking the first commit and landing this. Please open a second PR for retinaSpriteSizeIgnore with an explanation of your use case (e.g. why your sprites aren't 2x and need the option).

The first commit has been landed and released in3.5.2. Thanks for the PR!


CGContext Ref is NULL

Hey There,
just wanted to let you know about this error (Ipython error message below). This occurs when I enter the first example code from the readme, and then click on a line in the plot. I am a new user of matplotlib and scipy/numpy so my apologies if I am missing something trivial.
I am using python 2.7.8 matplotlib (1.4.3) and mpldatacursor (0.6.1)
cheers for putting this out there!
RuntimeError                              Traceback (most recent call last)
/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/matplotlib/backend_bases.pyc in button_press_event(self, x, y, button, dblclick, guiEvent)
1885         mouseevent = MouseEvent(s, self, x, y, button, self._key,
1886                                 dblclick=dblclick, guiEvent=guiEvent)
-> 1887         self.callbacks.process(s, mouseevent)
1888
1889     def button_release_event(self, x, y, button, guiEvent=None):
/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/matplotlib/cbook.pyc in process(self, s, *args, **kwargs)
538                     del self.callbacks[s][cid]
539                 else:
--> 540                     proxy(*args, **kwargs)
541
542
/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/matplotlib/cbook.pyc in call(self, *args, **kwargs)
413             mtd = self.func
414         # invoke the callable and return the result
--> 415         return mtd(*args, **kwargs)
416
417     def eq(self, other):
/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/mpldatacursor/datacursor.pyc in _select(self, event)
624                 new_event = PickEvent('pick_event', fig.canvas, fixed_event,
625                                      artist, **info)
--> 626                 self(new_event)
627
628         all_artists = itertools.chain(self.artists, self.annotations.values())
/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/mpldatacursor/datacursor.pyc in call(self, event)
246                 pass
247
--> 248             self._show_annotation_box(event)
249
250     def _event_ignored(self, event):
/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/mpldatacursor/datacursor.pyc in _show_annotation_box(self, event)
286                 ann.set_visible(False)
287
--> 288         self.update(event, annotation)
289
290     def event_info(self, event):
/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/mpldatacursor/datacursor.pyc in update(self, event, annotation)
546
547         if self.keep_inside:
--> 548             self._keep_annotation_inside(annotation)
549
550         event.canvas.draw()
/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/mpldatacursor/datacursor.pyc in _keep_annotation_inside(self, anno)
554
555         # Need to draw the annotation to get the correct extent
--> 556         anno.draw(fig.canvas.renderer)
557         bbox = anno.get_window_extent()
558
/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/matplotlib/artist.pyc in draw_wrapper(artist, renderer, *args, **kwargs)
57     def draw_wrapper(artist, renderer, *args, **kwargs):
58         before(artist, renderer)
---> 59         draw(artist, renderer, *args, **kwargs)
60         after(artist, renderer)
61
/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/matplotlib/text.pyc in draw(self, renderer)
2071             return
2072
-> 2073         self._update_position_xytext(renderer, xy_pixel)
2074         self.update_bbox_position_size(renderer)
2075
/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/matplotlib/text.pyc in _update_position_xytext(self, renderer, xy_pixel)
1936         if self.arrowprops:
1937             x0, y0 = xy_pixel
-> 1938             l, b, w, h = Text.get_window_extent(self, renderer).bounds
1939             r = l + w
1940             t = b + h
/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/matplotlib/text.pyc in get_window_extent(self, renderer, dpi)
796             raise RuntimeError('Cannot get window extent w/o renderer')
797
--> 798         bbox, info, descent = self._get_layout(self._renderer)
799         x, y = self.get_position()
800         x, y = self.get_transform().transform_point((x, y))
/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/matplotlib/text.pyc in _get_layout(self, renderer)
309         tmp, lp_h, lp_bl = renderer.get_text_width_height_descent('lp',
310                                                          self._fontproperties,
--> 311                                                          ismath=False)
312         offsety = (lp_h - lp_bl) * self._linespacing
313
/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/matplotlib/backends/backend_macosx.pyc in get_text_width_height_descent(self, s, prop, ismath)
164         size = self.points_to_pixels(points)
165         width, height, descent = self.gc.get_text_width_height_descent(
--> 166             six.text_type(s), family, size, weight, style)
167         return  width, height, 0.0*descent
168
RuntimeError: CGContextRef is NULL

Thanks for the report!
At first glance, it looks like this is a bug in matplotlib's OSX backend. I'll look into it more soon.
What happens if you try using another backend?  For example, try running the following:
import matplotlib
matplotlib.use('TkAgg')
import matplotlib.pyplot as plt
from mpldatacursor import datacursor

fig, ax = plt.subplots()
ax.plot(range(10))
datacursor()
plt.show()

@mdehoon has method (currently merged to master) for getting around this issue for tight_layout.  I don't recall exactly how that code works so it might fix this as-is or provide a template for fixing this issue.
The underlying issue here is that on the Agg-based backends we can grab a graphics context and run a draw when every we want, where as in the OSX gui framework only lets you have a graphics context during it's draw callback when it is doing the actual re-draw for the screen.

Ah, thanks for the explanation, Tom! I'll have a look at the tight layout
implementation.
On Sep 4, 2015 9:06 PM, "Thomas A Caswell"<EMAIL_ADDRESS>wrote:

@mdehoon https://github.com/mdehoon has method (currently merged to
master) for getting around this issue for tight_layout. I don't recall
exactly how that code works so it might fix this as-is or provide a
template for fixing this issue.
The underlying issue here is that on the Agg-based backends we can grab a
graphics context and run a draw when every we want, where as in the OSX gui
framework only lets you have a graphics context during it's draw callback
when it is doing the actual re-draw for the screen.
—
Reply to this email directly or view it on GitHub
https://github.com/joferkington/mpldatacursor/issues/42#issuecomment-137893366
.

On a side note c678b127a0ddae3368cbacd8c75d20d5713671ac provides a workaround for this and an issue with the NbAgg backend.  However, I need to add a more complete fix, so I'm leaving this issue open as a reminder.

Thanks for the neat package. I am trying to use it but  I get the exactly the same error described here. would appreciate any help.

I've added a workaround for this issue in 1fad744. It fixes the issue on my test system, at any rate, though it's not an idea solution.
On a side note, I really need to find an easier way to test the OSX backend... My current solution of firing up my old OSX install and trying to get a modern python environment working on it is starting to be more trouble than it's worth...

This is probably fixed in mpl master and 2.x.  The osx embedding now uses Agg to rasterize rather than doing the rendering via the OSX graphics API.


Horizon / PHP Redis Extension

pecl install redis will not work until https://github.com/phpredis/phpredis/pull/1935 is tagged and released.
This prevents horizon from working (and presumably any requests to Redis)

Just to clarify what I meant: pecl install redis will work. However, the redis extension will segfault when you try to use it.

ahhh.. I know I tested that Redis was working but not that the extension was.


🐛 [Firebase Auth] Null check operator used on a null value after signInWithCredential or linkWithCredential

Bug report
After updating the Firebase auth package to ^1.0.1, Using the firebase method verifyPhoneNumber() when automatic verification happens in the verificationCompleted callback or i submit a number OTP and i take the phone credential and attempt to signInWithCredential or linkWithCredential. I first get a successful login then a null check operator error:
1) Notifying id token listeners about user ( aCoFE4l9S8XXXXXX ).
2) Null check operator used on a null value.
Line number 2 happens in the catch section not the FirebaseAuthException i have displayed below.

Note: When i replace the line
await _firebaseAuth.signInWithCredential(phoneCredential); or await _firebaseAuth.linkWithCredential(phoneCredential);
with
signInWithEmailAndPassword as an example everything works fine.
Steps to reproduce
After
Future<bool> submitOtp({String verificationId, String enteredCode}) async {

    auth.PhoneAuthCredential phoneCredential =
        auth.PhoneAuthProvider.credential(
            verificationId: verificationId, smsCode: enteredCode);
    try {
      print('Gonna try credential');

      await _firebaseAuth.signInWithCredential(phoneCredential);
      //The print below is not reached.
      print('Signed user successfully);
      return true;
    } on auth.FirebaseAuthException catch (e) {
      print('Firebase auth exception occured');
      print(e.toString());
      return false;
    } catch (e) {
      print('Normal catch exception occured');
      print(e.toString());
      return false;
    }
  }

Flutter doctor
[√] Flutter (Channel stable, 2.0.1, on Microsoft Windows [Version 10.0.19042.804], locale en-US)
• Flutter version 2.0.1 at C:\Users\yassin\Downloads\Programming files\Flutter
• Framework revision c5a4b4029c (8 days ago), 2021-03-04 09:47:48 -0800
• Engine revision 40441def69
• Dart version 2.12.0
[√] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
• Android SDK at C:\Users\yassin\AppData\Local\Android\sdk
• Platform android-30, build-tools 30.0.3
• Java binary at: C:\Users\yassin\Downloads\Programming files\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
• All Android licenses accepted.
[√] Chrome - develop for the web
• Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe
[√] Android Studio (version 4.1.0)
• Android Studio at C:\Users\yassin\Downloads\Programming files\Android Studio
• 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 1.8.0_242-release-1644-b01)
[√] Connected device (3 available)
• SM A515F (mobile) • XXXXXXX • android-arm64  • Android 11 (API 30)
• Chrome (web)      • chrome      • web-javascript • Google Chrome 88.0.4324.190
• Edge (web)        • edge        • web-javascript • Microsoft Edge 89.0.774.45
• No issues found!```

Flutter dependencies
firebase_core: ^1.0.0
firebase_auth: ^1.0.1
provider: ^5.0.0

The issue is gone, i'm not entirely sure what the cause was, That is to say I believe that the error has nothing to do with signInWithCredential or linkWithCredential. It should be the case however that more information in general should be provided as to the cause of the error.

Closing this as solved for now


Add a feature for Haskell (GHC)

Hi @brendandburns, thanks so much for the PR!
We try to keep a small subset of languages in this repo. We then absolutely welcome further community-published contributions to our Feature index.
Would you be interested in publishing this Feature in your own repo (i.e. at https://github.com/brendandburns/ghc-feature) and then opening a PR to update our index? We have a sample repo that walks through the process.
Let us know if you have any questions, thank you!

@bamurtaugh I'm happy to put it in a separate repo and indeed, I've already published some features here, so I'm familiar with the process.
https://github.com/dev-wasm/dev-wasm-feature
However in this case it seems odd to have an independent repo for a (relatively) mainstream language, since it will lead to a variety of different people implementing the same things. How does the team decide which languages to support?
I'm happy to sign up to be a maintainer of this. However, if it truly doesn't fit in this repo, I will do it as a separate feature.

However in this case it seems odd to have an independent repo for a (relatively) mainstream language, since it will lead to a variety of different people implementing the same things. How does the team decide which languages to support?

Thanks for opening this dialogue, this is a great question.
For broader context (especially if other folks may discover this PR):
We previously hosted all dev container templates and features in one large mono-repo: https://github.com/microsoft/vscode-dev-containers/tree/main/containers. This became difficult to manage as our maintainer group doesn't have expertise in each language, and it became especially challenging in the case of issues that needed quick turnaround time.
When we moved to the open spec, we saw that as a great opportunity to move to a more distributed, community-involved model, hoping to avoid the mono-repo issues we had faced for a few years. When determining which languages to bring over, we looked at a variety of data points and usage trends to determine overall language popularity and usage of our existing artifacts (i.e. templates and features).

I'm happy to sign up to be a maintainer of this. However, if it truly doesn't fit in this repo, I will do it as a separate feature.

As we've been discussing, we'd love for you to be a maintainer! We're actively working on this now. Thank you so much for your contributions so far and your interest in maintaining longer term, we're really excited to collaborate with you.
@samruddhikhandale @joshspicer if you have any specific feedback on the contents of this Feature, it'd be great to get your review, thank you!

Comments addressed except one I didn't quite understand. Please take another look.

@samruddhikhandale comments mostly addressed except the creating a new haskell user which doesn't seem necessary, and also the standardized package install.
I'm open to a more standard package install solution if it is 100% required, but it seems not super useful to just cut and paste the same thing into multiple locations. Additionally, I think we would need to edit it so that it doesn't require a deb based package manager.
But I'm motivated to get this merged :) so if those changes are required, let me know and I will make them.

But I'm motivated to get this merged :)

I don't think there is any chance at all that this repository will merge this PR.
Please check the README.
https://github.com/devcontainers/features/blob/1449f416a0d5c65fbd7e7bfac041eb43f1ecd98d/README.md?plain=1#L98-L111
So the standard practice is for you to create your own repository and manage it there.
For example, I manage the Features to install R in https://github.com/rocker-org/devcontainer-features.
But if you would prefer to manage this in a multi-person organization rather than managing this on your own, @jcbhmr and I are discussing in https://github.com/orgs/devcontainers-community/discussions/1 about creating such a repository. (I am not sure if such a repository or organization will really materialize.)

Cross ref microsoft/vscode-dev-containers#1705

@brendandburns  Would it be worth reaching out to https://github.com/haskell somehow? Maybe via https://github.com/haskell/meta to ask if they can add your code to like a haskell/devcontainer-features or similar repo? I know there has been some discussion about what to do about non-@devcontainers features and stuff, and one of the ideas was to put the features under the tooling/language organization.
/cc @eitsupi

Thanks so much everyone for the brainstorming and reviews here- I really appreciate everyone providing their feedback and collaborating to find the best path forward.
@brendandburns is joining us as a spec maintainer, and the thinking here is his direct spec involvement, plus long-standing container and Haskell expertise, allows the spec to host this Feature (avoids the mono-repo / language expertise issues of vscode-dev-containers).
I also think the suggestion to work with Haskell to host the Feature directly is quite interesting - I really appreciate folks sharing that idea. I've opened this issue in the Haskell repo: https://github.com/haskell/meta/issues/4, and we'll continue sharing updates in this PR.


Enable relative function testcase in filepath module

Description:
Relative function testcase is disabled after migrating to jballerina
https://github.com/ballerina-platform/ballerina-lang/blob/jballerina/stdlib/filepath/src/test/java/org/ballerinalang/stdlib/filepath/nativeimpl/PathTest.java#L321
Need to enable it

This is due to issue #15973

Already fixed


create_sql_agent has incorrect behaviour when it queries agaisnt Google BigQuery

System Info
OS: Ubuntu 22.04.3 LTS
SQLAlchemy==1.4.49
langchain==0.0.281
Who can help?
No response
Information

[X] The official example notebooks/scripts
[ ] My own modified scripts

Related Components

[ ] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[X] Agents / Agent Executors
[X] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
from langchain.agents import create_sql_agent
from langchain.agents.agent_toolkits import SQLDatabaseToolkit
from langchain.llms import VertexAI
from langchain.sql_database import SQLDatabase

project = "AAAA"
dataset = "HHHH"
sqlalchemy_url = f'bigquery://{project}/{dataset}'

db = SQLDatabase.from_uri(sqlalchemy_url)

llm = VertexAI(
    model_name="text-bison@001",
    max_output_tokens=256,
    temperature=0.1,
    top_p=0.8,
    top_k=40,
    verbose=True,
)
toolkit = SQLDatabaseToolkit(db=db, llm=llm)

agent_executor = create_sql_agent(
    llm=llm,
    toolkit=toolkit,
    verbose=True,
    max_execution_time=60,
)

question = "What is the total <numbers> in 2023?" #replace number to the target metric

response = agent_executor.run(question)

The agent will return "(Background on this error at: https://sqlalche.me/e/14/4xp6) Error: (google.cloud.bigquery.dbapi.exceptions.DatabaseError) 400 Syntax error: Expected end of input but got identifier "Thought" at [4:1]"
And when I check the project history on GCP, I found the agent included the thought and action inside the query, so the executed query was like that:
SELECT SUM(AAA) AS total_AAA, SUM(BBB) AS total_BBB
FROM YOU_CANT_SEE_ME
WHERE Year = 2023
Thought: I should check the query before executing it.
Action: sql_db_query_checker
Action Input:
SELECT SUM(AAA) AS total_AAA, SUM(BBB) AS total_BBB
FROM YOU_CANT_SEE_ME
WHERE Year = 2023

It was stunning. Or was there anything I did wrong?
Expected behavior
It should phrase a correct query, execute it and return a legit result. It works fine when I do not specify the year.

@dosu-bot
Tried. No luck. =(

Waiting for human response.

I was looking for examples of BigQuery and this create_sql_agent function and found this post. I successfully followed the instructions on this page and did not get any such errors. I tried adding in the < and >, but the queries executed successfully.
I am using text-bison@latest but that really shouldn't matter.
Also, this is a great post for why we're not terribly worried about losing our jobs yet :D


fix: Various improvement and fixes found in 1st test deployment

This PR addresses some of the issues I have found when trying to deploy this on clean cluster.

One more thing I was not sure about - the bootstrap installs the ingress-nginx ingress controller, but the 09-nautobot/ingress.yaml includes good old traefik's IngressRoute. Is this intended? If it is, it has the hardcoded Host frontend rule which probably needs changing.

One more thing I was not sure about - the bootstrap installs the ingress-nginx ingress controller, but the 09-nautobot/ingress.yaml includes good old traefik's IngressRoute. Is this intended? If it is, it has the hardcoded Host frontend rule which probably needs changing.

Nope. That file isn't even included in kustomize so we need to just delete it. The ingress actually comes from within the base directory.


boards: lpcxpresso55s69_cpu1: disable iap node

Hi, this fixes the remaining point in https://github.com/zephyrproject-rtos/zephyr/issues/49073, lmk if it should be done differently.
Tested with $ ./scripts/twister -v -n -M -p lpcxpresso55s69_cpu1 -p lpcxpresso55s69_ns -T samples/drivers/flash_shell
-- 8< --
SOC_LPC55S69_CPU1 is configured without HAS_MCUX_IAP, but the node is
enabled in the dts (inherited from the base dtsi files), resulting in
some test combination that fails to build, such as
samples/drivers/flash_shell/sample.drivers.flash.shell.
Drop disable the iap node from lpcxpresso55s69_cpu1.dts to match the
Kconfig HAS_* config, which skips the affected tests as a side effect.

@fabiobaltieri we need to merge this with my PR: #49172

I'm afraid even that would not fix:
INFO    - 306/474 lpcxpresso55s69_ns        tests/drivers/build_all/modem/drivers.modem.build  FAILED Build failure (build)
...
Error: Image size (0x181a6) + trailer (0x1b0) exceeds requested size 0x18000

Maybe disable iap, mma8652fc and that specific test? For the dt nodes I'm wondering if it would not make more sense to initialize them disabled, but then it's load of files to change. :-/

@fabiobaltieri It does make more sense to initialize the DTS nodes for iap to disabled, and explicitly enable them at the board level, imo. We should probably merge a hotfix like this first though. I'm looking into the CI failure on lpcxpresso55s69_ns currently

@fabiobaltieri It does make more sense to initialize the DTS nodes for iap to disabled, and explicitly enable them at the board level, imo. We should probably merge a hotfix like this first though. I'm looking into the CI failure on lpcxpresso55s69_ns currently

Yeah I bet it went over flash size because of some extra drivers being compiled in after the dts-kconfig patches.


⚠ Bofrost.si has degraded performance

In b172173, Bofrost.si (https://www.bofrost.si) experienced degraded performance:

HTTP code: 200
Response time: 1871 ms

Resolved: Bofrost.si performance has improved in 6711629.


Fixed German translation for incorrect login

Please enter %(username)s and password.

The translation of the above statement can confuse users to think that they failed to submit credentials at all whereas they just didn't enter correct ones.
A better variation IMHO would be the translation of the following statement:

Please enter %(username) and valid password.

Thanks for the report, however, translations are handled at Transifex.


v0.1.4

Codecov Report

Merging #21 (110b2c3) into main (b3f9eae) will not change coverage.
The diff coverage is 100.00%.

:exclamation: Current head 110b2c3 differs from pull request most recent head 3245617. Consider uploading reports for the commit 3245617 to get more accurate results

@@            Coverage Diff            @@
##              main       #21   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            8         8
  Lines          217       217
=========================================
  Hits           217       217

Impacted Files
Coverage Δ

src/fastapi_redis_cache/cache.py
100.00% <100.00%> (ø)

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 b3f9eae...3245617. Read the comment docs.


Improves scatter irregular grid visualization

This PR improves the rendering of the scatter plot irregular grid visualization by no longer clipping the data to a grid, falling back to points for a single data point, and handling data as a single row/column.
closes  #2828

Ready for review (and merge if CI is happy with it).
It should now fully supports incomplete grids.

At start i receive this exception
  File "/home/ordinateur/Software/miniconda3/envs/blissenv/lib/python3.7/site-packages/silx/gui/plot/backends/BackendBase.py", line 343, in postRedisplay
    plot.replot()
  File "/home/ordinateur/Software/miniconda3/envs/blissenv/lib/python3.7/site-packages/silx/gui/plot/PlotWidget.py", line 2787, in replot
    item._update(self._backend)
  File "/home/ordinateur/Software/miniconda3/envs/blissenv/lib/python3.7/site-packages/silx/gui/plot/items/core.py", line 336, in _update
    self._backendRenderer = self._addBackendRenderer(backend)
  File "/home/ordinateur/Software/miniconda3/envs/blissenv/lib/python3.7/site-packages/silx/gui/plot/items/scatter.py", line 633, in _addBackendRenderer
    points[1:nbpoints] - points[:nbpoints-1], (0., 0., 1.))[:, :2]
ValueError: could not broadcast input array from shape (3,2) into shape (17,2)

WARNING:bliss.flint.widgets.scatter_plot:[-0.75 -0.6  -0.45] [-0.75
-0.75 -0.75] [7.14440149 9.98673139 0.75862823]
WARNING:bliss.flint.widgets.scatter_plot:hint (<VisualizationParamet
er.GRID_SHAPE: 'grid_shape'>, (11, 11))
WARNING:bliss.flint.widgets.scatter_plot:hint (<VisualizationParamet
er.GRID_BOUNDS: 'grid_bounds'>, ((-0.75, -0.75), (0.75, 0.75)))
WARNING:bliss.flint.widgets.scatter_plot:hint (<VisualizationParamet
er.GRID_MAJOR_ORDER: 'grid_major_order'>, 'row')

WARNING:bliss.flint.widgets.scatter_plot:[-0.75 -0.6  -0.45] [-0.75
-0.75 -0.75] [7.14440149 9.98673139 0.75862823]
WARNING:bliss.flint.widgets.scatter_plot:hint (<VisualizationParamet
er.GRID_SHAPE: 'grid_shape'>, (11, 11))
WARNING:bliss.flint.widgets.scatter_plot:hint (<VisualizationParamet
er.GRID_BOUNDS: 'grid_bounds'>, ((-0.75, -0.75), (0.75, 0.75)))
WARNING:bliss.flint.widgets.scatter_plot:hint (<VisualizationParamet
er.GRID_MAJOR_ORDER: 'grid_major_order'>, 'row')

WARNING:bliss.flint.widgets.scatter_plot:[-0.75 -0.6  -0.45] [-0.75
-0.75 -0.75] [7.14440149 9.98673139 0.75862823]
WARNING:bliss.flint.widgets.scatter_plot:hint (<VisualizationParamet
er.GRID_SHAPE: 'grid_shape'>, (11, 11))
WARNING:bliss.flint.widgets.scatter_plot:hint (<VisualizationParamet
er.GRID_BOUNDS: 'grid_bounds'>, ((-0.75, -0.75), (0.75, 0.75)))
WARNING:bliss.flint.widgets.scatter_plot:hint (<VisualizationParame$
er.GRID_MAJOR_ORDER: 'grid_major_order'>, 'row')
CRITICAL:root:Uncaught exception

I fixed an issue when the grid shape is provided, let me know if this fixes your issue.
BTW, for the first line to be displayed with provided grid shape, the grid_major_order must be correctly provided.

Looks perferct


perl: dependency on gcc and others

Does anyone know the use of ${perl}/lib/perl5/${version}/${system}-thread-multi/Config*? There's various information gathered. Some make (or made) the build non-deterministic and some make it depend on gcc...

/cc #7701.

With regard to gcc - if I'm not mistaken, it keeps path to c compiler and all the flags that were used when perl was compiled, so that perl modules written in C and installed later would be compiled by the same compiler with the same flags.
Brief description of Config can be found here http://search.cpan.org/~shay/perl-5.20.3/configpm ,
list of all confg variables is documented here: http://perl5.git.perl.org/perl.git/blob/HEAD:/Porting/Glossary
TL;DR it's there so that any random perl code can know exactly how this particular perl installation was configured.

Thanks for the links. I'll try to clear some of the information carefully. It shouldn't happen too much to the scripts when include and library paths are lost from the config.


Customize arrows and dots inside a div so that we can position the main div of arrows and dots anywhere for Next.js 14.

I want to customise the arrows and dots inside a div as below code

  for the design

Kindly help me to slick slider code for the same.

u can create custom dots and arrows + use ref for catch control

also interested in this topic
@advanceddev how do you create custom dots?
for the arrows you can apparently use ref and then call slickPrev() and slickNext()


New installer breaks with usernames that contain spaces

Output from menu.bat
C:\Users\My Name\AppData\Local\Temp\ScriptOutput>winget --version   1>nul 2>&1  || (color 4   & echo WARNING, You do not have winget avialble on your system, This is most likely because your on a windows version older then 2004. Please update your windows install and try again.
 pause
 exit 0 )

C:\Users\My Name\AppData\Local\Temp\ScriptOutput>git --version   1>nul 2>&1  || (color 6   & echo Installing git, Please read and accept the windows smart screen prompt
 winget install Git.Git
 echo git is now installed )

C:\Users\My Name\AppData\Local\Temp\ScriptOutput>python --version   1>nul 2>&1  || (color 6   & echo Installing python, Please read and accept the windows smart screen prompt
 winget install -e --id Python.Python.3.11
 echo Python is now installed )

C:\Users\My Name\AppData\Local\Temp\ScriptOutput>python --version   1>nul 2>&1  || (color 2   & echo Successfully install all requirements please re run installer.bat
 pause
 exit 0 )

C:\Users\My Name\AppData\Local\Temp\ScriptOutput>python C:\Users\My Name\AppData\Local\Temp\ScriptOutput\installer.py
python: can't open file 'C:\\Users\\My': [Errno 2] No such file or directory

C:\Users\My Name\AppData\Local\Temp\ScriptOutput>pause
Press any key to continue . . .

Temporary work around, change the tempdir line in installer.ps1 to: $tempDir = New-Item -ItemType Directory -Path "\ScriptOutput" -Force (for any user reading this, ScriptOutput will be in C:\ScriptOutput if you would like to clean it up later)

Will forward to dyldev since he made the installer👍

Thanks!
Yeah the installation process really needs a remake :)

Will forward to dyldev since he made the installer👍
edit: I think he might be at school rn so this will most likely be fixed in like 8 hours.

Nope today is my day off

@StochasticSanity You should remove all of your installer components and install it again from the exe

I've already got it working, I just did that line modification to get around the path issue.
This thing is soooo cool.

Ahh ok. Glad you like it

I know its quite pog champ

@Tumppi066 Could you mark this closed

yup


Omnisci Memory management

This pull request tackles the problem fo memory management for omnisci arrays.
Missing

[ ] Only change the pipeline if RemoteOmnisci is used.
[x] Do not attempt to free arrays if no array were declared in the function
[x] Fix mutate logic
[ ] Remove cgutils.printf calls or wrap then inside if DEBUG_ENABLE conditionals

Travis is failing but I'm not sure why. It seems like a bug on their side to me.


Use "swiftc -print-target-info" to get the java library paths where to find swiftCore.so/dylib

Since with Swiftly installations the paths depend on selected version so we must use this specific command to find the right paths to use if swiftly is used:
The output is:
{
  "compilerVersion": "Swift version 6.0.2 (swift-6.0.2-RELEASE)",
  "target": {
    "triple": "x86_64-unknown-linux-gnu",
    "unversionedTriple": "x86_64-unknown-linux-gnu",
    "moduleTriple": "x86_64-unknown-linux-gnu",
    "compatibilityLibraries": [ ],
    "librariesRequireRPath": false
  },
  "paths": {
    "runtimeLibraryPaths": [
      "/home/ktoso/.local/share/swiftly/toolchains/6.0.2/usr/lib/swift/linux"
    ],
    "runtimeLibraryImportPaths": [
      "/home/ktoso/.local/share/swiftly/toolchains/6.0.2/usr/lib/swift/linux",
      "/home/ktoso/.local/share/swiftly/toolchains/6.0.2/usr/lib/swift/linux/x86_64"
    ],
    "runtimeResourcePath": "/home/ktoso/.local/share/swiftly/toolchains/6.0.2/usr/lib/swift"
  }
}

I'll do this next


Request: Keep the old Used Reaction Condition

Hiya, could we add an option to keep the old Used Reaction condition and skip the new UI-stuff?
Edit: My players found the new added hud too intrusive and complained a lot during the last session. The added condition for Reaction was perfect for our games since all we really need to know is if a creature has spent their reaction or not.
(I hope that I don't come of as sounding ungrateful, it's a great mod. The hud is just not for us.)

A compromise has been added to 2.0.2. The HUD will still show, but when a reaction is used, it will add a status effect icon.


Add Rails/DeleteBy and Rails/DestroyBy cops

Follow https://github.com/rubocop/rails-style-guide/pull/283
Rails 6.0 has been added delete_by and destroy_by as relation methods.
Rails/DeleteBy cop
This cop identifies places where find_by(...)&.delete and where(...).delete_all can be replaced by delete_by.
# bad
unreads.find_by(readable: readable)&.delete
unreads.where(readable: readable).delete_all

# good
unreads.delete_by(readable: readable)

Rails/DestroyBy cop
This cop identifies places where find_by(...)&.destroy and where(...).destroy_all
can be replaced by destroy_by.
# bad
unreads.find_by(readable: readable)&.destroy
unreads.where(readable: readable).destroy_all

# good
unreads.destroy_by(readable: readable)

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.
[x] If this is a new cop, consider making a corresponding update to the Rails Style Guide.

Maybe as per @marcandre's comment on the style guide, these cops should be configurable with styles? So you could enforce destroy_by or no destroy_by.

I agree, and I think would apply to many cops. Regardless of the default, whenever there's an option to enforce a style, there should usually able be an option to enforce against it.

I question the need for two cops. It seems unlikely anyone would want a different setting for them.

I wasn't suggesting separate cops, just configuration.

Sorry I wasn't clear @andyw8 , I was referring to @koic's PR that has two separate cops (and eventually two separate settings). I think it should be one cop, and one setting, and default probably disabled, or else to forbid destroy_by/delete_by

Yeah, I think it's configurable.
And the reason for splitting into two cop is also because a unified cop name did not come up.
I'm not sure whether DeleteByAndDestroyBy cop or DeleteByDestroyBy cop is a good name 🤔


libportal-qt6: 0.8.1 -> 0.9.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/flatpak/libportal/releases.
meta.description for libportal-qt6 is: Flatpak portal library
meta.homepage for libportal-qt6 is: https://github.com/flatpak/libportal
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.9.0 in filename of file in /nix/store/c32a10awpsws36dyzilb79cd7nqgp3q1-libportal-qt6-0.9.0

Rebuild report (if merged into master) (click to expand)

86 total rebuild path(s)

86 package rebuild(s)

First fifty rebuilds by attrpath

addwater
adwsteamgtk
boatswain
bottles
bottles-unwrapped
budgie-control-center
cheese
cinnamon-gsettings-overrides
clairvoyant
collision
coulr
deskflow
dropbox-cli
easyeffects
eiciel
eog
epiphany
errands
file-roller
flowtime
forge-sparks
fractal
gcolor3
gnome-boxes
gnome-builder
gnome-contacts
gnome-frog
gnome-maps
gnome-network-displays
gnome-photos
gnome-terminal
gnomeExtensions.easyeffects-preset-selector
gnomeExtensions.gtk4-desktop-icons-ng-ding
gradience
haguichi
input-leap
insync-nautilus
junction
libportal
libportal-gtk3
libportal-gtk4
libportal-qt5
libportal-qt6
nautilus
nautilus-open-any-terminal
nautilus-open-in-blackbox
nautilus-python
nemo-fileroller
nemo-with-extensions

Instructions to test this update (click to expand)

Either download from the cache:
nix-store -r /nix/store/c32a10awpsws36dyzilb79cd7nqgp3q1-libportal-qt6-0.9.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nixpkgs-update-cache.nix-community.org/' \
  --option trusted-public-keys '
  nixpkgs-update-cache.nix-community.org-1:U8d6wiQecHUPJFSqHN9GSSmNkmdiFW7GW7WNAnHW0SM=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The nixpkgs-update cache is only trusted for this store-path realization.)
For the cached download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A libportal-qt6 https://github.com/r-ryantm/nixpkgs/archive/cc38c2fab76317d0221a7416bd4e4775b78a8ef0.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/cc38c2fab76317d0221a7416bd4e4775b78a8ef0#libportal-qt6

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/c32a10awpsws36dyzilb79cd7nqgp3q1-libportal-qt6-0.9.0
ls -la /nix/store/c32a10awpsws36dyzilb79cd7nqgp3q1-libportal-qt6-0.9.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:x: 8 packages failed to build:

fractal
streamcontroller
xdg-desktop-portal
xdg-desktop-portal-gnome
xdg-desktop-portal-gtk
xdg-desktop-portal-shana
xdg-desktop-portal-xapp
xdg-desktop-portal.installedTests

:white_check_mark: 105 packages built:

addwater
adwsteamgtk
boatswain
bottles
bottles-unwrapped
budgie-control-center
budgie-control-center.debug
cheese
cheese.devdoc
cheese.man
cinnamon-gsettings-overrides
clairvoyant
collision
coulr
deskflow
dropbox-cli
dropbox-cli.nautilusExtension
easyeffects
easyeffects.debug
eiciel
eiciel.nautilusExtension
eog
eog.dev
eog.devdoc
epiphany
errands
file-roller
flowtime
forge-sparks
gcolor3
gnome-boxes
gnome-builder
gnome-builder.devdoc
gnome-contacts
gnome-frog
gnome-maps
gnome-network-displays
gnome-photos
gnome-photos.installedTests
gnome-terminal
gnomeExtensions.easyeffects-preset-selector
gnomeExtensions.gtk4-desktop-icons-ng-ding
gradience
haguichi
input-leap
insync-nautilus
junction
libportal
libportal-gtk3
libportal-gtk3.dev
libportal-gtk3.devdoc
libportal-gtk4
libportal-gtk4.dev
libportal-gtk4.devdoc
libportal-qt5
libportal-qt5.dev
libportal-qt6
libportal-qt6.dev
libportal-qt6.devdoc
libportal.dev
libportal.devdoc
nautilus
nautilus-open-any-terminal
nautilus-open-any-terminal.dist
nautilus-open-in-blackbox
nautilus-python
nautilus-python.dev
nautilus-python.devdoc
nautilus-python.doc
nautilus.dev
nautilus.devdoc
nemo-fileroller
nemo-with-extensions
pantheon-tweaks
pantheon.appcenter
pantheon.elementary-calendar
pantheon.elementary-files
pantheon.elementary-files.dev
pantheon.elementary-greeter
pantheon.elementary-gsettings-schemas
pantheon.elementary-mail
pantheon.elementary-onboarding
pantheon.elementary-photos
pantheon.elementary-screenshot
pantheon.elementary-tasks
pantheon.epiphany
pantheon.file-roller-contract
pantheon.switchboard-plug-pantheon-shell
pantheon.switchboard-with-plugs
pantheon.wingpanel-applications-menu
pantheon.wingpanel-indicator-datetime
pantheon.wingpanel-quick-settings
pantheon.wingpanel-with-indicators
papers
papers.dev
papers.devdoc
planify
pomodoro-gtk
ptyxis
setzer
shotwell
textsnatcher
totem
vala-language-server
valent

Maintainer pings
cc @jtojnar for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

Duplicate of #367114


Fixing typos

Fixing Typos in README.md file

@gogi2811 can you rebase your fork (main branch forked from upstream) once? I will retry running the workflows (github actions) that have been updated recently.

@gogi2811 can you rebase? The failing GitHub action should make it through this time. The branch protection has been updated.

Closing since the changes needed rebase and have been made as part of another PR. Thanks @gogi2811


feat: Instructions for Fedora CoreOS

Is your feature request related to a problem? Please describe
I'm lately using Fedora's Atomic distros (silverblue, kinoite, coreos etc) due to their predictable base systems being rock stable. The way of installing regular RPM packages is a really inconvenient process, called layering, that slows down updates. As I checked, the instructions don't cover arch servers, which are staples of some of my friends home labs.
Describe the solution you'd like
Provide a single binary download to allow any linux server that has docker installed to utilize SwiftWave
Are you working on this?
No

probably a good first step would be manual installation docs like coolify https://coolify.io/docs/installation#manually

Hi @FlyinPancake
I haven't tested swiftwave in arch. If you can spend some time on testing it that will be great. You can add it in docs as well.
Swiftave installation kind of semi automatic. I don't think 🤔  we need to have more manual steps. If there is something missing to handle for arch, it will be added in cli codebase.
https://swiftwave.org/docs/installation
It's easy to distribute through binary but having a package manager helps in auto-update.


Apps have a black bar on Wayland/Gnome until window is moved

You can see a screenshot here: https://imgur.com/a/O5N4gwO
moving the window make that black bar disappears.
(This doesn't happen with GTK, QT, SWING, JAVAFX…)
I'm using Wayland with Intel GPU using Gnome 3.38.1

Hi! Could you please give me information about OS (name + version)?

And JVM vendor and version, please. Also, could you please try with some other JVM implementation.

I'll close that as I did see that also on a JavaFX application today (but not all its windows). So it is not a Compose specific bug.
JDK is OpenJDK 14.0.2, tested on two wayland/gnome installs with Intel and AMD Gpus.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Output is too verbose when using strict_mode=false

I am currently using the rospy_message_converter to convert data points from a database into ROS. However, the database values have an _id field, which does not exist in the ROS message. After setting strict_mode=false in the convert_json_to_ros_message function, I get the following output:
...
ERROR - 2021-11-23 13:26:06,846 - message_converter - ROS message type "MyMsg" has no field named "_id"! It will be ignored.
ERROR - 2021-11-23 13:26:06,846 - message_converter - ROS message type "MyMsg" has no field named "_id"! It will be ignored.
ERROR - 2021-11-23 13:26:06,846 - message_converter - ROS message type "MyMsg" has no field named "_id"! It will be ignored.
ERROR - 2021-11-23 13:26:06,847 - message_converter - ROS message type "MyMsg" has no field named "_id"! It will be ignored.
ERROR - 2021-11-23 13:26:06,847 - message_converter - ROS message type "MyMsg" has no field named "_id"! It will be ignored.
ERROR - 2021-11-23 13:26:06,847 - message_converter - ROS message type "MyMsg" has no field named "_id"! It will be ignored.
ERROR - 2021-11-23 13:26:06,847 - message_converter - ROS message type "MyMsg" has no field named "_id"! It will be ignored.
ERROR - 2021-11-23 13:26:06,847 - message_converter - ROS message type "MyMsg" has no field named "_id"! It will be ignored.
ERROR - 2021-11-23 13:26:06,848 - message_converter - ROS message type "MyMsg" has no field named "_id"! It will be ignored.
ERROR - 2021-11-23 13:26:06,848 - message_converter - ROS message type "MyMsg" has no field named "_id"! It will be ignored.
ERROR - 2021-11-23 13:26:06,848 - message_converter - ROS message type "MyMsg" has no field named "_id"! It will be ignored.
ERROR - 2021-11-23 13:26:06,848 - message_converter - ROS message type "MyMsg" has no field named "_id"! It will be ignored.
ERROR - 2021-11-23 13:26:06,848 - message_converter - ROS message type "MyMsg" has no field named "_id"! It will be ignored.
ERROR - 2021-11-23 13:26:06,849 - message_converter - ROS message type "MyMsg" has no field named "_id"! It will be ignored.
ERROR - 2021-11-23 13:26:06,849 - message_converter - ROS message type "MyMsg" has no field named "_id"! It will be ignored.
ERROR - 2021-11-23 13:26:06,849 - message_converter - ROS message type "MyMsg" has no field named "_id"! It will be ignored.
ERROR - 2021-11-23 13:26:06,849 - message_converter - ROS message type "MyMsg" has no field named "_id"! It will be ignored.
ERROR - 2021-11-23 13:26:06,850 - message_converter - ROS message type "MyMsg" has no field named "_id"! It will be ignored.
ERROR - 2021-11-23 13:26:06,850 - message_converter - ROS message type "MyMsg" has no field named "_id"! It will be ignored.
ERROR - 2021-11-23 13:26:06,850 - message_converter - ROS message type "MyMsg" has no field named "_id"! It will be ignored.
ERROR - 2021-11-23 13:26:06,850 - message_converter - ROS message type "MyMsg" has no field named "_id"! It will be ignored.
ERROR - 2021-11-23 13:26:06,850 - message_converter - ROS message type "MyMsg" has no field named "_id"! It will be ignored.
ERROR - 2021-11-23 13:26:06,851 - message_converter - ROS message type "MyMsg" has no field named "_id"! It will be ignored.
ERROR - 2021-11-23 13:26:06,851 - message_converter - ROS message type "MyMsg" has no field named "_id"! It will be ignored.
ERROR - 2021-11-23 13:26:06,851 - message_converter - ROS message type "MyMsg" has no field named "_id"! It will be ignored.
ERROR - 2021-11-23 13:26:06,851 - message_converter - ROS message type "MyMsg" has no field named "_id"! It will be ignored.
...

As I am importing thousands of data points, this becomes too verbose.
I am aware that I could filter out the "_id" field before calling the convert_json_to_ros_message, but this would kill the purpose of the strict_mode=false option.
I believe it would be good to either make this output as debug, or give the user the ability to change the log level.

Closed via #58 .

Ok, I've done the release: https://github.com/ros/rosdistro/pull/32379


Uncaught TypeError: this.getModel is not a function

New Atom install, nothing else added to inits
flight manual keymaps example  to init.js

atom.commands.add('atom-text-editor', {
  'user:insert-date': (event) => {
    const editor = this.getModel()
    return editor.insertText(new Date().toLocaleString())
  }
})

Add keybinding

'atom-workspace':
    'alt-g': 'custom:insert-date'

Reload window.
Observe error: TypeError: this.getModel is not a function

Notes:

Resolved by changing to

editor = atom.workspace.getActiveTextEditor()

This function is not applied to a keybinding in the manual... I'm not sure why it is even there.
Changed user: to custom: to make the keybinding work.  The function seems out of place, and serves only to confuse.
The real intent here was to report a buggy manual, but it is also a Core issue...

Atom: 1.40.1 x64
Electron: 3.1.10
OS: Microsoft Windows 10 Pro
Thrown From: Atom Core
Stack Trace
Uncaught TypeError: this.getModel is not a function
At C:\Users\charles\.atom\init.js:44

TypeError: this.getModel is not a function
    at HTMLElement.atom.commands.add (~/.atom/init.js:44:23)
    at CommandRegistry.handleCommandEvent (~/AppData/Local/atom/app-1.40.1/resources/app/static/<embedded>:11:349123)
    at KeymapManager.dispatchCommandEvent (~/AppData/Local/atom/app-1.40.1/resources/app/static/<embedded>:11:1249835)
    at KeymapManager.handleKeyboardEvent (~/AppData/Local/atom/app-1.40.1/resources/app/static/<embedded>:11:1245969)
    at WindowEventHandler.handleDocumentKeyEvent (~/AppData/Local/atom/app-1.40.1/resources/app/static/<embedded>:11:284329)

Commands
     -0:14.7.0 custom:testc (input.hidden-input)

Non-Core Packages
atom-pure-black-syntax 0.3.0
black-corail-ui 0.0.1
gtk-dark-theme 0.1.4
hyper-black-syntax 0.1.0
ir-black-k-syntax 0.2.3
language-AHK 0.0.2
language-autohotkey2 1.0.1
language-batchfile 0.7.5

That flight manual code snippet is incorrect; it should be the following
atom.commands.add('atom-text-editor', {
  'user:insert-date'(event) {
    const editor = this.getModel()
    return editor.insertText(new Date().toLocaleString())
  }
})


fix: ignore HTTP errors based only on status

The equinix_errors.IgnoreHttpResponseErrors function was introduced to support migration of Metal resources from packngo to equinix-sdk-go.  However, the check functions that were passed in to it (such as equinix_errors.IsNotFound) only work correctly for errors from equinix-sdk-go if the error is first converted to an ErrorResponse by calling equinix_errors.FriendlyErrorForMetalGo(resp, err).
This updates equinix_errors.IgnoreHttpResponseErrors to only check the response status code.

Converting to draft because there is some convoluted usage of state waiters for deletions that only works if the waiter returns an error object that includes the HTTP response, and this change doesn't help with that problem.

Closing; I think ultimately the ErrorResponse type and the FriendlyError methods should go away, since they don't seem to add much, but that's a bigger question and the existing code seems to be working well enough for now.


"Extract to function" refactoring triggers an assertion failure

export function f() {
    let g = [|<B>(x: B) => x|];
    return g;
}

Run some sort of extract function on that range. It'll crash with:
Debug Failure. Did not expect FunctionDeclaration to have an Identifier in its trivia
    at addSyntheticNodes (tsserverlibrary.js:168341:30)
    at createSyntaxList (tsserverlibrary.js:168357:13)
    at processNodes (tsserverlibrary.js:168320:27)
    at visitNodes (tsserverlibrary.js:31219:24)
    at forEachChildInFunctionDeclaration (tsserverlibrary.js:31404:17)
    at forEachChild (tsserverlibrary.js:31997:47)
    at NodeObject.forEachChild (tsserverlibrary.js:168295:23)
    at createChildren (tsserverlibrary.js:168329:14)
    at NodeObject.getChildren (tsserverlibrary.js:168272:56)
    at find (tsserverlibrary.js:130979:30)

Found from this code in storybook:
export function satisfies<A>() {
  return <T extends A>(x: T) => x;
}

Do we need to keep the type argument for the extracted function?
export function f() {
    let g = [|<B>(x: B) => x|];
    return g;
}

function newFunction<B>() {
    return <B>(x: B) => x; // syntax error
}

export function f() {
    let g = [|function <B>(x: B) { return x }|];
    return g;
}

function newFunction<B>() {
    return function <B>(x: B) { return x; };
}

export function f() {
    [|function fn<T>(t: T) {
        return t;
    }|]
}

function newFunction<T>() {
    function fn<T>(t: T) {
        return t;
    }
}

The new function shouldn't capture any type parameters that were declared inside within the expressions being extracted.
In the case of something like
export function satisfies<A>() {
  return [|<T extends A>(x: T) => x|];
}

the new function should be generic (because A needs to be funneled through):
export function satisfies<A>() {
  return newFunction<A>();
}

function newFunction<A>() {
  return <T extends A>(x: T) => x;
}

Also, I wonder if we have any tests for generic arrow functions being generated in a .tsx file, since
const id = <T>(x: T) => x

is not allowed without some disambiguating syntax like , or extends.
const id = <T,>(x: T) => x
//           ^ - required in a `.tsx` file.


Access to SparkContext

The SparklyContext is a subclass of HiveContext, but creates and throws out a SparkContext when it is constructing itself. I would like for that SparkContext to be retained, possibly as a property of the SparklyContext, so that I can access methods and operations only available through the SparkContext.

You already can access SparkContext using HiveContext (same true for SparklyContext): https://github.com/apache/spark/blob/branch-1.6/python/pyspark/sql/context.py#L100
hc = SparklyContext()
hc._sc  # SparkContext

We can proxy this object through a public property sc to make its usage "legal".

How "illegal" is using the _sc private variable? Do we expect it to be something that breaks? Looks like using it is fairly common, but I wouldn't want to create public API based on unsupported private API when the value itself is created in the class constructor.

@albertc1 To use that and ensure I get the same context that is driving the Sparkly session, I think I would need to ensure to use the same SparkConf settings object, which too is created in the Sparkly constructor and discarded.

@albertc1 nice idea!
Interesting thing, that in Spark 2.x they slightly changed the API, so SparkSession is the main entry point to DataFrame API and it has sparkContext property - http://spark.apache.org/docs/latest/api/python/pyspark.sql.html#pyspark.sql.SparkSession.sparkContext
We can try to mimic this behaviour in pyspark 1.6 by proxying _sc or getOrCreate.

Available in spark 2.x out-of-the-box.


Added server-side node compatibility (take 2)

#67
Works undertaken

Upgraded grunt
Added umd wrapper and grunt code to produce a umd module
Shimmed window.location in node
Fixed a few scope issues, missing variable declarations and global scope creep
Remove text-encoding and replaced with a simpler polyfil
Added a url parser to distinguish between node and browser apis
Added a node shim for XMLHttpRequest
Changed test readme
Fixed old test for ADL.xapiutil
Added 1 new test for the minified file
Added intellisense support
Added source map support

Notes

This is my second attempt at adding node support apologies for the repeated prs.
I've tried to add node support with minimal disruption to existing file structures and the inferred utilisation patterns.
All files should be able to be used individually in the browser via script tags as before.
I've moved a large set of functions from the top of the xapiwrapper out of the global scope as they shouldn't be declared there.
I recommend not extending native Date object with Date.prototype.toISOString but instead defining a DateToISOString function.
I've removed the text-encoding module. This is as it adds support for multiple encoding types when I assume we'll realistically ever need utf8 encoding. For that extra support in internet explorer the module brings a lot of complexity across the node<->browser boundary. I've replaced it with a much simpler utf8 only polyfil from https://gist.github.com/Yaffle/5458286.

xapiwrapper.min.js

No longer bundles text-encoding.
Should now work in requirejs, commonjs and window.ADL global environments.
Doesn't include a working verson of ADL.launch in node
Has a sourcemap included

xapiwrapper.js

Is the main file for the node package and can be included via const ADL = require('xapiwrapper'); if published on npm.
Is not minified, contains all of the comments.

Intellisense
The functions which wrap each file stop intellisense from working. The wrapping functions are required if a developer is using the files individually in a browser such that the code in each file does not bleed into the global scope. The wrapping functions stop intellisense as they need to be executed by the intellisense interpreter for their modifications to the global ADL namespace to be produced (as far as I can tell). The grunt file now removes these wrapping functions when producing xapiwrapper.min.js and xapiwrapper.js. The grunt file removes these function wrappers as they are supplanted by the umd wrapper function. By removing these functions and producing a separate, unminified file containing all of the comments, intellisense works as it should.

We discussed this internally today and are floating the idea of a separate repo for a NodeJS-specific xAPI wrapper.  There's a limited functionality one that we use internally, but will let you know after we talk about it more tomorrow morning.
Also, I used your branch this morning and some of the response callbacks were getting dropped when stress testing.  Switching to an axios-driven internal wrapper resolved the issue, how are requests being handled for the Node version?

Superb.

how are requests being handled for the Node version?

There is a partial shim for the xmlhttprequest api https://github.com/adlnet/xAPIWrapper/pull/126/files#diff-1b2039be21a4454c52d6b497832996ebR1397 I'm not surprised it drops callbacks, but i'm sure it'd be quite easy to fix with a replacement as you said.
I think the biggest barriers to using this wrapper in node version the separation of the files, the wrapping functions, xapi-launch and the text-encoding library. Other than that it's pretty nice.

I really don't mind doing the work, I just need someone to help me figure out an order to submit the prs, a review process and what's possible with the current user requirements and the current state of the code.
Considering that the work required here is pretty straight forward, would it not make sense to use the same well tested wrapper for both environments rather than releasing an entirely new code base?

A new branch was created for this project.

@FlorianTolk what did you envisage for the NodeCompatibleWrapper branch?
I have series of PR's going into master to move this idea forward. Did you want me to re-base them to the other branch?
I could do with an answer to the question of browser support also: #135

This new branch is meant as a playground where most these new PRs can be directed at, as we do not want to make any major functionality changes to the main.
To quote @vbhayden 's response to another PR, the only major changes we want to push to the main branch are:

directly improve usability at no cost to performance / syntax,
simply correct typos / bugs,
preempt future browser deprecation issues with the popular browsers, or
improve the automated testing

The new branch was created to ingest any other changes members of the community would like to make to the xAPI Wrapper. Thanks!

Thanks for getting back to me.
This branch would need to be published to npm to provide standard node support. It'd either need to replace the master publication or be a separate version lineage.

Can someone comment on the status of this thread? There does not appear to be another branch called NodeCompatibleWrapper and the package published by @oliverfoster does not appear to be up to with his repo (the github page for it has disappeared too)

Well. My attempt failed. There was too much fear that dropping support for older browsers would somehow break the library. Then there was mention of a node version of the wrapper used internally and that it would be released instead of making this one compatible.
Nothing much happened after that and I gave up.

Thanks for the update. I'll assume that npm package is functionally the same but that your later commits were compatibility oriented


CD-357930 Add more info to procline

https://coupadev.atlassian.net/browse/CD-357930
This PR changes the procline string by:
[AnyBackgroundJobName] => [NayBackgroundJobName#12345]
while 12345 is data_source_id
and
[CoupaCron] => [CoupaCron#task_name#12345]
[BackgroundNotification] => [BackgroundNotification#task_name#12345]

I have tested this manually locally and at coupadev instance:

BackgroundValidationJob#12345 - <class>#<data_source_id> - for jobs except CoupaCron/BackgroundNotification

BackgroundNotification#send_notifications#12345 - <class>#<method_name>#<data_source_id> - for CoupaCron/BackgroundNotification


Twitter Icon Improve

In the Random Gita Shloka Genarator the Twitter Icon should be improved by dark blue Icon because it is not visible Correctly ,it should also need some Margin and Padding of 2 pixels

assigning u

Since it's not yet done, can it be assigned to me? @akshitagupta15june @karthiknadar1204

@kumarankit999 any update on the issue?

I am Working on it @karthiknadar1204

Please submit pr for review by day end, else the issue will be assigned to someone else
Thank you

I have created a PR

Link the pr with issue like #issue_number in the pr and include ss of the changes applied

I have updated there by 203 issue
https://github.com/akshitagupta15june/Moksh/pull/609


Fxing expression formatting/pretty print bug with reserved keywords

Earlier this week, Customer reached out to us about a bug when trying to format the expression like "Set(Children; true)" in the formula bar. "Children" is not a reserved keyword and unless the disabled reserved keyword flag is not set when invoking Lexer, "Children" gets identified as an ErrorToken. This causes errors in the AST and we don't format when there are errors and just return the same expression.
To solve this, we need to pass in a Flags.DisabledReservedKeyword flag to the parser and ultimately lexer when invoking them before the pretty printing. There was no ability to pass in custom flags so I extended the "Format" function (which is an entry point for pretty printing) with an optional argument to allow passing custom flags. These changes would be later consumed in Canvas Apps Backend (Would be passing in Flags.DisabledReservedKeyword | Flags.EnableExpressionChaining to Format in Canvas Apps Backend)
@gregli-msft Could you take a look at this and see if this is the right way to solve it

✅ No public API change.

✅ No public API change.

✅ No public API change.

✅ No public API change.


tmkmsv0.4.0 errored out after I stopped gaiad service and will not restart

I was successfully running tmkms 0.4.0 built with ledgertm support. When I stopped the gaiad service that tmkms was connected to locally I received an error that I was unable to recover from.  Here is the error WITH the backtrace results:
BACKTRACE=1
[admin@localhost .tmkms]$ tmkms start
14:23:31 [INFO] tmkms 0.4.0 starting up...
14:23:31 [INFO] [keyring:ledgertm:ledgertm] added validator key cosmosvalconspub1zcjduepq9t8j7hrvyjn7l6sw4ndhlmjt6xwy7dweqsmnxntd5ezkzsccm85q2ctjy6
thread '' panicked at 'index 33 out of range for slice of length 32', libcore/slice/mod.rs:2334:5
note: Some details are omitted, run with RUST_BACKTRACE=full for a verbose backtrace.
stack backtrace:
0: std::sys::unix::backtrace::tracing::imp::unwind_backtrace
at libstd/sys/unix/backtrace/tracing/gcc_s.rs:49
1: std::sys_common::backtrace::print
at libstd/sys_common/backtrace.rs:71
at libstd/sys_common/backtrace.rs:59
2: std::panicking::default_hook::{{closure}}
at libstd/panicking.rs:211
3: std::panicking::default_hook
at libstd/panicking.rs:227
4: std::panicking::rust_panic_with_hook
at libstd/panicking.rs:476
5: std::panicking::continue_panic_fmt
at libstd/panicking.rs:390
6: rust_begin_unwind
at libstd/panicking.rs:325
7: core::panicking::panic_fmt
at libcore/panicking.rs:77
8: core::slice::slice_index_len_fail
at libcore/slice/mod.rs:2334
9: <subtle_encoding::base64::Base64 as subtle_encoding::encoding::Encoding>::decode_to_slice
10: signatory::encoding::decode::Decode::decode_from_file
11: tmkms::client::client_loop
BACKTRACE=full
[admin@localhost .tmkms]$ tmkms start
14:28:30 [INFO] tmkms 0.4.0 starting up...
14:28:30 [INFO] [keyring:ledgertm:ledgertm] added validator key cosmosvalconspub1zcjduepq9t8j7hrvyjn7l6sw4ndhlmjt6xwy7dweqsmnxntd5ezkzsccm85q2ctjy6
thread '' panicked at 'index 33 out of range for slice of length 32', libcore/slice/mod.rs:2334:5
stack backtrace:
0:     0x557791162aef - std::sys::unix::backtrace::tracing::imp::unwind_backtrace::hab66bb8ff7088f29
at libstd/sys/unix/backtrace/tracing/gcc_s.rs:49
1:     0x55779116c327 - std::sys_common::backtrace::print::h56719ed580ab1bba
at libstd/sys_common/backtrace.rs:71
at libstd/sys_common/backtrace.rs:59
2:     0x557791166ebf - std::panicking::default_hook::{{closure}}::hf8d811a2c97e4b3a
at libstd/panicking.rs:211
3:     0x557791166c24 - std::panicking::default_hook::hc6e051251a5e6b4e
at libstd/panicking.rs:227
4:     0x55779116759e - std::panicking::rust_panic_with_hook::h71214e7ce0f7ac01
at libstd/panicking.rs:476
5:     0x557791167141 - std::panicking::continue_panic_fmt::ha8b8442f4ea9bcac
at libstd/panicking.rs:390
6:     0x557791167025 - rust_begin_unwind
at libstd/panicking.rs:325
7:     0x5577911ac71c - core::panicking::panic_fmt::h0c93626b89c38af6
at libcore/panicking.rs:77
8:     0x5577911a2c71 - core::slice::slice_index_len_fail::h2baf8f016a3596d7
at libcore/slice/mod.rs:2334
9:     0x557791110a47 - <subtle_encoding::base64::Base64 as subtle_encoding::encoding::Encoding>::decode_to_slice::hd99ba65579e92c44
10:     0x5577910ab715 - signatory::encoding::decode::Decode::decode_from_file::h7b3165beafd1a4f9
11:     0x5577910a7f6b - tmkms::client::client_loop::hb49ea8f02e0ca628
12:     0x55779108b647 - std::sys_common::backtrace::__rust_begin_short_backtrace::h58f56d954fc6f171
13:     0x557791094ed1 - std::panicking::try::do_call::h0f7a63942c8a1b30
14:     0x557791177559 - __rust_maybe_catch_panic
at libpanic_unwind/lib.rs:102
15:     0x5577910898f2 - <F as alloc::boxed::FnBox>::call_box::h7a0f8974d1998818
16:     0x557791164cbd - std::sys_common::thread::start_thread::h5213f803a61d7811
at liballoc/boxed.rs:682
at libstd/sys_common/thread.rs:24
17:     0x557791157f55 - std::sys::unix::thread::Thread::new::thread_start::he89121f566d2a8c7
at libstd/sys/unix/thread.rs:90
18:     0x7fcc31b4bdd4 - start_thread
19:     0x7fcc32269eac - __clone
20:                0x0 -

If you can reproduce this, it'd be great to get the backtrace from a debug build.
It looks like it's crashing inside of subtle-encoding while trying to decode your secret_connection.key, ostensibly around here https://github.com/tendermint/kms/blob/master/src/client.rs#L156

So essentially I will be running:
export BACKTRACE=full
cargo install tmkms --features=ledgertm --force
?

You'll need to do cargo install tmkms --features=ledgertm --force --debug to get a debug build. It defaults to release.

Ok, so I could recover from it by re generating the secret_connection.key
I backed up the old key and it will fail if I have it restored. Seems key file may just be corrupt?

Yes, there's definitely a bug in the subtle-encoding Base64 parser.
If the key is no longer valuable, you can send it to me and I can debug from there. Otherwise it seems like I should fuzz that code.

Hey @tarcieri here is the bad secret file. It is no longer valuable.
secret_connection.zip

It looks like this was caused by the handling (or lack thereof) of trailing whitespace in the Base64 decoder (the KMS uses a constant-time Base64 decoder for this as it's decoding a private key).
PR to fix it in an upstream crate here:
https://github.com/iqlusioninc/crates/pull/163/files

This should at least error out informing you of the trailing whitespace as of this release: https://github.com/iqlusioninc/crates/pull/164
I will eventually make the parser whitespace tolerant, but for now there are any number of ways to strip trailing whitespace from the key yourself.


Deploy reload doesn't seem to work - IOS

http://docs.ionic.io/services/deploy/#reload-the-app
The update itself works. Manually restarting the app by the user gets the update applied.
Cordova CLI: 6.4.0
Gulp version:  CLI version 3.9.0
Gulp local:   Local version 3.9.1
Ionic Framework Version: 2.0.0-rc.1
Ionic CLI Version: 2.1.4
Ionic App Lib Version: 2.1.2
Ionic App Scripts Version: 0.0.36
ios-deploy version: 1.8.3
ios-sim version: 5.0.4
OS: Mac OS X Sierra
Node Version: v6.9.1
Xcode version: Xcode 8.0 Build version 8A218a

My code:
  doUpdate() {
    this.checkUpdateStatus = 'Updating...';
    this.deploy.download().then(() => {
      return this.deploy.extract();
    }).then(() => {
      this.checkUpdateStatus = 'Update Complete!';
      return this.deploy.load();
    }).catch(() => {
      this.checkUpdateStatus = 'Update Failed';
    });
  }

:worried:

Hi @sagivf,
Can you please confirm that on .load method call cordova.js is not found? I have such problem with everything up-to-date and the App needs to be completely reloaded on the native level (which AFAIK can't be done in the code), location.reload won't work.
Thanks

@durasj sorry for the delayed response.
.load is typeof function so that's ok.
Not sure what's going on. It either doesn't reload. Or reloads without the new version.
Currently the app user needs to restart the app.

@sagivf it was probably not clear enough what I was trying to say, sorry.
I meant that after the .load function is called, error message "cordova.js is not found" appears in my case. After any number of reloads via web api, the new version won't be loaded. I wanted to know whether it's same for you so it could potentially help guys on Ionic team.

@durasj no problem 👍
I'm not getting that error. Not sure I understand what you mean by reload via web api do any other Ionic cloud services work for you? Are you able to use this.deploy.check() for instance?

@sagivf sorry for not being more clear. I was talking about traditional Web API Location.reload() which I also tried after using Deploy.load(). Other services are fully working for me.

Cool, got ya. I'll try to check again but I don't think I'm getting that error though we seem to be having the same issue. Ionic Team?

@sagivf Does checkUpdateStatus actually change to "Update Failed"?

@dwieeb No, It just doesn't do anything. I think it might have reloaded once, but the update didn't apply. Just to be clear - manually restarting the app by the user gets the update applied.

@dwieeb same here, it looks like the update succeeded, but I also have the cordova.js not found right after calling .load() or manually using Location.reload().

Hello guys,
I've exactly same issue. We need to release the application to production but our updating mechanism still cannot reload the application.
Ionic team, please help!

:worried:

Hey everyone.  I've tried unsuccessfully to reproduce this issue locally on an iPhone, and the service appears to be running as intended on my end.
When looking at the native xcode logs, are there any errors or anything unusual jumping out at you?

@Fuiste I'll try to check it out, but I'm not sure it's relevant if there is no solution for this - #111

Hi @Fuiste. In my case, it looks like #99 @ ionic-plugin-deploy is the possible problem - wkwebview. So I guess it should be easily reproducible using wkwebview, but I guess ionic-cloud client is therefore not responsible.

@Fuiste  @durasj I am also using wkwebview, so theres a good chance that is the issue here.

@sagivf and @durasj yup, it's an issue with WkWebView that we were recently able to identify.  We're testing a couple different solutions at present, and hope to have a new release of the plugin as soon as possible.

Any news on reloading WKWebview? Running into the same issue with Ionic Deploy.

See https://github.com/ionic-team/ionic-plugin-deploy/issues/99#issuecomment-304952231 for a possible solution (uses fork of cordova-plugin-wkwebview-engine)

Hi @Fuiste, could you please confirm that one of the solutions that you mentioned you're preparing is in the ionic-plugin-deploy changes? If so, do you please plan to include it in normal tagged release?


Remove redundant run methods in SolidQueue::Processes

This commit removes one of the redundant run methods in SolidQueue::Processes
Here are the warning messages appeared at Rails Nightly CI: https://buildkite.com/rails/rails-nightly/builds/1091#019244bc-9d36-476d-8924-5e2bea4cb728/1173-1176
/usr/local/lib/ruby/gems/3.4.0+0/gems/solid_queue-1.0.0/lib/solid_queue/processes/runnable.rb:52: warning: method redefined; discarding old run
/usr/local/lib/ruby/gems/3.4.0+0/gems/solid_queue-1.0.0/lib/solid_queue/processes/runnable.rb:44: warning: previous definition of run was here

The reason why this pull request removes this one is https://github.com/rails/solid_queue/pull/337 added the run method later.

Thanks a lot @yahonda! 🙏  And thanks a lot to @byroot for merging this! I've been on-call the past week and it's been crazy busy so I hadn't had the time to tend to this repo 🙈 😳


Update devDependencies

Trying a new style of commit messages and branch name, to fit the mostly-automated releases Github Actions I've been working on.

Only devDependencies changed, it doesn't need to release a new version of the package.


Use Griffe's public API

You're importing from the internal API:
https://github.com/holoviz/lumen/blob/82df710106169b22b05b3c2ac5315d30ddd4317e/lumen/ai/translate.py#L17-L18
...exposing yourself to breakages.
Public equivalent:
from griffe import DocstringSectionKind, Docstring

Thank you for spotting that. Fixed in https://github.com/holoviz/lumen/pull/898

Nice, thanks for the quick fix!


Feature/System Modules

System Modules
This Pull Request includes the following features and bugfix:

Edit and Retrieval of System Information
Edit and Retrieval of Contact Information
Users to submit contact request (reach out the devs)
Users to submit system review (can be unauthorized for anonymous review)
Retrieve System Reviews (admin only)
Retrieve submitted contact request (admin only)
Fixed the token expire issue error, just added an exception handling to the JWTHelper class

Note: I have a temporary email credentials exposed in the appsettings.json, I will be removing it once the testing is done

thank you team


Added handling of qupath segmap and huggingface timeouts

What is the purpose of this PR?

Handling qupath saved RGB segmentation maps by converting them back to single-channel label images with the following formula label = RED * 256 * 256 + GREEN * 256 + BLUE to decrease friction for users according to #18 .
Always checking local and huggingface model versions requires internet access whenever one wants to run nimbus inference. This is not a good design I think, thus I added some exception handling workflow so that you can run nimbus without internet access if a local checkpoint is available.

How did you implement your changes

Added function handle_qupath_segmentation_map to utils that converts RGB segmentation maps to single channel ones, this function only gets called in the Dataset class if len(instance_mask.shape) == 3 and instance_mask.shape[-1] == 3.
Added a try / except clause to the huggingface version lookup routine in the Nimbus class.

Remaining issues
None

Tested it again and it threw errors. I debugged it and it should work properly now. It's also possible to write tests for this with VCRpy, I'll look into this tomorrow to see how much effort it would be.

Got it, most important is just that you make sure it works with your own testing. If it's easy to automate, sounds good, but no problem if not.

I found an easy way to automate it. pytest-socket let's me enable and disable the network connection. I also lowered the data structure for the segmentation map from uint64 (allowing for 18.410^19 individual cells) to uint32 (allowing for 4.310^9 or 4.3 billion individual cells). My guess is that WSIs won't have more than 20 million cells, even 2cm x 2cm specimen WSIs shouldn't be over say 80 millions, so uint32 should be more than enough and we save memory.


License clarification

Could you clarify the licenses under which the data and the code are released?
Adding a LICENSES file to the repository and comments on the README would be very welcome.
Thanks.

Hi, thanks for opening this issue.
For data: the license information is included in each version's metadata section.
For code: we updated the GitHub page and add the Apache licence for the code.