Ignore isn't working the way I expect it to

I have set up paper_trail to ignore all changes except a Paperclip attachment, so that any of the metadata can change (title, description) without forcing a new version, but if a new version of the image is uploaded, the new version is created. No matter what I do, I can't seem to get it to ignore non-image changes. Could this be related to the fact that I also have a meta key on the paper trail?
  has_paper_trail ignore: [:user_id, :project_id, :title, :description, :approval_status, :genre, :created_at,
    :updated_at, :key, :version_number], :meta => {:version_number => :version_number, :key => :key}

Any suggestions about this?

What versions of PaperTrail and ActiveRecord are you using?

PaperTrail 3.0.6 and ActiveRecord 4.1.8.
Walter
On Jan 28, 2015, at 12:36 AM, Ben Atkins<EMAIL_ADDRESS>wrote:

What versions of PaperTrail and ActiveRecord are you using?
—
Reply to this email directly or view it on GitHub.

So what exactly is the issue?  It is generating new versions when you don't expect it to?

Yes, it makes a new version every time I save, even when the only thing (I think) that should trigger a version is me uploading a different image to the model. I change the name or another of the ignored attributes, I get a new version.

Can you try taking :created_at and :updated_at out of your ignored attributes list and seeing if that fixes the issue? It shouldn't be necessary to have those.

They were added after I noticed the problem. The only thing I can think may be affecting this is the version_number method, which counts the number of versions and adds 1. When you choose to ignore a version, does it exist? In the life-cycle of the object, if you make (ignored) changes to the object, then call versions.length (before you save) does the version count increment before those ignored fields are discounted? Or does the ignore code simply avoid counting those attributes in its calculation of what version we are currently in before anything else is considered? I guess I need to read the source and see how you figure these out.

Apologies for delayed response, but if you can provide a dummy app or template, I can probably give you a hand debugging this better.
But no, a version doesn't get built until after the fact of the update, and it will only start being built if the object has "changed notably", in other words, if an attribute that is not being ignored/skipped has changed.
And you would only need to ignore the timestamp attributes if you are making explicit calls to update those attributes, and those only, like this:
ModelKlass.create!(created_at: Time.now)
# or
model_instance.update_attributes!(updated_at: Time.now)

Okay, I will go back through my branches and find a version that still has this on it (I had to move ahead, and dug back in time to vestal_versions to get things working for my client). I'll try to put something together for you tonight or tomorrow.

Closing this for now since I never heard back from you @walterdavis, if you do end up digging up your project and you can provide a bit more information, feel free to ping me/us again and we'll see if we can fix your problems.


Building on JDK9 fails

A number of issues have beed detected related to building on JDK9

No longer relevant.


Result set on Functional Scores should be ordered according to Date Modified

The table should be sorted in reverse chronological order according to Date Modified/Signed column.

Tested and closed. @lpeyton


Coreutils Missing Bitcode

It is strange that the Coreutils [ program, for example, has no bitcode, I think we need to figure out why and ask klee-dev about it if we could not obtain the bitcode file ourselves.

Thanks @feliciahalim .


Completed objects assessment

5 / 5
Good job, Charlie. Would you say that you generally feel good about this stuff?


Add RevisionQuery support to Sync for Advanced Test Panel

What does this Pull Request accomplish?
Add RevisionQuery support to Sync for Advanced Test Panel. I based this off of the existing RevisionQuery method that Scope has implemented.
Why should this Pull Request be merged?
So that the Sync Advanced Test Panel Works
What testing has been done?
New test passes:
admin@NI-PXIe-8861-031B3401:~# ./SystemTestsRunner --gtest_filter="*RevisionQuery*"
Note: Google Test filter = *RevisionQuery*
[==========] Running 1 test from 1 test suite.
[----------] Global test environment set-up.
[----------] 1 test from NiSyncDriverApiTest
[ RUN      ] NiSyncDriverApiTest.RevisionQuery
[       OK ] NiSyncDriverApiTest.RevisionQuery (13 ms)
[----------] 1 test from NiSyncDriverApiTest (13 ms total)

[----------] Global test environment tear-down
[==========] 1 test from 1 test suite ran. (13 ms total)
[  PASSED  ] 1 test.

I need to merge and resolve conflicts with https://github.com/ni/grpc-device/commit/7e171ac3fee5a1151bfba0090eb05703ac502042

I need to merge and resolve conflicts with 7e171ac

Done!


Copy journal files when creating new masters in integration test

Cherry-picked 60f14a3 to branch-2.0
Currently in some integration tests we create a master replica from
journal directory to check journal status. This can cause problems if
the original master is not stopped when the new master is created. This
change solves the issue by copying the journal to a separate directory
and creating the new master based on the copy.
pr-link: Alluxio/alluxio#9778
change-id: cid-ea6e0d655899da607950833c6e0840ba4f3a73ec

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/5262/
Test PASSed.

im closing this to try the cherry-pick feature from the bot


mono camera - *(sensor) calibration - suggestions for external calibration tools.

Hi everyone,
We have had several issues with the camera-camera and camera-lidar calibration tools provided on the Github page. I would really like to move away from this for extrinsic calibration until there is an update.
Any suggestions on open source tools (ROS or otherwise) for mono camera-camera and mono camera - lidar calibrations?
Thanks,
Sneha

Hey Sneha,
Would you please kindly to report those issues regarding calibration tools or get involved in the development is more than welcome and appreciated!
Since I said that, I'd like to help from the technical point of view.
Thanks,
-Jeff

Hey Jeff,
Please see #4732, #4241 and #4511. We have been trying other calibration tools, but trying to land on a good pattern that works simultaneously for the 6mm and 25mm camera setup in Apollo.
Thanks,
Sneha


Fixed GameJson analysis field

Description
This pull request updates the GameJson struct for exporting games by user. This change fixes the analysis field, which is supposed to be Array of objects (GameMoveAnalysis) as stated in the Lichess API.
Changes Made

Modified the GameJson.analysis field in src/model/games/mod.rs to Option<Vec<GameMoveAnalysis>>.
Added missing data to the test file tests/data/response/game_json.json.

Looks like an oversight on my recent change - thanks for the fix 🙂


Add Kafka consumer group id set by Quickwit in the docs

We should document the Kafka group.id set by Quickwit is quickwit-{index-id}-{source-id}.
(we can also make it possible to configure but we can do that later)

Closed via #2248.

Nice!


Support nonlinear problems?

Hi, I tried to test the exponential cone constraint feature in Mosek 9.1.2 with a simple example: min exp(x) s.t. x>=0 and I use the equivalent formulation: min t s.t. t>=exp(x), x>=0. But I got the error:"The solver does not support nonlinear problems (i.e., NLobjective and NLconstraint)." The code I used is the following:
using JuMP, LinearAlgebra, MosekTools
M = Model(with_optimizer(Mosek.Optimizer))
@variable(M, x)
@variable(M, t)
@NLconstraint(M, t-exp(x)>=0)
@constraint(M, x>=0)
optimize!(M)
Could you please suggest a way out of this? The test was performed in Julia 1.1.1 with JuMP v0.20.1, MosekTools v0.9.3
Thanks!

You need to use the conic form:
using JuMP, MosekTools
model = Model(Mosek.Optimizer)
@variable(model, x >= 0)
@variable(model, t)
@constraint(model, [t, 1, x] in MOI.ExponentialCone())
optimize!(model)

Here is a list of cones that you can use: https://www.juliaopt.org/MathOptInterface.jl/stable/apireference/#MathOptInterface.ExponentialCone

You need to use the conic form:
using JuMP, MosekTools
model = Model(Mosek.Optimizer)
@variable(model, x >= 0)
@variable(model, t)
@constraint(model, [t, 1, x] in MOI.ExponentialCone())
optimize!(model)

Here is a list of cones that you can use: https://www.juliaopt.org/MathOptInterface.jl/stable/apireference/#MathOptInterface.ExponentialCone

Thank you! It works. Do you know if we can do something similar with JuMP v0.18.6?  I tried your code and I got "UndefVarError: MOI not defined".

No. You cannot do this in JuMP 0.18.6. Better support for conic sets was one of the primary motivations for rewriting JuMP.

No. You cannot do this in JuMP 0.18.6. Better support for conic sets was one of the primary motivations for rewriting JuMP.

Thank you! I'm bit confused because I do see MathProgBase support Exponential cone "https://mathprogbasejl.readthedocs.io/en/latest/conic.html". I just don't know the correct syntax to use it with JuMP 0.18.6.

I just don't know the correct syntax to use it with JuMP 0.18.6.

As @odow said, there is no syntax to use the exponential cone in JuMP 0.18.6. It was defined in MathProgBase but not exposed through JuMP.


NestedScrollView physics not working

Steps to Reproduce

Execute flutter run on the code sample
Just scroll the view

Expected results:
Scroll should not work in this example
Actual results:
User can scroll. We lost the ability to have custom physics like Clamping or Bouncing.

Code sample

import 'package:flutter/material.dart';

void main() {
  runApp(const MaterialApp(home: Home()));
}

class Home extends StatelessWidget {
  const Home({super.key});

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      backgroundColor: Colors.white,
      body: NestedScrollView(
        physics: const NeverScrollableScrollPhysics(),
        headerSliverBuilder: (BuildContext context, bool innerBoxIsScrolled) => <Widget>[
          SliverOverlapAbsorber(
            handle: NestedScrollView.sliverOverlapAbsorberHandleFor(context),
            sliver: const SliverAppBar(
              pinned: true,
              elevation: 0,
              backgroundColor: Colors.blue,
              toolbarHeight: kToolbarHeight,
              expandedHeight: 200,
              flexibleSpace: FlexibleSpaceBar(
                title: Text("Physics not working"),
              ),
            ),
          ),
        ],
        body: Builder(
          builder: (BuildContext context) => CustomScrollView(
            physics: const NeverScrollableScrollPhysics(),
            slivers: <Widget>[
              SliverOverlapInjector(
                handle: NestedScrollView.sliverOverlapAbsorberHandleFor(context),
              ),
              SliverList(
                delegate: SliverChildBuilderDelegate(
                  (context, index) => Card(
                    child: ListTile(
                      leading: const Icon(Icons.search),
                      title: Text(index.toString()),
                    ),
                  ),
                  childCount: 200,
                ),
              ),
            ],
          ),
        ),
      ),
    );
  }
}

Logs

[✓] Flutter (Channel stable, 3.3.4, on macOS 12.6 21G115 darwin-arm, locale fr-FR)
    • Flutter version 3.3.4 on channel stable at /Users/earminjon/fvm/versions/3.3.4
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision eb6d86ee27 (2 weeks ago), 2022-10-04 22:31:45 -0700
    • Engine revision c08d7d5efc
    • Dart version 2.18.2
    • DevTools version 2.15.0

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/earminjon/Library/Android/Sdk
    • Platform android-33, build-tools 33.0.0
    • ANDROID_HOME = /Users/earminjon/Library/Android/Sdk
    • Java binary at: /Users/earminjon/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/213.7172.25.2113.9123335/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14A400
    • CocoaPods version 1.11.3

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

[✓] Android Studio (version 2021.3)
    • Android Studio at /Users/earminjon/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/213.7172.25.2113.9123335/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

[✓] IntelliJ IDEA Ultimate Edition (version 2022.2.3)
    • IntelliJ at /Users/earminjon/Applications/JetBrains Toolbox/IntelliJ IDEA Ultimate.app
    • Flutter plugin version 70.2.5
    • Dart plugin version 222.4345.14

[✓] IntelliJ IDEA Ultimate Edition (version 2022.2.3)
    • IntelliJ at /Users/earminjon/Library/Application Support/JetBrains/Toolbox/apps/IDEA-U/ch-0/222.4345.14/IntelliJ IDEA.app
    • 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

[✓] Connected device (3 available)
    • sdk gphone64 arm64 (mobile) • emulator-5554 • android-arm64  • Android 13 (API 33) (emulator)
    • macOS (desktop)             • macos         • darwin-arm64   • macOS 12.6 21G115 darwin-arm
    • Chrome (web)                • chrome        • web-javascript • Google Chrome 106.0.5249.119

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

Triage report
I can reproduce this issue on Master (3.5.0-9.0.pre.32)

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

void main() {
  runApp(const MaterialApp(home: Home()));
}

class Home extends StatelessWidget {
  const Home({super.key});

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      backgroundColor: Colors.white,
      body: NestedScrollView(
        physics: const NeverScrollableScrollPhysics(),
        headerSliverBuilder: (BuildContext context, bool innerBoxIsScrolled) =>
            <Widget>[
          SliverAppBar(
            pinned: true,
            elevation: 0,
            backgroundColor: Colors.blue,
            toolbarHeight: kToolbarHeight,
            expandedHeight: 200,
            flexibleSpace: FlexibleSpaceBar(
              title: Text("Physics not working"),
            ),
          ),
        ],
        body: CustomScrollView(
          physics: const NeverScrollableScrollPhysics(),
          slivers: <Widget>[
            SliverList(
              delegate: SliverChildBuilderDelegate(
                (context, index) => Card(
                  child: ListTile(
                    leading: const Icon(Icons.search),
                    title: Text(index.toString()),
                  ),
                ),
                childCount: 200,
              ),
            ),
          ],
        ),
      ),
    );
  }
}

yes, i also have this problem.

@Piinks

I have the same problem, Has anyone solved this problem?

I thought this might be related to https://github.com/flutter/flutter/issues/45619, but this seems separate from that to me.

It looks like we apply clamping scroll physics to whatever physics is already supplied:
https://github.com/flutter/flutter/blob/34d2c8d0302cc07ea48dac204de9f328e4a75d6a/packages/flutter/lib/src/widgets/nested_scroll_view.dart#L437
@xu-baolin has been looking at a similar issue where DraggableScrollableSheet automatically applies AlwaysScrollableScrollPhysics (#121021 | #123109)


Attend Page shows Absences

Hey. So I wrote some code to show the user's missing attendances on the attend page. I'm new to this stuff so feedback is welcome. William made the absences show up red, but the UI is up for debate of course. Also, I had to change the select property to true on the attendance code for the smallgroup schema because the code is the only way to compare with the user's attendance objects. Is that a security vulnerability or something? I saw that it was just recently changed to not be included in the queries. Issue #513

Hey looks great cowang4! I'll give feedback shortly! Sorry for the delay. Super appreciative of the work!

That was a security issue where the current attendance code was being sent out and one could simply load the list of all codes and get the current day's code. This could be addressed differently if you need it for comparison. Maybe we should have another identifier that isn't the code, or just override the code if it is for the current day so the user can't see the most recent code.

Oh okay, I figured it might be something like that. Overriding it should be simple and easy. Or maybe we can remove all the codes from the objects before they get returned. But are there other functions that read from that database? Some other identifier would probably be cleaner in the long run.

I think it would best to do it by date, small group / large group & bonus/non-bonus.  That would allow students who switched small groups to not show a marked-absent for any days they attended in the old group.  Also, if you need the small group code, its best to leave the select:false and pull the codes just for your endpoint (see https://github.com/rcos/Observatory3/blob/master/server/api/attendance/attendance.controller.js#L371)

Okay, I changed the smallgroup comparison to be date and bonusDay based. Does that make more sense?

I think the group and bonus day check is fine. We don't save the attendance object if it's the wrong code as far as I know.

Okay. How about now?

Looks good except for one tiny change. We are sorting by datetime and you don't initialize that variable. We can either change the sort to be just date or add a datetime parameter to the codes you are adding. I'm not sure why we have date and datetime as two unique fields. Kinda ugly but once you change that it looks great. Tested it and works well! Will merge after the fix, would have done it myself but i don't have push rights to your branch

the date vs datetime difference is kinda ugly, not your design!

Yeah it displays date but sorts by datetime... pretty strange. I'll change it to sort by date.


Total ammo weapon specific weight adjustment

Sliders for each weapon category showing how much of the total % it would be. For example: primary 40%, secondary 30%, grenades 20%.
"overly complex" is an understatement

The experimental version is hidden on mod.io currently, if you want admin on it let me know.


deep merge dicts

# Source: https://stackoverflow.com/questions/7204805/how-to-merge-dictionaries-of-dictionaries/7205107#7205107
def merge(a, b, path=None):
    "merges b into a"
    if path is None: path = []
    for key in b:
        if key in a:
            if isinstance(a[key], dict) and isinstance(b[key], dict):
                merge(a[key], b[key], path + [str(key)])
            elif a[key] == b[key]:
                pass # same leaf value
            else:
                raise Exception('Conflict at %s' % '.'.join(path + [str(key)]))
        else:
            a[key] = b[key]
    return a

# works
print(merge({1:{"a":"A"},2:{"b":"B"}}, {2:{"c":"C"},3:{"d":"D"}}))
# has conflict
merge({1:{"a":"A"},2:{"b":"B"}}, {1:{"a":"A"},2:{"b":"C"}})

# Source: https://stackoverflow.com/questions/7204805/how-to-merge-dictionaries-of-dictionaries/7205107#7205107
def merge(a, b, path=None, overwrite=False):
	"merges b into a"
	if path is None: path = []
	for key in b:
		if key in a:
			if isinstance(a[key], dict) and isinstance(b[key], dict):
				merge(a[key], b[key], path + [str(key)])
			elif a[key] == b[key]:
				pass # same leaf value
			else:
				if overwrite:
					a[key] = b[key]
				else:
					raise Exception('Conflict at %s' % '.'.join(path + [str(key)]))
		else:
			a[key] = b[key]
	return a


fix could not find mice/mouse dropdown title

currently shows: could not find 1 mice
should be 1 mouse (not mice)

fixed


Bridge for Arduino IDE

Hi, I'm a beginner and I was looking for examples for a Pico USB-UART Bridge using the Arduino IDE.
Do you have any advice about it?

AFAIK, most of the code here can be reused for replicating the project in Arduino IDE.
However, this isn't the intention of this project, so I'm closing this issue.


Feature Request: IWR: Add "Vulnerability" RE

This would be useful for handling scarecophagus "Vulnerable to Positive Energy".
https://2e.aonprd.com/Monsters.aspx?ID=1953

Thinking about whether this would be generally useful or if we could get away with an actor property that gets checked similar to how negative healing is handled.


Exclude hydrogens from being mapped

In order to allow us to run perses with bonds to hydrogen constrained, we are forced to exclude hydrogens from mapped atoms so that they are always unique. Otherwise, we would not be able to permit constrained H-X bonds of different lengths in initial and final states.

Currently being addressed in #398

This wasn't actually fully addressed.

From a slack discussion with @jchodera, he suggested that we could set the atom mapping criteria in a way that doesn't allow ring breakage, and also doesn't map hydrogens.
DEFAULT_ATOM_EXPRESSION = oechem.OEExprOpts_Aromaticity | oechem.OEExprOpts_RingMember | oechem.OEExprOpts_Degree | oechem.OEExprOpts_AtomicNumber | oechem.OEExprOpts_EqAromatic | oechem.OEExprOpts_EqHalogen | oechem.OEExprOpts_EqCAliphaticONS

@jchodera adds:

oechem.OEExprOpts_AtomicNumber won't map one element onto another, BUT:
oechem.OEExprOpts_EqAromatic lets you change atoms within aromatic heterocycles as long as they are still aromatic (so no benzene to cyclohexane)
oechem.OEExprOpts_EqHalogen treats all halogens as equivalent, and
oechem.OEExprOpts_EqCAliphaticONS treats aliphatic atoms we care about as equivalent

Longer term, the correct fix is to annotate the bonds for molecules going into _get_mol_atom_map to label them as constrained or unconstrained, then use that information to not map bonds that change constraint.
But it will take more refactoring to make that longer-term change

We could use the user-defined IntType for labeling constrained bonds as 1 and unconstrained bonds as 0 and the OEExprOpts_IntType for checking they are the same:
https://docs.eyesopen.com/toolkits/python/oechemtk/OEChemConstants/OEExprOpts.html#OEChem::OEExprOpts::IntType

I believe that #441 fixes this

Let's keep this open so we can implement the second solution later:

Longer term, the correct fix is to annotate the bonds for molecules going into _get_mol_atom_map to label them as constrained or unconstrained, then use that information to not map bonds that change constraint.
But it will take more refactoring to make that longer-term change
We could use the user-defined IntType for labeling constrained bonds as 1 and unconstrained bonds as 0 and the OEExprOpts_IntType for checking they are the same:
https://docs.eyesopen.com/toolkits/python/oechemtk/OEChemConstants/OEExprOpts.html#OEChem::OEExprOpts::IntType

We are sorted on how we map/don't map hydrogens


Unconfirmed Bug: The example setup_environment_tutorial.ipynb breaks when ran

Error tracing
IndexError                                Traceback (most recent call last)
 in
3 env = TradingEnvironment(portfolio=portfolio,
4     action_scheme='managed-risk',
----> 5     reward_scheme='simple')
E:\anaconda\envs\STA\lib\site-packages\tensortrade\environments\trading_environment.py in init(self, portfolio, action_scheme, reward_scheme, feed, window_size, use_internal, **kwargs)
86         logging.getLogger('tensorflow').disabled = kwargs.get('disable_tensorflow_logger', True)
87
---> 88         self.compile()
89
90     def compile(self):
E:\anaconda\envs\STA\lib\site-packages\tensortrade\environments\trading_environment.py in compile(self)
104         self.feed = self.feed + create_internal_feed(self.portfolio)
105
--> 106         initial_obs = self.feed.next()
107         n_features = len(initial_obs.keys()) if self.use_internal else len(self._external_keys)
108
E:\anaconda\envs\STA\lib\site-packages\tensortrade\data\stream\feed.py in next(self)
83
84         for node in self.inputs:
---> 85             self._next(node)
86
87         feed_data = {name: self._data[name] for name in self.names}
E:\anaconda\envs\STA\lib\site-packages\tensortrade\data\stream\feed.py in _next(self, node)
77             for output_node in node.outbound:
78
---> 79                 self._next(output_node)
80
81     def next(self):
E:\anaconda\envs\STA\lib\site-packages\tensortrade\data\stream\feed.py in _next(self, node)
77             for output_node in node.outbound:
78
---> 79                 self._next(output_node)
80
81     def next(self):
E:\anaconda\envs\STA\lib\site-packages\tensortrade\data\stream\feed.py in _next(self, node)
71
72     def _next(self, node: Node):
---> 73         outbound_data = node.next()
74         if outbound_data:
75             self._data.update(outbound_data)
E:\anaconda\envs\STA\lib\site-packages\tensortrade\data\stream\node.py in next(self)
111         self._call_count = 0
112
--> 113         data = {self.name: self.forward(self._inbound_data)}
114         outbound_data = _flatten(data, sep=self._sep) if self.flatten else data
115         self.propagate(outbound_data)
E:\anaconda\envs\STA\lib\site-packages\tensortrade\data\stream\transform.py in forward(self, inbound_data)
79     def forward(self, inbound_data):
80         if not self.key:
---> 81             self.key = list(filter(self.selector, inbound_data.keys()))[0]
82             self.name = self.key
83         return inbound_data[self.key]
IndexError: list index out of range
Describe the current behavior
self.key = list(filter(self.selector, inbound_data.keys()))[0] will cause index error when the list created is empty, especially happening in train_and_evaluate.ipynb. Further found that this empty list happens when using create_internal_feed(portfolio) in the TradingEnvironment.complie() method. Actually the third node of the created feed is <Node: name=free>, which will trigger the bug above.
Also I feel confused that the complie method will create an internal feed for initializing the env no matter the argument for 'use_internal' is True or False.
Describe the expected behavior
First, the bug in the create_internal_feed() should be fixed. Second, the internal_feed method should not be implemented in complie() method when the trading environment is established.
Code to reproduce the issue
from tensortrade.wallets import Wallet, Portfolio
""
portfolio = Portfolio(USD, [
Wallet(coinbase, 10000 * USD),
Wallet(coinbase, 10 * BTC),
Wallet(coinbase, 5 * ETH),
Wallet(bitstamp, 1000 * USD),
Wallet(bitstamp, 5 * BTC),
Wallet(bitstamp, 3 * LTC),
])
from tensortrade.environments import TradingEnvironment
env = TradingEnvironment(
feed=feed,
portfolio=portfolio,
action_scheme='managed-risk',
reward_scheme='simple',
window_size=15,
enable_logger=False
)
""
This is the codes(part) in the example. I have implemented some experiments, using the following codes:
""
from tensortrade.data.internal.helpers import create_internal_feed
from tensortrade.instruments import USD, BTC, ETH, LTC
from tensortrade.wallets import Wallet, Portfolio
portfolio = Portfolio(USD, [
Wallet(coinbase, 10000 * USD),
Wallet(coinbase, 10 * BTC),
Wallet(coinbase, 5 * ETH),
Wallet(bitstamp, 1000 * USD),
Wallet(bitstamp, 5 * BTC),
Wallet(bitstamp, 3 * LTC),
])
new_feed=create_internal_feed(portfolio)
for i in range(len(new_feed.inputs)):
print(i)
new_feed._next(new_feed.inputs[i])
""
The above codes will trigger the same bug when i=2(i.e., the third node in new_feed.inputs.
Other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

The example is currently working with the latest version of the repo. Make sure you have installed from the Github source:
pip install git+https://github.com/notadamking/tensortrade.git -U


Pond docs are not available for translation

Can anybody make Pond documentation available for translation at TranslateWiki? I've made my own portuguese translation by editing the generated.js file btw.

I would like to make a translational version in hindi. Please tell if that would be a great idea?


Question : how many simplecoin we have?

I'm trying to making my own cryptocurrency with learning this project.
I have  a question : how many simplecoin we have and how can I change to for example 120,000,000,000

ok so you dont want to use this to create ur currency use something else
I can give you a generator if you want

https://github.com/sandoche/Proof-of-Stake-Cryptocurrency-generator is a repository for creating a coin it self that will actually run this code here isn't safe and can e broken easily like said in the video its not recommended to start a actual coin on this

looks like we have infinite coin fot now

Exactly, this project has the bare minimun things to get an idea of how things work. So there is no way to create coins out of nowhere (at least not without exploiting one of the missing features).

Tankye

vriy good

@Ariya383323 can i tag along with the coin creation
?
I've tried it myself Im a NOOB


HTML comments are included in p tag

When using
var tokens = marked.lexer(text, options)
marked.parser(tokens)

HTML comments will get included into a p tag and so be visible in the output!

Is there any update on this one?
Testing with:
<!-- Test Comment -->

Result:
<p><!-- Test Comment -->
</p>

Bump?

Bump?

+1 :+1:

+1

#985


Is it safe to delete Azure app SharePointPnP.ProvisioningApp.Tenant?

What is the context of your question?

[X] Installation
[ ] Administration
[ ] Custom Content
[ ] M365 learning pathways web part

What is your question?

Can the Azure enterprise application SharePointPnP.ProvisioningApp.Tenant be safely deleted after initial provisioning?

Does the Microsoft Learning Template use the Azure app in any way post installation? My understanding is it does NOT, but I just want to double-check.

Provide related screen-shots to help us understand your question
Azure administrator recognized that the app has a lot of permissions, and understandably does not want unnecessary apps in the system.

Yes, you can delete the enterprise application and no the template doesn't use the azure app in any way after installation.


Still trying to dyld a dependency?

Cocoapods 1.2.0
Amimono 0.0.7
I've added to my podfile
plugin 'cocoapods-amimono'
and
post_install do |installer|
    require 'cocoapods-amimono/patcher'
    Amimono::Patcher.patch!(installer)
end

When I run my application, though, I'm getting a crash during the dyld step.  The library referred is one that is internal, and I would expect would have no issue loading the object files into the executable.
dyld: Library not loaded: @rpath/WxIcons.framework/WxIcons
  Referenced from: /Users/xxxxx/Library/Developer/CoreSimulator/Devices/A8223266-E751-4255-9CCE-7C091F47C9F7/data/Containers/Bundle/Application/508E1A23-04A6-4827-8F6C-242C9012F786/xxxxx.app/xxxxx
  Reason: image not found

When I look in the [Amimono] Create filelist per architecture build phase, I see the framework listed as one of the dependencies.
declare -a DEPENDENCIES=(... 'WxIcons' ...);
IFS=" " read -r -a SPLIT <<< "$ARCHS"
for ARCH in "${SPLIT[@]}"; do
  cd "$OBJROOT/Pods.build"
  filelist=""
  for dependency in "${DEPENDENCIES[@]}"; do
    path="${CONFIGURATION}${EFFECTIVE_PLATFORM_NAME}/${dependency}.build/Objects-normal/${ARCH}"
    if [ -d "$path" ]; then
      search_path="$path/*.o"
      for obj_file in $search_path; do
        filelist+="${OBJROOT}/Pods.build/${obj_file}"
        filelist+=$'\n'
      done
    fi
  done
  filelist=${filelist%$'\n'}
  echo "$filelist" > "${CONFIGURATION}${EFFECTIVE_PLATFORM_NAME}-${TARGET_NAME}-${ARCH}.objects.filelist"
done

So, at this point I'm stumped...
Thoughts?
By the way, our app has several dependencies that are precompiled frameworks.  Is it appropriate to place these frameworks in the Embedded Binaries list for our application target, or should they go somewhere else?

I apologize... The issue was on my end; apparently I had inadvertently added the WxIcons.framework to the list of linked frameworks...


Table Row/Cell Styles

Is your feature request related to a problem? Please describe.
When I have a table with at has rows that span multiple lines it can be difficult to distinguish one row from the next.
Describe the solution you'd like
It would be nice to have the ability to have table rows/cells with background color.  This would allow for striped rows. It would also be nice to be able to have a border on each row similar to the border on the header/footer.
Describe alternatives you've considered
N/A
Additional context
N/A

@shaneray Sounds like a good idea! Would you like to take a stab at this?

@shaneray Sounds like a good idea! Would you like to take a stab at this?
Getting the API right might be difficult since we're currently don't allow configuration by row, but happy to discuss it more.

I would love to contribute... However, I am working 2 jobs at the moment and not sure when I would be able to get around to it.


fix(ITs): configuration-governance ITs fixed

:tada: This PR is included in version 0.0.8 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Should mideleg[32] be hard-wired to 0 in Smsdia ?

According to Priv. Spec.: Machine Trap Delegation Registers (medeleg and mideleg):
mideleg holds trap delegation bits for individual interrupts, with the layout of bits matching those
in the mip register (i.e., STIP interrupt delegation control is located in bit 5).

Per discussion in https://github.com/riscv/riscv-smmtt/issues/34.
We may not want to delegate MSDEI to less privilege mode.
If this is true, it's helpful to explicitly mention that mideleg[32] should be hard-wired to 0 if CPU supports Smsdia.

Will add that MSDEI cannot be delegated.

That's helpful, thanks!

related to the issue #34 we will not hardwire mideleg[32] to 0. This interrupt will be renamed to SDEIP and can be delegated if needed to an SD.

Close this issue, should be discussed in #34.


Update autorest version to prerelease 3.0.0-alpha.20240627.2

Generated by Autorest Regen Preview build 20240627.2Triggered from pull request: https://github.com/Azure/autorest.csharp/pull/4871

API change check
API changes are not detected in this pull request.


add an example of using DataFrame to create a subquery

Which issue does this PR close?

it is difficult to write the same logic based on the DataFrame API as SQL.
Therefore, I created this example hoping to help others.
The reason for the difficulty is,

https://github.com/apache/arrow-datafusion/issues/5791#issuecomment-1502692953
https://github.com/apache/arrow-datafusion/issues/5789
https://github.com/apache/arrow-datafusion/issues/5771#issuecomment-1488253981
https://github.com/apache/arrow-datafusion/pull/5686#issuecomment-1486618404

Maybe it's just my personal reason.
Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

I think we can refine this example further but it is better than what we have at the moment

Thanks again @jiangzhx


Not an issue, but can't wait until this supports new Let's Encrypt Wildcard certs!

https://community.letsencrypt.org/t/acme-v2-and-wildcard-certificate-support-is-live/55579

Support for wildcard certs won't be possible, see #319

Shame.


Fix/remove npm engine specifier

The package.json includes an "engines" field declaring a requirement for node 16.15.1.
If you start an empty project with the following package.json:
{
  "name": "testproj",
  "version": "1.0.0",
  "dependencies": {
    "@bitmovin/player-integration-yospace": "^2.3.0"
  }
}

running node 18, you will receive the following warning:
lkaalim@N1003643 testproj % npm i @bitmovin/player-integration-yospace
npm WARN EBADENGINE Unsupported engine {
npm WARN EBADENGINE   package: '<EMAIL_ADDRESS>npm WARN EBADENGINE   required: { node: '16.15.1' },
npm WARN EBADENGINE   current: { node: 'v16.14.0', npm: '9.8.1' }
npm WARN EBADENGINE }

As the @bitmovin/player-integration-yospace package is mostly intended to run on browsers and video-playing-environments, theres no actual requirement for consumers of this package to have a specific version of node. In fact, if you have any version of node apart from 16.15.1 specifically, npm will report that warning.
If using yarn, the error is more fatal:
lkaalim@N1003643 testproj % yarn
yarn install v1.22.21
[1/4] 🔍  Resolving packages...
[2/4] 🚚  Fetching packages...
error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version "16.15.1". Got "18.19.0"
error Found incompatible module.
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.

When installing this own package for development purposes and during test runs in CI, you also see the same warning as this project itself uses node 18 as defined in it's .nvmrc:
v18.13.0

The engines field isn't relevant for this project, as the nvmrc file determine what version of node someone developing this package should use, and the version of node that a consumer happens to use to install the project has no relevance on the actual runtime expected from this package.
If there is somehow a requirement for clients to use specific versions of node, the engine field should be updated to be either:

"node": "~16.15.1" (for node 16 specifically)
"node": ">16.15.1" (for node 16 and above)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Luke Kaalim seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Luke Kaalim seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Add Grumpkin to mina/curves

[ ] add grumpkin/bn254 pairing parameters to add (max 1h)
[ ] get rid of ark_bn254 dep and use mina_curves::bn254 and grumpkin in zkVM

@dannywillems and @fabrizio-m should we still do this one?

All done but still needs pairing

PR https://github.com/o1-labs/proof-systems/pull/1806 is closed for the moment. It adds the curves Grumpkin and BN254 to the mina-curves repo, and the two tasks of this issue is the follow-up.


Copy dandietag code from dandi-cli

Closes #57.

ok, let's get this merged -- test coverage is improved although not 100% yet, and dandi-cli errors are due to newer metadata shema version which is not yet supported by dandi-api


Gradle build fails - update xalan

Gradle is throwing bad UTF-8 byle 9e at offset 00000002 from org/apache/xalan/res/XSLTErrorResources_ru.class.
http://stackoverflow.com/questions/20836047/xalan-jar-error-when-build-in-android-project claims to be fixed in Xalan 2.7.2

the main question is, what dependency is adding it, because from what i have seen we have not added it.

Selendroid Standalone -> selenium-java -> selenium-htmlunit-driver -> xalan

i've excluded that dependency for now then :)

@galaxyfreak can you verify on the snapshot build that it works fine for you?

@DominikDary  Please could you build jar to be sure? I did, but size is about half of 0.16 release, and init example is not actually working (crash), so I assume I'm missing something, as I never used Maven before.

@galaxyfreak it's here:
http://oss.sonatype.org/content/repositories/snapshots/io/selendroid/selendroid-standalone/0.17.0-SNAPSHOT/

@lukeis so my build is fine, size did decrease much in latest changes.
Well, my issue is sorted but I'd like to get it up and running for a start.
Logcat: http://pastebin.com/NEWKVuhP
Code: http://pastebin.com/FM6QgXTk
Fails on driver = new SelendroidDriver(caps);

what device or emulator are you running this on?
"java.lang.NullPointerException: Attempt to invoke virtual method 'boolean java.util.concurrent.atomic.AtomicBoolean.compareAndSet(boolean, boolean)' on a null object reference"
implies it couldn't find that method which is provided by the Android API library... which would imply you're not running in a proper android environment?

Sony Xperia Z3, Android 5.0.2

so, that's lollipop... the error makes no sense to me, why would it think a core library isn't available. Something might be funky with that device?
Can you test against an emulator?

@lukeis tested, same error.

i'm now entirely confused... what are you building with gradle and why is it including selendroid? Can you give us a way to reproduce the issue?

Nothing yet. Step by step. that onCreate code is only code I have for now.

How do you run it? From what I see, you can't use stock httpclient on Android, but have to use android port. Yet you have httpclient in all POM files..


Output of tokenizer.encode has None for space

I downloaded the pretrained and fine-tuned model from https://s3.amazonaws.com/models.huggingface.co/transfer-learning-chatbot/finetuned_chatbot_gpt.tar.gz
tokenizer_class, model_class = GPT2Tokenizer, GPT2DoubleHeadsModel
tokenizer = tokenizer_class.from_pretrained(args.model_checkpoint)
tokenizer.encode('good morning')
The output is [3454, None, 1054, 40164]. Get None for space in the list.

I think I have wrong tokenizer_class and model_class. But what is the class for this fine-tuned model?


EF Core 1.0 with ASP.NET Core 1.0 MVC WebApi on IIS Server 7.0 Error 404

Hello,
I have a small application that uses EF Core 1.0 with ASP.NET Core 1.0 MVC WebApi, is working perfectly in kestrel self-hosting scenarios (IIS Express) .When i publish on IIS 7.0, i get 404 error when i call method use EF Core, but when i call method that does not use EF Core it work perfectly. What is the configuration for using EF Core 1.0 with ASP.NET Core 1.0 MVC WebApi on IIS Server. Thanks for your help. You would find at this url an application answering the same criterion that will give a 404 error after being pulse on IIS 7.0
https://github.com/FabianGosebrink/ASPNET-Core-Entity-Framework-Core

Can you try posting this to SO? This is not a support forum.

take a look at https://docs.microsoft.com/en-us/aspnet/core/publishing/iis

Excuse me, but I do not know where ask my question. Before asking my question I look at the doc aspnet-core; Publishing / IIS, fundamentals / server and EF Core. In addition all the examples that I found in microsoft or the web in general have the same problem when I deploy them on IIS.
The problem is on EF Core AspNet Core webapi on IIS, if don't have EF Core all is ok.
Another time Excuse me.


Remove deprecated startup/shutdown directives

Now that startup and shutdown have been replaced by on and have not been documented for some time, let's remove those directives completely (AFTER the 0.10.11 release).
We can remove this folder then: https://github.com/mholt/caddy/tree/master/startupshutdown
Eldin, would you like to submit a PR for this when you have a chance in the next couple weeks?

@mholt Sure :)


create a new indeed project to submit for the training or instructor job

create a new indeed project to submit for the training or instructor job

http://www.indeed.com/cmp/Iconic-Technology-Group/jobs/Software-Instructor-baaa78285a967807


Solar Armor and Celestial Stole has no ability to stack Energy Resistance.

Exceptions to allow Energy Resistance stacking are not functional.  Suggest adding a flag to allow stacking of the ER amounts.

Similar Issue with Resistance Mode for Electrical Attunement for Damage Reduction that has an exception for this calculation.

Energy resistance (and DR) stacking not being available is a known issue. Can be manually adjusted for by editing the modifiers (editing the one that is stackable seems best for tracking). This does mean its open to user error, but for now it's what we have.


Integration features

When moving cursor to integral, the value should be shown instead only coordinates (or a pop up value appears or sth like that). Also, each integral should be possible to be defined to a value (e.g. setting a specific signal’s integral to 1), the initial default value for a signal should be rather 1 than a big number.

@lpatiny @nes-cgn
I display the integral value when the mouse come over it under the x axis

looks good to me! that is definitively a useful improvement
Zitat von hamed-musallam<EMAIL_ADDRESS>
@lpatiny @nes-cgn
I display the integral value when the mouse come over it under the x axis


Youtube player

Give please the html code for inserting (youtube) plyr video into the site. JS and CSS I have already configured.

Read the docs.


Widevine licence setup causes glitch is currently playing audio

I tried contacting widevine about this but they just responded with:

Please create an issue at the Shaka Player github page. We have developers who monitor the form regularly and can respond to your question directly: https://github.com/google/shaka-player/issues

So sorry if this is considered a trash issue for the shaka-player project...
I actually think what I'm seeing is a recent Chromium bug, which we think also has existed pre Chrome v72. I tried several old shaka-player versions and I experience this glitch in all of them. I haven't tried any other Widevide JS-player though.
Short summary: Having two shaka instances on the same origin across different tabs, or on the same page. One is playing, the other one is responding to load(). Both playing widevine content. When the widevine licence response is handled in the load()ing shaka instance, the currently playing shaka instance get audio and video glitches.
More details: https://bugs.chromium.org/p/chromium/issues/detail?id=1193437
At TIDAL we got a lot of user reports on this, some of them here. So we had to disable the preloading of next track in a hotfix.

Yes, this doesn't really apply to us, this is definitely a Chrome bug.  There probably isn't anything we can do about it.  It looks like all the correct people are on the Chrome bug, so it should get handled soon.  So I'm going to close this issue.
If you want to implement that workaround they suggested, you may be able to hook the update method to delay calling into the browser.  I think it work correctly:
const orig = MediaKeySession.prototype.update;
MediaKeySession.prototype.update = async function() {
  await makeDelayPromise();
  return orig.apply(this, arguments);
};


Unable to quit the app (by ⌘Q)

After pressed ⌘Q, the option "Quit Planet", it greyed out.
And the Planet Application is still alive.
Planet Version 0.12.0 (995)

Maybe I just need to wait for a while.
It quitted after a while, sometimes longer.
Anyway, it is great to have this app, thank you.

This time it takes more than 30 minutes, and is still running.
So I think I'd better reopen this issue.
Further information:
After pressed ⌘ Q, Planet became Offline.

There are two ipfs process in Activity Monitor:

After killed these two ipfs processes, Planet quitted automatically.
Maybe it was doing some cleanup work, but due to some reason it just couldn't finish.
Potential reason:
My computer can not accept any incoming connection.


implement design layout for header

Closed in favor of: https://github.com/abhay-raizada/nostr-forms/pull/73


Remove rustfmt config again

This contains unstable options, which results in warnings when running
stable rustfmt. That doesn't seem like a good experience to me.
@chris-laplante what do you think? IMO this is a little too ugly (for outside contributors, too) to keep like this. Alternatives:

We can enforce nightly rustfmt in CI, but sucks if local rustfmt is different from CI
We could ignore rustfmt.toml, so you can keep it around and I'll review manually?

Ah, darn. I think option 2 might be easiest. I can just leave the rustfmt.toml in my home directory and it will be picked up according to the docs.


Problem with server extensions

Im having this problem, im new using jupyter and don't know what info could be relevant for the issue, please let me know if there's any extra data I can provide:
[W 19:07:07.908 NotebookApp] server_extensions is deprecated, use nbserver_extensions
[W 19:07:08.549 NotebookApp] Error loading server extension jupyter_nbextensions_configurator

The notebook extensions are just nice to have, but if you are experiencing difficulties installing them, you can just ignore them (they're mostly to have nice table of contents, that's all).
If you want to use them, can you please give me more information about your system (O.S., python version, using pip or anaconda? etc.)?
Did you follow the installation instructions?

Sure, thanks for the quick response. I followed the instructions yes, maybe it has something to do with having jupyter previously installed outside virtualenv?
This is my history:

virtualenv env
source ./env/bin/activate
pip3 install --upgrade -r requirements.txt
jupyter contrib nbextension install --user
jupyter nbextension enable toc2/main
jupyter notebook index.ipynb

Well, I don't know what fixed it but I fixed it. Tried many things like installing jupyter_nbextensions_configurator from github and also updating all pip packages.

Great! Thanks for letting me know. Enjoy the book.


Special characters such as \n are not interpreted

Special characters with backslash such as \n \r \t \v are not treated. Looks like string literals are actually just a byte stream.

Goby

a = "Hello\nWorld"
puts(a)
# => "Hello\nWorld"

Ruby

a = "Hello\nWorld"
puts(a)
# => Hello
# => World

#189 meets some sort of same problem, such as when implementing the String#strip, it need to handle the case such that:
"\n\t Hello World\t \r\n".strip # => "Hello World"

Actually, I met the same issue when implementing String#strip ;-)
Well, I suppose we should also change string type in Golang to rune type.

I'll let @Maxwell-Alexius work on this first.

@st0012 @hachi8833 I'm not sure whether to also convert string type to rune type in initStringObject? What are your opinions?

I'm not very familiar with rune type, need to spend some to do some research.

Me, too. There's a lot to learn, then leave this issue for a moment. (I'll keep on implementing other string api in #184 )

@hachi8833 I think @Maxwell-Alexius already fixed this, can you help verify?

Confirmed \n \t works now!
I close the issue.


iOS 17.6 beta 2 gives an error: "Which text would you like to match?"

My iphone 13 mini is on beta updates (public beta) channel and beta 17.6 beta 2 was rolled out on 1 Jul 2024. After that, noticed SW-DLT shortcut started to give an error "Which text would you like to match?" when trying to download a video.
To reproduce:

Update iOS to 17.6 public beta 2
Copy any link, for example: https://www.youtube.com/watch?v=s3_kKPtXAQs
Launch SW-DLT

Actual outcome:
SW-DLT invokes top banner "Which text would you like to match?" with Text being "MediaURL".
I tried reinstalling dependencies (deleteAll = True), using a-shell (full) instead of mini and doing a reboot, as well as returning from iOS Beta back to stable (Settings > General > Software Update > Beta Updates and tap Off.)
I think I just need to wait this out as full return to stable would next involve removing the beta by erasing and restoring my device, so this bug report is just a heads up, in case iOS 17.6 beta 2 introduced some change to that "MediaURL" that ends up to stable. Then this needs to be fixed.
With an older iPhone 8 (iOS 16.7.8) and beta updates off, video download is initiated fine.

Beta 4 was just released. Updated and the error is no longer witnessed. Closing.


Move TOC under main content and show sidebar toggle by default

https://miniohq.slack.com/archives/C013NDUJKMK/p1627054788180600

https://miniohq.slack.com/archives/C013NDUJKMK/p1627054788180600

Private links @rushenn let's avoid them - publicly not readable. Lets paste the actual content here.


[MM-16042] Added aria-labels to profile picture settings buttons

Summary
Added aria-label tags to the buttons under the Profile Picture settings in Account Settings. Also changed the tags from being anchors to actual buttons so that they can be tabbed correctly.
NOTE: The part where the screen reader says ‘read-only’ before the Select button is read is being fixed by https://github.com/mattermost/mattermost-webapp/pull/3100
Ticket Link
https://mattermost.atlassian.net/browse/MM-16042

Creating a new SpinWick test server using Mattermost Cloud.

Mattermost test server created! :tada:
Access here: https://mattermost-webapp-pr-3101.test.mattermost.cloud
Test Admin Account: Username: sysadmin | Password: Sys@dmin123
Test User Account: Username: user-1 | Password: User-1@123

tab to the profile pic edit button, hit enter
tab once to the select button

Observed: read out says "Dialog account settings dialog read only read only select button"
Expected: "Select button"

Select button is not getting proper visual focus box like the save and cancel buttons

Spinmint test server destroyed

Creating a new SpinWick test server using Mattermost Cloud.

Mattermost test server created! :tada:
Access here: https://mattermost-webapp-pr-3101.test.mattermost.cloud
Test Admin Account: Username: sysadmin | Password: Sys@dmin123
Test User Account: Username: user-1 | Password: User-1@123

Spinmint test server destroyed

Creating a new SpinWick test server using Mattermost Cloud.

Mattermost test server created! :tada:
Access here: https://mattermost-webapp-pr-3101.test.mattermost.cloud
Test Admin Account: Username: sysadmin | Password: Sys@dmin123
Test User Account: Username: user-1 | Password: User-1@123

New commit detected. SpinWick upgrade will occur after the build is successful.

Mattermost test server updated!
Access here: https://mattermost-webapp-pr-3101.test.mattermost.cloud

@devinbinnie the remove profile picture action is not accessible through the keyboard, we should make sure you are able to also remove the profile picture that way.

New commit detected. SpinWick upgrade will occur after the build is successful.

Spintmint test server creation failed. See the logs for more information.

New commit detected. SpinWick upgrade will occur after the build is successful.

Mattermost test server updated!
Access here: https://mattermost-webapp-pr-3101.test.mattermost.cloud

@devinbinnie the remove profile picture action is not accessible through the keyboard, we should make sure you are able to also remove the profile picture that way.

Done!

New commit detected. SpinWick upgrade will occur after the build is successful.

Mattermost test server updated!
Access here: https://mattermost-webapp-pr-3101.test.mattermost.cloud

New commit detected. SpinWick upgrade will occur after the build is successful.

Mattermost test server updated!
Access here: https://mattermost-webapp-pr-3101.test.mattermost.cloud

Spinmint test server destroyed

@amyblais curious why is the cherrypick approved label on here? This is for v5.14


Fix incorrect comments

Signed-off-by: magiceses

[ ] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes

Verification

You can sign the DCO using
git commit --amend --no-edit --signoff

Thank you :bow:


Nx plugin setup

Motivation
As all the good stuff will come from the community plugins it pays of to give them the best DX possible.
Next to good docs and examples etc i see a Nx plugin very useful as it can support us in the following areas:

skufholding

setup
unit testing
e2e testing

migration
publishing
integration
documentation
we could leverage the power of Nx CLI

This should make it as easy as possible to start a custom plugin project.
The plugin can also help use with advertisement as:

we can generate branded readme files etc with the generate commands
we can publish it under Nx community plugins

Todos

[ ] Basic setup for Nx plugin package
[ ] Execute command collect to

[ ] execute the CLI
[ ] maintaining configurations partially in project.json

[x] Generate command init to

[ ] setup code pushup in a specific project

[ ] Generate command plugin to create a custom plugin inc:

[ ] project with esbuild & vitest` setup for the plugin logic
[ ] project with vitest setup for the plugin e2e tests

@BioPhoton please create more smaller issues from this one.

done


Trailing slashes in URLs

How to handle trailing slashes in URLs (when creating documents/handling requests)? This could result in different URLs for the same resource (duplicate content), which should not be allowed.
Suggestions:

No trailing slashes in documents.
A request with a trailing slash in the URL, should be redirected (status 301) to the URL without the trailing slash.

Why does this matter? A given resource may be exposed at any number of URLs anyway.

A resource is identified by the URL, so I think it should be unique.
REST API Design Rulebook:
https://books.google.de/books?id=4lZcsRwXo6MC&pg=PA12&lpg=PA12&source=bl&ots=F4CtaBkuaP&sig=zPd3g8wd-1DTixB4WaXdrN1IQVA&hl=de&sa=X&ei=eU82VfvwHYitswHph4LQBg&ved=0CDEQ6AEwBg

JSON API resources are not resources in the REST/HTTP sense. They are identified by their type + id combination, not by a unique URI.
To see why, consider that a response document from a single URI may contain any number of JSON API resources.

The specification makes no requirements on URL representation, so this isn't part of the spec per-se.
In theory, we could add something like this as a Recommendation, but I'm not sure it's worth it right now.

It's true that different URLs can result in the same content being returned, also without trailing slahes.
And when following the spec it's allowed to use an empty string as id (it's set - present and not null - and it's a string), so this case can only be represented with a trailing slash:
http://example.com/resource/{id}
would result in
http://example.com/resource/ (id = empty string)
which is different from
http://example.com/resource (no id set)
Will close it... but perhaps empty strings should not be allowed as id, as this could be confusing.


Watchdog forensics

Attempts to dump a forensic stack trace of the applet on OS boot.
This is intended to help track down causes of the watchdog being triggered.

I'll close this for now, it'll still be around if we need it in future.


please add a note about ApplicationAttribute Debuggable

on Android, SDK detects debug/release by reading android:debuggable from app manifest;
and this is set to TRUE by default even in release appcenter builds
Please add a note that one needs to include [assembly: Application(Debuggable=false)], it would've saved us a couple of days debugging if it was documented.
Thanks in advance!

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

ID: 83d269a0-0b09-c2f6-d37a-319831bcf782
Version Independent ID: f2c17a3a-87bb-0c75-9efb-de6ffb6f7738
Content: App Center Distribute for Xamarin - Visual Studio App Center
Content Source: docs/sdk/distribute/xamarin.md
Service: vs-appcenter
GitHub Login: @TroubleMakerBen
Microsoft Alias: bereimol

(debuggable:false is required for in-app updates to work)

@wizzard0 Thanks for the info and apologize for any inconvenience. I filed a bug for this one, we will review the step that you mentioned is required or investigate if it is caused by something else.

In Xamarin builds it's false in debug and true in release, including in App Center builds. It does require more investigation why this specific application does not behave the standard way.

Hi @wizzard0, normally if you build in release mode (including in AppCenter build), the application does not end up having debuggable=true by default. The default is false for release builds and we have tested with apps of our own.
Do you have any kind of customization in your app that would have switched this flag incorrectly to true? Like in the xml manifest or another file.


*: rebase controller runtime refactor onto master and add ao commands back

/cc @johnkim76 and @dymurray can you guys help me make sure the ansible operator commands are correct.

dep ensure as part of the new command, it always gives me the following warning dep: WARNING: redundant prune option "unused-packages" set for "k8s.io/code-generator"

@LiliC Previously we were pruning all unused pkgs by default in Gopkg.toml. But I've removed that since new dependencies for the controller-runtime are added when you run operator sdk add api or add controller.  So rather running dep ensure again after each cmd I just disabled the pruning of unused pkgs.
The warning is due to the redundant unused-package=false option for code-generator. That's not needed since we're not pruning anymore.
https://github.com/operator-framework/operator-sdk/blob/refactor/rebase-master-add-AO-commands/pkg/scaffold/gopkgtoml.go#L87


ws2811_init failed with code -5 (mmap() failed)

Hello,
I am using rpi_ws2812 and just copy past rpi_ws281x-python program to my python program but it is giving me error
ws2811_init failed with code -5 (mmap() failed).
if I commect out
LED_PIN        = 10      # GPIO pin connected to the pixels (10 uses SPI /dev/spidev0.0).
LED light stays white and not changing colors.
Please Help.

If you are not using the SPI interface you need to run as root.

Thank you David
Thanks and Regards,
Priti Bhatt

From: David Greaves @.>
Sent: Sunday, March 14, 2021 12:23 PM
To: rpi-ws281x/rpi-ws281x-python @.>
Cc: bhattpriti @.>; Author @.>
Subject: Re: [rpi-ws281x/rpi-ws281x-python] ws2811_init failed with code -5 (mmap() failed) (#61)
If you are not using the SPI interface you need to run as root.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/rpi-ws281x/rpi-ws281x-python/issues/61#issuecomment-798954599, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJS2YFCPMCC4HU47XJQVOWLTDT5I5ANCNFSM4YNGZ23A.

If you are not using the SPI interface you need to run as root.


Modal not showing

On version 0.3.4, and I was using TailwindCSS on my project. No configuration, basically just install Oruga to test it
I was following Open Modal Clip Scroll
https://oruga.io/components/Modal.html#examples
But the modal is not showing. Nothing happen
When inspect the vue dev tools, the o-modal active already true

Are you importing Oruga CSS?

@jtommy sorry its oruga for Vue 2.

Can you reproduce on codepen or similar?

@cyberfly any update on this?

@jtommy is there any codepen template for Oruga UI that I can quickly spin off?
@astagi I revisit the modal after 2 weeks, and looks like the Modal only works if the isCardModalActive is set to true by default.

<button type="button" @click="isCardModalActive = true">
        Open modal (clip scroll)
      </button>

<o-modal :active.sync="isCardModalActive" :width="640" scroll="clip">
      <p>
        laoreet et metus. Aenean placerat turpis nec tincidunt tempus.
      </p>
    </o-modal>

data() {
    return {
      search_data: {},
      total: 0,
      loading: false,
      current_user: {},
      isCardModalActive: true
    };
  },

@cyberfly yes, active prop must be true


Integration with yosys

Hello. Since the open source version of yosys cannot read system verilog, may I ask how can I unse UHDM to complement this? That is to say, could you explain more on how to feed system verilog to UHDM and generate a file to yosys so that yosys can synthesize it? Thanks!

See https://antmicro.com/blog/2020/12/ibex-support-in-verilator-yosys-via-uhdm-surelog/

@peijunh please ping @kgugala directly for instructions. This is still WIP, it won't support any design quite yet. We have Ibex and OpenTitan supported as this point.


Implemented paths

Told Face to do it, He muted me for pinging him so I did it myself.

How all i did was implement the system?

I understand that i got muted i was just joking in this pr... still did implement it

it fully works


List Searching functions Endpoint Argument

The list searching functions can have an endpoint argument with default value equal to the list size - 1.
A choice exists between whether to use inclusive or exclusive endpoint index.
Originally posted by @DK96-OS in https://github.com/DK96-OS/MathTools/pull/32#discussion_r795601961

This is an easy feature to implement.


Add mcu_connection_status commands

This PR adds the new mcu_connection_status (0x25) commands.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

13 of 13 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 83.734%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
6723

Relevant Lines:
8029

💛  - Coveralls

Codecov Report
Base: 83.70% // Head: 83.73% // Increases project coverage by +0.02% :tada:

Coverage data is based on head (bf83816) compared to base (071190e).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #2054      +/-   ##
==========================================
+ Coverage   83.70%   83.73%   +0.02%
==========================================
  Files         249      249
  Lines        8016     8029      +13
==========================================
+ Hits         6710     6723      +13
  Misses       1306     1306

Impacted Files
Coverage Δ

zhaquirks/tuya/mcu/__init__.py
99.51% <100.00%> (+0.03%)
:arrow_up:

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

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


General question of the github app roadmap!

Hi guys, been a little while since I last checked in. This is probably best attentioned for: @BasThomas and @Huddie
Genuinely curious about the general roadmap of carrying this great app forward.
I handpicked several awesome issue tickets that I was excited for that either seemed to have been reviewed, ready for review or just seemingly pretty close to fully fleshed out:
#2348 - edit title
#2393 - templates
#2482 - sub/unsub
#2687 - open/closed toggle
#2106 - even this one which is at a much different progress point than the others. But at some point would be so awesome to run with designs already mocked than to keep waiting on more without realizing them
Seems like the torch has been passed to you guys. Back when Ryan was more active, the development cycles were great and frequent. I understand things change and he’s on to newer and great things, but in the most positive way possible I’m a little sad that the cycles seem to have slowed down so much. A good example is the API status error, the patch has been set, it’s been in testflight for a while. But curious what is the general holdup to eventually pushing it to the appstore (it doesn’t constitute a ton of beta testing and haven’t seen any beta testers have any issues), but sadly the app hasn’t been updated in 6+ months. And with the items I listed above, if any/most are actually good to go I’d be stoked to test them out in a new upcoming beta.
You two have been some of the most active users in this community for quite some time (and I know a lot of what we have already is due in large part to your contributions!) so please take this as a proactive and not a complaint post. But just curious if we can be a little more aggressive with taking merge requests that seem to be ready and running with them since things seen a little static lately.
Thoughts?

Hey gang. I will get back to this along with some general updates ASAP.

Awesome and thanks for the reply @rnystrom ! Hopefully the tone was properly conveyed as excited for what’s your come! To be honest, really hoping you might be working some magic over at your new gig to potentially influence an official app maybe built off githawk 😉
P.S. to hijack myself, I know you used to work at IG, any chance to glance at this tweet.
https://twitter.com/ian_myers/status/1152717567225675776?s=21
Cheers and thanks again for popping in!

Just going to drop this one here for those who haven't seen it
https://github.com/mobile
Probably answers the "roadmap" question

:) signed up around noon et today and it was already full. Hoping some spots pop up to help test during the beta

Ryan was the one to announce it at Universe. I suspect he gained at least some of his skills for the official app from this.


There is a bug in emmet extension

We have written the needed data into yHello . Good time . VS Code is my favorite editor and I am using this editor for programming python and django but I have found a bug in emmet extension . when we install Django extension that written by Baptiste Darthenay and configure setting.json same as description of the extension we can not use the auto suggestion of html tags in django templates because emmet extensions does not load when we open html document  html tags do not appear in auto suggestion menu . I figured out If I add "onLanguage : django-html" statement to the activationEvent of package.json file of emmet extenstion this bug will be removed and we can use html tags in django templates and use html tags will appear in auto suggestion menu.,
please solve this problem .
Thank you for your favor .

Duplicate of #137012
It should be fixed in the next release, which comes out in under two weeks.
Alternatively, you can switch to Insiders.


[stable/opencart] Add apache exporter

What this PR does / why we need it:
This PR adds an Apache exporter to be used with Prometheus.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

/ok-to-test
/lgtm

/ok-to-test
/lgtm

/retest


Various README fixes

Depending on your preferences, @casey, I can open this as multiple PRs. I believe it will be faster to get them in place with one PR. As each change builds on the last but is otherwise isolated, I recommend reviewing this PR a commit at a time. What follows is a summary of the commits based on the commit messages for each.
Prefer ATX headings over SETEXT headings
The two heading types are equivalent, but SETEXT headings only support two levels (=== [h1] and --- [h2]), and there are enough other headings to make it worthwhile to use ATX headings exclusively.
Replace Unicode punctuation with ASCII punctuation

Replace ’ with ' globally. The Unicode version is more correct, but most modern Markdown converters apply SmartyPants conversions to automatically convert ' to ’.

Replace instances of … in code blocks with .... Left the Unicode … characters in place in prose as I don’t think that SmartyPants conversions handle those.

Changed in-document URLs to not go remote
Removed the prefix https://github.com/casey/just from in-document URLs so that the links work in local Markdown viewers without redirecting to GitHub.
As an example, [much more](https://github.com/casey/just#manual) becomes [much more](#manual).
Automatic prettier formatting

This fixes various small formatting issues.

Make the styling of justfile consistent
Modified the styling of justfile to always be presented in code format, including for the plural as justfiles. There are a couple of places where this has not been followed, such as in the function group title #### Justfile and Justfile Directory.
Fix a couple of link errors
Make justfile syntax indentation consistent
Changed indentation of justfile syntax to two spaces consistently.
Fix various errors

Fixed the use of the en-dash (–) where the em-dash (—) belongs.
Fixed the unnecessary use of backticks inside a syntax block.
Fixed a couple of sentence errors.
Fixed cases where text should have been been wrapped in code blocks and weren’t or were wrapped in other syntax markers.

Clean up the installation table
Convert the in-line links to reference links. Note that the references immediately below the table are not sorted, but are presented in the order of appearance in the table.
Fix a couple of wrong-level headings
Move Dotenv Integration section
This was placed under the Functions section, but belongs under the Features section. I moved it to a place that seems to make sense.

I’ll revert back to SETEXT later (I have to disable Neoformat because prettier automatically changes SETEXT headings to ATX headings), but I’d like you to look at the commit I’m pushing up now where I am doing a replace of all ... with …. This only affects code wrapped text (`text`) and fenced blocks (```…```) to see if there are spots where you don’t want … instead of ....

Sweet, the change you just pushed looks good to me.

OK. I’ll get the headings changed and finalize this PR in a few hours.

This should finalize the cleanup PR.


pull request for e2e test at 01/29/2017 15:26:12

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


handle homebrew's bump to mysql@v9 on macOS

pscale shell is semi broken as of recently, because homebrew bumped mysql to v9, which doesnt contain the .so required for y'all's shell integration.
See https://stackoverflow.com/questions/78938322/mysql-authentication-plugin-issues-on-macos
https://gist.github.com/aybabtme/4fd51e181942405052d8e4645773a2d1
Might be good to catch that in the CLI and handle it cleanly, perhaps suggesting how to fix it at a minimum, so people don't wonder what's up.

Fixed!


Uncaught Exception: Attempting to call a function in a renderer window that has been closed or released. Function provid

An uncaught exception was reported.
Info:
OS: win32
Arch: ia32
GPMDP Version: 3.0.1
Time: Fri Mar 11 2016 07:51:13 GMT-0600 (Central Standard Time)
Error:
Attempting to call a function in a renderer window that has been closed or released. Function provid
Error: Attempting to call a function in a renderer window that has been closed or released. Function provided here: Object.<anonymous> (C:\Users\ross.varrette\AppData\Local\GPMDP_3\app-3.0.1\resources\app\build\inject\GPMInject\interface\mouseButtonNavigation.js:8:8.
    at BrowserWindow.callIntoRenderer (C:\Users\ross.varrette\AppData\Local\GPMDP_3\app-3.0.1\resources\atom.asar\browser\lib\rpc-server.js:201:19)
    at emitTwo (events.js:92:20)
    at BrowserWindow.emit (events.js:172:7)

This issue was created automatically inside the uncaughtException handler

Fixed in #370


operator keysight-wap-msg-broker (0.0.90)

@nicomola Unfortunately, your bundle project was created with the incorrect distribution type (I assume you did this programmatically using the API).
The external distribution mode is not supported, and is what's causing all of your bundle images to fail to actually publish.
You will need to create new projects with the distribution mode set to "Red Hat" which will make the bundles available via the expected registry.connect.redhat.com/loadcore/keysight-xxxx-bundle pull spec. The internal or backend registry quay.io/redhat-isv/ is not publicly accessible, and can only be accessed using your partner-specific registry key.

Trying this again, as I've changed the project distribution mode to rhcc from external. The release-pipeline failed again despite this, yet the bundle should get published as a result of the pipeline run.
I'm trying this again, just to avoid having to manually publish the operator bundle in the data backend.


Please cache project creation progress

When I am in the middle of creating a project and I accidentally press ESC, I have to restart the whole process again. None of my information already entered is saved.

Dear @subwaystation ,
can you explain, what has lead to the accidental ESC button interaction? The ESC is default behaviour to escape any current action, so we assume users want to abort.

I agree that I want to abort the current window (aka creating a new project), but in my flow at the time of testing, I was hoping I could continue at this state.
alternatively, you could provide a button: continue project (for the last unfinished one)

Dear @subwaystation,
can you elaborate, why you left the registration process and wanted to continue later? Have you been unsure about the experimental setup?
We would like to understand your workflow better, to see if that is an important use case.
The accidental ESC action can be addressed by the implementation of  an additional confirmation modal.

We track down the abort confirmation modal, but are currently not planning to implement a project creation cache. The creation of a project is quick once you have the required information.


The highlighting of 'apply changes' button is cut for Beam Remote Node

Now the highlighting of 'apply changes' button is cut for Beam Remote Node from Settins menu.

The highlighting should be done as it's done for Bitcoin submenu:

fixed

checked


Gantt chart doesn't load

Hello, I have recently installed the newest version of the Devliery Gantt add on but it just loads indefinitely. Have tried various different methods to try and get something to load but nothing is pulling through.
Console info attached.

Can you please advise on how to resolve this issue asap?
Thanks

Hi @Rearna could you elaborate on where exactly this is happening? also tell me you have created at least one gantt .. since at the first boot there is always a spinner until at least one gantt is created.

It happens to me too, just created my Gantt and still keeps loading...

Refused to connect to 'https://browser.events.data.microsoft.com/OneCollector/1.0/?cors=true&content-type=application/x-json-stream&w=0' because it violates the following Content Security Policy directive: "connect-src .visualstudio.com wss://.visualstudio.com https://web.vortex.data.microsoft.com .dev.azure.com dev.azure.com wss://.dev.azure.com wss://dev.azure.com *.vsassets.io ms.gallery.vsassets.io vsassetscdn.azure.cn ms.gallerycdn.vsassets.io ms.gallerycdn.azure.cn *.blob.core.windows.net https://login.microsoftonline.com".


Dine In table selection on fulfillment page for kiosk

Is your feature request related to a problem? Please describe.
Currently Dine table selection available on checkout page for kiosk. If there is no table available then the selection of product is totally become waste for customer.
Describe the solution you'd like
Table selection should be on fulfillment page. When customer select the Dine In fulfillment, in a next step they are able to select there table. If there is no table then they can't proceed from that step.

You have the wrong repo.


Issues creating documentation for node.js (ts-node) app

Having a problem I can't solve actually when trying to document my code
which uses node specific typings.
Included "@types/node": "^7.0.0"
First doc generator crashes at __dirname ->  Cannot find name '__dirname'.
Then it also crashes at a readFileSyncwhich is included this way
import {readFileSync} from 'fs'; -> Cannot find module 'fs'.
I don't know what's going wrong here because any other aspects around typings are working (run the app, runt mocha tests and coverage report).
Do I have to bind the typing differently or so?

I am having similar problems. TypeScript (v2.1.5) is able to compile correctly and without errors simply by include @types/node in node_modules but running typedoc (on v0.5.5) will produce
Cannot find name 'process'.

Is there a specific step needed to ensure that TypeDoc's execution of the TypeScript compiler includes @types definitions?

These both seem like issues with arguments being passed to TypeScript. There were multiple changes made that may have fixed these.
I will close this issue, but if you are still encountering problems, please let us know how we can reproduce the issue. Thanks!


Fix wrap_match_arms resulting in a missing comma

This fixes issue #1127. This is my first dive into rustfmt, so I'll explain my changes to make sure they're agreeable.
In terms of diagnosing the issue, I focused on the only 3 occurrences of context.config.wrap_match_arms (all are in expr.rs).
The first was on line 1256. Removing the config condition had no effect on the output, so I stopped focusing on this section.
The next occurrence was on line 1284, inside a block which starts at line 1271. This whole block would only effect match arms which could be formatted into a single line; so none of that block effects the case described in this bug.
The only remaining use of this configuration variable is line 1312, near the end of the method.
        let (body_prefix, body_suffix) = if context.config.wrap_match_arms {
            if context.config.match_block_trailing_comma {
                ("{", "},")
            } else {
                ("{", "}")
            }
        } else {
            ("", "")
        };

        /* ... */

        Some(format!("{}{} =>{}{}{}\n{}{}",
                     attr_str.trim_left(),
                     pats_str,
                     block_sep,
                     indent_str,
                     next_line_body,
                     shape.indent.to_string(context.config),
                     body_suffix))

It basically becomes clear that this bug is a missing comma before the newline in the format argument. In order to not start changing too many thing, I split the return into two different format! branches—the original, and a new one which removes the empty line this bug was producing.

I think looks right, but I must admit I am not entirely sure from the diff. Why can you remove the \n from the result in the non-wrap case? Or alternatively where is the extra blank line coming from?
This looks like a fine fix, I wonder if we might refactor to make this cleaner in some way - it feels like the non-wrap case should be handled the same way as the single line case (which also needs a comma but no braces). This may or may not be possible and doesn't need to be fixed in this PR, but might be interesting for the future.
Could you add a test for this PR please?

It wasn't posted as a code example, but the actual issue referenced produced the following format, where it was missing both the comma and had a blank line added:
match (&self.state, try!(self.de.ch())) {
            (&Attribute, EmptyElementEnd(_)) => return Ok(None),
            (&Attribute, StartTagClose) => 0,
            (&Attribute, AttributeName(n)) =>
                return Ok(Some(try!(KeyDeserializer::deserialize(try!(self.de.check_utf8(n))))))

            (&Element, StartTagName(n)) =>
                return Ok(Some(try!(KeyDeserializer::deserialize(try!(self.de.check_utf8(n))))))

            (&Inner, Text(_)) => 1,
            (&Inner, _) => 4,
            (&Value, EndTagName(_)) => return Ok(None),
            (&Value, Text(txt)) if txt.is_ws() => 3,
            (&Value, Text(_)) => return Ok(Some(try!(KeyDeserializer::value_map()))),
            (&Element, EmptyElementEnd(_)) => 2,
            (&Element, Text(txt)) if txt.is_ws() => 5,
            (&Element, EndTagName(_)) => return Ok(None),
            (&Element, EndOfFile) => return Ok(None),
            _ => unimplemented!(),
        }

As @oli-obk said:

the two long match arms get their comma removed and there's a trailing empty line with trailing whitespace after each of them.

In this case, the new line can be removed because it's only used when context.config.wrap_match_arms is true to add the indent and body_suffix which is only a closing } or },. When wrap_match_arms is false, the original code was resulting a line with just indent and an empty string for body_suffix.
Regarding refactoring to make it cleaner: I was trying to think of a way to do this—potentially making the tuple above into (body_prefix, new_line_indent, body_suffix) or something to that effect, but trying to concatenate shape.indent.to_string(context.config) with a \n felt messy, too so I got kind of stuck. I'm perfectly happy to implement changes if you have any recommendations for handling that.
I'll try to get a test or two up over the weekend. Full disclosure: I was going to try, but the tests on master are failing on Windows, so I'll try to whip something up over the next couple days when I can set this up on a Linux box.

the original code was resulting a line with just indent and an empty string for body_suffix

Do you mean something else? body_suffix was previously just "" and now ",", so I don't think it can be the source of the newline.
I don't think you necessarily need to change your code, but I would like to understand why we get an extra newline in the !wrap case, but not the default.

trying to concatenate shape.indent.to_string(context.config) with a \n felt messy

why does this feel messy? I'm not sure exactly where you want to do it, but it feels fine to me.

The tests on master are failing on Windows

huh, interesting, we usually don't have cross-platform problems with Rustfmt. Which tests are causing problems.

Do you mean something else? body_suffix was previously just "" and now ",", so I don't think it can be the source of the newline.

Sorry for the poor explanation.
So, this was the original format code:
Some(format!("{}{} =>{}{}{}\n{}{}",
             attr_str.trim_left(),
             pats_str,
             block_sep,
             indent_str,
             next_line_body,
             shape.indent.to_string(context.config),
             body_suffix))

when wrap_match_arms is true, it will produce a block like this:
        (&Element, StartTagName(n)) => {
            return Ok(Some(try!(KeyDeserializer::deserialize(try!(self.de.check_utf8(n))))))
        },

Here is an annotation of each of the format strings and their contribution to that output:
attr_str.trim_left(),                     "        "
pats_str,                                 "(&Element, StartTagName(n))"
block_sep,                                " {\n"
indent_str,                               "            "
next_line_body,                           "return Ok(Some(try!(KeyDeserializer::deserialize(try!(self.de.check_utf8(n))))))"
[new-line char from the format string]
shape.indent.to_string(context.config),   "        "
body_suffix                               "},"

So, before this fix and with wrap_match_arms set to false, the annotation would be this:
attr_str.trim_left(),                     "        "
pats_str,                                 "(&Element, StartTagName(n))"
block_sep,                                " \n"
indent_str,                               "            "
next_line_body,                           "return Ok(Some(try!(KeyDeserializer::deserialize(try!(self.de.check_utf8(n))))))"
[new-line char from the format string]
shape.indent.to_string(context.config),   "        "
body_suffix                               ""

Notice the last three lines: the blank line that manifested itself as part of this bug—just to be clear, the main bug was the missing comma, which caused compile errors—is the line which would have contained the }, characters if wrap_match_arms were enabled; because that setting was disabled, the result was a new-line with just an indent (and an empty body_suffix).

I went ahead and added a test case for this, so let me know if you see anything else that needs to be addressed.
On a tangent note, regarding tests on Windows:

Which tests are causing problems.

Basically all of them
running 9 tests
test format_lines_errors_are_reported ... ok
test rustfmt_diff_make_diff_tests ... ok
test rustfmt_diff_no_diff_test ... ok
test checkstyle_test ... ok
test stdin_formatting_smoke_test ... ok
test idempotence_tests ... FAILED
test coverage_tests ... FAILED
test system_tests ... FAILED
test self_tests ... FAILED

thread 'self_tests' panicked at 'Empty diff? Maybe due to a missing a newline at the end of a file?', tests\system.rs:323
stack backtrace:
   0:     0x7ff6ca1253f8 - std::panicking::default_hook::{{closure}}
                        at C:\bot\slave\nightly-dist-rustc-win-msvc-64\build\src\libstd\panicking.rs:356
   1:     0x7ff6ca124882 - std::panicking::default_hook
                        at C:\bot\slave\nightly-dist-rustc-win-msvc-64\build\src\libstd\panicking.rs:361
   2:     0x7ff6ca1282bd - std::panicking::rust_panic_with_hook
                        at C:\bot\slave\nightly-dist-rustc-win-msvc-64\build\src\libstd\panicking.rs:545
   3:     0x7ff6c99530d7 - std::panicking::begin_panic<&str>
                        at C:\bot\slave\nightly-dist-rustc-win-msvc-64\build\src\libstd\panicking.rs:507
   4:     0x7ff6c99a2124 - system::handle_result
                        at C:\Users\Marcus\Source\rustfmt\tests\system.rs:323
   5:     0x7ff6c99a0bc4 - system::idempotent_check
                        at C:\Users\Marcus\Source\rustfmt\tests\system.rs:256
   6:     0x7ff6c999ec74 - system::check_files<core::iter::Chain<core::iter::Chain<core::iter::Map<core::iter::Chain<std::fs::ReadDir, std::fs::Read
Dir>, fn(core::result::Result<std::fs::DirEntry, std::io::error::Error>) -> collections::string::String>, core::option::IntoIter<collections::string
::String>>, core::option::IntoIter<collections::string::String>>>
                        at C:\Users\Marcus\Source\rustfmt\tests\system.rs:181
   7:     0x7ff6c999daf0 - system::self_tests
                        at C:\Users\Marcus\Source\rustfmt\tests\system.rs:126
   8:     0x7ff6c9fc835a - test::{{impl}}::call_box<(),closure>

It has to do with Window's new line characters; I'm going to open a separate issue for that.

Thanks for adding the test, and sorry this slipped off my radar for a while.


Add plasma technology table

resolve #127.
I've essentially followed the established standard. Please let me know if there are any issues 👍

Changes are looking good, awesome work. 👍 I have merged the PR. Thanks again for your contribution!


Feature cobra cmd

#236

you can run the commands as such:
go run main.go master --config
go run main.go worker --config

I will do a bit of a thorough review tomorrow, but at a glance, there are a few more things we will need to account for if worker and master become subcommands of the single executable. Specifically, platforming package:
(docker-swarm)
https://github.com/ScoreTrak/ScoreTrak/blob/30304a78a6a3884b76152214493680d9413132cf/pkg/platform/docker/docker.go#L201
(kubernetes)
https://github.com/ScoreTrak/ScoreTrak/blob/30304a78a6a3884b76152214493680d9413132cf/pkg/platform/kubernetes/kubernetes.go#L61
(docker)
https://github.com/ScoreTrak/ScoreTrak/blob/30304a78a6a3884b76152214493680d9413132cf/pkg/platform/docker/docker.go#L228
can not be referencing ./worker as the executable

Also, lets remove Copyright lines from .go files. I think the LICENSE file in the root directory is sufficient

At a glance seems fine. I will test them out and fix the formatting once the platforming files are fixes.
Also go lint github action works by highlighting code at: https://github.com/ScoreTrak/ScoreTrak/pull/248/files (scroll down in file, you will see)

Also, lets remove Copyright lines from .go files. I think the LICENSE file in the root directory is sufficient

yeah, the copyright lines were generated by the cobra cli auto generator. That can be removed

Neat job. Lets test this on a kind cluster with helm, and push it up. Good job!

working on it 👍

no longer crashes as https://github.com/ScoreTrak/ScoreTrak/blob/feature-cobra-cmd/pkg/runner/runner.go#L114

yeah, i am testing it to rn

After having a call, we decided that this pull request introduces too many changes, and it is too hard to keep track of everything. We are instead splitting this PR into two different PRs:

Introduce Cobra + Viper, however, keep the config package exactly the same as it was before(functionality-wise). This way platform package will only need to change its commands, like specified in this comment: https://github.com/ScoreTrak/ScoreTrak/pull/248#issuecomment-973893146
Refactor Config, and any other packages that would be affected by a refactor
Misc PR that introduces unrelated things like license, etc.


I'm having deployment issues

Hi DeliciousBiscuits, Applications are unable to run on file systems that are formatted as exFat. This is unfortunately a limitation of Microsoft Windows.
To get the program to run, please move the files onto a hard disk that is formatted with NTFS.
Please let me know if you require any more help with this.
Regards
Jack

Hi Jack
Thank you, The application started running when I reformatted the hard disk to NTFS just like you said.
Regards,
Kizzie

Good job peopleeeee


Error doing job: Task exception was never retrieved

Hi,
I can see that I had some errors in the last night, probably because the server of Huawei was offline, or network issues.
first errror:
Logger: custom_components.fusion_solar.sensor
Source: helpers/update_coordinator.py:182
Integration: FusionSolar ([documentation](https://github.com/tijsverkoyen/HomeAssistant-FusionSolar), [issues](https://github.com/tijsverkoyen/HomeAssistant-FusionSolar/issues))
First occurred: 04:00:58 (1 occurrences)
Last logged: 04:00:58

Error requesting FusionSolarKiosk data: HTTPSConnectionPool(host='region02eu5.fusionsolar.huawei.com', port=443): Max retries exceeded with url: /rest/pvms/web/kiosk/v1/station-kiosk-file?kk=XXXXXXXXXX (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7f49a3dcb7c0>: Failed to establish a new connection: [Errno 111] Connection refused'))

second error:
Logger: homeassistant
Source: custom_components/fusion_solar/fusion_solar/energy_sensor.py:67
Integration: FusionSolar (documentation, issues)
First occurred: 04:10:58 (21 occurrences)
Last logged: 07:30:59

Error doing job: Task exception was never retrieved
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 182, in _handle_refresh_interval
    await self._async_refresh(log_failures=True, scheduled=True)
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 330, in _async_refresh
    self.async_update_listeners()
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 135, in async_update_listeners
    update_callback()
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 390, in _handle_coordinator_update
    self.async_write_ha_state()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 566, in async_write_ha_state
    self._async_write_ha_state()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 607, in _async_write_ha_state
    state = self._stringify_state(available)
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 572, in _stringify_state
    if (state := self.state) is None:
  File "/config/custom_components/fusion_solar/fusion_solar/energy_sensor.py", line 67, in state
    return float(current_value)
ValueError: could not convert string to float: 'unavailable'

Probably we need to avoid convert to float the state: unavailable
I'm not a programmer, but probably should be like:
              if realtime_power == '0.00':
                   _LOGGER.info(
                       f'{self.entity_id}: not producing any power, so not updating to prevent positive glitched.')
                   if current_value == 'unavailable':
                        None
                        return float(current_value)

Hi,
I just let you know, it did not fix it. Your changes in the code it should be before line no 67, no?


Did you try to use the true global state as input in LBF?

Dear Authors,
To train QMIX on LBF, I found it uses the concatenated obs as the states. For fully observable settings, there is no problem doing so. I am curious about did you try to use the true global state as input in LBF? Is the performance good?
Thank you in advance.

Hi @GoingMyWay
Do you mean extracting a global state from the partially observable versions of LBF?
No, we only looked into concatenating the observations, which are just an approximation of the global state (in the partially observable environments). There should be no change in the fully-observable ones.

Thanks for the clarification.


Docs: add documentation for cli.py

This is a handy tool that isn't mentioned anywhere. Add it to the docs and mention it in the install script.

Perhaps we could also auto-generate CLI docs? https://click.palletsprojects.com/en/7.x/documentation/

Turns out we already had the first part -- renamed the appropriate page to make it more obvious (#217).

Expanded #217 to add better CLI docs in click


After packaging with Electron, an error occurred when starting the application that says "detect_libc.familySync is not a function".

node 14.0.0 -32bit
electron 11.5.0
When using Electron-Vue to package an application, an error message "detect_libc.familySync is not a function" is displayed in the console when attempting to run the packaged application.

Any solution?


Package context available in the standard library as of Go 1.7

Shall we make a sweep update for package context in all the subdirectories? Replacing golang.org/x/net/context with context.
"As of Go 1.7 this package is available in the standard library under the name context. " (Source: godoc)
Go 1.7 was released in Aug 2016.

Very soon. November 2018 is when 1.6 is turned down for GAE.
We probably could do this now, since 1.9 is GA on GAE.
On Fri, Aug 17, 2018, 4:09 PM Tianzi Cai<EMAIL_ADDRESS>wrote:

Shall we make a sweep update for package context in all the
subdirectories? Replacing golang.org/x/net/context with context.
"As of Go 1.7 this package is available in the standard library under the
name context. " (Source: godoc https://godoc.org/context)
Go 1.7 was released in Aug 2016.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/golang-samples/issues/575, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AABhlsJERXYjT8uBNUBBOJMs4rAJJySKks5uR01CgaJpZM4WCTvg
.

We should probably wait a bit longer before dropping 1.6. November is a couple months away.

Fixed by #601.