Tail call optimize JSON list renderer

From the forum: Attempting to serialize a large list to string with ShowListsAsJSON = true results in a StackOverflowError from runtime.scm. The issue is in the implementation of join-strings, which Kawa does not appear to be optimizing for tail recursion, resulting in the overflow on a sufficiently large list.

I'm unable to duplicate this error.  I created a list of 100,000 numbers and computed its (string) length with ShowListAsJSON= True.   It took a long time, but gave an answer.   Am I not doing a sufficient test case?

Here is the link to the test AIA Abraham posted. I confirmed the bug using this AIA and the emulator. Click the button in the app to load a CSV file as a list. The code then attempts to get the length of the string representation of the list, and in the coercion from list to string the stack overflow occurs (see logcat for details). Turning off the ShowListAsJson property on Screen1 and repeating the procedure will not result in a stack overflow.
https://groups.google.com/group/mitappinventortest/attach/5cd934eb86996/text_overflow_bug.aia?part=0.2

On Aug 8, 2017, at 21:37, hal<EMAIL_ADDRESS>wrote:
I'm unable to duplicate this error. I created a list of 100,000 numbers and computed its (string) length with ShowListAsJSON= True. It took a long time, but gave an answer. Am I not doing a sufficient test case?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Closed via e3d1a87


[SU-41] Display notice in Notebooks tab when notebooks are actively being copied over

Also plays nicely with the page if you happen to create a notebook before the transfer is complete:

<!--
Hello, friend!
Remember to mention what sort of testing/verification you performed in the course of building this PR, i.e. checking functionality in the browser locally.
Thanks!
--!>

@MatthewBemis Notebooks.js will be replaced by Analyses.js once the Analysis Tab feature is released soon so can we apply the changes to Analyses.js too? You can enable the Analysis Tab feature by running `configOverridesStore.set({ isAnalysisTabVisible: true }).


Can't add Font Awesome icon

What happened?
Hosted by github page
Use latest version
I add Font Awesome Icon to my page as below
https://github.com/Reevid/Reevid.github.io/commit/77ef05fab8e1485f5e90f9583032d9fb1b0f3864
But it shows wrong image

https://reevid.github.io
Expected behavior
I want use all free font awesome icons
Steps to reproduce the behavior
See this page https://reevid.github.io
Other
No response

Hey! I just came across a related (basic) issue. Can you tell me how you managed to change the font? So far I tried everything I know.
I recently started with jekyll and the discussion forum seems to be dead. Consider my repo is a "fresh" clone of BB...


Missing Shouminkan turn

./akochan-reviewer.exe --lang en -e -n 0.005 "https://tenhou.net/0/?log=2021082908gm-0089-0000-369d5ba3&tw=0"
In South 4:

The review jumps from Turn 2 to Turn 5, the turn when I made pon into kan is missing.

Same with build from source. Missing Shouminkan on Hatsu. Is this reviewer's problem or algorithm?
WARNING: unable to find player's action in akochan's return, expected to find: [Kakan { actor: 3, pai: Hatsu, consumed: [Hatsu, Hatsu, Hatsu] }], list: [[Dahai { actor: 3, pai: Hatsu, tsumogiri: true }], [Dahai { actor: 3, pai: Man9, tsumogiri: false }], [Dahai { actor: 3, pai: Sou6, tsumogiri: false }], [Dahai { actor: 3, pai: AkaPin5, tsumogiri: false }], [Dahai { actor: 3, pai: Man7, tsumogiri: false }], [Dahai { actor: 3, pai: Man6, tsumogiri: false }], [Dahai { actor: 3, pai: Pin9, tsumogiri: false }], [Dahai { actor: 3, pai: Haku, tsumogiri: false }]]
WARNING: unable to find player's action in akochan's return, expected to find: [Kakan { actor: 3, pai: Hatsu, consumed: [Hatsu, Hatsu, Hatsu] }], list: [[Dahai { actor: 3, pai: Pin1, tsumogiri: false }], [Dahai { actor: 3, pai: Pin7, tsumogiri: false }], [Dahai { actor: 3, pai: Hatsu, tsumogiri: true }], [Dahai { actor: 3, pai: Pin5, tsumogiri: false }], [Dahai { actor: 3, pai: Pin3, tsumogiri: false }], [Dahai { actor: 3, pai: Sou4, tsumogiri: false }], [Dahai { actor: 3, pai: Sou3, tsumogiri: false }], [Dahai { actor: 3, pai: AkaMan5, tsumogiri: false }], [Dahai { actor: 3, pai: Man6, tsumogiri: false }]]

It's an akochan issue.


Added a user setting for distrubute on split.

@alexr00 moving to backlog for now as we want to wait for users to provide feedback on this setting before introducing it just yet.


refactor: duplicate chain extensions

Description
Removing the extension from the devnet and testnet runtimes. Make sure that adding features in the future for one runtime does not impact the other runtimes.
CHANGELOG

Separate the extension from runtimes to standalone crate pop-runtime-extension in runtime/extension
Preview the integration of pop-runtime-extension crate to pop-runtime-testnet: https://github.com/r0gue-io/pop-node/pull/167
Encoding / decoding dispatch error test module is currently defined inside pop-runtime-devnet. This can be improved in the another PR to make something like TestSuiteBuilder that can be reused in multiple runtimes.

How feature can be added in the future?
Create a struct Extension and implement the ReadState trait and CallFilter trait for the Extension to handle the queries from the contract environment.

Got several blocks trying to separate the chain extension from the runtimes. The main issue is the required traits. Let say if Devnet implements fungible feature, it requires pallet_asset::Config but Testnet (and probably Prod) won't. My first attempt is do mutual exclusiveness, having two PopApiExtensionConfig trait that is flipped based on the feature. But feature unification will mess it up so the approach does not work.
// Conditionally use different implementations of `PopApiExtensionConfig` based on the active feature.
cfg_if::cfg_if! {
	if #[cfg(feature = "pop-devnet")] {
		pub use config::devnet::PopApiExtensionConfig;
	} else if #[cfg(feature = "pop-testnet")] {
		pub use config::testnet::PopApiExtensionConfig;
	}
}

My second attempt is to expose two different traits devnet::PopApiExtensionConfig and testnet::PopApiExtensionConfig which both requires PopApiExtensionConfigBase. However, this ends up with another blocker for impl has stricter requirements. Reason for that because StateReadHandler and CallDispatchHandler has PopApiExtensionConfigBase as a trait bound but we provide devnet::PopApiExtensionConfig which implements pallet_asset::Config (more requirements than the one asked by the methods)

Codecov Report
Attention: Patch coverage is 41.72662% with 162 lines in your changes missing coverage. Please review.

Please upload report for BASE (daan/api@1acbd21). Learn more about missing BASE report.

Files
Patch %
Lines

extension/src/lib.rs
8.00%
138 Missing :warning:

extension/src/tests.rs
88.03%
0 Missing and 14 partials :warning:

runtime/devnet/src/config/api.rs
0.00%
10 Missing :warning:

@@             Coverage Diff             @@
##             daan/api     #163   +/-   ##
===========================================
  Coverage            ?   31.43%
===========================================
  Files               ?       34
  Lines               ?     2885
  Branches            ?     2885
===========================================
  Hits                ?      907
  Misses              ?     1955
  Partials            ?       23

Files
Coverage Δ

extension/src/v0.rs
100.00% <100.00%> (ø)

runtime/devnet/src/config/contracts.rs
60.00% <ø> (ø)

runtime/devnet/src/lib.rs
17.31% <ø> (ø)

runtime/devnet/src/config/api.rs
0.00% <0.00%> (ø)

extension/src/tests.rs
88.03% <88.03%> (ø)

extension/src/lib.rs
8.00% <8.00%> (ø)


Question on choice of conditioning

Hi guys,
Once again, congrats on the work! This is a quite cool idea and something that I had been thinking of implementing myself before realising you were already on it.
In my planning, I envisioned using the additional lesion metadata coming with the LIDC-IDRI dataset as additional conditioning information for the lesion generation process. I understand here you only make use of the histogram info which you extract from the dataset and the specific lesion mask.
I wonder if you considered using that lesion data, and if it was a design choice, what was your rationing behind not using it.
Thanks again!
Best,
Pedro

Hi Pedro,
Thanks for your interest! We actually did consider the other attributes from the LIDC dataset. Incorporating these attributes directly alongside the histogram as conditioning information would indeed be straightforward, but we opted not to use them for a few reasons:

Our current focus with LeFusion is on general lesion segmentation tasks, and we wanted our method to be applicable across a wide range of lesion segmentation datasets. The histogram is something that can be derived from any dataset, which makes it a more generalizable choice compared to manually annotated labels like attenuation (texture) and other clinical attributes.

Clinical attribute conditioning is more suited for projects with a clinical focus. We are planning a large-scale extension of this method specifically for nodules, and designing a NoduleBank for generating a broad range of lung lesions. However, evaluating the generation based on clinical attributes poses its own challenges, which is why we’ve chosen to separate this from the current work that focuses on improving segmentation.

While LIDC is one of the largest publicly available lung lesion databases, it’s still relatively small with thousands of samples. This poses challenges for both training and evaluating generative models. Fortunately, we have internal datasets that are tens of times larger, which we’ll be using to scale up our efforts. Stay tuned!

Best,
JC

Thanks very much for the context! Makes sense! Looking forward to see the next iterations of this work.


In this confirmation, we currently include the time at which it is due to be sent, this time is in UTC.
I propose we refactor this part of the code to instead use an Embed, with a the timestamp field.
https://github.com/python-discord/bot/blob/ce819ade482e82ecbc474bce5fb8ac9dd8b37b40/bot/exts/moderation/modpings.py#L107
This would mean that the time would automatically get converted to the user's current time zone by Discord.

I think this should be done, given that all information is contained within the embed.

Yeah, that sounds useful


Support for swift package manager

Would it be possible to support the swift package manager? It'll make the dependency management seamless.

Noticed that it already exists!


feat: Use 'pip-secure-install' recommendations

Use Brett Cannon's recommendations for pip-secure-install to ensure that the environment is fully reproducible and can be installed as securely as possible. This works well with pip-tools!
c.f.:

https://www.python.org/dev/peps/pep-0665/#secure-by-design
https://github.com/brettcannon/pip-secure-install
https://twitter.com/brettsky/status/1486137764315688961

The options used to follow pip-secure-install recommendations are:

A requirements file must be specified to make sure all dependencies are known statically for auditing purposes (--requirement).
No dependency resolution is done to make sure the requirements file is complete (--no-deps).
All requirements must have a hash provided to make sure the files have not been tampered with (--require-hashes).
Only wheels are allowed to have reproducible installs (--only-binary :all:).

In the case of the libraries termcolor and jax, which only ship sdists, the --only-binary option will fail and so the --no-binary option needs to be given explicitly for them.

Thanks @brettcannon! This pattern works really nicely with pip-tools, as you pointed out! :+1:

Glad it worked for you! Thanks so much for letting me know!


Paginated Queries fail with ROUND_ROBIN_REQUEST

OrientDB Version: 3.0.13
Java Version: jdk 8 v191
OS:
Expected behavior
Should send request for OResultSet.hasNext() back to the original server.
Actual behavior
Sends request to a new server who can't find the 'query ' as it has been made on the server from which the first page has been fetched.

Hi,
This should already be fixed in a more recent version of 3.0.x.
Closing


Resume Bot Framework V4 dialog from external service

🚨 The issue tracker is not for implementation questions 🚨

[Question]
Hi,
I am trying to implement payment dialog in Bot Framework V4 and direct line to connect to the bot. The payment should work as the following:
in the chatbot, the user click on a "pay now" button and the payment web page will popup. after the the payment finish, should resume the Bot Framework V4 dialog and here where I am stuck. and idea how can have a call back to the chat bot ?

'How to' questions such as this one are better suited for Stack Overflow. Please feel free to post other questions you have about developing your own features over there so the community at large may help out. If you need help with a Web Chat implementation, you can post a question to the Web Chat tag. Thank you!


Does locust support ‘CSV Data Set Config‘ feature like jmeter

I am newcomer from Jmeter background, I want to get some help.
just like the example in [Quick start] section in doc of locust,
how could I share the thousands  of pairs of userid and password between all threads like jmeter do?
Does locust support ‘CSV Data Set Config‘ feature like jmeter?

Just in case anyone stumbles upon this, there is a longer blog post on this topic at https://www.blazemeter.com/blog/how-to-run-locust-with-different-users/

user_credentials = read_user_credentials_from_csv()
can you please provide code for "read_user_credentials_from_csv()" function too.


JSON support

The article does not mention at all that JSON is also supported. E.g. to define variables you can use both yml and JSON

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

ID: 6724abea-bbdc-bf66-ed5e-3214fa6c3e66
Version Independent ID: 4f8dab21-3f0e-da32-cc0e-1d85c13c0065
Content: Templates - Azure Pipelines
Content Source: docs/pipelines/process/templates.md
Product: devops
Technology: devops-cicd-process
GitHub Login: @juliakm
Microsoft Alias: jukullam

@yahorsi Do you mean using PowerShell to create a JSON object with Convert-FromJson? Please give an example.

No, I mean that you can use JSON to pass parameters to, for instance template

I'll reach out to the product team. I'm not seeing JSON as an officially supported data type.

JSON is supported variouse ways in the various places.
Sample 1 we use right now:
parameters:

name: DeploymentType
displayName: Type of the deployment to be performed
type: string
default: Deployment
values:

Deployment
DeploymentWithoutDb
Maintenance

trigger: none
pr: none
resources:
pipelines:

pipeline: build-pipeline
source: 'build-code-web'
branch: GMI-Edge-Staging
trigger: none

variables:

group: code-global-vars
template: variables.json

extends:
template: ../code-template.yml
parameters:
DeploymentType: ${{ parameters.DeploymentType }}
IsLegacyPipeline: true
Please note in the 1st case we use template: variables.json...
This is very important as it let's you to share same variables across different deployment pipelines, e.g. code deployment pipeline and infrastructure deployment pipeline


Cannot Write, unknown type

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

@immutable
@CopyWith()
@HiveType(typeId: 1)
@JsonSerializable(nullable: false)
class User extends Equatable {
  /// Display name of this.
  @JsonKey(name: "displayName")
  @HiveField(0)
  final String name;

  /// Score of this.
  @HiveField(1)
  final int score;

  /// Level of this.
  @HiveField(2)
  final int level;
  // more code...
  /// Creates a [User] from Json.
  factory User.fromJson(Map<String, dynamic> json) => _$UserFromJson(json);

  /// Creates a Json from [User].
  Map<String, dynamic> toJson() => _$UserToJson(this);

To Reproduce
Steps to reproduce the behavior:
i think it's too much code, it worked prior on 3.1.0 perfectly, but now with hive i don't know what else to change.
Expected behavior
persisting data like in 3.1.0
Screenshots

**Logs **
I/flutter (10060): [2020-06-24 20:54:45.136155 | ConsoleHandler | INFO]
I/flutter (10060): [2020-06-24 20:54:45.136688 | ConsoleHandler | INFO] ------- STACK TRACE -------
I/flutter (10060): [2020-06-24 20:54:45.147581 | ConsoleHandler | INFO] #0      Bloc.onError.<anonymous closure>
package:bloc/src/bloc.dart:146
I/flutter (10060): [2020-06-24 20:54:45.147917 | ConsoleHandler | INFO] #1      Bloc.onError
package:bloc/src/bloc.dart:147
I/flutter (10060): [2020-06-24 20:54:45.148223 | ConsoleHandler | INFO] #2      HydratedBloc.initialState
package:hydrated_bloc/src/hydrated_bloc.dart:38
I/flutter (10060): [2020-06-24 20:54:45.148455 | ConsoleHandler | INFO] #3      CafeteriaBloc.initialState
package:tc_frontend/…/cafeteria/cafeteria_bloc.dart:33
I/flutter (10060): [2020-06-24 20:54:45.148553 | ConsoleHandler | INFO] #4      new Bloc

Paste the output of running flutter doctor -v here.
[√] Flutter (Channel stable, v1.17.3, on Microsoft Windows [Version 10.0.18363.836], locale de-DE)
    • Flutter version 1.17.3 at C:\flutterSDK\flutter
    • Framework revision b041144f83 (3 weeks ago), 2020-06-04 09:26:11 -0700
    • Engine revision ee76268252
    • Dart version 2.8.4

[√] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at C:\androidsdk
    • Platform android-29, build-tools 28.0.3
    • ANDROID_HOME = C:\androidsdk
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)
    • All Android licenses accepted.

[√] Android Studio (version 3.2)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 31.2.1
    • Dart plugin version 181.5656
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)

[√] VS Code, 64-bit edition (version 1.45.0)
    • VS Code at C:\Program Files\Microsoft VS Code
    • Flutter extension version 3.11.0

[√] Connected device (1 available)
    • AOSP on IA Emulator • emulator-5554 • android-x86 • Android 9 (API 28) (emulator)

Additional context
i added the hive annotations, what else do i need to do to make it work again?
Cheers and thanks for the effort :)

If  _$UserToJson produces valid json, hive should be fine to save it. Don't forget to call user.toJson() from hydrated bloc's overridden toJson method
When using adapter call Hive.registerAdapter(UserAdapter()); from your main, where UserAdapter is class UserAdapter extends TypeAdapter<User>

Closing this for now since there isn't enough information to reproduce the issue locally. Feel free to comment with a link to a sample which reproduces the problem and I'm happy to continue the conversation 👍


Task hook adjustments + cli

#102 Closes
Adds a new, optional hook_admin to the InstantiateMsg and Config of the task-queue.
This hook_admin is the only user authorized to add/remove hooks for a given receiver.
CLI now has a command for adding and removing task hooks.
TODO:

Also add some docs how to do that.

Should we add this on the CLI readme or start establishing some more  extensive docs in the WIKI for specific commands?

Noah's requests

yeah for sure. i can see reasons why it would be nice to have global hooks, but i think the admin should also be able to enable the task submitter to add a hook/callback with their submission

for my immediate use with the AI bouncer, task-specific callbacks would be ideal, so that multiple DAOs can use the same AVS AI backend, instead of each having to spin up their own AVS

TODO: allow task submitter to add a hook with their submission

If Noah likes it, then it is good.


Docker image tshock-5.0.0 does not exist.

Just what the title says. beardedio/terraria:tshock-5.0.0 is not available.

this is fixed with my latest updates


API Routing

Must have both "Get" and "post" routes

First set of routes completed for main, dashboard, signup and login.

Routes from the homepage to the other pages are the next steps in this process.

Routing is complete


Adapt damage screen for low-information scenario

We have to choose what information to present in detail, and what information to not present to players in a scenario without information. E.g., are the effects of all measures known? Is the exact house protection level known? Where necessary, values can be replaced by ? on the view-damage screen.

Probably the low-info scenario will not be used anymore, so this has been labeled as not urgent.


ValueError: Axis limits cannot be NaN or Inf

Here is an issue from Flint with some curve using a monitor containing a 0.
Traceback (most recent call last):
  File "/users/valls/workspace/bliss.git/bliss/flint/widgets/utils/refresh_helper.py", line 55, in __update
    self.__aggregator.flush()
  File "/users/valls/workspace/bliss.git/bliss/flint/utils/signalutils.py", line 105, in flush
    callback(*args, **kwargs)
  File "/users/valls/workspace/bliss.git/bliss/flint/widgets/curve_plot.py", line 388, in __structureChanged
    self.__redrawAllScans()
  File "/users/valls/workspace/bliss.git/bliss/flint/widgets/curve_plot.py", line 643, in __redrawAllScans
    self.__redrawScan(currentScan)
  File "/users/valls/workspace/bliss.git/bliss/flint/widgets/curve_plot.py", line 684, in __redrawScan
    self.__updatePlotItem(item, scan)
  File "/users/valls/workspace/bliss.git/bliss/flint/widgets/curve_plot.py", line 870, in __updatePlotItem
    self.__updatePlotZoom(updateZoomNow)
  File "/users/valls/workspace/bliss.git/bliss/flint/widgets/curve_plot.py", line 874, in __updatePlotZoom
    self.__view.plotUpdated()
  File "/users/valls/workspace/bliss.git/bliss/flint/widgets/utils/view_helper.py", line 55, in plotUpdated
    self.__plot.resetZoom()
  File "/users/valls/Software/miniconda3/envs/blissenv/lib/python3.7/site-packages/silx/gui/plot/PlotWidget.py", line 2890, in resetZoom
    self._forceResetZoom(dataMargins=dataMargins)
  File "/users/valls/Software/miniconda3/envs/blissenv/lib/python3.7/site-packages/silx/gui/plot/PlotWidget.py", line 2854, in _forceResetZoom
    self.setLimits(*newLimits)
  File "/users/valls/Software/miniconda3/envs/blissenv/lib/python3.7/site-packages/silx/gui/plot/PlotWidget.py", line 2321, in setLimits
    self._backend.setLimits(xmin, xmax, ymin, ymax, y2min, y2max)
  File "/users/valls/Software/miniconda3/envs/blissenv/lib/python3.7/site-packages/silx/gui/plot/backends/BackendMatplotlib.py", line 1052, in setLimits
    self.ax2.set_ylim(min(y2min, y2max), max(y2min, y2max))
  File "/users/valls/Software/miniconda3/envs/blissenv/lib/python3.7/site-packages/matplotlib/axes/_base.py", line 3622, in set_ylim
    bottom = self._validate_converted_limits(bottom, self.convert_yunits)
  File "/users/valls/Software/miniconda3/envs/blissenv/lib/python3.7/site-packages/matplotlib/axes/_base.py", line 3158, in _validate_converted_limits
    raise ValueError("Axis limits cannot be NaN or Inf")
ValueError: Axis limits cannot be NaN or Inf

Here is the content of xmin, xmax, ymin, ymax, y2min, y2max
(-0.02, 1.02, -inf, inf, -inf, inf)

It would be nice to filter inf?

Nonetheless i don't know why both min and max of y1 is inf here. With my data, only one side should be inf.

No matter of the data, when it reaches matplotlib in should not be inf... do you have some data to reproduce that ?

Monitor data have to be filtered at the application side prior to perform the normalization.

Yes, but there is/should be some defensive testing at the silx level.

Like ignoring the request to set the limits without raising an exception but a warning?

I will try to reproduce it with pure silx components. For now i just have 2 curves, and 1 with inf values. But there is other objects in the scene.

@vasole yes, to me it is similar to handling negative values with log scale axes, just discard those and log a warning.

How to reproduce:
from silx import sx; import numpy
sx.plot(numpy.arange(4), numpy.array([0, 1, 2, float('inf')]))

It's funny we didn't had this issue so far...

Could be part of a bug fix release.

Well, since matplotlib is rising a exception it was not needed to handle it at our side. Basically every application is/was aware thus making sure the behavior was correct prior to set the limits.

You mean filtering HDF5 raw data before displaying it?

When you are using a monitor you have to get the indices of the non-zero monitor values and send to the plot the x values and the normalized y values of those indices. Handling the use of monitors at the plot side (as old Newplot was doing) was more complex than making a "dumb" plotting widget that only know about showing x and y values and putting the responsibility on the plot caller.

This is a general issue: any curve data with infinite values no matter where it comes from will raise an exception. That should be avoided... and it's easy to do.

Setting the limits to infinity is what raises an exception.
Matplotlib has no problem at all showing the curve you have shown.

From my backtrace, silx set itself the limits to infinite.

Yes, but silx has some problem.
To me it is an expected behavior to raise an exception if on calls setLimits with infinite values and that should not change, but it is a bug if this exception is raised when one calls PlotWidget.addCurve with data containing infinite values.

The silx calculation of the limits then is buggy.

Yes, it ignores NaN but not inf, that's the issue.

I made a fix, I had some tests and make a PR.


Improve linting

Feature Description
Currently, k6-operator uses only yaml-lint for manifests. Add Golang linters.
Suggested Solution (optional)
k6 repo can be used as a main reference.
Already existing or connected issues / PRs (optional)
No response

@yorugac  Hi, I'd like to work on this issue.
Can we use golangci-lint?

@u5surf sure. As mentioned in description, k6 repo is the reference point - it uses golangci-lint.


Code style, format and README.

Add original author to License.
Improve code style.
Improve format and usage instructions.

@randomdude999


Rename arXiv (CoRR) to arXiv

Minor data clean up on arXiv papers.

Track related changes to other publication venues in #6.


DOC: minor edits to Array chunk documentation

What does this PR implement?
It properly formats compute_chunk_sizes and to_dask_array. Before this PR, they were mis-formatted (italic).
It also adds more details on compute_chunk_sizes.
This modifies the docs from #5312.

[n/a] Tests added / passed
[n/a] Passes black dask / flake8 dask

Taken care of in #5390. Just restarted the CI, so hopefully things pass now


When using with Redux, does not passing childContext from Provider to child components

Redux recommends that you wrap your Router with a Provider from react-redux which passes the store to all child components via React's context. Container components then find this store and pull state off of it. The router setup looks like this:
<Provider store={store}>
  <Router history={browserHistory}>
    <Route path='/' component={IndexContainer}>
  </Router>
</Provider>

However, react-engine builds a RouterContext based on the matched params from react-router. This does not pass any available context to the children component, making the recommended use case of Redux with React blow up.

@ryanbahniuk redux support isn't available yet. We are working on it.

@ryanbahniuk any work arounds you came up with?
@samsel any eta on this?

@SOSANA I ended up not using Provider or connect from react-redux and instead hooking up the store manually and subscribing to its change events in my containers. It's not ideal, however, because I am duplicating some of the basic logic of connect without some of its more advanced performance based logic.

@ryanbahniuk thanks for sharing, gonna try and do the same till a fix is applied

@samsel has this issue been resolved? I have a project we are kicking off tomorrow and would like to include this dev boilerplate.
any gotchas or things I should know to have a smooth implementation? @ryanbahniuk @taosbeta @samsel ? I noticed the docs haven't been updated in a while and just wanted to make sure I wire it up correctly. I will be implementing this with express.js

@SOSANA Hey, here is a simple sample: https://github.com/taosbeta/react-engine-starter.
Basically there are 2 places need to setup the redux env:

config for server side rendering: https://github.com/taosbeta/react-engine-starter/blob/master/config/config.json#L16
client side rendering bootstrap: https://github.com/taosbeta/react-engine-starter/blob/master/public/views/client.js

The idea behind is that now react-engine wraps the component to render (e.g react router) with Provider if these config/settings above are present.

@SOSANA any comments on the approach?

we can take this approach and merge to mainstream if you guys are ok

@samsel yes we are using the provider approach to this project, I have to double check the links you provide but could of sworn one of them was based off kraken
here's an example on client side
ReactDOM.render(
  <Provider store={store}>
    <Router history={browserHistory} routes={getRoutes} />
  </Provider>
  , document.querySelector('.container'));

@samsel thanks so much for your quick response! Also if all goes well could add updated boilerplate for your project or another example with express.js implementation

@taosbeta can I still wire this up like you did using express.js? if not suggestions? Thanks bro for replying back so soon!

@SOSANA yah, we can wire using express.js: by providing the routes config in the create arguments.
https://github.com/paypal/react-engine/blob/master/README.md Usage On Server Side -Setup in an Express app section.

@taosbeta @samsel @ryanbahniuk For this client project doing microservices archecture so haven't implement FE yet, just wrapping up the server api as we speak.
One thing I noticed and maybe you guys could chime in but with other server-side mounted on inital render you will notice a delay in css or a flicker, basically serves the inital load from the server before the css loads on client by a few seconds, have you guys experienced this at all? and if so any hacks to avoid this issue? haven't implemented this for the FE app yet but will be soon and just want to avoid this, hoping no issues with this :) I have experienced this with other serverside mounting before

Using React on the server-side should limit any kind of flash from JS hookup because it will diff on the client-side and not re-render. I've never noticed any real CSS flicker with server side rendering.

progress on redux?

Any news on this guys? Redux integration with new ReactRouter DOM?


Add and_then combinator

Intended to solve #22. I think the addition of the Other variant is an useful regardless but I am a bit vary of the and_then function since there is some overlap with then then function and I could also think of a third function which would have a signature like:
fn and_then<F, O>(self, f: F) -> AndThen<Self, F>
         where F: FnMut(Self::Input, Self::Output) -> ParseResult<O, Self::Input>;

Nifty, thanks!


Bamboo plugin installation failing on supported version of Bamboo

The latest version 1.11.0 of the plugin states that our version of Bamboo 6.10.4 should work. However when installing we get an error (logs attached). We have contacted Atlassian Support (email text attached) and they were able to replicate the error on the same version of Bamboo. Can you update the plugin or at least the supported versions of the latest version to avoid this confusion?
email.txt
logs.txt

I am also seeing the same error when I install the allure-bamboo plugin on bamboo 6.3.0

I ran into the same problem last night, and discovered that it is due to the published 1.11.0/1.12.0 were built for Java 9+.  Our Bamboo instance is running on Java 8, so of course barfs when trying to deploy the plugin.
If I could find the right branch that 1.12.0 was built from I'd rebuild it myself, but the master is currently at 1.11.0-SNAPSHOT.  That's not in any way encouraging.

For any doubt on that: the pom does indicate <compiler.version>1.8</compiler.version>, however, if you take the jar file and crack it open:
$ wget https://github.com/allure-framework/allure-bamboo/releases/download/1.12.0/allure-bamboo-1.12.0.jar
$ unzip allure-bamboo-1.12.0.jar
$ file module-info.class
module-info.class: compiled Java class data, version 53.0
version 53.0 == Java SE 9
see: Java class file

From my testing, it was the change from net.sf.json-lib to com.fasterxml.jackson.core in 1.11.0 that is causing this issue.
https://github.com/allure-framework/allure-bamboo/compare/1.10.0...1.11.0
Reverting the change in the pom.xml and AbstractAddInfo.java removes the inclusion of the module-info.class in the allure-bamboo.jar, and the jar successfully deploys on Bamboo 6.9.1 running on Java 1.8.  I'll leave it to the module authors to officially address this.

@battmush this is correct, will be restore in future release.
The versions 1.11 and 1.12 in not supported for versions < 7


Adding a footer which contain contact info

Description
(Write your answer here.)
Checklist

[x] I am making a proper pull request, not spam.
[x] I've checked the issue list before deciding what to submit.

Related Issues or Pull Requests
(Write your answer here.)
Add relevant screenshot or video (if any)
(Add here)

Great to solve this issue, please go through my pull request


[fixes #841] ensure testEnvironment.{beforeEach,afterEach} are released

@cibernox & @stefanpenner

Thank you for the contribution. Can you please sign our CLA? http://contribute.jquery.org/CLA/

Thank you for the contribution. Can you please sign our CLA? http://contribute.jquery.org/CLA/

Send it to legal, lets see what they have to say.
@jzaefferer can you provide feedback now, I can atleast iterate to make sure this lands quickly. This leak is causing grief on many large apps, so we should try to get this in as soon as possible.

The actual change looks good. I can fix the style issue when landing this, if you don't get around to it.

@jzaefferer style fixed, I'll push for +1 on the CLA quickly.
Are we looking at a quick turn-around on release once merged?

I was looking at it, but then I had to stop for some scheduled meetings. It took me a while because I went to the original sources to take a deep look.
This fix seems good to me.

It also seems ok to release QUnit 1.19.0 after this.

I'm ready to merge jquery/qunit#842 I just need to confirm if you signed the CLA.
Merging it, I'll release QUnit 1.19, probably on Monday morning

Ya, still pending legal :+1:.

sorry for the delay folks, still pending legal :+1:

ok, I'm holding the 1.19 release until we get the approval.

Sorry about the delay, I have just gotten the legal :+1: and signed the CLA.
Apparently there was a small concern about some specific verbiage, if anyone is interested I can put them in contact with them. On a positive note, anyone at Yahoo! now has :+1: to contribute.

Thanks, @stefanpenner! It's merged now.

sorry for the delay


Android namespace issue

Incorrect package="com.singular.flutter_sdk" found in source AndroidManifest.xml: /Users/****/.pub-cache/hosted/pub.dev/singular_flutter_sdk-1.3.2/android/src/main/AndroidManifest.xml.
Setting the namespace via the package attribute in the source AndroidManifest.xml is no longer supported.

Thanks for notifying us regarding the issue.
Fixed with released version 1.4.1


OneDrive storage type doesn't work on Android/Chrome

I open draw.io in chrome on Android v9 phone (Note9). I select OneDrive storage and open existing. This opens a separate OneDrive browser window and I navigate to the library/folder/file I want to open from OneDrive and all it does is download the file onto my android device's downloads folder.
Why doesn't this behave like you'd expect, i.e. why doesn't this open the selected drawing in draw.io and enable auto saving directly to oneDrive?
I have confirmed that Google Drive storage type works fine in this same environment.

It will be fixed in the next release

It will be fixed in the next release


WIP Auto-authenticate Tessel with your computer

I'm not especially inclined to trust the keypair library after a quick read of the source. Incredibly overcomplicated ways of doing important things like obtaining random numbers that can be done with simple Node APIs, and in general doing too much easy-to-subtly-break crypto in unaudited JS.
Most of the other libraries just shell out to ssh-keygen, which won't work on Windows, though...

It seems like this should be broken up into two components before being merged:

Using a postInstall script tag in the package.json, create HOME/.tessel if it doesn't exist and place a new SSH key there. That way, it's automatically created when the tessel package in installed.

Running tessel setup --name OPTIONAL_NAME will transfer that SSH key over to the Tessel and optionally set the hostname of the device.

@kevinmehall @Frijol thoughts?

@johnnyman727 that sounds appropriate

@johnnyman727 What's going on with this file organization? Should I move init and setup into the tessel folder?

I placed tessel specific commands in the tessel folder. Higher level CLI commands can be in the lib folder. Setup.js can stay where it is.

It would be better to create .tessel and the SSH key on tessel setup if they don't exist, rather than on postinstall. Postinstall will lead to permissions issues if people sudo to install globally, or if we choose to install the package with a system package manager.

@kevinmehall why would you run into permissions issues on postinstall but not tessel setup?

Just for the record, we talking about this IRL and @kevinmehall explained that the issue would arise when someone runs sudo npm install -g tessel which sets root permissions on the file that gets created (and which users do all too often). By not using postInstall, we can avoid permission issues (unless someone does sudo tessel setup, at which point, they've probably got bigger issues).

Replaced by https://github.com/tessel/t2-cli/pull/58 and https://github.com/tessel/t2-cli/pull/59


Unnecessary move animation in nested

Version
3.2.30
Reproduction link
github.com
Steps to reproduce
npm run dev
en :  and click green or red button to change sort
中文 : 然后点击绿色或红色的按钮调整列表顺序
What is expected?
en : Only the outermost div has transition animation
中文 : 只有最外层的 div 有过渡动画
What is actually happening?
en : Inner div also has transition animation
中文 : 内部的 div 同样产生了过渡效果

en : i find add a css can fix this bug
中文 : 我发现添加一个 css 可以修复这个 bug
.scx-group-list-move .scx-group-list-move {
transition: unset;
}

I would not see this as a bug, it's a side effect of nesting the same transition into itself that needs handling from the developer.

🧐

Playground

Here my example of bleeding custom class triggers, since my bug report has been closed: Vue SFC Playground example


Fix race condition on task retry

Related issue:
https://github.com/python-arq/arq/issues/482

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##             main     #487      +/-   ##
==========================================
- Coverage   96.27%   96.10%   -0.18%
==========================================
  Files          11       11
  Lines        1074     1078       +4
  Branches      209      190      -19
==========================================
+ Hits         1034     1036       +2
- Misses         19       21       +2
  Partials       21       21

Files with missing lines
Coverage Δ

arq/worker.py
97.37% <100.00%> (+0.20%)
:arrow_up:

... and 1 file with indirect coverage changes

Continue to review full report in Codecov by Sentry.

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

@samuelcolvin @chrisguidry 🙏


Rename the track for the sake of consistency

See the discussion https://github.com/exercism/discussions/issues/161

Looks like it broke exercism.io.

Ahhh! Yes, there's a good chance that this is what caused it. I'll fix that ASAP and report back here. So sorry about the mess.


Ledger API registerCredentialDefinition behaviour when ledger or wallet are reset

While adapting AATH for 0.3.0, I've noticed that there is a change in Ledger API's registerCredentialDefinition method internal behaviour that makes a bit difficult to manage the case where either agent's wallet or ledger are reset (mostly common in development environments):

If we attempt to register a credential definition that exists on the ledger but not in the wallet, it will throw an error (as opposed to 0.2.x, where it just returns the credentialDefinitionId silently);
If we attempt to register a credential definition that exists on the wallet but not in the ledger, it will return the stored credential definition. However, no actual credential definition will be registered on the ledger

This new logic makes sense for production deployments, but I'm wondering if there is a way to allow this inconsistencies in an elegant way (in AATH I've called IndyLedgerService directly, but this will not store the CredentialDefinitionRecord in the wallet).
A few more details (some not strictly related to this) that I don't want to forget analyzing:

Why registerSchema won't throw an error when an schema is present in ledger but not in the wallet?
./docs/getting-started/ledger.md is outdated and the example is wrong: sets didIndyNamespace fields when it should be indyNamespace.
By the way, why there are still a few getting started documents here (ledger.md and proofs.md, which is also outdated) and not directly in aries-framework-docs repo?

@TimoGlastra of course


Filling Trapped Chests locks them

The title basically says it: in MCEdit when using the fill tool (hotbar slot 4) to fill a selection with trapped chests (in my specific case replacing stone with trapped chests) makes them inaccessible to the player after saving the world and opening it in Minecraft.
Minecraft: 1.8 with Forge+Liteloader
MCEdit: window-title says "Unified v<IP_ADDRESS> for Minecraft 1.8/1.9"
OS: Linux Mint 17.1 Rebecca, Kernel: 3.13.0-37-generic

There's an error in the data for the trapped chests.
Editing it's NBT data and changing the id to Chest (instead of Trap) makes the trapped chests accessible in game. (1)
This will be fixed in the next release.
(1) Place trapped chests with any method you want. Select the aera where you placed the chests. Then select the NBTEdit filter and search for TileEntities in the selected aera. In the results panel, deploy each element and change the id tag to Chest.


Registry-sweeper upgrade for multitenant registry

💡 Description
Using opensearch serverless
We now have one single opensearch serverless URL.
Sweeper still takes a single node as argument, AWS infrastructure takes care of running the needed sweepers (one for each node). There will be one task definition per node.
Create new roles read/write access that will be associated to the sweeper's ECS task.
We should not need to signed the HTTP requests to connect to OpenSearch and to do it we should use a AWS SDK (see https://docs.aws.amazon.com/opensearch-service/latest/developerguide/serverless-sdk.html). @sjoshi-jpl has example of using this code.
If we want to run sweeper from a local laptop we would still need the signed URLs to be implemented.
⚔ Parent Epic / Related Tickets
No response

Alex is making progress on this ticket.

Status: Sagar to take a look at necessary roles to perform this action.

Currently being tested on MCP - image has been pushed to ECR but need ECS settings to continue.  Will follow up with @sjoshi-jpl tomorrow
Backwards compatibility with non-MT registry has been manually tested.

Deployed in dev but issues with 403 errors.

status: auth is handled, but lidvids appear to be typed as text instead of keyword, causing sweepers to fail

@alexdunnjpl is unblocked on this ticket and will resume the testing when he is done with the resolution of the data migration.

Status: nontrivial refactoring required.
Need to look at EC2 scratch scripts in aoss branch and existing work in multitenancy-update branch, merge the work (i.e. cherry-pick anything relevant from aoss - *DON'T FORGET THE SEARCH-AFTER POLLUTION GUARD), and poach whatever's relevant into the cognito wrapper work, too.
Separate credential instantiation from auth/opensearch-py client instantiation.
Will be straightforward, but if I don't write it down now I'll need to figure it out again later.

So more work is needed.

Unsure about current state/status - probably will need to cherrypick/reimplement PR from new branch
https://github.com/NASA-PDS/registry-sweepers/pull/122

@jordanpadams pulling from sprint for now - but if this is a high priority, feel free to add it back.  I've lost track of where this sits in terms of priority and being blocked by other work.

Cherrypick complete(?) and open in #130, but not tested yet

Status: blocked in testing this on prod.

Status: IAM role created, but looks like permissions need tweaking. Working with @sjoshi-jpl to resolve this.

@jordanpadams @tloubrieu-jpl @sjoshi-jpl Is the ECS sweepers deployment in another ticket?  If not, feel free to re-open this if you don't want to create a separate one.

@jordanpadams @tloubrieu-jpl psa ancestry sweeper terminated on mcp prod1 due to out-of-memory condition.
All other sweepers/nodes have completed successfully.  Recommend against attempting to re-run on prod1 - it can wait until we have it deployed to ECS (where we can size accordingly).


Tagui recording issue

I am trying to record the desktop/browser action using the TAGUI setup which was shared. but i am not able to auto record the actions performed by the User.
Kindly suggest me is that auto  recording of the Desktop/Browser is possible using TAGUI for Windows.

Hi @vasanthaiwan, you may have missed the Chrome extension section in readme. To record user actions on web browser, you'll need to install TagUI Chrome extension from Chrome web store.
The readme link above has more details on usage and limitations. If you want to record desktop actions, I recommend checking out WorkFusion or UiPath - they are commercial RPA apps with recording functions.


Models2

Supersedes #93, which supersedes #88.
Should probably be squashed due to several false starts.  Sorry for the messy history.

@ethanwhite Doesn't look like the history is any cleaner this way? Sorry if I did something silly.

Actually, maybe they've updated something and I just didn't notice given that there's a lot going on here. Are any of the commits from the PR I merged this morning listed here or is this just the new commits for this PR?

I feel like the diff looks different actually. I wonder if it just took a few minutes to get updated after the merge (which would be a great change in the GitHub interface if that happened).

Oh, I think you're right.

Yeah, it's definitely clean now and I'm pretty sure it wasn't when I first look. Sorry for the confusion. I'm on it.

Closing since #93 was actually fine. GitHub just keeps getting better.


Contact

@capull0 : Could you explain to me how to proceed to reset a crystal skylander imaginator ( portal xbox 360) with ubuntu ?
I don't understand the steps to obtaint the reset ?
Coul'd you help Me ?
Thank'a a lot.
Regards.

Intall libusb and dependencies
sudo apt-get install libusb 1.0.0 libusb-1.0-0-dev
To compile  in / SkyDumper-master
Create build directory
mkdir build
to get in
cd build
compile
cmake ..
make
After that it works the easiest way to paste the created SkyDumper file into the bin directory
sudo thunar
Call him then with sudo to not get bored with the rights


OTR: Typo in term name GO:0034897

GO:0034897 4-(1-ethyl-1,4-dimethyl-pentyl)phenol monoxygenase activity
I think it should be monooxygenase not monoxygenase. This is consistent with other child terms of GO:0016705.
Also, the cross-reference to EC on GO:0034897 and GO:0016705 (and maybe others) goes to a non-existent page (1.14). Should this instead be 1.14.-.- (https://enzyme.expasy.org/EC/1.14.-.-) ?
Thanks.

Hi
I can fix the typo.
At this time, due to the way the EC2GO file is generated, we have removed 3 digit ecs with a -.- from xrefs because if they were assigned to a specific enzyme, the EC2GO file got the mapping of GO:0034897 would get all of the enzymes that fell under EC 1.14
So all of the enzymes on https://enzyme.expasy.org/EC/1.14.-.- would map go  GO:0034897
In fact, you cannot add a 3 digit with -.- ; Travis Checks will flag it.
I am in the process of removing all three digit ecs from anything other than grouping terms where they make more sense, and replacing them with a 4 digit EC if available.
Does this make sense?

Yes, I think so. Thanks Harold.

Hi again
I may have garbled this a bit, but also when we had EC:x.x.-.-
MGI (and Zfin that I know of) pick up UniProt records to map to our genes. For these we get interpro domains and EC numbers. . If the UniProt record has an EC:x.x.-.- (which it might if either there as no better EC (4 digit), or not fully curated (ie, Trembl records) , then what happens is that when we then run an EC2GO script, a gene was getting any GO id that had an EC that mapped under the EC:x.x.-.-; this resulted in many incorrect GO assignments based on EC number.
I can fish out the original ticket, which might have actually be an old sourceforge thread.

Fixed misspelling and removed 3 digit EC xref


Attempt to fix powerlevel9k on systems where the perl fallback fails

This is a hacky workaround I've been playing with on OSX when messing with the prezto integration. readlink doesn't have -f and perl appears to return an empty string... so I fell back to python which may be relatively slow but it should be effective.
Please feel free to leave suggestions... I'm not really happy with how this looks at the moment, even if it works.

On OS X this part should work realpath -P $POWERLEVEL9K_INSTALLATION_PATH. It does for me

I installed the coreutils using brew install coreutils for this to work

However, that requires an additional package which is never mentioned in the documentation. This attempts to work around the issue if you don't have coreutils installed.
That being said, I do appreciate the workaround. It's nice to have something that works until there's some sort of fix merged in.

If you want a pure macOS approach, try stat -f%N $POWERLEVEL9K_INSTALLATION_PATH

@belak Thanks for the fix.
I agree that coreutils should not be a dependency for P9K. Could you try the following code:
0=${(%):-%N}
dir=${0:A:h}

This piece of code comes from this comment. I always wanted to test it, but never found time for it.. It may be a good way to get rid of a lot of the old cruft. :)

Thanks for the comments, @dritter and @onaforeignshore! And yes, agreed, we can't add coreutils as a dependency.  Let us know what you find from the test @dritter suggested!

That's actually very similar to what I tried initially... however, when loaded with prompt_powerlevel9k_setup, the usage of that is inside a function (an autoloaded prompt_powerlevel9k_setup which gets put into the fpath) and $0 ends up being the pwd, not the location of the script. We actually had this issue with prezto as well and ended up storing the result of ${0:h} as ZPRESTODIR so it could be used inside the function itself.

Another option may be changing the options for readlink because that's found on a default OSX install as well.

@belak what is a bit odd to me is that the whence -v $0 trick does not work for you.. This is the only (more or less reliable) way of finding the source file of a defined function in ZSH (without using external tools). It should be available if you are using ZSH > 5.0.8.
See source
Hmm. Maybe it is a chicken-egg problem. We ask in line 29 if prompt_powerlevel9k_setup is defined as function. The actual function definition happens much later in the script. Could you change that line to whence -w $0?

I'm actually closing this in favor of #494 because it removes all external dependencies to read symlinks.
@dritter My issue wasn't with getting the filename. That part worked fine. POWERLEVEL9K_INSTALLATION_PATH ended up getting set to /Users/belak/.antigen/bundles/sorin-ionescu/prezto/modules/prompt/functions/prompt_powerlevel9k_setup using whence $0 branch (this is following the set up from the prezto instructions of this repo). The issue was with resolving that symlink.

@belak Ah. I see. Apologies for getting you wrong.


[AutoPR containerinstance/resource-manager] Fixing Error in Naming Convention for Rows and Cols in Terminal Size

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/2651

(message created by the CI based on PR content)

Codecov Report

Merging #2169 into restapi_auto_containerinstance/resource-manager will increase coverage by 1.04%.
The diff coverage is 33.33%.

@@                                 Coverage Diff                                 @@
##           restapi_auto_containerinstance/resource-manager    #2169      +/-   ##
===================================================================================
+ Coverage                                            52.05%    53.1%   +1.04%
===================================================================================
  Files                                                 5149     4778     -371
  Lines                                               132333   119598   -12735
===================================================================================
- Hits                                                 68889    63514    -5375
+ Misses                                               63444    56084    -7360

Impacted Files
Coverage Δ

...nce/models/container_exec_request_terminal_size.py
57.14% <33.33%> (ø)
:arrow_up:

...ork/azure/mgmt/network/v2017_10_01/models/probe.py
33.33% <0%> (-8.78%)
:arrow_down:

...etwork/v2017_11_01/models/packet_capture_result.py
31.25% <0%> (-8.75%)
:arrow_down:

...etwork/v2017_10_01/models/packet_capture_result.py
31.25% <0%> (-8.75%)
:arrow_down:

...17_10_01/models/verification_ip_flow_parameters.py
35.71% <0%> (-8.74%)
:arrow_down:

...re/mgmt/network/v2017_10_01/models/ipsec_policy.py
35.71% <0%> (-8.74%)
:arrow_down:

...re/mgmt/network/v2017_11_01/models/ipsec_policy.py
35.71% <0%> (-8.74%)
:arrow_down:

...17_11_01/models/verification_ip_flow_parameters.py
35.71% <0%> (-8.74%)
:arrow_down:

...rk/v2017_11_01/models/connection_monitor_result.py
26.31% <0%> (-8.47%)
:arrow_down:

.../network/v2017_10_01/models/load_balancing_rule.py
26.31% <0%> (-8.47%)
:arrow_down:

... and 557 more

Continue to review full report at Codecov.

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


bugs Calendar divide page is error when max row count is 5

              app:cv_maxRowCount="5"
               app:cv_hasBoundaries="true"

               app:cv_orientation="horizontal"
               app:cv_outDateStyle="endOfGrid"
               app:cv_scrollMode="paged"

-===============================================
I think there are bugs in this code in fun generateWeekDays in MonthConfig.kt
// Add more rows to form a 6 x 7 grid //// 6 is fixed ??
if (outDateStyle == OutDateStyle.END_OF_GRID) {
while (weekDaysGroup.size < 6) {
val lastDay = weekDaysGroup.last().last()
val nextRowDates = (1..7).map {
CalendarDay(lastDay.date.plusDays(it.toLong()), DayOwner.NEXT_MONTH)
}
weekDaysGroup.add(nextRowDates)
}
}

Could you explain the current vs expected behavior? A screenshot would also help.

is there a solution? i see last week half cut off on my calendar


Display of Dense Layers

Is there a better way to display dense layers to be more visually appealing?

All the other layers are visually clear but the layer of 256 nodes is hard to interpret.  Is there a way to reshape the visualization? Or do I need to add a reshape layer to the model?

This is the model I'm using:

@vee-upatising yeah that's a good question. What would you suggest here? Reshaping to a rectangle or a square shape?

Would there be a way to change the shape to a square shape like the single output layer? Although this would be a little misleading since the nodes aren't actually arranged this way.

It works if num_units is a square. For example 10 would not work. So I guess something like a square of size int(sqrt(num_units)) + 1 would make sense if num_units is larger than a number say 16.

@vee-upatising if you have time to write a small pr it would be appreciated!

PR: https://github.com/philipperemy/keract/pull/113

Pushed in 4.2.4


Parse a section at a time - very large file

Hi,
Sorry if I'm asking this question in the wrong place:
I have a very large XML file - 10GB - and I'm using the following code to parse it:
function parsexml(callback) {
    fs.readFile(__dirname + '/sample_larger.xml', function (err, data) {
        parser.parseString(data, function (err, result) {
            pData = result
            console.log('====> Done Parsing XML');

            callback(null)
        })
    })
}

Then I have second function addVertices which uses pData - to make a Gremlin query to add that data into my Azure Cosmos DB.
My question is - copying the 10GB into that variable pData - seems like a bit of a waste - is it possible instead to parse one section at a time - for example if I specified the XML header I'm after.
Assume my xml looks something like:
<songs>
  <song>
   //details I want
  </song>
</songs>

Is there something like:
function parsexml(callback) {
    fs.readFile(__dirname + '/sample_larger.xml', function (err, data) {
        parser.parseSection("song", function (err, result) {

            //do my gremlin query into my Cosmos DB

            callback(null)
        })
    })
}

Any advice/help appreciated.
Thanks

copying the 10GB into that variable pData - seems like a bit of a waste

Remember, when you do pData = result, you're not copying anything - pData then holds a reference to the same object as result. both of them are pointing to the same object

Right - so nothing really to worry about.
Nevertheless, is it possible to parse a block of XML at a time - based on a tag I specify?

@psmod2 I had the same problem. I've made a function that does this and follows the xml2js format (it's pretty slow, but doesn't run out of memory):
https://github.com/tfso/njs-tfso-xml/blob/master/src/streamParse.js
Usage: https://github.com/tfso/njs-tfso-xml/blob/master/test/testStreamParse.js
It returns in the same format as the xml2js version here:
https://github.com/tfso/njs-tfso-xml/blob/master/src/parse.js
Would be awesome to get this functionality in xml2js as well.

See https://github.com/Leonidas-from-XIV/node-xml2js/issues/137 and https://github.com/Leonidas-from-XIV/node-xml2js/issues/102

@psmod2  any workaround?


Add VNLA

added the folder and the logo image

Please check now.  ive updated everything including the one for testnet in

remitano repo
Please check now. ive updated everything including the one for testnet in remitano repo thanks, kind regards.
@ngocbv thanks.


HSEARCH-2242 Convert FacetManager implementation to no longer use Fil…

…ters

https://hibernate.atlassian.net/browse/HSEARCH-2242

I have pushed a new version to address @gsmet 's feedback, but hadn't seen the comments from @gunnarmorling yet..

re-pushed with improved names. I'll merge it if tests are green..


Add activating vaults and amount to stk home

close #203

ACK 1bf3837 -- i tested this on the demo (merged on Daniela's demo branch)


Multiple Devise Models. One using token

So I have a rails app where Admins can login and also Clients, but Clients can only login from a separate app that uses j-toker to sign them in. The problem is allowing both. I created a seperate API controller for devise_token_auth but it doesn't seem to get called it always seems to call the standard ApplicationController's :authenticate_admin! method which tells me its using ApplicationController. Any ideas on how to fix the problem?
Update: From what I can tell its because this gem's main controller inherits from DeviseController which inherits from Devise.parent_class which is auto set to ApplicationController. Still can't get anything to work but if that helps anyone help me than awesome.

Bump, I also have two models for Auth, and would like one of them to use tokens and the other to use normal Devise auth. Is there a way to do this right now?

Hi there @pducks32 and @frankmarineau   ,
In an effort to cleanup this project and prioritize a bit, we're marking issues that haven't had any activity in a while with a "close-in-7-days" label. If we don't hear from you in about a week, we'll be closing this issue. Obviously feel free to re-open it at any time if it's the right time or this was done in error!
If you are still having the issue (especially if it's a bug report) please refer to our new Issue Template to provide some more details to help us solve it.
Hope all is well.


Feature   make available field

Added a new boolean field (available_item) that maintains item availability.

older branch


Take 2 on #[ram] soundness

Submission Checklist 📝

[x] I have updated existing examples or added new ones (if applicable).
[x] I have used cargo xtask fmt-packages command to ensure that all changed code is formatted correctly.
[ ] My changes were added to the CHANGELOG.md in the proper section.
[x] My changes are in accordance to the esp-rs API guidelines

Extra:

[x] I have read the CONTRIBUTING.md guide and followed its instructions.

Pull Request Details 📖
Description
Closes #1110, closes #1650, closes #1649
New PR as this is a full rewrite. It also fixes #1650 since both #[ram(persistent)] and #[ram(zeroed)] require the same check that the type can take an all-zero bit pattern. I used bytemuck::Zeroable since it seems like the most widely used, rather than implement. It did mean adding bytemuck as a dependency of esp-hal, but I used version 1.0.0 to maximize compatibility with whatever version is probably already in users' dependency graphs.

Error message for usage on non-Zeroable types
error[E0277]: the trait bound `NonZero<u8>: bytemuck::zeroable::Zeroable` is not satisfied
   --> src/bin/ram.rs:40:18
    |
40  | static NON_ZERO: NonZeroU8 = NonZeroU8::MIN;
    |                  ^^^^^^^^^ the trait `bytemuck::zeroable::Zeroable` is not implemented for `NonZero<u8>`
    |
    = help: the following other types implement trait `bytemuck::zeroable::Zeroable`:
              bool
              char
              isize
              i8
              i16
              i32
              i64
              i128
            and 73 others
note: required by a bound in `assert_is_zeroable`
   --> …/esp-hal/esp-hal/src/lib.rs:229:40
    |
229 |     pub const fn assert_is_zeroable<T: bytemuck::Zeroable>() {}
    |                                        ^^^^^^^^^^^^^^^^^^ required by this bound in `assert_is_zeroable`

For more information about this error, try `rustc --explain E0277`.

If exposing the #[doc(hidden)] function name in the error message is undesirable, I use the method I did in #1649.
Testing
The updated ram example ran on an S3 dev board. I do not have any of the RISC-V ESPs, so someone else will need to test that once it's implemented.
Additional To-dos

[x] Update esp-riscv-rt

It will now only zero persistent ram after initial boot, since, I think[^1] that's the only time the RTC ram is not preserved. IMO, this should be expanded to all the reset reasons that could theoretically happen before the init had finished. It seems to me that that would be anything not caused by a watchdog or software (for S3: brown out, clock glitch, efuse error, and usb uart/jtag resets).
I also updated esp-riscv-rt to match the xtensa behavior. I don't have any risc-v esps to test on, but it did seem to work for esp32c3 in qemu.
Additionally, I noticed the rv-init-data and rv-init-rtc-data features (and the esp-riscv-rt features they enable) seem to have only been used for the old direct boot support. I also attempted to find uses of them in a GitHub-wide search and did not see any outside of forks of esp-hal. How do you feel about removing these features and perhaps merging the existing zero-rtc-fast-bss and new zero-rtc-fast-persistent into one rtc-ram feature?
[^1]: Based on the Reset and Clock chapter of the ESP32, ESP32-S3, and ESP32-C3 reference manuals

Additionally, I noticed the rv-init-data and rv-init-rtc-data features (and the esp-riscv-rt features they enable) seem to have only been used for the old direct boot support. I also attempted to find uses of them in a GitHub-wide search and did not see any outside of forks of esp-hal. How do you feel about removing these features and perhaps merging the existing zero-rtc-fast-bss and new zero-rtc-fast-persistent into one rtc-ram feature?

I tend to agree - maybe good to hear others opinion on that, too

Looks quite good overall - I guess after adding a CHANGELOG.md entry this should be good to go
One thing maybe worth considering would be to have a (doc-hidden) function in esp-hal like should_reset_rtc_persisted_ram (naming is debatable) which decides on the reset reasons for both - Xtensa and RISC-V. Would make it easier to change the decision when to zero memory (if we need to reconsider the condition). Not mandatory - just an idea

Great. My last concern is with the edge case of an externally triggered RTC RAM-preserving reset (brown out, power glitch, jtag reset, etc.) occurring very early in the first boot such that the zero initialization gets skipped, causing undefined behavior. If this is a worry, which kinds of resets should be added to the list to trigger zeroing?
I wrote the condition in assembly to match the rest of the risc-v init, since I'm not sure how it interacts with the soundness issues that prompted the decision to avoid Rust there.

Probably all the reset reasons which are chip-reset or system-reset should always trigger zeroing:
(from C6 TRM)

I wrote the condition in assembly to match the rest of the risc-v init, since I'm not sure how it interacts with the soundness issues that prompted the decision to avoid Rust there.

Yes - makes sense. In theory when being cautious writing such a function it should be okay but sure - in assembly it's easier to know what really happens 👍

If that looks good, I'll update the changelog and rebase.
The risc-v code could also use some testing on real chips—I've only been able to test on an S3 and qemu.

Seems like the description in the TRM fooled me a bit and "System Reset: resets the whole digital system, including LP system." doesn't mean LP/RTC RAM is reset. So only "Chip Reset: resets the whole chip." seems to be a reason to zero the memory. (Sorry for the confusion)
Maybe we should just assume "Brown-out system reset" is "Chip Reset" (it can also be System Reset according to the TRM) and only zero out memory for these two reasons

I interpreted that to mean that a minor brown out could cause a system reset (preserving RTC RAM) and would give a reset code of 0x0F, while a more severe brownout could cause a chip reset with code 0x01. If it could give code 0x0F even after causing a chip reset, then I agree—definitely need to init after that.
I had expanded the list to guard against something like this:

Initial boot starts (reset reason = ChipPowerOn)
While the bootloader is running, something triggers a reset
Second boot starts and completes with a non-ChipPowerOn reset reason, meaning persistent ram is not zeroed
Undefined behavior

I am unsure which resets could occur at step 2, hence the long list. Perhaps anything that could trigger a reset that early would result in ChipPowerOn? Would a JTAG reset be possible that early?
Perhaps the time between a reset that could cause the above sequence becoming possible and the ram init finishing is simply so short that it should not be considered. I personally dislike that solution as a soundness-purist, but I'll follow your lead there.

t.b.h. I guess the above scenario is not completely impossible in a development setup but maybe unlikely in production - not too sure
At least not zeroing on e.g. WDT resets can be useful, I guess. On the other hand, if code is manipulating data stored in RTC RAM and gets interrupted by a WDT reset (or any other data preserving reset) the data might be invalid then. I start to wonder if there would be a 100% solution other than e.g. using checksums?

What about this?

Update the docs to warn about the potential for resets during writes and to avoid resets immediately after first boot
Require persistent statics to impl AnyBitPattern
Zero after ChipPowerOn and SysBrownOut

Maybe also some set of SysClkGlitch, CoreEfuseCrc, and/or CorePwrGlitch?

Also, from reading this ESP-IDF source, it looks like some cases of brownout resets are detected by the IDF, so esp-hal will report them as CoreSW. Could those be the system brown-out resets described in the TRM?

Shall I split the fix for #1650 out into its own PR so that isn't held up by finalizing these specifics?

Update the docs to warn about the potential for resets during writes and to avoid resets immediately after first boot

Yes, we probably should explain these things more

Require persistent statics to impl AnyBitPattern

That is basically just MaybeUninit, right? But I agree it's probably the only safe thing to do then

Zero after ChipPowerOn and SysBrownOut

Yes, sounds good.
Regarding splitting the PR: Ideally, I would love to see all of this in the next release - not sure if splitting out parts of the PR helps or causes more work for you. I'm fine with both I guess

I noticed this note on SocResetReason::ChipResetReason:

In ESP-IDF this value (0x01) can also be ChipBrownOut or ChipSuperWdt, however that is not really compatible with Rust-style enums.

Combined with this part of the IDF, I think that a chip level brownout will not give SysBrownOut. Based on this assumption, I'm only initializing after ChipPowerOn.

That is basically just MaybeUninit, right?

I guess it's close, yeah. But this can be used without any unsafe:
#[ram(rtc_fast, persistent)]
static BOOT_MODE: AtomicU8 = AtomicU8::new(0);

match BOOT_MODE.load(order) {
	// ...
}

Well, I missed the note in more recent versions of the AnyBitPattern docs that explicitly forbids interior mutability because of bytemuck::cast_ref, etc. I've replaced it with a new esp_hal::Persistable trait that matches the actual requirements for #[ram(persistent)]. It's implemented for a minimal set of integers, floats, portable_atomic::Atomic*, and arrays that should cover most usage.

Looks good but I'm on vacation this week -not sure if and when I will get to this before next week. So, if anyone likes to review this: feel free


Add Keimyung University

Hey,
We had a hard time the first time we had to set our password at Keimyung University, so I thought this would be a good place to post that.

Thanks for the PR.
Can you distinguish between the 2nd and 3rd screenshots? What are the messages?

Thanks for the PR.
Can you distinguish between the 2nd and 3rd screenshots? What are the messages?

One is telling you that you can't have more than 3 times the same characters and the other that you can't have more than 2 consecutive numbers in a row.

Any news?
If you need more information and if you need me to change something, don't hesitate to ask.

Thank you for the contribution!


Jinghan/merge list/describe feature to get meta feature

This PR combines list feature and describe feature to get meta feature.
related to #651

Does the -o yaml option wait until after the migration?

Does the -o yaml option wait until after the migration?

@lianxmfor yes, this should be done in another PR, tracked in issue #694

The command describe feature can now be deleted?

The command describe feature can now be deleted?

deleted!


Negative regex matches

Carto already has =~, !~ is its complement.

Not so sure it would be an enhancement, in XML this can be very useful:
<Filter>![key].match('.*value.*')</Filter>


update search lib & tem disable algolia

Purpose

Update Algolia client to V3 latest
Temporarily disable search
For 4.1.0 docs

Related to https://github.com/wso2/docs-apim/issues/5395


Add mrcnn as a dependency instead of copying the code

https://github.com/matterport/Mask_RCNN/tree/master/mrcnn is used in your repository. Why do you include it by copying? The module is pip-installable. Alternatively, a reference to the GitHub repository could be added.

We have done some changes to the original scripts present in mrcnn folder. Specifically, the config.py file has a lot of changes. Since we are using Tensorflow 2.0 now, so we had to change some functions in model.py as well. Although, unnecessary scripts that are not required during inference could be deleted.

@khurramHashmi Great work. Getting mrcnn running with Tensorflow 2.0 is a great achievement (cf. https://github.com/matterport/Mask_RCNN/pull/1896 and the PRs referenced there)! Please consider to propose your modifications for the upstream mrcnn. Many, many people would benefit from that.


This world has been deactivated to prevent cascading errors.
Generic word length does not define a method for the fixnum class.

(O) Method: M\ object before=?
(O) Method: M\ sequence <=>
(O) Method: M\ object length
(O) Word: no-method
(O) Method: M\ object throw
(U) Quotation: [
        OBJ-CURRENT-THREAD special-object error-thread set-global
        current-continuation -> error-continuation set-global
        [ original-error set-global ] [ rethrow ] bi
    ]

The exact error is "2.0" has-version? because GL_VERSION glGetString is returning f.

This is fixed now.

How was this fixed?

Fixed when we fixed #1185 in this patch 20a1c09a, I believe.
My AMD Radeon graphics cards Linux machines no longer crash on startup, which was the bug.

What about the better error idea here:
https://github.com/slavapestov/factor/issues/926#issuecomment-32686946

That's in the repo. d6140845
:cake: :fish_cake:

Hah okay!


Monitoring - Thanos images are out of date

What kind of request is this (question/bug/enhancement/feature request):
enhancement
Steps to reproduce (least amount of steps as possible):

run rancher v2.4-head or master-head
edit the system-chart catalog to branch dev-v2.5
enable the cluster monitoring v0.1.1 with Thanos
after all workloads are active, go to check the logs of the workload prometheus-operator-monitoring-operator

Result:
the following warning is in the log
level=warn ts=2020-05-27T19:16:12.682742854Z caller=operator.go:1199 component=prometheusoperator msg="prometheus key=cattle-prometheus/cluster-monitoring, field spec.baseImage is deprecated, 'spec.image' field should be used instead"
level=warn ts=2020-05-27T19:16:12.682776323Z caller=operator.go:1209 component=prometheusoperator msg="prometheus key=cattle-prometheus/cluster-monitoring, field spec.thanos.baseImage is deprecated, 'spec.thanos.image' field should be used instead"
how to enable Thanos in monitoring: https://github.com/rancher/rancher/issues/24214#issuecomment-569356843

the bug fix is validated on rancher:master-f1a410a97e2abf8c81c62195a7b40f10007398f0-head single install.
Steps:
enable the cluster monitoring v1 0.1.3 on a downstream cluster with Thanos enabled
Results:
the warning does not see in the logs of prometheus-operator-monitoring-operator.


add Windows port of perf.c

i do not agree on this : the purpose of the perf counter is to be fast.
for the 1st comment, i use a conditional branch which will almost always be true (if there is no memory for 48 bytes, we are in a very bad shape)
for the 2 other comments, i don't understand the interest of allocating on a stack a variable and using it. I don't  know if the compiler can optimize it and remove the stack variable
on the other hand, i see a bug : the 'start' field is set in lexbor_perf_end() while it should be 'end'

@vtorri
My remarks are related solely to the readability of the code.
In the first comment it's better to check that the memory is not allocated and you need to go out. In this entire project, these checks are used.
For the rest of the comments. Not critical. You can leave it as it is, but for better readability ...

Awesome!
Thanks!


WebEX Teams Space Link Not Working

The WebEX Teams Space link doesn't seem to be working:  https://eurl.io/#bWHTbWz1Z

That link shows up here: https://github.com/CiscoDevNet/cml-community#getting-help
It's part of Hank's first commit for this repository.  I'm not sure what the intent was.  We do not have a WebEx space for general purpose CML support.
Since it shows up right under the section about trying CML in the DevNet Sandbox, I guess that link is supposed to open the Cisco Modeling Labs (CML) Sandbox Support space.  But I'm not sure whether it's really appropriate to post that link in the cml-community repository's README.  That WebEx space is specifically for help using CML in the DevNet Sandbox, and we do not want CML users trying to ask support questions there.
I propose that we fix it by removing that line.  We could maybe add some additional links, such as

CML Documentation
Introduction to Network Simulations with Cisco Modeling Labs - Free training on Cisco U
CML Videos and Webinar Recordings

@tombry Can you either commit your desired changes or offer a PR?


Compile modules exporting packages that have no Java source, but have other JVM source

Allow compiling modules with a module-info.java that exports a package that has no Java source, but that has source in another JVM language (like Kotlin) in the same build.
Should be achieved by using --patch-module instead of --module-path for the outputs of source sets in the same build that are in JavaCompile#classpath.

Gradle 6.4 now has Module support built-in. Please open an issue on https://github.com/gradle/gradle for this use case and how you wound imagine this to work in Gradle.
You can already do it by adding --patch-module arguments to the corresponding compile/run/test tasks - e.g.:
tasks.compileJava {
    // this is needed because we have a separate compile step in this example with the 'module-info.java' is in 'main/java' and the Kotlin code is in 'main/kotlin'
    options.compilerArgs = listOf("--patch-module", "org.module.kotlin=${sourceSets.main.get().output.asPath}")
}


Error in Description of filters of Contents argument

It says that "" (empty string) would be copy all files and "" (single asterisk) is copy all files recursively. But if I remember correctly, it shoudl be "" (single asterisk) for copying all files and "**" (double asterisk) for copying all files recursively!

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

ID: 81f02378-b85b-d3a4-d3c8-f07d935367da
Version Independent ID: fab57f76-35d2-4024-6f7f-1740d7b24743
Content: Copy Files task - Azure Pipelines
Content Source: docs/pipelines/tasks/utility/copy-files.md
Product: devops
Technology: devops-cicd
GitHub Login: @vtbassmatt
Microsoft Alias: macoope

Good catch, thanks. From looking at the HTML tag soup, I suspect we did some kind of automated copy from another content system and this got mangled. I'll fix it.


Fix next-auth CVE-2023-27490 and some dependencies issue

Fix next-auth CVE: https://github.com/advisories/GHSA-7r7x-4c4q-c4qf

Remove @types/stripe because package has been deprecated: This is a stub types definition. stripe provides its own type definitions, so you do not need this installed

Move some @types package and balel plugin package to devDependencies

Thanks!

Uh-oh. Looks like there are issues with next-auth 4.20.1:
https://github.com/nextauthjs/next-auth/issues/6949

Uh-oh. Looks like there are issues with next-auth 4.20.1:
nextauthjs/next-auth#6949

Okey, I will track this issue, and make a new PR when this issue had fixed.


feat: multiply season points by 100 to remove decimal places

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Hero Section Redesign

I abstracted all the hero banner to a customized component to keep the UI consistent.
Need to redesign the font size and layout and sentences.

This section uses in many pages, Board, ManageEvent, FAQ, EventDetails, GettingStarted, Help, MyDashboard, PrivacyPolicy, Waivers.
@XiaoStevens , Do you interested in this task?

related to #1159

I redesigned those hero section, including sentences. Please check this Figma file. (Gray Rectangle covered are not changed.)

finished the layout. wait for the response from marketing regarding the sentences.


AArch64 support on Linux

Thanks to @afonso360 Cranelift now supports everything on AArch64 that cg_clif needs when compiling for Linux. CI is currently testing cross-compilation. Native may work too, though I don't know if that would hit https://github.com/bjorn3/rustc_codegen_cranelift/issues/1060.

🎉  I'm happy to see this! I have ran into some issues running test_rustc_tests.sh natively, but I haven't had time to properly investigate. At a first glance it didn't seem to be anything aarch64 specific though.
I'll look into it again