App not displaying

Describe the bug
Keep getting the No preview available message
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-z9i7bkbl
Steps to reproduce
Cannot preview the app at all. Numerous errors to resolve.
Expected behavior
App should display
Screen Recording / Screenshot

Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 131
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 145604715,
  "usedJSHeapSize": 81753483,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = RySmeth
Chat ID = 4c542d8169cd

Additional context
No response

Hi @codewithtyler - thanks for helping.
Apologies I'm not a developer so not sure what is useful and what isn't.
There are a lot of messages there when the app tries to initiate so I've saved the code into a Word doc and attached.
Hopefully a few common threads here that I might be able to help direct Bolt how to fix?
Thanks again
CodeErrors.docx

Sorry somehow this got missed in my notifications. Can you go to the terminal and run the following command:
npm install @next/react-refresh-utils --save-dev
and then see if you can get the preview to load? if that doesn't load we can look at your new error and see if anything has changed.


Don't run tests tagged with ignore

Tests will not be run if one of the tags is Ignore. This will be logged to the console.

@jharewinton is this needed? if not close it (or let me know and I'll close it!)

Looks like a relic of Marc's. I'll get rid of it as we've not had any
issues using ignore tags
On 23 March 2015 at 15:01, John Duffell<EMAIL_ADDRESS>wrote:

@jharewinton https://github.com/jharewinton is this needed? if not
close it (or let me know and I'll close it!)
—
Reply to this email directly or view it on GitHub
https://github.com/guardian/scala-automation/pull/35#issuecomment-85038351
.

--

Visit theguardian.com. On your mobile and tablet, download the Guardian
iPhone and Android apps theguardian.com/guardianapp and our tablet editions
theguardian.com/editions.  Save up to 57% by subscribing to the Guardian
and Observer - choose the papers you want and get full digital access.
Visit subscribe.theguardian.com
This e-mail and all attachments are confidential and may also be
privileged. If you are not the named recipient, please notify the sender
and delete the e-mail and all attachments immediately. Do not disclose the
contents to another person. You may not use the information for any
purpose, or store, or copy, it in any way.  Guardian News & Media Limited
is not liable for any computer viruses or other material transmitted with
or as part of this e-mail. You should employ virus checking software.
Guardian News & Media Limited is a member of Guardian Media Group plc. Registered
Office: PO Box 68164, Kings Place, 90 York Way, London, N1P 2AP.  Registered
in England Number 908396


Logfilters or improved

This supersedes #160.
In this PR compared to the original contribution the code duplication of several JSONRPCClient functions is removed from the tests. Instead, a mock transport instance is used in the rpc client instance.

Coverage increased (+2.2%) to 53.227% when pulling 528627d1332d559254ffafbe649330dc7bd04b83 on logfilters_or_improved into 60a56d249b029819cc6c49873a0953374e711c8d on develop.


Extend documentation for the minimal tflite example.

Add tflite:: prefix for all tflite types for consistency.
Extend documentation of how to read the input/output buffers of the interpreter.

Bonus:

Adds missing entry for android_arm64 in .baselrc.

Are there any more required changes or what is the status for this PR?

Are there any more required changes or what is the status for this PR?

@nordstroem  Nothing to do at this moment. This is waiting for reviewer approval.  Thank you!


Fatal: react-responsive.js: Unknown option: .../node_modules/react/react.js.Children

Hi, I'm trying to use this library with react-native-web, as recommended here:
But I'm immediately getting a fatal error when I try to import the dependency.
Line 4 of my file LoginScreen.js says:
import MediaQuery from 'react-responsive'

Which is throwing a fatal error in webpack:
webpack: Compiling...
Hash: 7910906c4a439e4c0a80
Version: webpack 2.3.3
Time: 1770ms
    Asset     Size  Chunks                    Chunk Names
bundle.js  2.71 MB       0  [emitted]  [big]  main
chunk    {0} bundle.js (main) 2.61 MB [entry] [rendered]
  [747] ./~/react-responsive/dist/react-responsive.js 2.13 kB {0} [built] [failed] [1 error]
  [863] ./src/LoginScreen.js 4.31 kB {0} [built]
     + 900 hidden modules

ERROR in ./~/react-responsive/dist/react-responsive.js
Module build failed: ReferenceError: [BABEL] my_project/node_modules/react-responsive/dist/react-responsive.js: Unknown option: my_project/node_modules/react/react.js.Children. Check out http://babeljs.io/docs/usage/options/ for more information about options.

A common cause of this error is the presence of a configuration options object without the corresponding preset name. Example:

Invalid:
  `{ presets: [{option: value}] }`
Valid:
  `{ presets: [['presetName', {option: value}]] }`

For more detailed information on preset configuration, please see http://babeljs.io/docs/plugins/#pluginpresets-options. (While processing preset: "my_project/node_modules/react/react.js")
    at Logger.error (my_project/node_modules/babel-core/lib/transformation/file/logger.js:41:11)
    at OptionManager.mergeOptions (my_project/node_modules/babel-core/lib/transformation/file/options/option-manager.js:226:20)
    at my_project/node_modules/babel-core/lib/transformation/file/options/option-manager.js:265:14
    at my_project/node_modules/babel-core/lib/transformation/file/options/option-manager.js:323:22
    at Array.map (native)
    at OptionManager.resolvePresets (my_project/node_modules/babel-core/lib/transformation/file/options/option-manager.js:275:20)
    at OptionManager.mergePresets (my_project/node_modules/babel-core/lib/transformation/file/options/option-manager.js:264:10)
    at OptionManager.mergeOptions (my_project/node_modules/babel-core/lib/transformation/file/options/option-manager.js:249:14)
    at OptionManager.init (my_project/node_modules/babel-core/lib/transformation/file/options/option-manager.js:368:12)
    at File.initOptions (my_project/node_modules/babel-core/lib/transformation/file/index.js:212:65)
 @ ./src/LoginScreen.js 4:21-48
 @ ./src/App.js
 @ ./index.web.js
 @ multi (webpack)-dev-server/client?http://localhost:3000 ./index.web.js
webpack: Failed to compile

I'm going to keep investigating. Hope this bug report will be helpful, and in case anyone else comes across this issue.

Since it's a babel error, that's only being triggered once this library is imported, I wonder if it could be related to the babel.presets section of this lib's package.json?
Deleting the contents of that array resolves the build error, and the library works! 🎉
Now I wonder what sort of longer-term fix there is, rather than needing to manually edit the dependencies after they've been downloaded.

You should ignore node_modules from your babel config in webpack
Add exclude: /node_modules/ to the babel-loader config

Thanks, that worked with a little tweak. I had to use
exclude: /node_modules\/react-responsive/,

because some of my other dependencies do require babel.
Thanks for your help and the great library. 😄

@dsernst Typically the pattern people use is to pre-compile their libraries, which reduces compile time and doesn't make any assumptions that people are using tooling like webpack. Don't think I've ever run into a dependency that requires you to babel it, but glad it worked for you.

Yeah, that makes sense.
It looks like the main problem is react-native-tabs, which probably assumes it's being run in react-native (with its default webpack config), not react-native-web.


Add Psych Texture Atlas resources

Links to 2 videos and an example script to assist in using Texture Atlas in Psych Engine

Absolutely kind of a self plug (i made the resources) but figured it would be cool to add since it's not a very documented part of Psych Engine


Fix #553: Add support for TonY jobs

Implemented TonyFetcher, TaskMemoryHeuristic, TonyMetricsAggregator.
Added unit tests for each.
Disabled TonyFetcher by default in the included FetcherConf.xml.
Bumped source and target JDK versions to 1.8 since TonY requires that.

Please ignore the "Unknown" max memory, as that is due to the ProcfsBasedProcessTree that TonY uses only working on Linux, not Mac. On a Linux machine, max memory metrics are collected and emitted.

 I will complete the review by tomorrow.

Hi @mkumar1984 ,
Thanks for reviewing the code. I believe the CI build failed due to Checkstyle issues. I've pushed some new commits to fix them. Hopefully the next CI run passes.
Regarding com.linkedin.tony.events, they are part of the tony-core jar.

LGTM.


Feature/line chart supports legend

This PR allows for the addition of legend items to a Line Chart.
LineChart will now accept an optional legend property on its data object which should be an array of strings as observed below.
const data = {
  labels: ["January", "February", "March", "April", "May", "June"],
  datasets: [
     {
       data: [20, 10, 4, 56, 87, 90],
       color: (opacity = 1) => `rgba(0, 255, 255, ${opacity})` // optional
     },
     ...
   ],
   legend: ["Rainy Days", "Sunny Days", "Snowy Days"] // optional
 };

Colors for the icons associated with a legend item are derived from the corresponding color defined in the dataSet. The color of the first item in the dataSet will also be assigned to the first legend item's icon and so forth. Since the legend property is optional, a Line Chart created without a legend property will appear the same as it did previous to this PR.
For testing, a reviewer may add or remove legend items from the data object in data.js, observing the changes to the application as they do. Legend items should:

Be evenly spaced as the number of legend items is increased or decreased.

Cause no affects to the Line Chart if they are not present.

Be centered by the mid point of their text, rather than their icon. For example, if there are two legend items, the mid point of their texts should be centered on 33% and 66% of the legend's width respectively. Centering by the icon rather than the text makes the legend appear right-justified.

Future work: Adding many legend items or legend items with very long labels will lead to overlap between items. This is generally not noticeable in 1-4 legend items but becomes an issue when 4 or more items are present. Future work could include a dynamic resizing of the legend to account for multiple rows of legend items if their length grows too great. Additionally, we could use rotation on the texts, similar to how we do with the labels for the x-axis
This PR should close #158.

Thank you for your contribution Simon!
This is now available in v4.2.0


PYSCC - Wait_for evaluate error messages at run-time

For wait_for method errors, if there is any count check being performed return the # of items as evaluated at run-time in the error message.

There's no plausible way to evaluate the errors at run time unless we pass a callable object that will evaluate the string for us. In which case, what I would rather do is update the default error messages to include the number of items found, current location, and current window title to help with debugging.

https://github.com/neetjn/py-component-controller/pull/67


Added Marky Mark

Added Markymark library to the 'Text' category
Project URL
https://github.com/m2mobi/Marky-Mark
Description
Offers a unique way of parsing and displayed Markdown with many layout possibilities. Can render into native views or plain NSAttributedString. Markdown flavor, layout and rendering can be extended and overwritten to customise the parser into any way needed.
Why it should be included to awesome-ios (optional)
None of the Markdown parsers in the list offer parsing into native layout which is highly customisable, including remote images, nested lists and custom bullets.
Checklist

[x] Only one project/change is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 9 / tvOS 10 or later
[x] Supports Swift 3
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

          1 Error

:no_entry_sign:
Found 2 link issues

Link issues by awesome_bot

Line
Status
Link

1765
301
https://github.com/gizmoboy7/VCFloatingActionButton redirects tohttps://github.com/giridharvc7/VCFloatingActionButton

2536
Error
https://www.ioscreator.com/  Connection refused - connect(2) for "www.ioscreator.com" port 443

  Generated by :no_entry_sign: Danger

Thank you for contributing to awesome-ios 😄


blueprint-compiler: 0.6.0 -> 0.8.1

Description of changes
v0.8.1

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

So this update will break 2  (giara und textpieces) of 20 (bottles bottles-unwrapped denaro dialect dynamic-wallpaper elastic giara gnome-extension-manager gradience halftone health identity imaginer junction raider tangram textpieces video-trimmer whatip) packages.
How do i proceed with this? Do i have to patch these packages in this pr? @dasj19 @zendo
20 packages updated:
blueprint-compiler (0.6.0 → 0.8.1) bottles bottles-unwrapped denaro dialect dynamic-wallpaper elastic giara gnome-extension-manager gradience halftone health identity imaginer junction raider tangram textpieces video-trimmer whatip

$ nix --experimental-features nix-command build --no-link --keep-going --option build-use-sandbox relaxed -f /home/bene/.cache/nixpkgs-review/rev-12239f8be59ef127ca04c2a8243555a39bdf271d/build.nix
error: builder for '/nix/store/h065kmrjbs7kjz9sn08i3m76wn4gjlpm-giara-1.0.1.drv' failed with exit code 1;
       last 10 log lines:
       >   26 |        item { custom: profile; }
       >      |                       ^
       >
       > error: Unexpected tokens
       > at ../data/ui/headerbar.blp line 26 column 22:
       >   26 |        item { custom: profile; }
       >      |                     ^
       >
       > 2 errors
       > ninja: build stopped: subcommand failed.
       For full logs, run 'nix log /nix/store/h065kmrjbs7kjz9sn08i3m76wn4gjlpm-giara-1.0.1.drv'.
error: builder for '/nix/store/lcy9gc3g9mf1y089qg75v3pmxd3zlzvw-textpieces-3.4.0.drv' failed with exit code 1;
       last 10 log lines:
       > at ../resources/ui/CustomToolPage.blp line 43 column 3:
       >   43 |  .TextPiecesToolSettings tool_settings {
       >      |  ^
       >
       > error: Cannot convert 2.5 to integer
       > at ../resources/ui/Editor.blp line 45 column 26:
       >   45 |          margin-bottom: 2.5;
       >      |                         ^
       >
       > ninja: build stopped: subcommand failed.
       For full logs, run 'nix log /nix/store/lcy9gc3g9mf1y089qg75v3pmxd3zlzvw-textpieces-3.4.0.drv'.
error: 2 dependencies of derivation '/nix/store/b9d57n4nsj2l9yhvb22z9ydm1n2v7516-review-shell.drv' failed to build
2 packages failed to build:
giara textpieces

18 packages built:
blueprint-compiler bottles bottles-unwrapped denaro dialect dynamic-wallpaper elastic gnome-extension-manager gradience halftone health identity imaginer junction raider tangram video-trimmer whatip

Hi @BenediktBroich it seems like "giara" expects version 0.6 of blueprint-compiler, I even tried bumping the version of giara to 1.1.0 but it still relies on version 0.6 of blueprint-compiler.
So I guess we have to ask upstream for 0.8 support.
As I unfortunately don't have much spare time at the moment, and I no longer use giara, maybe we should consider dropping giara from nixpkgs so you can carry on with the update of blueprint-compiler.

How do i proceed with this? Do i have to patch these packages in this pr?

or mark it broken

How do i proceed with this? Do i have to patch these packages in this pr?

or mark it broken

inside this pr?


Refactor \Client and add models to better manage data and to utilize usage of SDK

ps. this PR breaks BC


🏃 Update test scripts

What this PR does / why we need it:
Adds golint.sh and codegen.sh and make gofmt.sh fail if files
are changed

/test-integration

/cc @kashifest
/cc @fmuyassarov
/cc @furkatgofurov7

/test-integration

/test-integration


Discussion: adding a createProviderFromHook utility

I absolutely love this library, and am using it in several big projects.  So let me start with a thank you!
I wanted to suggest a small feature.  I am more than happy to create a PR, but wanted to discuss it first.
The problem: boilerplate
When working with Context (both the native React context and the Context of this library), especially with hooks, I find myself repeating this same exact boilerplate:

Create a context
Export a custom Provider that wraps context.Provider
Export a custom hook that wraps useContext/useContextSelector but throws if the Provider is missing
Create a type definition for the context's value

For example, here's the boilerplate required for this repo's Usage example:
const MISSING_PROVIDER = Symbol('MISSING_PROVIDER');
const context = createContext<CounterInfo>(MISSING_PROVIDER);

type CounterInfo = [ { count1: number, count2: number }, setState: any ];

export const CounterProvider = ({ children }) => (
  <context.Provider value={useState({ count1: 0, count2: 0 })}>
    {children}
  </context.Provider>
);

export const useCounterSelector = (selector: any) => {
  return useContextSelector(context, (value) => {
    if (value === MISSING_PROVIDER) throw new Error("Missing `CounterProvider`");
    return selector(value);
  });
};

The solution: createProviderFromHook
All of this boilerplate can be eliminated!  Implementation details aside, here's the finished result:
export const [ CounterProvider, useCounterSelector ] = createProviderFromHook(() => useState({ count1: 0, count2: 0 }));

This syntax is inspired by the wonderful constate library,.
A few noteworthy things about this createProviderFromHook function:

The raw context is kept private, so you don't have to remember to import createContext from the right place
The custom Provider boilerplate is eliminated.
The custom useCounterSelector boilerplate is eliminated, and automatically checks for missing providers.
Strongly typed!  All type arguments are correctly inferred, no manual type defs needed.

Let me know what you think about adding this utility.  The implementation would not be big or cause bloat, and this feature would be extremely useful to anyone using this library.

Here's an implementation of createProviderFromHook for reference.  I can create this as a PR, and add tests etc, if desired:
src/createProviderFromHook.ts:
import { createElement, FC, PropsWithChildren } from 'react';
import { createContext, useContextSelector } from './index';

const MISSING_PROVIDER = Symbol('MISSING_PROVIDER');

export default function createProviderFromHook<THookValue, TProviderProps>(
  providerHook: (props: TProviderProps) => THookValue,
  defaultValue: THookValue | typeof MISSING_PROVIDER = MISSING_PROVIDER,
) {
  const context = createContext(defaultValue);

  const Provider: FC<PropsWithChildren<TProviderProps>> = (props) => {
    const value = providerHook(props);
    const { children } = props;
    return createElement(context.Provider, { value } as any, children);
  };

  const useSelector = (selector: any) => useContextSelector(context, (value) => {
    if (value === MISSING_PROVIDER) throw new Error('Missing `CounterProvider`');
    return selector(value);
  });

  return [Provider, useSelector] as const;
}

That's exactly what I do in react-tracked with useSelector.
So, I don't want the duplication.
More importantly, this library is something that will be obsoleted when React provides a native solution. So, we shouldn't add a new feature.
If react-tracked doesn't fit for you (as it also has useTrackedState), you might want to create a new library. (and list in https://github.com/dai-shi/use-context-selector#projects-that-use-use-context-selector)

Cool, totally understand.  I'll copy-paste this utility in my projects for now, especially since it's just a tiny file.
I'm looking forward to a native React solution!  Are you referring to useMutableSource, or some other API that I should be aware of?
I'm surprised React has been missing this feature for so long ... this library provides such a good performance-friendly, code-friendly API, that it really does need to be a native React feature.

I'm referring to the useContextSelector being experimented in React.
https://github.com/reactwg/react-18/discussions/73

Oh, that's awesome to hear.  I wasn't aware of this experiment; I will eagerly start watching for that!  Thanks for the heads up.


Rule override between inherit classes

System Details

FluentValidation version: FluentValidation.AspNetCore (8.4.0)
Web Framework version: ASP.NET Core 2.2

Issue Description
Is there a way to do rule override between parent and children class, I search for quite some time and the closest thing I found is this https://github.com/JeremySkinner/FluentValidation/pull/59#issuecomment-102386781
Let say I have 2 classes
public interface IRequest { }

public class SomeRequest : IRequest
{
   public string FirstName { get; set; }
   public string LastName { get; set; }
   public string Description { get; set; }
}

I created a validator for IRequest so I can set a default max length for all the string properties
And then I added validator for SomeReqest to specify one property to have a special handling
public class StringValidator : AbstractValidator<IRequest>
{
    public StringValidator()
        => RuleFor(x => x)
            .Custom((req, ctx) =>
            {
                if (ctx.Rule.Validators.OfType<MaximumLengthValidator>().Any()) return;

                var invalid = req
                    .GetType()
                    .GetProperties(BindingFlags.Instance | BindingFlags.Public | BindingFlags.GetProperty)
                    .Where(x => x.PropertyType == typeof(string))
                    .Select(x => new { x.Name, Value = (string)x.GetValue(req) })
                    .FirstOrDefault(x => x.Value != null && x.Value.Length > 255);

                if (invalid == null) return;

                ctx.AddFailure(new ValidationFailure(invalid.Name, "String too long"));
            });
}

public class SomeRequestValidator : AbstractValidator<SomeRequest>
{
    public SomeRequestValidator()
        => RuleFor(x => x.Description)
            .MaximumLength(5000);
}

However, this does not work.
The validation for SomeRequest.Description is still maxed at 255 characters and not override by SomeRequestValidator
I have tried adding if (ctx.Rule.Validators.OfType<MaximumLengthValidator>().Any()) return; to skip the 255 validation when .MaximumLength(...) exist but since it is under context of IRequest, it doesn't know about the rule for SomeRequest.Description
Is there option to override existing rule from SomeRequestValidator
Or is there some methods we could use in StringValidator to check the existing rule for each property, so I can skip validation when MaximumLengthValidator for that property is found?

Hi there, FluentValidation doesn't support any form of rule overriding like this. Rules in the inherited class are always added to the rules provided by the parent class (parent rules are added first, child rules are added after). The usual way for combining rules from multiple classes is not to use inheritance, but to use Include Rules. This gives much more control over what you want included.
Honestly I would really recommend you don't go down this route. FluentValidation is a tool for building strongly typed rules. If you want dynamic rules built with reflection like you're doing, then FluentValidation really isn't the right tool to be using.


[sil-opaque-values] OwnershipVerifier support.

@swift-ci smoke test.

Oops. Accidentally reused the same branch. Closing.


fix: benchmark links should point to new testdata folder

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


Initialise in the component?

How do I initialise in the component.
For example:
const video = document.createElement('video'); body.appendChild(video);
How do I add the directive to this video tag? I want to make it draggable.

Maybe you can build up an angular component for your video DOM, and then add directive into its template


Using relative paths in development mode.

I didn't find how to get relative paths for my assets in development mode while retaining the ability to use absolute paths in my code so I came up with my own solution. I'm using a simple helper that I put in the root controller:
helpers do

  def depth
    root = File.dirname __FILE__
    relative_path = Dir.pwd.sub root, ''
    relative_path.count File::Separator
  end

  def asset(path)
    return path unless ENV['MODE'] == 'dev' && path[0] == '/'

    if depth == 0
      '.'
    else
      depth.times.map { '..' }.join(File::Separator)
    end + path
  end

end

This could be a worthy addition to Stasis I think, but it could probably be done in a better way in Stasis internals.

Hello. I'm not sure I understand your original problem. Could you expand upon that, maybe with some code examples?

Say I have an image in a HTML page in a subdirectory of my static site and that all my images are in the /images/ directory, I'd do:
%img{ src: '../images/foo.png' }

But using relative path eveywhere is error prone and become a real mess when I move things around, that why we normally use absolute path in most case with dynamic web applications, like:
%img{ src: '/images/foo.png' }

With a static website though, I'm not using a web server during development so using absolute path doesn't work. One trick I've used in other similar project is to make a helper that automatically generate the relative path from an absolute one, that is the code I'm showing in this ticket. With it I can use:
%img{ src: asset('/images/foo.png') }

and get the best of both worlds.

Ah I see what you mean. Thank you for the explanation. Will give some thought to this.

I decided that the solution to this was to provide a web server in dev mode. Start a web server on port 3000 like so:
stasis -d 3000

See: http://stasis.me/#usage


"unrar" license is not correct

The license in unrar/default.nix is declared as GPL2, but the license certainly is not the GPL, it is not even a free software license as it contains this restriction on use:
   2. UnRAR source code may be used in any software to handle
      RAR archives without limitations free of charge, but cannot be
      used to develop RAR (WinRAR) compatible archiver and to
      re-create RAR compression algorithm, which is proprietary.
      Distribution of modified UnRAR source code in separate form
      or as a part of other software is permitted, provided that
      full text of this paragraph, starting from "UnRAR source code"
      words, is included in license, or in documentation if license
      is not available, and in source code comments of resulting package.

The "unrar" package should thus be marked as non-free.

Hmm, the license was changed by 8fbc082d1, /cc @wkennington. I'm rather certain that it truly is unfree. What is worse, I can't see any free alternative except for what debian packages but that one can't handle newer archive versions. (All other SW I've found is based on the restricted sources of this unrar, including p7zip.)

I forgot to /cc maintainer @ehmry.

Worst case it look like it is unfreeRedistributable.
On Sat, Jan 2, 2016 at 1:49 AM Vladimír Čunát<EMAIL_ADDRESS>wrote:

I forgot to /cc maintainer @ehmry https://github.com/ehmry.
—
Reply to this email directly or view it on GitHub
https://github.com/NixOS/nixpkgs/issues/12084#issuecomment-168376459.

Yes, I believe that's the best specification from what we have in lib.licenses.

And, thanks for reporting this!

I have found this one, which might work with newer RAR versions:
https://bitbucket.org/WAHa_06x36/theunarchiver/overview
It's under LGPL 2.1.

Hmm, yes, in comparison to the one I posted it claims to support RAR 2.9 and 3.0. Not (yet) the 5.0 format of the original unrar, but it's likely free, as it's approved by FSF.

https://bitbucket.org/WAHa_06x36/theunarchiver/overview

That requires an objective C compiler and run-time and sounds like a lot of work.

Well, I personally don't plan to invest much energy into any RAR-connected stuff, but that shouldn't prevent anyone else interested...
IIRC our gcc has flags to enable Objective C, only I'm afraid they may be rotten.

IIRC our gcc has flags to enable Objective C, only I'm afraid they may be rotten.

Oh, I actually spent an hour or so doing just that.
I soon found myself packaging the abomination that is GNUstep, just to get error messages about missing header files that should have been included with the shiny new gc(obj)c.
I noped out of there pretty quickly and regret trying.


docs(moses): fix typo

Pull Request Test Coverage Report for Build 199

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.07%) to 28.208%

Totals

Change from base Build 198:
-0.07%

Covered Lines:
299

Relevant Lines:
1060

💛  - Coveralls

Pull Request Test Coverage Report for Build 199

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.07%) to 28.208%

Totals

Change from base Build 198:
-0.07%

Covered Lines:
299

Relevant Lines:
1060

💛  - Coveralls

Pull Request Test Coverage Report for Build 199

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.07%) to 28.208%

Totals

Change from base Build 198:
-0.07%

Covered Lines:
299

Relevant Lines:
1060

💛  - Coveralls

Thanks for the PR!


Add new attribute on branches (future,hotfix)

Git flow has additional attributes (-S,--[no]squash squash feature during merge). Can You add it to the settings?

Duplicate of #74


feat: winit dnd

This implements DnD via window-clipboard for iced. iced-sctk is not refactored here except to update to the latest version. I think a separate PR updating iced-sctk along with an effort to update the iced-sctk libcosmic apps might be better timed after the alpha release? In any case, this should make it possible to add DnD to cosmic-files.

Would it make sense to submit this to upstream Iced?

Would it make sense to submit this to upstream Iced?

I think so, @jackpot51 asked Hector if he was interested in expanded mime types and DnD support before I started working on it.

ok this should be ready to merge now. I had to fix color format and do a couple fixes for multi-window.


cannot find exe file in zip

Sounds silly, but can someone please send me a pointer? I have gone through the folders and cannot find the exe file to start the program. Thanks

Check out this issue #24

https://github.com/starik222/BooruDatasetTagManager/releases/tag/v1.6.3
Head to release page and download the newest version. There are assets within exe file.

Strangely that exr file does not appear in the zip i downloaded from GitHub 𝐑⒜𝐟 𝕊t𝐚ｈⅇl @.*** https://www.instagram.com/rafstahelin/?hl=enOn 9 May 2023, at 10:24, Kiter1210 @.***> wrote:
https://github.com/starik222/BooruDatasetTagManager/releases/tag/v1.6.3
Head to release page and download the newest version. There are assets within exe file.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Did you download the BDTM_V1.6.3_NetCore60_AllInclude.zip
or
BDTM_V1.6.3_NetFramework472.zip ?

No, I didn't.
got it now thanks so much. Where were these posted in github, i could not
see them
On Wed, May 10, 2023 at 5:37 AM Kiter1210 @.***> wrote:

Did you download the BDTM_V1.6.3_NetCore60_AllInclude.zip
https://github.com/starik222/BooruDatasetTagManager/releases/download/v1.6.3/BDTM_V1.6.3_NetCore60_AllInclude.zip
or
BDTM_V1.6.3_NetFramework472.zip
https://github.com/starik222/BooruDatasetTagManager/releases/download/v1.6.3/BDTM_V1.6.3_NetFramework472.zip
?
—
Reply to this email directly, view it on GitHub
https://github.com/starik222/BooruDatasetTagManager/issues/25#issuecomment-1541340437,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APNXTFV6MODYQHG63YGUPXDXFMLQNANCNFSM6AAAAAAX26G7KQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
thanks
best regards, raf

No, I didn't. got it now thanks so much. Where were these posted in github, i could not see them
…
On Wed, May 10, 2023 at 5:37 AM Kiter1210 @.> wrote: Did you download the BDTM_V1.6.3_NetCore60_AllInclude.zip https://github.com/starik222/BooruDatasetTagManager/releases/download/v1.6.3/BDTM_V1.6.3_NetCore60_AllInclude.zip or BDTM_V1.6.3_NetFramework472.zip https://github.com/starik222/BooruDatasetTagManager/releases/download/v1.6.3/BDTM_V1.6.3_NetFramework472.zip ? — Reply to this email directly, view it on GitHub <#25 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/APNXTFV6MODYQHG63YGUPXDXFMLQNANCNFSM6AAAAAAX26G7KQ . You are receiving this because you authored the thread.Message ID: @.>
-- thanks best regards, raf

At the Code page, there is a "Release" place in the right side.


Ibexa 4

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 91.70%. Comparing base (389f113) to head (c2f825f).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             ibexa-4      #34   +/-   ##
==========================================
  Coverage      91.70%   91.70%
  Complexity       185      185
==========================================
  Files             24       24
  Lines            446      446
==========================================
  Hits             409      409
  Misses            37       37

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Log the response when an abort status is received

This is a small change that adds the contents of the response to the log message when the download is aborted with one of the error codes specified with the command-line option --abort-on.
I find the contents to be useful, sometimes it just says "You've been logged out", sometimes something like "Challenge required". And it would help in the case of other errors as well.
Is it just a proof of concept? No
Is the documentation updated (if appropriate)? Not necessary
Do you consider it ready to be merged or is it a draft? It's ready
Can we help you at some point? Not necessary

An HTML page would be bad, I agree.
Running my version with this change, however, I only get JSON errors, which are much smaller (and useful).
Maybe a check could be added and only output the body if the content type is JSON, but it seems unnecessary.

What about saving the content in a html file instead of dumping it in the console output?

What about saving the content in a html file instead of dumping it in the console output?

I think that would be a good and useful solution.

Saving it to a file would probably require a command-line option to specify the destination (and enable the option, as not everyone might want it), and so seems overly complicated.
I've added a check to only include the body it it's a JSON (the exception that might print the body is raised in a method called get_json, but I've added an extra check), and it's also truncated it it's too long (I used 500 characters, but this can be changed - in my experience, the longest I've seen is about 300 characters). Hopefully this can be merged.

Yes this looks good and I also think it can be merged. Thanks!


Gracefully handle initialization errors in oracle and monitor

fixes #1848
If we fail to process one CFD we should not break the loop because we might not process the other CFDs leading to problems.
We don't fail the application if we run into an error with initializing the monitor or the oracle, so we should handle the initialization errors inside the actor.
I think fallible tasks when starting an actor that is not supervised should be avoided because we don't handle the errors and it is unclear what the behavior of the application would be.

clarification for the last sentence commit description: fallible tasks are orthogonal to the supervisor actor - even if the actor was supervised, this fallible task would not have resume processing under any circumstances. They can be used in conjunction when the fallible task does stop the whole actor, causing a restart if it's supervised (which was not the case here!).
IMHO it would be better if we removed the sentence, as it's a bit misleading, but it's not that big of a deal :)

bors r+


Develop a calculator app using any language of your choice and submit as a PR

See README.md for details.

@AravindHari Sure go ahead, its open to anyone to submit a PR :)


mason not recognizing updated tag

Hello, I don't know if this is a code bug, a misconfiguration, or perhaps should be a feature request.
If you look here on line 13:
https://github.com/mason-org/mason-registry/blob/main/packages/drools-lsp/package.yaml#L13C11-L13C37
, you can see that it references the @latest tag. However, "latest" is a moving target. For example, "latest" was updated here just recently as the results of a CI build kicked off from a recent PR merge:
https://github.com/kiegroup/drools-lsp/tags
The problem is that when I ask Mason to check for (and install) updates, it doesn't recognize that "latest" was updated. I think it might be because Mason's just looking at the name of the tag, and thinks, "I've already downloaded latest - no need to do so again", but it's not taking into consideration that the timestamp on that tag has changed and it should re-download and replace it.
Now, I can understand the argument that tags shouldn't change, however "latest" is special, and I can envision other, similar moving targets. Is there anyway to configure mason to notice the updated timestamp?
Aside, yes, I recognize that there are no numbered releases in drools_lsp yet. Just the "latest" tag at this point. Whereas the Drools LSP server does function at a basic level, it is still in its early stages. Perhaps the answer is for us to just start doing numbered releases? (Like 0.1.0, 0.1.1, etc.?) I still think being able to set it up somehow to recognize tag timestamp changes would be a useful capability.
Thank you so much! :)

Hmmm... Do you think it would behave similarly with updated branches as it is here with tags? Does Mason on deal with incremented version numbers for updates? (Sorry for my ignorance.)

Moved to https://github.com/williamboman/mason.nvim/issues/1458


data augmentation-ground-truth sampling

Hi, Tianwei. I read you paper for CenterPoint++. "We adopt ground-truth sampling [15] on Waymo to deal with the long tail class distribution, which copies and pastes points inside an annotated box from one frame to another frame."
how did you handle the time_lag dimension? Thanks in advance!

Thanks for your quick reply! The database points for multi sweep was collected as :
point_indices = box_np_ops.points_in_rbbox(points, gt_boxes)
points = sensor_data["lidar"]["combined"], all points stacked together with time
annos = sensor_data["lidar"]["annotations"] , gt_boxes = annos["boxes"] ,the gt_boxes is for the current frame
yes, the points will already contains multi-frame point cloud with the time_lag feature. But only partial for previous frame, because the gt_boxes is only for current frame. Is my understanding correct? I don't know whether this will enfluence db-sampler effect.
Thank you in advance!

you are right. it is only partial. I guess we can do some interpolation to get box location in previous frame and then crop the point cloud. I don't know if this will be better


Improve grafana plugin updates, close #443

Improved handling of grafana plugin updates in upgrade.sh.
Closes #433

Thanks @BuongiornoTexas  - running tests...

Testing this for v4.1.0

Tested by simulating an upgrade using the new upgrade.sh script, new grafana.env.sample, etc. per the files for the merge.
Note the git pull may look different to a normal upgrade, as I tested this from a private repository to try to simulate a valid upgrade.
Plugin list before running upgrade:
# ls -l grafana/plugins/
drwxr-sr-x    3 admin    admin         4096 Mar 10 18:23 fetzerch-sunandmoon-datasource
drwxr-sr-x    6 admin    admin         4096 Mar 10 18:23 grafana-influxdb-flux-datasource
drwxr-sr-x    4 admin    admin         4096 Mar 10 18:23 grafana-piechart-panel
drwxr-sr-x    3 admin    admin         4096 Mar 10 18:23 simpod-json-datasource
drwxr-sr-x    8 admin    admin         4096 Mar 10 18:23 yesoreyeram-boomtable-panel

Ran the upgrade:
# ./upgrade.sh
Upgrade Powerwall-Dashboard from 4.0.5 to 4.1.0
---------------------------------------------------------------------
This script will attempt to upgrade you to the latest version without
removing existing data. A backup is still recommended.

Upgrade - Proceed? [y/N] y

Resetting Timezone to Default...

Pull influxdb.sql, dashboard.json, telegraf.conf, and other changes...

No local changes to save
remote: Enumerating objects: 11, done.
remote: Counting objects: 100% (11/11), done.
remote: Compressing objects: 100% (1/1), done.
remote: Total 6 (delta 4), reused 6 (delta 4), pack-reused 0
Unpacking objects: 100% (6/6), 1.98 KiB | 337.00 KiB/s, done.
From https://github.com/mcbirse/testing
   9350def..7c70119  main       -> origin/main
Updating 9350def..7c70119
Fast-forward
 Powerwall-Dashboard/VERSION            |  2 +-
 Powerwall-Dashboard/grafana.env.sample | 13 ++++++++++++-
 Powerwall-Dashboard/upgrade.sh         | 64 +++++++++++++++++++++++++++++++++++++++++++++++++++++-----------
 3 files changed, 66 insertions(+), 13 deletions(-)

Updating 'grafana.env' structure. '.old' backup file created.

Grafana plugin list has changed.

You appear to be running an older Grafana configuration with
plugins that are different to the current recommended list.
This script can update your plugins to the current recommendations.
If you have installed custom plugins or want to keep it as
it is, answer 'n' below and the script will provide instructions for
manual checks/updates.

Updating Powerwall-Dashboard stack...
Including powerwall.extend.yml
Running Docker Compose...
[+] Running 5/5
 ✔ Container pypowerwall  Running                                                                                0.0s
 ✔ Container influxdb     Running                                                                                0.0s
 ✔ Container weather411   Running                                                                                0.0s
 ✔ Container grafana      Started                                                                                1.7s
 ✔ Container telegraf     Running                                                                                0.0s

Waiting for InfluxDB to start...
 up!

Add downsample continuous queries to InfluxDB...
2024/03/10 07:34:21 error: continuous query already exists
2024/03/10 07:34:21 error: continuous query already exists

Deleting old pyPowerwall...
pypowerwall
pypowerwall

Deleting old telegraf...
telegraf
telegraf

Deleting old weather411...
weather411
weather411

Restarting Powerwall-Dashboard stack...
Including powerwall.extend.yml
Running Docker Compose...
[+] Running 5/5
 ✔ Container pypowerwall  Started                                                                                2.0s
 ✔ Container influxdb     Running                                                                                0.0s
 ✔ Container weather411   Started                                                                                2.1s
 ✔ Container grafana      Running                                                                                0.0s
 ✔ Container telegraf     Started                                                                                3.5s

---------------[ Update Dashboard ]---------------
Open Grafana at http://localhost:9000/

From 'Dashboard\Browse', select 'New/Import', browse to /home/admin/testing/Powerwall-Dashboard/dashboards
and upload 'dashboard.json'.

Please note, you may need to select data sources
for 'InfluxDB' and 'Sun and Moon' via the
dropdowns and use 'Import (Overwrite)' button.

The revised plugin list after upgrade no longer includes "grafana-influxdb-flux-datasource":
# ls -l grafana/plugins/
drwxr-sr-x    3 admin    admin         4096 Mar 10 18:34 fetzerch-sunandmoon-datasource
drwxr-sr-x    4 admin    admin         4096 Mar 10 18:34 grafana-piechart-panel
drwxr-sr-x    3 admin    admin         4096 Mar 10 18:34 simpod-json-datasource
drwxr-sr-x    8 admin    admin         4096 Mar 10 18:34 yesoreyeram-boomtable-panel

Plugins were re-downloaded and registered when grafana container started:
# docker logs grafana
✔ Downloaded grafana-piechart-panel v1.6.4 zip successfully

Please restart Grafana after installing plugins. Refer to Grafana documentation for instructions if necessary.

✔ Downloaded yesoreyeram-boomtable-panel v1.5.0-alpha.3 zip successfully

Please restart Grafana after installing plugins. Refer to Grafana documentation for instructions if necessary.

✔ Downloaded fetzerch-sunandmoon-datasource v0.3.0 zip successfully

Please restart Grafana after installing plugins. Refer to Grafana documentation for instructions if necessary.

✔ Downloaded simpod-json-datasource v0.6.3 zip successfully

Please restart Grafana after installing plugins. Refer to Grafana documentation for instructions if necessary.

logger=plugin.manager t=2024-03-10T07:34:23.59679441Z level=info msg="Plugin registered" pluginId=input
logger=plugin.manager t=2024-03-10T07:34:23.715230233Z level=info msg="Plugin registered" pluginId=grafana-piechart-panel
logger=plugin.manager t=2024-03-10T07:34:23.715577783Z level=info msg="Plugin registered" pluginId=simpod-json-datasource
logger=plugin.manager t=2024-03-10T07:34:23.715957717Z level=info msg="Plugin registered" pluginId=yesoreyeram-boomtable-panel
logger=plugin.manager t=2024-03-10T07:34:23.716077987Z level=info msg="Plugin registered" pluginId=fetzerch-sunandmoon-datasource

All worked okay from what I can tell. Plugin list was pruned and re-downloaded core plugins only, and Grafana + Dashboard working.
During the upgrade I see an error about some cq's already exists. I'm not sure why, and I don't think this will cause an issue... maybe this always happens and I've never noticed!?

Thanks @mcbirse !

During the upgrade I see an error about some cq's already exists. I'm not sure why, and I don't think this will cause an issue... maybe this always happens and I've never noticed!?

Yes, this is a known issue. We have always run the full influxdb.sql file during each upgrade without deleting the CQs first. Only new CQs in the file will be accepted with a warning about the existing ones.

Thanks, @BuongiornoTexas ! Good addition.
I merged this as version 4.1.0 since it removes plugins that could be breaking if someone made additions assuming flux datasource was available.

I have an old Linux box (Ubuntu 18.04.6 LTS) that I use for a final test.  It did fail during the upgrade. I somehow didn't get the right PR during tests before the merge. 😞

Error: ✗ Plugin not found (Grafana v9.1.2 linux-amd64)

This Linux box uses docker-compose version 1.17.1.  The issue appears that the old docker compose doesn't support variable substitution so that GF_INSTALL_PLUGINS=${PWD_CORE_PLUGINS} will always be empty.  Only Docker-compose 1.5+ support variables substitution. I know it is an edge case but this linux box was
I keep this old box around because this was a fairly common distro when we first started the project. I would dismiss it but I wonder if there are others in the community with this version of docker-compose that will face this same problem (either on a Linux box, RPi or NAS). The only fix is to revert or to remove the substitution.  I'm going to investigate the impact of removing the substitution.

It appears that anyone still stuck on the older docker-compose versus docker compose will be impacted by this. I'm going to revert for now until we can figure out best next steps.

Reverted the added upgrade logic but kept the new plugin list. Also, adjusted logic in case someone ran 4.1.0 on a docker-compose system. It will remediate it if they update. Right now the logic is simple: looks for tag in grafana.env "Updated v4.1.1" to trigger prompt to let user update to full grafana.env.sample.  Not as elegant as it will remove any customizations.
Release v4.1.1: https://github.com/jasonacox/Powerwall-Dashboard/releases/tag/v4.1.1

@jasonacox - I do understand your desire to continue supporting as many OS versions as possible, but I'll also note that even with 18.04 being an LTS, it is now out of its extended support period for nearly a year and is only getting security maintenance. I think you could make the case that anyone sticking with 18.04 is probably doing it for good reason and should be capable of installing docker updates to get things running properly for their system (the current major LTS is due for release shortly and its up to  version 24!).
I don't have any real skin in the game, but this is the second time I've been caught by long tail issues in this project (the last one was also related to docker compose V1 - while V2 hasn't been out quite as long as Ubuntu 18.04, it was released 3-4 years ago!). These issues are going to make it harder and harder to make useful contributions to the code base in the long term.

If it was just Ubuntu, I agree with you and would have kept as is, possibly just added a warning note to upgrade. However, as I investigated, I discovered it was due to the docker-compose which is not tied to Ubuntu 18.04.  Latest version can include docker-compose (even if they don't need it). We currently accommodate it in our setup and ./compose-dash.sh. I debated doing the complete upgrade and removing docker-compose support with a warning that the user needs to update docker and remove docker-compose.  That is still worthwhile effort (we need to do that) but I wanted to get your changes in as I think removing those unused plugins is overwhelmingly helpful and a needful cleanup.
Thanks for your help @BuongiornoTexas . I recognize the frustration, but I hope to convey that I genuinely appreciate your contributions. I'm also keen to ensure the upgrades effective (and mostly issue free) for our large community. And yes, we do have a growing tail. :)

I discovered it was due to the docker-compose which is

No problem - I understood the issue. It was much more a generic comment about the problems of maintaining LONG backward compatibility while adding new features (if influxdb 3.0 improves functionality dramatically, I can see it resulting in a major crunch - not holding my breath on that, but hopeful at this point).
It's also frustrating that the thing that is holding us back on this issue is the docker stack itself - which is kind of defeating the purpose of containerisation in the first place!


Rigid IDE integration

I appreciate that Rome is rigid in areas where flexibility is futile, such as your intent to make linting fully inflexible.
I'm curious if you considered to go as far as applying the same principle to IDE integration? E.g. having a vscode Rome extension that is fully opinionated. Rome could go as far as even creating its own (strongly opinionated) modal editing, e.g. https://github.com/johtela/vscode-modaledit. With TypeScript, this could lead to some interesting super-powered editing experience.
Same as linting, vscode Rome extension would be opt-in and users are free to use neovim, emacs, etc.
I can see many people wanting to volunteer; the barrier of entry for contributing is low.
AFAICT the added workload on Rome core would be low.
Thoughts?
I can't wait to see Rome liberating us from trivialities.

We actually have an implementation of the LSP and  VSCode extension that was implemented in #207. The only special sauce that's in the VSCode extension is finding the Rome binary and then executing it. You should be able to use any other editor that supports LSP and it just needs to run rome lsp and connect over stdin.
Right now I don't think there's any additional planned functionality beyond what the LSP already provides, but we could do some interesting and powerful things with our diagnostics format as it's significantly richer than what the LSP offers.

Closing since the question has been answered. LSP will be the implemented protocol. LSP additionally allows custom methods which we could expose for additional functionality.


TODO: Core: Asset: Chinese characters in the name field throw an exception in the VS debugger

Chinese characters when syncing / serializing asset will cause an exception.  This is only an issue during debugging, likely due to VS encoding.

@gubanotorious Could you check if this issue still happen?


Configuration of ssh-simple using Mock SSH Responses

I am able to run ssh<EMAIL_ADDRESS>id nobody on the command line. But I have yet to figure a way to configure using ssh-simple. Is it possible and how? Below results in a timeout error.
function getSSH (req, res) {
  const SSH = require('simple-ssh');
  const fs = require('fs');
  const key = fs.readFileSync('/Users/ben.pingilley/.ssh/mocked-localhost');
  const ssh = new SSH({
    host: 'mock-answers.yml',
    user: 'ben.pingilley',
    key: key
  });

  ssh.exec('id nobody', {
    exit: function(code, stdout, stderr) {
      res.send(code, stdout, stderr);
    }
  }).start();
}

module.exports = { getSSH };

Hi there. simple-ssh is using the ssh2 client in the background, which is not going to consult your ~/.ssh/config file.
That's not a problem, as all this is doing is seeing you're trying to ssh to mock-answers.yml and replace the hostname with localhost and the key with the correct one for the connection.
In your example above try changing
  host: 'mock-answers.yml',

to
  host: 'localhost',

and give it another go. I'm finding that to work with your example

Thanks for the update. I thought I had updated my comment with how to do it. Figured it out a few minutes after posting.

On Jan 24, 2017, at 4:55 PM, Matt Summers<EMAIL_ADDRESS>wrote:
Hi there. simple-ssh is using the ssh2 client in the background, which is not going to consult your ~/.ssh/config file.
That's not a problem, as all this is doing is seeing you're trying to ssh to mock-answers.yml and replace the hostname with localhost and the key with the correct one for the connection.
In your example above try changing
host: 'mock-answers.yml',
to
host: 'localhost',
and give it another go. I'm finding that to work with your example
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/mdsummers/mock-ssh-responses/issues/1#issuecomment-274977894, or mute the thread https://github.com/notifications/unsubscribe-auth/ANVo4uc6SxKmi2WI7TI-8ebYsvA9rG2Xks5rVo-KgaJpZM4Ls36L.


Removing mariadb support (Issue #29)

Attempts to solve and close Issue #29. Open to suggestions and changes since I am not quite sure of the code changes

Thank you. Before this PR can be merged a PostgreSQL related PR needs to be solved first here https://github.com/fossasia/eventyay-tickets/issues/28

@mariobehling is the postgresql PR identical to https://github.com/pretix/pretix/pull/3435/files? If yes then I shall attempt to solve that too

@mariobehling  is the postgresql PR identical to https://github.com/pretix/pretix/pull/3435/files? If yes then I shall attempt to solve that too

@mariobehling is the postgresql PR identical to https://github.com/pretix/pretix/pull/3435/files? If yes then I shall attempt to solve that too

It is correct that we need to achieve the same outcome, but as Pretix changed the license we cannot just copy changes. Of course changes that follow the documentation of MariaDB or PostgreSQL would be ok, but other changes would have to be implemented independently in order not to infringe other licenses.

Please make a new PR against the development branch. Closing this therefore. Thanks for your contribution.


Horizontal bars are sometimes needlessly showing

The broken behavior can be observed on https://grsmto.github.io/simplebar/
I can also observe it with latest beta
Current Behavior

Expected behavior

Horizontal bar to not show
Reproducible example
Open https://grsmto.github.io/simplebar/
Suggested solution(s)
No idea
Additional context

Software
Version(s)

SimpleBar
Same as on https://grsmto.github.io/simplebar/

Browser
Version 70.0.3538.77 (Official Build) (64-bit)

npm/Yarn
None

Operating System
Linux / openSUSE Tumbleweed

Interesting. I'm not having this.
That might be a Linux + Chrome specific issue. Unfortunately I don't have Linux and it's not available on Browserstack so I can't test :/

I encountered similar issue with IE 10. I fixed it with fix similar to #174. This one could be same issue.
I think cause is that size calculations are not exact in some browsers if container or content sizes are not integers in pixels.

I have the same issue. In debug mode, getting rid of the width: inherit seems to fix it, but since this is coming from sata-simplebar I cant change it, or am I getting something wrong here?
testable here ;)
www.hinz.design

@ChrisSympatient you are setting overflow-y: scroll, that breaks the plugin.
Also your own css is added before SimpleBar's css, that's why your width is overriden.

@Grsmto Thank you very much for the quick reply!!! :)
I commented 	/* overflow-y: scroll;*/ out (style.css Line 1480)
and changed the css rel in the index file so simplebar is first:

<link rel="stylesheet" type="text/css"  href="css/style.css">
<link rel="stylesheet" type="text/css" href="css/responsive.css">

...
But now the formating of the form is fully ***ed :D

My endless programming skills are quiet limited :D i guess Im missing out on major things here.

Yeah that's the height: inherit from SimpleBar breaking your layout (we might not need this property...). Apply height: auto to your div, that will fix it.
But you need to remove overflow-y: scroll or you will necessarily have the scrollbar track visible.

Oh but I want the y bar to show. Because when a user enters a really long message and receives my "fake reply" the content will scroll down. Since the vertical scrollbar is there already, I dont have to bother rearranging the content. My concern was regarding the yellow marked horizontal scroll bar shown in the first screenshot i posted.
So actually everything is working just fine now, thanks to your advice rearranging the css rel order. Im just worried I actually miss your point and sound like a douche.

@Grsmto It is reproduced not only on Linux: https://github.com/Grsmto/simplebar/issues/231#issuecomment-439344211

Reproduced on IE11 on https://grsmto.github.io/simplebar/  by setting container width to 171.78px

I don't get it, just tested on browserstack now and I can't reproduce this :/

Any other things you're doing to get this? Like browser zoom level etc.?
What Windows version?

Went to check my zoom level but now I cannot reproduce it too :/
IE: 11.345.17134.0
Windows 10: 1803

By the way it's worth noting that you can work-around this issue by explicitly setting overflow-x: hidden to your element, if you don't need horizontal scroll at all of course.

@Grsmto I tried in different system & found out that when windows 10 system resolution is 150% then horizontal scrollbar is always visible. Maybe this will help you identify the issue.

Hey guys,
FYI I just released<EMAIL_ADDRESS>I would appreciate if you could test that out and let me know if that fixes the issue! 🤞
Thanks!

Hi @Grsmto, I'm using<EMAIL_ADDRESS>and I still see this issue on Win10/Edge running in Fusion on a Mac. Not sure what's causing it though, and I haven't found an easy fix yet. For example if I set width: 99% and select/deselect it a couple times, sometimes it disappears, but it mostly doesn't.

@briice if you could provide a reproductible case using the jsfiddle template that would help a lot to debug this issue further: https://jsfiddle.net/Grsmto/57obe5w2/1/
Thanks!

Is this fixed? I am also having problem with it on edge.
There is nothing to scroll from yet the scrollbar is visible.

I am using ^3.1.0-beta.2

Updated to 4.1.0
Now it just sticks up top.

I am experiencing the same issue on both Edge and IE11, but for vertical scrollbar.
They are shown even when there is no need.
I noticed the size calculations are not accurate to the pixel when there are elements in the scrollable area with margins/paddings.
Maybe something like this would help with subpixels differences:
this.axis.y.isOverflowing = Math.floor(this.contentWrapperEl.scrollHeight) > Math.ceil(this.contentWrapperEl.offsetHeight)

Likewise for horizontal axis.

I encountered same issue (needless scroll) on IE 11.
The "blue browser" is calculating width funny: on specific window widths (e.x. 1276) I get scrollWitdh: 1090 offsetWidth: 1089 and on window width 1260 it's already accurate: scrollWitdh: 1076 offsetWidth: 1076.
What's even more surprising, there's no native scrollbar in both cases, but simplebar is placing it's scroll in the first one.
I don't know sandboxes still supporting IE, so can't provide you with working example just yet.
I guess IE just rounds offsetWidth and scrollWidth differently and there's not much to be done, but...
Has someone dealt with this before?

I noticed a similar problem in FireFox. When calculating the value from scrollHeight, it is greater than offsetHeight. Because of this, scroll appears. This happens when scrolling horizontally. FireFox version is 125.0.3 64 bit
 this.axis.y.isOverflowing = contentElScrollHeight > contentWrapperElOffsetHeight;


Add version in global place

Version didn't match between setup.py and __init__.py

If this gets in, it would be excellent if you could tag and release versions


Sugar Migration Issue

I am having issues with database migration using SugarORM (1.4) in my Android project.
I want to add 2 columns in an existing table (NOTE).
In the migration SQL script (assets/sugar_upgrades/2.sql), I have:
ALTER TABLE NOTE ADD COLUMN created_by INTEGER REFERENCES USER(id);
ALTER TABLE NOTE ADD COLUMN is_synced INTEGER DEFAULT 0;

I also added 2 integers in Note.class, which extends SugarRecord (createdBy, isSynced).
This adds 2 columns to the table. The problem is when I try to retrieve data from NOTE table.
List<Note> note = Note.listAll(Note.class); : returns an empty list.
List<Note> allNotes = Lists.newArrayList(Note.findAll(Note.class)); : returns the records, but if I write:
allNotes.get(0).setTitle('title');
allNotes.get(0).setBody('body text');
allNotes.get(0).save();

it inserts a new row in the table instead of updating the first row. Further looking into it, I found that:
allNotes.get(0).getId(); : returns null.
Putting a blank upgrade SQL file does not add columns to the table, even if the new integers/columns are defined in Note.class. I also tried putting a blank 1.sql and a 2.sql containing the migration SQLs. Did not work.
In case of creating new tables the migration works fine.
Please let me know what the issue is. If it is a bug please look into it.
Thanking in advance.

Did you update the Schema version in the meta-data from  AndroidManifest file? Every time you make an upgrade or alter to your schema you need to increase the version.

Yes i did. Oherwise the migration SQL file would not have executed. Any thoughts on what may be wrong?

I wil try to take a look at it. I don't know what could be the reason of your problem, but I will try to reproduce it and solve it.

@jonisaa hey.. could u manage to check it? any update on the issue?

Hi, i got the same problem, have you solve the problem? How?
Really need help :( Thanks

Hey @FalinCh ,
Sugar 1.5 does not have such migration issues. in fact sugar 1.5 does not require any migration sql script. new columns are created once a new field has been added to the entity class (extends SugarRecord). I have added Sugar 1.5 in my project and removed all the migration scripts and its working fine now :) check the 1.5 documentation and changeLog for details.
add compile 'com.github.satyan:sugar:1.5'  to your buuild.gradle file


[RIP-19] Pop Consuming (wip)

Make sure set the target branch to develop
What is the purpose of the change
Add pop consuming method between client and broker.
Brief changelog

Add "Pop" "Ack" "ChangeInvisibleTime" request.
Add "SetConsumerMode" to admin tool.
Add relevant code to push-consumer to make same Consumer adapt to both pull and pop mode.

Verifying this change
Create a topic and a consumer group, switch this consumer group to pop mode, use two clients of this consumer group to consume messages from topic while producing some messages to one message queue of this topic, verify both clients are able to receive messages from same message queue.
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[ ] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

This PR is too large, can you split this pr into serveral smaller ones? @ayanamist

This PR is too large, can you split this pr into serveral smaller ones? @ayanamist


Custom Syntax not dysplaying selected color

Hello,
Great Package!
I have the following issue: I am trying to highlight some specific keywords in my log. I have created a debug entry and selected the colors and icons.
The icon is displayed, and the Summary show the line that matches the expression, but doe not matter what I do, the Log does not show in the color selected ( as in errors).
	// debug ___________________________________________________
	"debug" : {
		"enable"  : true,
		"summary" : true,
		"pattern" : [
			// [ "begin regex", "end regex" ] or [ "match regex", "" ]

			[ "^{{{LINK}}}?[^\\r\\n]*?(?i)onMessageReceived:",""], 	// a line including case-insensitive 'onMessageReceived'
			[ "^{{{LINK}}}?[^\\r\\n]*?(?i)wakeup", "" ],            // wakeup ... (single line)
			[ "^{{{LINK}}}?[^\\r\\n]*?(?i)Tap/Button", "" ],
			[ "^{{{LINK}}}?[^\\r\\n]*?(?i)isCallActivityVisible", "" ]
			// <-- Remove, Change, Add More Patterns Here -->
		],
		"color" : {
			"base"  : "#F94823", // debug message
			"link"  : "#23F9E5", // link in info message
			"quote" : "#F923CB "  // quote in info message
		},
		"icon"  : "Blueberry/bonus.png"    // remove "icon" not to use icon, "dot", "circle" "bookmark" are possible
	}

I think that you didn't run Log Highlight: Generate Custom Syntax & Theme  command.

H @poucotm ,
Thanks for the quick reply.
I did run the command and then have tried to run LogHighlight: Erase Custom Syntax & Theme, and Generate again, but I get the same:
I see the custom icons on the left side, and the matching expression shows in the summary but only errors get the colored.

In my computer, it works well...

I have the same issue. sublime version 3.2.2 Build 3211

I also use version 3211 Windows 10.

@poucotm , I am not sure how or why, but after I completely closed SublimeText3 (I had a reboot in between but I do not think it mattered), the custom logs start showing the color... (??)
Anyway, it is working for me now (screenshot below).
Thanks for the support!


Is it possible to embed icons in report instead of loading from CDN?

Currently the generated reports rely on two types of externally hosted data:

Google Fonts
Font Awesome icons (loaded from CloudFlare)

The fonts I am not so worried about because if the external font is not accessible the report will be displayed with the default system fonts.
The icons, on the other hand, worry me a bit more.  At this point they are all aesthetic, and you can use the report fine without them.  However, I expect that in the future we may want to include more icons that may be more functionally important.  (e.g. a button for the settings drawer)
Of, course I am not worried about the icons not being available on CloudFlare in the short-term. My concern is that reports used in an enterprise process have the potential of being around for a very long time (5+ years).  Unlike a normal hosted webpage or application, we would not be able to do anything about already generated reports if the Font Awesome URL ever changed.  (Can, of course, change it for new reports going forwards, but not for any that were previously generated).
So, we need to investigate the possibility of somehow embedding the icons with the report.  (First thing that comes to mind is encoding them in b64 like we do with the favicon, but maybe there is a better way...)

So, after some experimentation, I can confirm that embedding a base64 image into the HTML should work fine.  (Thanks to @TobiTenno for pointing me to http://fatopng.com/fontawesome-to-base64). That makes it easy to get the base64 text for a FontAwsome icon.  I am working on a PR now to embed the settings cog icon.  Just wanted to record a couple of other thoughts here for posterity sake:
The FontAwsome icon files are licensed under CC BY 4.0 (https://fontawesome.com/license/free). They are free to use but we need to give credit and provide a link to the license. I plan to include this as part of my PR.
Also, it is probably worth noting that we really only need to embed resources for things that are functionally essential.  If the report is still usable without the resource it is probably okay to load it from the web (e.g. I don't see any reason to stop using web-based fonts since the browser can always fall back to the system defaults)

Sounds like a great plan.


Update README.md

What type of PR is this?
Add one of the following kinds:

documentation

What this PR does / why we need it:
Comments with optional / alternative content of the "Release Information" section (got deleted in a previous PR)
Which issue(s) this PR fixes:

Fixes #

Hi Herbert,
I looked over this, it all looks good. I know you are requesting review from the admin team, so I will leave this pr open until the rest of the team gets to take a look.
Best,
Evan

@eharrison24 I had only restored some lines which accidentally got lost in a previous PR. Already merged.


Weather "age" is incorrect

I flight tested the code base from three days ago which includes color coded weather and the update to indicate how old a wether message is (rather than displaying a GMT timestamp).
Weather was displaying 2 days, 20 hours old in the WebUI.

Is this due to the Pi not picking up the correct time? Or are you able to account for that in the raw data?  I recall there was a fair amount of discussion on being able to pass the "time" in Issue #33

At this point I have no clue. I've not looked into it. Since I will have to debug this with playback data I'm going to have to first figure out what timestamps are available to me and what they originally meant when they were recorded.

More than likely the Pi is starting up with the incorrect time (w/o access to the 'net) unless the user manually sets it ... or a script is run to pull the time from a GPS. Found a number of articles on this ... a couple of links below that might be of of help (programmatically):

Q: I've got a USB GPS receiver permanently attached to my raspberry pi. At boot time, I would like raspbian to wait until the GPS has a signal, then set the OS's time to whatever the GPS says.

A: There's two "official" ways to do it.
gpsd can set the clock
ntp has a special driver <IP_ADDRESS>
The benefit of gpsd is that it gives you a nice interface to the GPS data for other purposes. If you use the ntp driver you've tied up your GPS receiver:

Raspberry Pi: Set time from GPS dongle
http://blog.petrilopia.net/linux/raspberry-pi-set-time-gps-dongle/

I would like to set the Raspberry PI's time clock on start up or once the GPS has a lock because the application this will be used on will not have access to the internet.

Script that sets system time through a GPS
https://code.google.com/p/gpstime/

I'm not sure what the answer is to get the correct "age" of a weather item. I have checked both the gen_gdl90 code and the WebUI. The UI calculate the age of weather by taking the difference between the two date/time values from gen_gdl90: one is the timestamp for the item and the othe is when Stratux received the item.
For some reason, this delta is huge: aka more than a day old.

arithmetic error somewhere?  Or data type conversion error maybe?
On 10/29/2015 04:45 PM, bradanlane wrote:

I'm not sure what the answer is to get the correct "age" of a weather
item. I have checked both the gen_gdl90 code and the WebUI. The UI
calculate the age of weather by taking the difference between the two
date/time values from gen_gdl90: one is the timestamp for the item and
the othe is when Stratux received the item.
For some reason, this delta is huge: aka more than a day old.
—
Reply to this email directly or view it on GitHub
https://github.com/cyoung/stratux/issues/85#issuecomment-152337491.

since I can not debug and fly at the same time I'll make a temporary build that dumps the various calculated and received datetime data so I can determine where the delta is coming from.

I'm scheduled to fly a couple of 100 mile legs on Sunday ... if I can be of any help let me know.

I did a series of tests and the issue comes back to not having a reasonable system time from the raspberry pi. I powered up my system, which has been sitting for a few days, and it started with with the date from November 1st. All of the weather reports were then calculated based on this reference.
I've go ahead and changed the code back to just showing the short date/time of the message. Down the road, if we can always get some sort of useful date/time as a reference point, then the 'age' format for weather can be revisited. I will close this issue and we can either leave it or have an "enhancement request" for somewhere down the road.

Probably not as elegant as you'd like, but in the short term, how 'bout giving the user the ability to enter in their local time via the UI?

silly me. The problem is only with the WebUI so I can ignore the base time that is sent with the weather messages from the raspberry pi and just use the mobile device's time.
The fix is pretty small (weather.js) I'll check in the code.
Can someone test it this weekend?
If it doesn't work as expected I can back out the code.

Just saw this ... leaving in about 45 minutes for the field. If you can get me the info I can test this morning.

I committed the code change yesterday. If you're set up to build Stratux then you'll get the new code. You won't need to compile anything. Just use "make www" to load the new WebUI. Then open your mobile browser as usual. On the weather page, the age should be correct.

Has anyone been able to confirm the "age" of weather reports in the WebUI is accurate (or at least reasonable close) and seems to make sense?
I'd like to go ahead and [re]closecthis issue.


details missing in googleapiclient.errors.HttpError

Run some code that makes an invalid request.
  File "/home/user/.local/lib/python2.7/site-packages/googleapiclient/http.py", line 840, in execute
    raise HttpError(resp, content, uri=self.uri)
googleapiclient.errors.HttpError: <HttpError 400 when requesting https://appengine.googleapis.com/v1/apps/<project>/services/default?alt=json returned "Invalid request.">

Now when you get the exception and its content the error becomes slightly more verbose.
{
  "error": {
    "code": 400,
    "message": "Invalid request.",
    "errors": [
      {
        "message": "Invalid request.",
        "domain": "global",
        "reason": "badRequest"
      }
    ],
    "status": "INVALID_ARGUMENT"
  }
}

Making the same request manually to the REST API returns a significantly more useful response.
{
  "error": {
    "code": 400,
    "message": "Invalid request.",
    "status": "INVALID_ARGUMENT",
    "details": [
      {
        "@type": "type.googleapis.com/google.rpc.BadRequest",
        "fieldViolations": [
          {
            "field": "service.split.allocations[1]",
            "description": "Allocation must be a value in the range (0, 1]."
          },
          {
            "field": "service.split.allocations",
            "description": "Traffic allocations must sum to 1."
          }
        ]
      }
    ]
  }
}

Seems like it might just be dropping the "details" section on the floor. I'm happy to review a patch to fix this, or we'll get to it when we can.

@jonparrott Thanks in advance for a review.


[feature request] insert with hint for TreeContainers

Is your feature request related to a problem? Please describe.
I want to update the element in the TreeContainer (e.g. OrderedSet), but it is prohibited.
It is reasonable because updated element could insert the different position.
However, users sometimes know the updated element is still the same position.
For example,
Before update:
    V
[1, 3, 5]

After update:
    V
[1, 4, 5]

When I want to update the element, I need to do the following steps:

container.find(key) the element's iterator. Then got it.
container.eraseElementByIterator(it)
Create the new element based on the gotten element and insert it.

Step1 and 3 are O(logN) time complexity.
I want to eliminate Step3 cost.
Describe the solution you'd like
js-sdsl's design refers to C++ STL.
C++ std::set::insert has insert(hint, value) overload.
https://en.cppreference.com/w/cpp/container/set/insert
See (3) (4) overloads.

Since C++ 11
3-4) Amortized constant if the insertion happens in the position just before the hint, logarithmic in the size of the container otherwise.

container.eraseElementByIterator(it) at the step2 returns the iterator just after the erased element.
So if I use the returned iterator as the hint, then the hint iterator is the insertion happens in the position just before the hint.
So the steps become as follows:

container.find(key) the element's iterator. Then got it.
it = container.eraseElementByIterator(it)
Create the new element based on the gotten element and insert it with hint. container.insertWithHint(it, 4)

Step3 becomes O(1) time complexity thanks to the hint it.
I guess that js-sdsl can implement similar way to the C++ one.
Describe alternatives you've considered
Directly modify the element value by user's risk. However, insert() with hint achieves the same goal (eliminate extra O(logN) time complexity) more elegant way.
Additional context
libc++ implementation of std::set::insert(hint, value)
https://github.com/llvm-mirror/libcxx/blob/2f961a057f1ecb29d40892fe9198fe4e4b5c5619/include/set#L704-L711

I've gone and looked at C++'s overloaded function for insert. But unfortunately, although it can improve efficiency, it is still O(logn) because it needs to self-balance after insertion.
I think you need a function which can change element by iterator while the tree is still balanced.
const st = new OrderedSet([1, 2, 3]);
// Change begin iterator's key to 0 but maintain balance.
// Then `st` will be [0, 2, 3].
st.updateByIterator(st.begin(), 0);

Is this what you want?

We will also consider adding the overloaded function of insert to the function.

I've gone and looked at C++'s overloaded function for insert. But unfortunately, although it can improve efficiency, it is still O(logn) because it needs to self-balance after insertion.

Ah, updating by remove and insert changes the number of elements. The change could trigger self-barance, I guess.
Hmm...

I think you need a function which can change element by iterator while the tree is still balanced.
const st = new OrderedSet([1, 2, 3]);
// Change begin iterator's key to 0 but maintain balance.
// Then `st` will be [0, 2, 3].
st.updateByIterator(st.begin(), 0);

Is this what you want?
Because this is done internally, user does not need to worry about its security, the function will judge it.

It is nice.
It is similar concept to Boost.MultiIndex's modify function.
See https://www.boost.org/doc/libs/1_80_0/libs/multi_index/doc/reference/ord_indices.html#modifiers
template<typename Modifier> bool modify(iterator position,Modifier mod);

I considered 2nd parameter.
If it is an updated value as follows, you can't access the original value.
st.updateByIterator(st.begin(), 0);

But if it is a function, you can access and modify the original value as follows:
st.updateByIterator(st.begin(), (value) => { --value; });

C++ can enforce the parameter as reference but I'm not sure JS can do that. If the value type is primitive type, then it has copy semantics so maybe object wripping would be required.
However, I think that the following approach is good enough. So modifer function version is not required. Value version is simple and good.
const it = st.begin();
const updated = it.pointer - 1; // 1 - 1 => 0
st.updateByIterator(it, updated);

We will also consider adding the overloaded function of insert to the function.

Ah, TypeScript can overload function if the number of parameters are different. It's nice.

The test version 4.1.4-beta.0 has been released, if there are no bug reports within a week, we will released the official version.


Do we need to change the license for the new release?

TU Delft has a new software policy for open-source software, and it has a specific way to write the license file. Do we adopt that? or do we keep the current license?

Very well. I will keep the current one


management NIC and host NIC can be the same for sidebanded systems

Describe the bug
TASK [control_plane_common : Verify different nics] fails when management NIC and host NIC are the same.
To Reproduce
Steps to reproduce the behavior:
ansible control_plane.yml
Expected behavior
This should not fail and allow  mngmnt_network_nic == host_network_nic
Screenshots
fatal: [localhost]: FAILED! => {
    "assertion": "mngmnt_network_nic != host_network_nic",
    "changed": false,
    "evaluated_to": false,
    "msg": "Failed. Incorrect nic information. public nic, management network nic and host network nic should not be the same"
}

@j0hnL, in case of LOM both management and host network can be same, however we need to differentiate this scenario from iDRAC Settings - Network  set to Dedicated and user giving same NIC for both management and host network, this is defective scenario.
May be we should introduce one more variable iDRAC-network=LOM or iDRAC-Network=dedicated. We should allow management NIC and host NIC same when iDRAC-network=LOM
I feel we should do this change in devel branch not in release-1.1. This needs some more testing.
@Shubhangi-dell , let's make the changes for this in devel

@sujit-jadhav @Shubhangi-dell did this already get resolved?

@j0hnL This is not yet resolved. We will target it for 1.2

@sujit-jadhav is this still on target for 1.2?

@j0hnL yes John we are targeting it for 1.2


Correction for raw affiliation College of Medicine, Department of Pharmacology & Experimental Therapeutics, Louisiana State University Health Sciences Centre 1901 Perdido Str., New Orleans, United States of America;; DMEM, INRA, Univ. Montpellier, Montpellier, France;

Correction needed for raw affiliation College of Medicine, Department of Pharmacology & Experimental Therapeutics, Louisiana State University Health Sciences Centre 1901 Perdido Str., New Orleans, United States of America;; DMEM, INRA, Univ. Montpellier, Montpellier, France;
raw_affiliation_name: College of Medicine, Department of Pharmacology & Experimental Therapeutics, Louisiana State University Health Sciences Centre 1901 Perdido Str., New Orleans, United States of America;; DMEM, INRA, Univ. Montpellier, Montpellier, France;
new_rors: 03rnk6m14;01qv8fp92;051escj72;00xffm983
previous_rors: 03rnk6m14;01qv8fp92;051escj72
works_examples: W2900755543
contact: d9de2c2709bcecb992dd9711e68abd47:844d4b307230a416ab4fe0f04a9a1363faef @ inrae.fr

This issue was accepted and ingested by the OpenAlex team on 2024-12-19. The new affiliations should be visible within the next 7 days.


No font style other than sans-serif is visible

(Autographa 2.0 beta 2)
Issue Description:-
Font style selector in the edit mode can't be changed from sans serif( open a project ->File menu-> default font family->san-serif)

It is working. Can you test it again?

looks fine moving to done.


ESAS: BehaviourRecording

https://github.com/ices-tools-dev/esas/issues/21
BehaviourRecording
Whether and how behaviour was recorded:
0 Behaviour not recorded
1 Typical detailed ship-based behaviour recording (see behaviour field)
2 Typical aeroplane-based activity recording (see activity field)
3 Other (only selected types of behaviour recorded, include more details in Notes)

Anna will make the descriptions more generic.

To be reviewed by ESAS

Request is cancelled, the field is removed from the format model


Moved creation of transaction to post request in type of business screen

Moved transaction post request to type of business post request, to prevent transaction being created if user does not start an application

CI: No security warnings found


Use webmock and to_rack in specs

Make wsoc a dependency of spidr and route all requests to it using webmock's .to_rack method. https://robots.thoughtbot.com/how-to-stub-external-services-in-tests

This was completed in e91c1190d7eea6645e3a9ceadd1c3918ad5a12fa and 0ecaffa38f7e59ae44e535f4439070fb055c9ce4.


add ecrecover precompile

signed-off-by: Vitali Grabovski<EMAIL_ADDRESS>Related to https://github.com/hyperledger/burrow/issues/1240

I've rolled this into #1420

I've rolled this into #1420

Would like to work on your notes but it seems like you have already done it by yourself @silasdavis

Ah hello! Sorry I didn't mean to take if off you if you were still interested in finishing it off. I did base more work off your commit.
When looking into it I realised a few other things were not being done correctly with our handling of secp256k1 outside of ecrecover.
I don't know if you are at all interested in implementing some of the other elliptic curve precompiles. There is an abandoned PR with some of the work here: https://github.com/hyperledger/burrow/pull/1284
If that has any value or interest to you I'd be very pleased to assist with merging a PR

I don't mind. That is nice that better Burrow version in now in master! Well done

I will take a look at https://github.com/hyperledger/burrow/pull/1284 This mab be close to feature I would like to be included in Hyperledger_Burrow


Lapce becoming very slow after hours of usage

Lapce Version
nightly-5aa3b61
System information
Linux thinkpad-t450s 6.1.1-1-MANJARO #1 SMP PREEMPT_DYNAMIC Wed Dec 21 23:21:50 UTC 2022 x86_64 GNU/Linux

i5-5300U with integrated graphics, 12GB RAM
Describe the bug
After hours of work in Lapce, editing a small C++ project (without any language server support or other running plugins), the editor becomes very slow, and I see a latency of around 1 second between a key press and the edits appearing on the screen. The laptop fan also ramps up and the system monitor shows heavy usage of all 4 cores of my system. The editor becomes in fact unusable, and the only way to continue with my work is to close and open it back again.
Additional information
I waited for the editor to reach this state, then ran perf record -p <pid> sleep 20 to capture data for 20 seconds. The report is the following:
# Samples: 79K of event 'cycles:u'
# Event count (approx.):<PHONE_NUMBER>6
#
# Overhead  Command          Shared Object                Symbol
    22.63%  lapce            lapce                        [.] lapce_core::buffer::Buffer::deletes_from_union_before_index
    18.30%  lapce            lapce                        [.] lapce_xi_rope::multiset::Subset::transform_shrink
    13.33%  lapce            lapce                        [.] lapce_xi_rope::multiset::Subset::transform
     6.28%  lapce            lapce                        [.] <rayon_core::job::HeapJob<BODY> as rayon_core::job::Job>::execute
     6.10%  lapce            libc.so.6                    [.] 0x000000000015c58a
     1.39%  lapce            libc.so.6                    [.] 0x00000000000959cc
     1.39%  lapce            libc.so.6                    [.] realloc
     1.27%  lapce            libc.so.6                    [.] malloc

I'll also attach the perf.data file, hoping it contains something useful. If you can instruct me on how to capture additional profiling data, or acquire other useful information, please tell me.
perf.data.zip
Thanks for the great work!

I waited for the editor to reach this state, then ran perf record -p  sleep 20 to capture data for 20 seconds.

I want to add that, during the 20 seconds of perf capturing samples, I kept typing and deleting characters to try hit the code causing the seconds-long delay between key presses and rendering.

Thanks for the perf, that made it a lot easier to find the issue :)
This seems to be due to editing/deletion growing slower as the number of revisions(?) grows larger and larger.
Random basic benchmark: https://gist.github.com/MinusGix/a87ccf90afa0342557142d3e98bc9986
Just insertions are fast (comment out the delete), but doing both insertion and deletion has them both grow progressively slower. is_pristine also seems to grow dramatically.

Judging from the event trace log, the issue lie entirely in lapce_core::buffer:Buffer::edit function,

Gonna have to learn more about ropes to fix this isssue.


How can I download every html webpage that I view within the WebView app? Offline view of previously loaded webpages?

I would like a feature where every webpage(not the whole entire website) that is visited automatically downloads the html of the single page that is loaded, so that it can be loaded offline. (Even webpages that are shown in Iframes)(Just text is fine)
And if possible, then you get to choose the amount of time it stays on the device before being deleted.
Can you create this or tell me what kind of code I would have to add?

Would adding something like this help?
webview.loadUrl("javascript:HTMLOUT.processHTML(document.documentElement.outerHTML);");


Upgrade React-intl and dedup packages

What's the purpose of this pull request?
This PR upgrades react-intl to the latest version. This upgrade will prevent users from installing a new version of react-intl and missing our webpack optimization rules.
Also, this PR adds gatsby-link as a peerDependency of store-ui fixing duplicate packages issue
How it works?
React-intl has a guide on reducing bundle sizes by pre-compiling messages. This guide has changed since last version and if users upgrade their react-intl version they may miss this optimization.
How to test it?
Open any deploy preview and check for bundle-stats.html that nothing new was added and gatsby-link was deduped
https://github.com/vtex-sites/marinbrasil.store/pull/505
https://github.com/vtex-sites/btglobal.store/pull/550
https://github.com/vtex-sites/storecomponents.store/pull/917
https://github.com/vtex-sites/carrefourbrfood.store/pull/687

we now miss the automatic gatsby parameter parsing
What do we miss by that?

huge benefits in stability and performance
Has this been measured?

we now miss the automatic gatsby parameter parsing

What do we miss by that?

huge benefits in stability and performance

Has this been measured? (the performance gains specifically)

Just moved this part to #729.


Fix failing sun tests with WebGL validation on.

For #2771.
The framebuffer was incomplete because we render the sun to a texture that is a fraction of the drawing buffer size. For the tests, the size is between 0 and 1. In practice, this will never happen, but I capped the minimum size to 1 for the tests. Since this never failed before, maybe there was an update where the texture width and height must be greater than or equal to 1?

I can confirm the tests are fixed.  @lilleyse or @pjcozzi can you merge if you're okay with this change.  Thanks.

@bagnell can you add a comment in master about why the min call is there.  Also, is there anywhere else that we need this?  Sounds like something the post-processing framework will need to support in general.


[Network] New Commands for RC6

Fixes #846. cc/ @DeepakRajendranMsft, @MikhailTryakhov for feedback.
Adds the following commands and tests:

nic get-effective-route-table
nic list-effective-nsg
vnet peering delete
vnet peering show
vnet peering create
vnet peering update (generic update)
vnet peering list
vnet check-ip-address

nic show-effective-route-table
Command
    az network nic show-effective-route-table: The get effective routetable operation retrieves all
    the route tables
    applied on a networkInterface.

Arguments

Resource Id Arguments
    --ids              : One or more resource IDs. If provided, no other 'Resource Id' arguments
                         should be specified.
    --name -n          : The network interface (NIC).
    --resource-group -g: Name of resource group.

nic list-effective-nsg
Command
    az network nic list-effective-nsg: The list effective network security group operation retrieves
    all the
    network security groups applied on a networkInterface.

Arguments

Resource Id Arguments
    --ids              : One or more resource IDs. If provided, no other 'Resource Id' arguments
                         should be specified.
    --name -n          : The network interface (NIC).
    --resource-group -g: Name of resource group.

vnet peering create
Command
    az network vnet peering create: Peerings in a VirtualNework resource.

Arguments
    --name -n           [Required]: The name of the VNET peering.
    --remote-vnet-id    [Required]: ID of the remote VNET.
    --resource-group -g [Required]: Name of resource group.
    --vnet-name         [Required]: The virtual network (VNET) name.
    --allow-forwarded-traffic     : Allows forwarded traffic from the VMs in the remote VNET.
    --allow-gateway-transit       : Allows gateway link to be used in the remote VNET.
    --allow-vnet-access           : Allows VMs in the remote VNET to access all VMs in the local
                                    VNET.
    --use-remote-gateways         : Allows VNET to use the remote VNET's gateway. Remote VNET
                                    gateway must have --allow-gateway-transit enabled for remote
                                    peering. Only 1 peering can have this flag enabled. Cannot be
                                    set if the VNET already has a gateway.

Though English is not my first language, but the help text looks pretty awkward, particular The list effective network security group operation retrieves..., should be shrunk to the operation retrieves. Also the line break after all the is unnecessary.
I know this is caused by we are loading the SDK doc. I suggest we should override it, unless we have a plan to update across all.
az network nic list-effective-nsg: The list effective network security group operation retrieves
    all the
    network security groups applied on a networkInterface.

The rest LGTM

Since we will need to improve help across the board, I went ahead and overrode the descriptions for the vnet and nic commands (not just the ones in this PR).

In reply to: 247488615 [](ancestors = 247488615)