Add latest government schemes for workers

#5 In this issue we do talk about having a worker's corner with government schemes for them.
Prepare a google document with the latest government scheme for workers and add the link in the comments.

can u elaborate a little  where we have to add or whether we have to clown goggle docs

Hi, @KaizenGirl1111 I want to work on this issue under GSSoC'22. Could you please assign it to me?

@kanchi12P assigned

I have added the link below . Please review it. Then I will make a pr for this.
https://docs.google.com/document/d/1ikySlKuXsKLYk2jKuHK_vhnBbSqIN8B0N6B6XGwEyfg/edit?usp=sharing

Hello contributors, I'm assigned as the mentor for this project. Reach me out for any queries on discord.


split dict decode from segment iterator

The logic of SegmentIterator is too complicated, so let's separate the dictionary-related Iterator to make the logic simpler

It's better to add a concise message to describe why to split the SegmentIterator.

@chaoyli done


How does parsing to vector<string> work?

In the following piece of code the goal is to parse a sequence of identifiers -- like "id1 id_ idx" -- into a vector<string>:
void parse_sequence_of_identifiers()
{
  namespace bp = boost::parser;

  auto id = bp::lexeme[+(bp::ascii::alnum | bp::char_('_'))];
  auto ids = +id;

  std::string str;
  std::vector<std::string> vec;

  bp::parse("id1", id, bp::ascii::space, str);      // (1)
  bp::parse("id1 id2", ids, bp::ascii::space, vec); // (2)
  bp::parse("i1 i2", ids, bp::ascii::space, str);   // (3)
}

(1) works fine. (2) fails to compile. (3) compiles and reports success, but the result is an empty string. This goes surely against the intuitive expectations.

I should have mentioned that (2) above works as expected in Boost.Spirit: https://godbolt.org/z/KsPxnnenz

Thanks for catching this.  Clearly I did not test enough cases.  (1) and (2) should work as of the patches that I just pushed to develop (if everything is fine, they'll land on master soonish, with a doc update).  (3) is a bone-headed mistake on my part that breaks probably everything if supported.  I've left (3) ill-formed for now.

Thanks. This Issue is actually about (2).
Regarding (3), I would never expect it to work. I was actually surprised that it compiled. If it now fails to compile, then I am happy.


Take extends Product with Serializable

This makes the inference a bit more accurate. In scenarios like the following
scala> import scalaz.zio.stream.{Take, ZStream}
<console>:11: warning: Unused import
       import scalaz.zio.stream.{Take, ZStream}
                                 ^
<console>:11: warning: Unused import
       import scalaz.zio.stream.{Take, ZStream}
                                       ^
import scalaz.zio.stream.{Take, ZStream}

scala> (ZStream.range(1, 10) map Take.Value.apply) ++ ZStream.succeed(Take.End)
res0: scalaz.zio.stream.ZStream[Any,Nothing,Product with Serializable with scalaz.zio.stream.Take[Nothing,Int]] = scalaz.zio.stream.ZStream$$anon$1@332829f8

Product with Serializable with scalaz.zio.stream.Take[Nothing,Int] is inferred for Take[Nothing, Int]. After the change it becomes
scala> import scalaz.zio.stream.{Take, ZStream}
<console>:11: warning: Unused import
       import scalaz.zio.stream.{Take, ZStream}
                                 ^
<console>:11: warning: Unused import
       import scalaz.zio.stream.{Take, ZStream}
                                       ^
import scalaz.zio.stream.{Take, ZStream}

scala> (ZStream.range(1, 10) map Take.Value.apply) ++ ZStream.succeed(Take.End)
res0: scalaz.zio.stream.ZStream[Any,Nothing,scalaz.zio.stream.Take[Nothing,Int]] = scalaz.zio.stream.ZStream$$anon$1@9b8469f

notice scalaz.zio.stream.ZStream[Any,Nothing,scalaz.zio.stream.Take[Nothing,Int]].
Anyway feel free to close the PR if this is not a priority.

@barambani Type inference is always a priority. 😄 Thank you! 🙏


Demo on Huggingface

Can a demo using Gradio on Huggingface be made?
It will be very nice if anyone was able to try your work.

Can a demo using Gradio on Huggingface be made? It will be very nice if anyone was able to try your work.

Hi, thanks for your interest in the work! Yeah, we've planed to make it more applicable for more people. However, the main authors have been busy running a recently established start-up company recently. I've get my finger crossed to work more on it.


Feature/34 add unit tests artist

Summary
Unit tests for Artist service & controller using Jest
Test Plan
All tests pass when run
Related Issues
#34

Looks really good, super comprehensive and I'm glad you tested the fail cases as well


translator:load doesn't load translations to db

Hi,
I have the same problem as decsribed here https://github.com/Waavi/translation/issues/9, locally (EasyPhp) an on the server (apache). Languages are defined in the languages table and I have the version with DIRECTORY_SEPARATOR. In the cache table, I have for example a key like this "laravelwaavi|translation|fr.nav.*", so I suppose that the package finds the translation files!?
I tried with debug on/off, different cache types, one language with one file...
I don't know really what else I could provide to help

@vpakg  Can u please provide to solve this issue? I have also this issues.

Hi there,
The following helped for me (nexanas Post that it worked the next day made me think about caching):
php artisan config:cache
php artisan cache:clear
php artisan optimize
composer dump-autoload


First draft of the dam project

This PR includes the first draft of the dam project. I added some explanation for each step.

You can check out a preview of the notebook here.


show only count option and disable jump labels?

Apologies If I am misunderstanding the options in the "Setup and description" section of the README.
If someone wanted to use this plugin without the jumping features is there a way to hide the [3N 2], [n 2] like tags, but leave the x/y results for the current line?
It seems like / + control g (do not hit enter) gives me the result I want, but using n/N is faster.
Context:
My Neovim is currently configured with "cmdheight = 0",  being able to quickly search a log file to count for occurences of say an error string, is the main feature I find I have lost since using cmdheight = 0.
This plugin is working great to add that functionality back, but I am wanting to hide the jump tags so that I only get the info I need so it is cleaner.

nearest_only = true


feat(Stream): switchMap

Type

[ ] Refactor
[x] Feature
[ ] Bug Fix
[ ] Optimization
[ ] Documentation Update

Description

Related

Related Issue #
Closes #

What is the benefit over flatMap(f, { switch: true })?

I guess just verbosity, not sure if this is common enough to justify it

RxJS also do have a switchMap operator, and it's widely used.
I decided to make it as a separate operator not just because of verbosity, but because of clarity and visualization.
This is a real code from my project
export const BinanceWsUnitLayer = Layer.effect(
  BinanceWsStreamTag,
  Effect.gen(function* () {
    const connection = yield* BinanceWsConnectionContextTag;
    const streams = new Map<
      string,
      Stream.Stream<
        unknown,
        | BinanceWsConnectionError
        | BinanceWsResponseError
        | BinanceWsConnectionCloseError
      >
    >();
    return {
      createStream(
        stream: string,
      ): Stream.Stream<
        unknown,
        | BinanceWsConnectionError
        | BinanceWsResponseError
        | BinanceWsConnectionCloseError
      > {
        return Stream.suspend(() => {
          debug(`Requested stream "${stream}"`);
          let stream$ = streams.get(stream);
          if (stream$) {
            return stream$;
          }
          stream$ = connection.pipe(
            Stream.flatMap(
              (ws) =>
                Effect.request(
                  BinanceWsSubscribeRequest({ stream, ws }),
                  RequestResolver.contextFromEffect(
                    BinanceWsSubscribeRequestResolver,
                  ),
                ),
              { switch: true, concurrency: 1 },
            ),
            Stream.flatten(),
          );
          streams.set(stream, stream$);
          return stream$;
        });
      },
    } satisfies BinanceWsStream;
  }),
);

It is not really easy to notice the options parameter for flatMap. When you fluently read this code, you only see Stream.flatMap, but default flatMap behaviour is different. In addition, there are no clear documentation about what does switch option means. On the site it is written

if the order of concatenation is not important for us, we can use the switch option.

And it doesn't describes this concept.
Actually, i thinks that the switch option changes the behaviour dramatically enough to extract it to the separate operator with it's own documentation and example, which i also provided in this PR.

@tim-smart what should we do? I am kind of in favour of accepting this given the semantic specificity

@tim-smart what should we do? I am kind of in favour of accepting this given the semantic specificity

I think we can add it back as flatMapSwitch (what it was called before we consolidated the apis).
We would then mark the switch option as depreciated and remove it in 4.0?

@mikearnaldi done

/rebase

@dilame can you pls fix the version in since to 3.5 and I'll merge it

Hi! Just wanted to remind that i changed the version in since that time to 3.5.0 and just now i changed it to 3.6.0 😃


[junit5] update pom so it gets both the jupiter engine and api artifacts

One-line summary
Got an error trying to use this library in another project because it needed the api jar as well
Types of Changes

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

part of #334


Turn on the artifacts output layout

I'd suggest turning on the artifacts output layout for the repo.
It just means adding this to Directory.Build.props:
  <UseArtifactsOutput>true</UseArtifactsOutput>

It's just much better experience once you start using it.

Thanks for the suggestion Premek


Original model not updated

I would like agility to update the original model object I give to the factory function '$$()'.
This helps especially with big trees of agility components (e.g. Tables with hundreds of components).
I would like to have only one source of truth for the model.
I made a patch that takes the original model object instead of doing '$.extend(...)' and and throws an exception if the new model is non-empty and the model inherited from the prototype is non-empty too.
This works great for me and should save some memory at runtime.
Of course this behaviour is undesired for views and controllers.
Here is my Test-HTML:
var globalModel = { value: 'bla' };
var input = $$(globalModel,
	'
<input type="text" data-bind="value" /> value:
', {
		'create': function() {
			// put the original value somwhere visible
			$('#message').append(globalModel.value);
		},
		'change:value': function() {
			// put the original value again
			$('#message').empty().append(globalModel.value);
			// show the original value and the value from this.model
			alert('global value: ' + globalModel.value + ', input.value: ' + this.model.get('value'));
		}
	}
);

$$.document.append(input, '#form');

Of course I would be willing to provide my patch (its quite small with two additional util functions).
But of course I am unsure if you like such a thing at all.
But for me its a plain bug since the view isn't bound to the model I provided.

Hi, thanks for bringing this up.
Right now Agility assumes your model lives inside Agility objects. This assumption works well for most things I can think of, but of course not all possible scenarios.
Can you please elaborate on your application/use case?

Hi, thanks for asking. :-)
Background:
We are using Agility to create highly dynamic web pages.
So we really have a quite big model describing the whole page with generic components I have built.
Every generic component gets its part of the model. So we easily have 100 components on the page.
Use cases:
Right now we would like to exchange the whole big model with the server.
This might be possible without a patch with the persist plugin. I haven't tested it so far.
On the more hypothetical side I can imagine that the components have to interact with each other somehow.
On the purely philosophical side I have a bad feeling watching my nice big model being ripped into lots of tiny models by the components. Plus these models get all inconsistent with each other over time.
On the hacker side the copying costs memory and CPU power.
At least for this month I don't think I will be able to work further on this part of the architecture.
So it will take some time until I am able to name a really important use case.
Thus this bug is so far more for food for thought and philisophical reasons. :-)
It might become more urgent in the future.
Best regards,
Ole

Hi there, I can start to imagine what your use case would look like.
So what you're saying is that your model is so big that you want
"sub-views" for different parts of the model, but at the same time you
want to update the whole thing on the server side? In other words, you
need many agility objects whose models are sub-parts of a big model?
By the way, are you using Agility in production already? If so, would
you mind if we featured your product in our agilityjs.com Gallery
page?
Thanks!
On Fri, Dec 9, 2011 at 7:24 AM, ole108
<EMAIL_ADDRESS>wrote:

Hi, thanks for asking. :-)
Background:
We are using Agility to create highly dynamic web pages.
So we really have a quite big model describing the whole page with generic components I have built.
Every generic component gets its part of the model. So we easily have 100 components on the page.
Use cases:
Right now we would like to exchange the whole big model with the server.
This might be possible without a patch with the persist plugin. I haven't tested it so far.
On the more hypothetical side I can imagine that the components have to interact with each other somehow.
On the purely philosophical side I have a bad feeling watching my nice big model being ripped into lots of tiny models by the components. Plus these models get all inconsistent with each other over time.
On the hacker side the copying costs memory and CPU power.
At least for this month I don't think I will be able to work further on this part of the architecture.
So it will take some time until I am able to name a really important use case.
Thus this bug is so far more for food for thought and philisophical reasons. :-)
It might become more urgent in the future.
Best regards,
Ole

Reply to this email directly or view it on GitHub:
https://github.com/arturadib/agility/issues/60#issuecomment-3078377

Hi,
so far I have only a small spike. I can't continue work on it right now.
We will decide until (probably) end of January how to go on.
I have 2 typical use cases:
A list page with a (large) table as the central component.
This table consists of head and body and those of rows and cells.
The cells finally contain components like links to the edit page, or just text or even input fields.
I don't know how to build a table with dynamic rows and columns without lots of subcomponents.
Is there an easier way???
A dynamic form with feedback and a configurable list of inputs.
Since the number, type and order of the input fields is configurable I need to use many small components.
Is there an easier way???
The configurability of the UI is its key feature. So I can't reduce on that.
Do you know of other tools that are more suitable? Or a better way to use Agility.js?
Thanks for your help!

Not really sure how to help you out here, sorry :(
Do keep me posted if you decide to go with Agility for your final product, and we can feature it in our Gallery page.


Add Easier APIs for Using Ion C

With #34, using the Ion C bindings is pretty low-level.  In particular, one needs to deal with iERR and also allocate and bind raw pointers.  It would be nice if we could expose safer versions of the low-level APIs that integrate with Result, make working with ION_TYPE/ION_READER/ION_WRITER/etc. easier.
This would not be a replacement for the traits we have in the native Rust implementation, but at least make interacting with IonC in its full capacity easier.

Resolving in favor of related issues that are more granular.


Support large documents in extension host

VSCode Version: 1.14.1
OS Version: Windows 7 and Windows 10, likely all OSes

When editing documents with very long lines, such as lines containing data-URIs, the property vscode.window.activeTextEditor will be undefined when it should be defined.
Steps to reproduce:

Create a new (blank) VSCode extension, and add this command to extension.ts:

'use strict';
import * as vscode from 'vscode';

export function activate(context: vscode.ExtensionContext) {
    let disposable = vscode.commands.registerCommand('bigDocTest.runDocTest', () => {

        let hasActiveTextEditor = (vscode.window.activeTextEditor !== undefined);

        if (hasActiveTextEditor) {
            vscode.window.showInformationMessage('Success, activeTextEditor is defined.');
        } else {
            vscode.window.showErrorMessage('Oops, activeTextEditor is not defined.');
        }
    });

    context.subscriptions.push(disposable);
}

export function deactivate() {
}

Launch this extension, load any normal text file, and run this command, you see the "Success, activeTextEditor is defined" information message appear.

But, try loading a file with a very long line contained inside it, such as this one: MetalRoughSpheres.gltf.  This file has long lines due to the use of embedded data URIs in the file.  If you run the above sample extension while editing this file, you see "Oops, activeTextEditor is not defined."

This makes it difficult for VSCode extensions to work with such files.  I'm the lead dev of the GLTF extension for VSCode, and this prevents my extension from running on GLTF files that embed large images as data URIs.

Yes, we unfortunately have to do that because very large documents (those with long lines or many lines) are very likely make the extension host choke and stall. The current limit is at 5MB. If a file exceeds that we don't sync it over to the extension host but only offer editing capabilities.

Thanks for replying.  Does the user have any way to configure the 5MB limit to be larger?  Could this issue be turned into an enhancement request for user-configurable max line length for extensions?
The current limit is particularly painful for extensions whose sole purpose is to augment editing of such files.

The limit isn't configurable and it is somewhat arbitrary, tho before increasing it we should check how the extension host behaves with large files. Obviously this is more complicated then just having a check for one file, because many little files can also be a problem...

Great, thanks @jrieken.  GitHub isn't showing me a reopen button, can this be reopened?

I am working on a extension that must work on large xml files which routinely are over 5MB. I just ran into this issue :-(. Making this configurable by the extension or offering an alternative scheme for commands to to still work would be helpful.

re https://github.com/Microsoft/vscode/issues/31078#issuecomment-316801062 - forgot to press reopen...

Anyone happen to know where this limit is specified in the source code?

Perhaps a very limited, restricted api.  IMHO, the primary thing that I and other should be able to get by with is:
Request the Uri and LineNumber of the Cursor for the active document.
The intent is for the vscode to tell the extension author, "Hey here is the what the user is up to, but I am not parsing that big file for you..have fun :-)"
window.activeTextEditorLocation(): Location | undefinded

Example usage:
'use strict';
import * as vscode from 'vscode';
import {MyCustomExtensionHelper} from './MyCustomExtensionHelper'

export function activate(context: vscode.ExtensionContext) {

    let disposableWordCounts = vscode.commands.registerCommand('bigDocTest.runDocTest', () => {
        let activeTextEditorLocation = (vscode.window.activeTextEditorLocation !== undefined);

        if (!activeTextEditorLocation) {
            vscode.window.showErrorMessage('No document is open.');
        }

        const helper = new MyCustomerExtensionHelper();
        helper.reportWordCount(activeTextEditorLocation.uri)
    });

    let textEditorCommand = vscode.commands.registerTextEditorCommand('bigDocTest.customMenuItem1', () => {
        let activeTextEditorLocation = (vscode.window.activeTextEditorLocation !== undefined);

        if (!activeTextEditorLocation) {
            vscode.window.showErrorMessage('No document is open.');
        }

        const helper = new MyCustomerExtensionHelper();
        const activeWord = helper.getWordTheUserCursorIsOne(activeTextEditorLocation);
        helper.findAllReferences(activeWord);
    });

    context.subscriptions.push(disposableWordCounts, textEditorCommand);
}

export function deactivate() {
}

This one little function would enable a fairly wide variety of use cases.

This would fix my ReferenceProviders
This would fix my DefinitionProviders
This would fix my TextEditorCommand handlers
This would fix my Command handlers

It would be great for the user to be able to configure it at least. I work with log files of max 200MB using Visual Studio Code and since there is no native filtering using line or regex (include/exclude), I can't filter the log file through third-party extension either...

@Leonid-Kokhnovych-Rivian  please see my extension https://github.com/mbehr1/vsc-lfs. It's really hackish but might be useful for your use-case especially if you want to filter a lot of content anyhow (see setting vsc-lfs.replacements ).

Solving this would be of great help!

Hello, any updates on this issue?

@emackey, this is controlled by the _MODEL_SYNC_LIMIT property of the TextModel class defined here: https://github.com/microsoft/vscode/blob/ecf479e662923a158c04ab58a835549a312442bf/src/vs/editor/common/model/textModel.ts#L176
On my system I was able to increase this limit by patching the workbench.desktop.main.js file. However, note that this might cause vscode to display a warning that the installation is corrupt. Still, this worked for me.
Here's the command I used (not that I recommend doing it):
$ sudo sed -iE 's/\._MODEL_SYNC_LIMIT=50\*1024\*1024,/._MODEL_SYNC_LIMIT=500*1024*1024,/g' /usr/share/code/resources/app/out/vs/workbench/workbench.desktop.main.js

Any progress on this? I want to use "Text Power Tools" (https://github.com/Microsoft/vscode/issues/31078) to filter large LOG files (size of them is out of my control) - and not having this function makes me to use Powershell to filter them before hand instead - which is a petty - as I rather use it inside vscode - when can we expect a fix for this?

Just stumbled into this issue myself. I have an extension that gives me emacs style keybindings. Of course, since it's an extension, whenever I open a file larger than 50MiB, they don't work. This makes me unable to navigate large files with the keyboard (even through the cursor keys).
An option to configure _MODEL_SYNC_LIMIT would go a long way. I'd see about implementing it myself, if Microsoft would be willing to merge it.

It's been 5 years now with this issue. Can we please get the possibility to edit the limit in the config. The default can stay the same, so most people wouldn't even notice.

It would be very useful to at least have find and replace for very large lines. Currently i have to use grep

The issue is still in place, and it would really be nice to make that limit configurable and let users decide whether they are ok with excessive CPU/MEM usage.

6 years, now

It's been 5 years now with this issue. Can we please get the possibility to edit the limit in the config. The default can stay the same, so most people wouldn't even notice.

The error shown for some extensions is unintuitive. For example for the Bookmarks extension with a large file, the error message is

Open a file first to toggle bookmarks

Hi everyone,
I found an extension that solves this issue!
The extension is called “Filter Line". While it has a limitation on large files, they provide a workaround.
Here’s the solution mentioned in their README:

Essentially, you need to create a file named filterline in the same directory as the file you want to filter. Then, use the tool as if you were working with the logs file. The filterline file will contain the filtered results.
I hope this helps!
Regards


404s

Issue in /src/pages/implement/server-side/sdk-guides/on-device-decisioning/index.md
Two 404s in this section:
https://developer.adobe.com/target/implement/server-side/sdk-guides/on-device-decisioning/#what-is-the-summary-of-steps-i-need-to-follow-to-ensure-my-on-device-decisioning-activities-are-delivered-successfully-via-adobe-targets-server-side-sdk

Resolved


Pricing model for Pro

I haven't decided on this yet, likely tiered, I'd be happy if small companies with little funding don't have to pay much, startups and established companies can afford more. Perhaps by company size? Maybe just by feature-set, we'll see, happy to hear suggestions!
Recording the time required to create Up, that might help gauge how much time it would save someone as well haha, most teams should not be wasting time developing tooling, engineers are pricey!

I don't know a whole lot about how Up is going to work (doesn't seem like SaaS, more like Apex++). I really like the Sketch pricing model... possibly per user to scale better with company size?

Yup! It's basically apex(1) but for apps/sites/apis instead of event processing. I like the idea of maybe even skipping a Pro version and licensing based on the size, but I imagine many people might just go ahead and not pay haha

How about capping projects? Any option is bound to make someone upset, but capping the total number of deployed projects seems like the most straightforward/honest metric.
Since Up is a self-hosted service, will be charging for every new app it's bootstrapped.
Eg: free -> 5 active apps; foo -> 10 active apps; pro -> unlimited active + unlocked features

@lukeed that might make sense, I suppose I'd have to have a way to "trial" the pro stuff anyway. Individuals tend to not care about licensing anyway, companies definitely do, so it might be a lost cause at the lower end haha.

Saw the edit -- like the sound of that.  At this point (hypothetically), you're only charging for convenience of managing deployments, it can be another straightforward pricing model of $3 per additional active project. You've got to play with the charges, ofc, but at some point it'll just make sense to go to the pro plan for unlimited deployments alone.

I'll be adding quite a few time-saving value-adding features as well hopefully. I'll give it some more thought, http://sidekiq.org's model is working really well for him, I think it's pretty fair to take a decent chunk from huge companies to make the project sustainable, they don't flinch at stuff like that.

You bet! You gotta do what works best for you, first and foremost. Just tossing around ideas

Appreciate it! Definitely new to this side of things

Maybe charge per feature, assuming it will be subscriptions you sell. People who really don't care about pricing will just click select all. And BTW I think everyone should pay at least a little bit. It's more important that projects like this get done than that they be free. Maybe support a --please flag which voids the need for a subscription for people who are trying it out or really can't afford anything.

@jkroso yeah I wonder if drawing the line at commercial use regardless of community/pro version makes sense. It would be nice not to manage two distinct versions.

Yeah that would work. But eventually I think you should just charge the marginal cost. Charging by feature enables you to drop the cost of old features while still having an incentive to pursue new features.

A pricing model similar to Caddy might also interesting for up: https://caddyserver.com/pricing

Considering your Apex "lifestyle" goals it seems like these pricing options might trap you in a business model that works against your overall business goals. Per-user, or any application level pricing models will require tracking licenses, license enforcement, lots of billing work (with associated chargebacks, etc) and a lot of small transactions best handled by a larger organization (at least larger than one person). A business/enterprise oriented model typically requires a sales team/person and much longer sales cycles. Many small startups starve to death before they can get the enterprise customer to finish their evaluations and budgetary cycles and get to a pay day.
In the short term, considering your large network/reach, what if you eliminated the Pro license and closed source headaches and funded the project using a Kickstarter? $1 tip jar, $30 t-shirt, $50 sponsor (name on website or something), $1,000 patron with some personal interaction, $5,000 spend a day with TJ, etc. Make the pro features "stretch goals". The goal is a "war chest" of cash up front without the hassle of actually running a business for the first 6 months to 1 year of the project. Maybe push to the open repo behind a 30 day delay with the actual working repo a private repo only exposed to kickstarter backers?
Spend the next X months building the solution using the kickstarter money, growing the user base, and finding the right corporate sponsor to fund further development. You can decide based on your needs whether that should just be a I'll-make-features-just-for-you sponsorship, brand-exposure-for-money sponsorship, or an employment sponsorship (ala node.js early vs late Joyent days).
Long term, partner with a consulting/support company to provide paid consulting/support (similar to Redis). If necessary, create a pro version at that point with the partner handling the gory details of licensing, billing, support, etc. (Again, ala Redis Enterprise).

I don't think it has to be super complicated, Sidekiq bills annually so that takes out many of the headaches. I've tried the whole donation thing, it didn't really work, so I don't think I'll ever go that route again it just isn't sustainable. I'll definitely have to make sure to keep things as simple as possible, TBH I don't think Mike from Sidekiq even enforces the licenses, though at least with a private binary it's easy to restrict access there.
You might be totally right that it could end up going against my lifestyle ideals, only one way to find out I guess! Maybe even having Pro OSS as well and just the honour system would be fine, companies don't seem to mind, individual developers tend to think they can quickly build any solution on their own, even if they're wasting hundreds of $$ doing so.

Keeping things simple and relying on the honor system is probably the right way to start. If you're really hurting from piracy then you can try to add some license security but as you say, it's probably not worth it.
I think Kickstarter would be great for marketing. It's also a nice way to bring in some early money. When Joyent used to need extra cash they would run an "unlimited lifetime" deal where you'd pay basically 2-3 years worth of hosting and get unlimited lifetime hosting. If the pro license is say $50/year you could run a kickstarter backer level that's something like $200 for an unlimited lifetime personal pro license (associated with a personal email addess).
I agree that scaling to company size is a good idea but could be hard to do while keeping things simple. Maybe have 2 "pro" levels and the free. Free lets you work on just one project at a time (somehow) or maybe doesn't come with the UI (it's command line only). Pro which can be very reasonably priced lets you have unlimited projects and/or the GUI. Both versions are pretty unlimited when it comes to the type of project and AWS resources you can use.
The expensive version is an "Enterprise" edition. For Enterprise features, focus on concerns that only larger organizations have rather than trying to limit the actual ability to do real world commercial deployments or access/use certain AWS features? There are many things that only crop up if you have a large number of users (like audit trails), or if you want/need to enforce workflows. For example, approval workflows: a build has to get approval from any 2 QA people and at least one release manager before it can actually be deployed. That's something a small company doesn't need at all, but would be "business critical" for enterprises.
Anyway, sorry for the long rant - I'm really excited to see what you come up with!

@badslug totally agree, much of that stuff is just a burden on smaller companies, and larger / enterprise workflows are definitely entirely different. I can't imagine anyone but small companies deploying locally via up, Pro users will likely be CI, and then Enterprise would be more like you mention. I like that split, Sidekiq does three as well, makes sense to me, I'll see what I can work out for features. Thanks again!

What if up positioned itself more like a tool rather than a service that you had the option to continually pay for? Sketch is the closest example to borrow from.
You could oversimplify Sketch as a glorified SVG editor and up as a glorified AWS API caller. Individuals could accomplish the final result of an SVG or API Gateway + Lambda by hand but it's much easier to use a specialized tool.
Sketch starts with a free trial so you can see all that it does. When you buy a Sketch license, you get one year of free updates (each license funds the continued development). If the version of Sketch you bought does everything you need, you never pay anything again. If you want to renew your license to get the latest features, you can do so at a discounted rate (encouraging ongoing support of Sketch). Volume discounts encourage companies to get a copy for everyone.
Replace "Sketch" with "up" in the previous paragraph and it's really appealing to me as a potential customer and perhaps meets most of your personal goals. (This model could also be applied to apex obviously.)

#375

I know this is closed but I would like to provide an alternative pricing scheme. I would love it if I could pay you, and only you. What I mean is, I run up and it deploys to your AWS account. I should be able to add custom domains, env variables, whatever else through the website and not ever have to worry about looking at AWS. You can then just take the server cost from AWS, add x% and charge me for it. Why do this? Well because I don't want to mess with AWS. It's too complicated for me to worry about and if you could just do it for me, I'd be willing to pay a premium. Additionally, if you were to add Docker support (deploy Dockerfiles), you'd be filling a gap that Zeit left with their upgrade to v2. This is sort of how Zeit currently works and I really like the method. And if you are wondering if this system works, yes it does. Zeit does it, wordpress.com does it, Gitlab does it. Make the tool free and make money from simplifying peoples lives even more.
Interested to hear what you think. It may be too late to do anything now, but might be interesting should you decide to release a v2.

@nahtnam appreciate the feedback, yeah it's certainly a possibility. I primarily chose to do it this way because most companies require (and prefer) the isolation for security purposes, and of course it shields me from malicious use since I'm not paying for the Lambda bill.
I'm personally not too interested in doing a PaaS version where I host everything, primarily because Heroku and others already exist, and they do nice job, and are actually cheaper than Zeit haha, even Zeit's logging would cost me $350/mo alone.
I'll definitely consider it though if it's something people really want, there are definitely some advantages to PaaS.

I think a simple affordable PaaS version would be super compelling - but a "run Up on your own AWS" option should always be in the mix for exactly the reasons you stated.
If you didn't want to do all the ops work you could partner with someone or bring on an "employee #2" that just does that piece and lives (or dies) off the PaaS revenue.

You said it yourself. Most companies require isolation and this is where you charge money for a pro edition. If an enterprise wants a good but easy solution, they'd be more than willing to pay for it. For someone like me who just wants to host a few side projects and my portfolio, I really don't need that. I'd rather just pay a dollar or so to get it done. Currently, my monthly bill for Zeit is around $0.7 with around 500,000 requests a month with v1. However, v1 is set to die soon and Zeit has been pretty unstable for me so I would love to switch.
The only downside I see is that if you host it for us, we won't get the 1 million free requests :( If you get big enough though, you could possibly subsidize this for the free plan.

There are many downsides to Up deploying your code on its account/servers. I thought about this initially too, but eventually agreed that the current model is the right choice for @tj's perspective. While it may be simpler on the user's end (dealing with AWS in any capacity is horrible, yes), TJ only inherits more cost and more liability. The price would have to increase to compensate, which a larger majority wouldn't be thrilled about.
This could still be offered, but IMO should be a separate, more expensive tier that users select.

@nahtnam even a thousand people giving me $0.7 for side projects isn't worth it though :p that's the problem, it's maybe ok for startups where "growth" and hype is a metric that might help them sell the team in the future to some other company, but as an indie developer it's probably not a great choice.

Good point, in that case the current system works well! :)


Error when: Pre-bundling them to speed up dev server page load...

vite v0.15.2
[vite] Optimizable dependencies detected.
⠧ Pre-bundling them to speed up dev server page load...
(this will be run only when your dependencies have changed)
<--- Last few GCs --->
[7138:0x3cd9580]   150843 ms: Scavenge 1370.5 (1435.6) -> 1355.8 (1437.1) MB, 10.5 / 0.0 ms  (average mu = 0.159, current mu = 0.131) allocation failure
[7138:0x3cd9580]   151002 ms: Scavenge 1371.0 (1437.1) -> 1356.3 (1439.1) MB, 34.5 / 0.0 ms  (average mu = 0.159, current mu = 0.131) allocation failure
[7138:0x3cd9580]   151088 ms: Scavenge 1371.5 (1439.1) -> 1356.8 (1440.6) MB, 8.4 / 0.0 ms  (average mu = 0.159, current mu = 0.131) allocation failure
<--- JS stacktrace --->
==== JS stack trace =========================================
0: ExitFrame [pc: 0x31f698edbe1d]

Security context: 0x14b87071e6e9
1: loadpkg [0x25832996c5f1] [/home/albertog/Documentos/work/myapp/node_modules/resolve/lib/async.js:~185] [pc=0x31f699148a09](this=0x3630f678d481 ,dir=0x258329975351 <String[77]: /home/albertog/Documentos/work/myapp/node_modules/tailwindcss/node_modules/fs>,cb=0x258329975239 <JSFunction onpkg (sfi = 0x27b135f93911)>)
2: load [0x25832...
FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory
1: 0x8f9d10 node::Abort() [/usr/local/bin/node]
2: 0x8f9d5c  [/usr/local/bin/node]
3: 0xaffd0e v8::Utils::ReportOOMFailure(v8::internal::Isolate*, char const*, bool) [/usr/local/bin/node]
4: 0xafff44 v8::internal::V8::FatalProcessOutOfMemory(v8::internal::Isolate*, char const*, bool) [/usr/local/bin/node]
5: 0xef4152  [/usr/local/bin/node]
6: 0xef4258 v8::internal::Heap::CheckIneffectiveMarkCompact(unsigned long, double) [/usr/local/bin/node]
7: 0xf00332 v8::internal::Heap::PerformGarbageCollection(v8::internal::GarbageCollector, v8::GCCallbackFlags) [/usr/local/bin/node]
8: 0xf00c64 v8::internal::Heap::CollectGarbage(v8::internal::AllocationSpace, v8::internal::GarbageCollectionReason, v8::GCCallbackFlags) [/usr/local/bin/node]
9: 0xf038d1 v8::internal::Heap::AllocateRawWithRetryOrFail(int, v8::internal::AllocationSpace, v8::internal::AllocationAlignment) [/usr/local/bin/node]
10: 0xeccd54 v8::internal::Factory::NewFillerObject(int, bool, v8::internal::AllocationSpace) [/usr/local/bin/node]
11: 0x116cede v8::internal::Runtime_AllocateInNewSpace(int, v8::internal::Object**, v8::internal::Isolate*) [/usr/local/bin/node]
12: 0x31f698edbe1d
Aborted
error Command failed with exit code 134.

Can you provide a reproduction? Need to see your deps to see what's going on.

Closing as duplicate of #146


Sublime Text 3: Emmet doesn't show custom tag on tab

What I wanted to do

Write 'b-row'
Press tab!
< b-row > < /b-row >

What it do

Write 'b-row'
Press tab!
b-row

Basically just the usual tab. Other tags like html, h1, etc works
However, when I do b-row[input="wow"], it shows up like this
< b-row input="wow" >< /b-row >
Sublime Text 3 Version: Build 3176
OS: Windows 64-bit
P.S.
This came up to my console..

Emmet: No need to update PyV8
Package Control: Skipping automatic upgrade, last run at 2018-12-13 10:07:03, next run at 2018-12-13 11:07:03 or after
Unable to open /C/Users/CP/AppData/Roaming/Sublime Text 3/Packages/Emmet/Emmet.sublime-settings
Unable to open /C/Users/CP/AppData/Roaming/Sublime Text 3/Packages/Emmet/Default (Windows).sublime-keymap

But when I try to upgrade packages using Package Control, and trying to upgrade the packages, it says "There are no packages ready for upgrade." Does it have anything to do with that?

In case of single element, Tab key works for known tags only, otherwise it will replace any work with tag, event if you try to use native Sublime Text snippets. You should either use Ctrl-E shortcut or update Tab key handler: https://github.com/sergeche/emmet-sublime#tab-key-handler

Thanks for replying, it works fine as I wanted now.


docs: add hint for useAsyncData() in local environment

When working with APIs on localhost it seems that you have to use real IPs in order to work with useAsyncData() in SSR mode.
A small hint in the docs might help a lot as it's hard to find any information on this.

Thanks for updating the documentation @jannescb . However, for me, even using the IP does not help because it still treats it as part of the nuxt local instance and a internal loop is created.
I had to change /etc/hosts and create IP -> domain mask to test local CMS (Strapi or some own system).
It's very problematic. Not sure if it shouldn't be an issue @danielroe ?

Yes, this sounds like a bug; this hint shouldn't be necessary. I would appreciate an issue/reproduction if you can. 🙏

I'm closing this for now as I think this shouldn't be required, but please do create an issue with a reproduction if you can 🙏
Alternatively I think there are some factors developers on Windows might need to know and we could possibly hint about them. (cc: @pi0)

@danielroe Actually, I haven't noticed this problem lately.
I use Linux every day and the problem probably solved itself when I updated Nuxt or packages and the /etc/hosts file.
I can't say now what caused the problem (at least in my case), but I suspect it could be due to a wrong system configuration somewhere.


[Event Change Request]  page 344 Navigate - OnAfterSetSource

Hello,  can you add another variable PostingDateFilter in Event OnAfterSetSource?
DocTypeEnable := SourceType2 <> 0;
        SourceTypeEnable := SourceType2 <> 0;
        SourceNoEnable := SourceType2 <> 0;
        SourceNameEnable := SourceType2 <> 0;

        OnAfterSetSource(SourceType2, SourceType, SourceNo, SourceName, PostingDateFilter);
    end;

    local procedure SetSourceForPurchase()

 [IntegrationEvent(true, false)]
    local procedure OnAfterSetSource(var SourceType2: Integer; var SourceType: Text[30]; SourceNo: Code[20]; var SourceName: Text[100]; var PostingDateFilter: Text)
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 53020.


Deprecated API call for iOS 13+

Remove deprecated API that crashes in iOS 13+.

thanks. but the maintained repo is https://github.com/jasonelle/jasonette-ios


[PE-D][Tester E] Validity of Phone Numbers

Right now when we add a new candidate into the system, it only supports the use of numbers as valid phone numbers. However i believe that it should accept spaces for convenience, ie "9238 8172" should also be accepted. Additionally, HRs may work with people from foreign backgrounds. Can also consider accepting phone numbers with country codes ie "+1 8271 8127 8172" for international workers. (Considering Singapore's diverse background)

Labels: severity.Low type.FeatureFlaw
original: waihin26/ped#9

I guess this is a small limitation. You may update the UG under the fields section


Fix foodcritic messages

:+1:
Why do you need to sandbox, though?

I just copied an example I found online. Didn't really get in too deep :smile:

Also it's behaving weirdly: I get different messages locally than I do from TravisCI and the ones I get locally are false positives. I have different Ruby versions setup but if foodcritic is dependent on the Ruby version then it's not really reliable as a linting tool for recipe.

weird... never had that issue, but sounds like it would be woth a look on friday


Silence lndir output in symlinkJoin

Motivation for this change
Recently, haskell-packages started using symlinkJoin for ghcWithPackages, and lndir's verbose output makes build output far less readable.  This PR silences that output.
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

It looks like lndir is still outputting some garbage on stderr, but I'm not sure if there's a good way to prevent that.  It looks like this:
/nix/store/xcsbbnz1jjsfp9py4a14nklzyz3nzaqm-time-locale-compat-<IP_ADDRESS>/nix-support:
propagated-native-build-inputs: /nix/store/1yk710b6q79935691189h1aqx8w97m5f-unordered-containers-<IP_ADDRESS>/nix-support/propagated-native-build-inputs

Thanks!


fix(mixins): Fix deprecation warning for sr-only

Overview
This was annoying.  Sorry for keeping it in so long.
PR Checklist

[ ] Related to Abstract designs:
[ ] Related to JIRA ticket: ABC-123
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

Description

📬Published Alpha Packages:@codecademy/gamut-labs@4.1.1-alpha.de0a33.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f6a1acb00678d126b9fccd5--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@4.1.2-alpha.7125d0.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f6a206b84b34900e7d42880--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@4.1.2-alpha.6358d2.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f6a221e0ca74c01588930a0--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@4.1.4-alpha.ad21cb.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f6a3e630a3ee323ecc77ec6--gamut-preview.netlify.appDeploy Logs


Request doesn't works

Hi,
i followed the instructions for this package. But when i execute a simple request i get this message:
"Error refreshing the OAuth2 token, message: '{
"error" : "invalid_grant"
}'"
What mean ?

I think you forgot to include ga: before your key!

Ok, i'll try again. Thanks


MINOR: Do not disable heartbeat during Rebalance

Allow the heartbeat thread to send hb request during rebalance; in turn when handling responses if the state is not in STABLE ignore the error.

Piggy-backing a log4j improvement on the broker coordinator for triggering rebalance reason, as I found it a bit blurred during the investigation.

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Checkstyle failure:
[ant:checkstyle] [ERROR] /Users/boyang.chen/code/secondary/kafka-2/clients/src/test/java/org/apache/kafka/clients/consumer/internals/AbstractCoordinatorTest.java:57:8: Unused import - java.time.Duration. [UnusedImports]```

@ableegoldman I'm re-targeting this against https://issues.apache.org/jira/browse/KAFKA-10134 now as I found the root cause of that ticket is actually because we do not send (and hence reset timer of) heartbeat during rebalance, hopefully I will have this PR in a final-reviewable manner for merge.

@ableegoldman @vvcephei @hachikuji Please take a look at the updated description on the top.

Cherry-picked to 2.6


Finger-printing assets?

Assets referenced in markdown will not work because they are finger-printed. Is there a way to do this with ember-remarkable?

Oops wrong repo sorry


[iOS] UiKitView is not receiving gestures after setState is called

UIKitViews containing a MKMapView are not getting gestures after setState is called. Initially the Map is displayed correctly and user interaction is normal, but after rebuilding the UIKitView only one more gesture is recognized by the MapView before it stops receiving them. No error is thrown, but running the App with Flutter run --verbose suggests that gesture actions are not send to the UIKitView anymore.

Example project can be found here: https://github.com/LuisThein/apple_maps_example

flutter doctor -v
[✓] Flutter (Channel stable, v1.17.5, on Mac OS X 10.15.6 19G73, locale en-GB)
    • Flutter version 1.17.5 at /Users/taha/Code/flutter_stable
    • Framework revision 8af6b2f038 (5 weeks ago), 2020-06-30 12:53:55 -0700
    • Engine revision ee76268252
    • Dart version 2.8.4

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.1)
    • Android SDK at /Users/taha/Code/sdk
    • Platform android-30, build-tools 30.0.1
    • ANDROID_HOME = /Users/taha/Code/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.6)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.6, Build version 11E708
    • CocoaPods version 1.9.3

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 48.0.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

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

[✓] Connected device (2 available)
    • SM M305F  • 32003c30dc19668f                     • android-arm64 • Android 10 (API 29)
    • iPhone 11 • 1EB615AF-5AB9-4AAA-A587-A6642BB79484 • ios           •
      com.apple.CoreSimulator.SimRuntime.iOS-13-6 (simulator)

• No issues found!

flutter doctor -v
[✓] Flutter (Channel dev, 1.21.0-1.0.pre, on Mac OS X 10.15.6 19G73, locale en-GB)
    • Flutter version 1.21.0-1.0.pre at /Users/taha/Code/flutter_dev
    • Framework revision f25bd9c55c (3 weeks ago), 2020-07-14 20:26:01 -0400
    • Engine revision 99c2b3a245
    • Dart version 2.9.0 (build 2.9.0-21.0.dev 20bf2fcf56)

⣾^[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.1)
    • Android SDK at /Users/taha/Code/sdk
    • Platform android-30, build-tools 30.0.1
    • ANDROID_HOME = /Users/taha/Code/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.6)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.6, Build version 11E708
    • CocoaPods version 1.9.3

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

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 48.0.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

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

[✓] Connected device (5 available)
    • SM M305F (mobile)  • 32003c30dc19668f                     • android-arm64  • Android 10 (API 29)
    • iPhone 11 (mobile) • 1EB615AF-5AB9-4AAA-A587-A6642BB79484 • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-13-6 (simulator)
    • macOS (desktop)    • macos                                • darwin-x64     • Mac OS X 10.15.6
      19G73
    • Web Server (web)   • web-server                           • web-javascript • Flutter Tools
    • Chrome (web)       • chrome                               • web-javascript • Google Chrome
      84.0.4147.105

• No issues found!

@cyanglaz do you know what might be going on?

Looks like it might be related how touch events is implemented in the MKMapView. I guess(only based on my observation of the behavior) is that MKMapView is tracking some state in onTouchBegin and onTouchEnded in some gesture recognizer.
It's worth to try the WaitUntilTouchesEnded blocking policy.
registrar(forPlugin: "Test")?.register(appleMapsViewFactory, withId: "example.com/appleMapsExample", gestureRecognizersBlockingPolicy:FlutterPlatformViewGestureRecognizersBlockingPolicyWaitUntilTouchesEnded)


Cap auto-created retention policy replica count at 3

Defaulting to the number of nodes in the cluster is doesn't make
sense with larger clusters. (e.g. 10 nodes = RF 10)

+1 on green build.


can we set table.consume.mode equal incremental or latest with presto session？

can we set table.consume.mode equal incremental or latest with presto session？

It could work since it calls the input format anyway ultimately. But dont know of anyone whos tried it. Most people use Spark or Hive for ETLs and Presto for snapshot queries.. Mind sharing your use-case?

It could work since it calls the input format anyway ultimately. But dont know of anyone whos tried it. Most people use Spark or Hive for ETLs and Presto for snapshot queries.. Mind sharing your use-case?

Thanks for your  reply, I just wanna use a same pipeline for  incremental or batch ETLs , the hive not allowed to be modified the config at the runtime,  and the hive policy can't allowed to be modified. For the moment, I have no easy answers to the problem, just want it set up, out-of-the-box.

So I checked this out. Seems like we cannot set custom variables on Presto like we can do in Hive.
https://hudi.apache.org/docker_demo.html#step-7--incremental-query-for-copy-on-write-table
if we can find a workaround for this, we can make it work.. @bhasudha as well to share any tips

Closing due to inactivity. Please reopen if needed


chore: Bump LimeSurvey docker image tag to 5.3.6+220321

Codecov Report

Merging #371 (b6956c4) into master (cba6601) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master      #371   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            6         6
  Lines          271       271
  Branches        31        31
=========================================
  Hits           271       271

Continue to review full report at Codecov.

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


Remove v0.9.0 deprecations

[x] I am familiar with the contributing guidelines
[x] Tests added
[x] Updates entries in docs/sphinx/source/reference for API changes.
[x] Adds description and name entries in the appropriate "what's new" file in docs/sphinx/source/whatsnew for all changes. Includes link to the GitHub Issue with :issue:`num` or this Pull Request with :pull:`num`. Includes contributor name and/or GitHub username (link with :ghuser:`user`).
[x] Pull request is nearly complete and ready for detailed review.
[x] Maintainer: Appropriate GitHub Labels (including remote-data) and Milestone are assigned to the Pull Request and linked Issue.

This PR removes things deprecated in v0.9.0 and marked for removal in v0.10.0:

SingleAxisTracker
the "pass-through" attributes for PVSystem and ModelChain
the celltemp and inverter PVSystem methods replaced by get_ wrappers
eta_m in temperature.pvsyst_cell

I just looked through these changes again, and reviewed the docs build logs, and things seem ok to me to.  Merging...


Canonicalize the conforming type of BuiltinConformances.

It turns out they aren't always canonical by construction (and they don't
really need to be), so we should canonicalize the type and requirements
when we canonical a conformance ref. Fixes rdar://94877954.

@swift-ci Please test

@swift-ci Please test


Add datagouvfr extension

 Issue by l-vincent-l
Thursday Aug 03, 2017 at 10:24 GMT
Originally opened as https://github.com/catalogueglobal/datatools-server/pull/28

We were not allowed to download a file when the user is not logged in.

l-vincent-l included the following code: https://github.com/catalogueglobal/datatools-server/pull/28/commits

 Comment by landonreed
Tuesday Aug 29, 2017 at 15:34 GMT

@l-vincent-l, we're going to merge this code and I will adapt it to match the changes in #32 that makes common more of the boilerplate code that was necessarily included in your original PR.  Please see #32 to follow this work.
One quick thing though: could you fix the new conflict to the configurations/server.yml that arose out of merging #26?

 Comment by l-vincent-l
Wednesday Aug 30, 2017 at 09:26 GMT

Done

 Comment by landonreed
Wednesday Aug 30, 2017 at 10:25 GMT

Sorry, @l-vincent-l, one more thing.  Could you explain the changes to FeedSource.java (code comments eventually, but a message here is ok)?
It looks like this is a pretty significant change to how fetch URLs are used and I wouldn't expect an extension to need this kind of change.  Perhaps this could be addressed with some of my changes in #32?

 Comment by l-vincent-l
Wednesday Aug 30, 2017 at 12:30 GMT

The thing is that I want to fetch url from an API endpoint (e.g. http://www.data.gouv.fr/api/1/datasets/horaires-des-tram-train-ter-pays-de-la-loire/)
In this endpoint I want to be able to download the dataset at that URL :  https://ressources.data.sncf.com/explore/dataset/sncf-tram-train-ter-pdl-gtfs/download?format=csv&timezone=Europe/Berlin&use_labels_for_header=true
So my URL isn't stored, but retrieved from the API.

 Comment by landonreed
Thursday Aug 31, 2017 at 17:32 GMT

If https://ressources.data.sncf.com/explore/dataset/sncf-tram-train-ter-pdl-gtfs/download?format=csv&timezone=Europe/Berlin&use_labels_for_header=true  is a static URL, is there any reason that it can't be stored during import rather than fetched each time it is needed?

 Comment by l-vincent-l
Monday Sep 04, 2017 at 12:35 GMT

It might change, so I want to reference the datagouv url.

 Comment by maukoquiroga
Tuesday Sep 05, 2017 at 16:57 GMT

News?

 Comment by abyrd
Thursday Sep 21, 2017 at 18:13 GMT

Hi @maukoquiroga and @l-vincent-l, I've switched on notifications for this repo now. I wasn't keeping up on the comments before. As stated elsewhere we're focused completely on replacing the MapDB backend, which is a large task. Landon had begun refactoring the extension points so we could reuse more code, and this should also make your implementation simpler. But we can't make that a priority until we truly complete this storage backend transition. I'm thinking that should be basically finished next week.

 Comment by maukoquiroga
Sunday Jan 14, 2018 at 18:24 GMT

Hi there, as I guess you've finished replacing the MapDB backend, could we have a visibility on this one? Thanks 😄 .

 Comment by l-vincent-l
Tuesday Jan 16, 2018 at 17:27 GMT

Not needed anymore.


tofu geom config loading and definition

Motivations:

The utility tool to load a default tofu config is used quite often and should more readily accessible
Several config exist in various versions now, and the most complete include PF coils, which are necessary for some uses only

Proposed solutions:

Implement a shortcut to call the config-loading routine from the root of tofu (so the user does not have to go into tofu.geom.utils)
Implement a convention: the configuration includes coils only if its name has '-coils' in it
Move default config definition to a separate dedicated file to make tofu.geom.utils.py lighter and cleanly separate input fro routines

Great idea !
To be clear, we are talking about tf.geom.utils.create_config right ?
And for the coils option, what about an optional parameter with_coils=False ?

Yes, we are talking about tf.geom.utils.create_config.
Good idea too, I don't know which is best honestly.
I have implemented the '-coils' option already (see PR #464 ), I propose we stick to it for a while and see if users complain about it ?


feat: Allow communication between pods on workers and pods using the primary cluster security group

PR o'clock
Description
Adds two security group rules allowing pods running on worker nodes to communicate with pods using the primary cluster security group (such as pods running on Fargate).
See https://github.com/terraform-aws-modules/terraform-aws-eks/pull/866#discussion_r428140829.
Checklist

[x] CI tests are passing
[x] README.md has been updated after any changes to variables and outputs. See https://github.com/terraform-aws-modules/terraform-aws-eks/#doc-generation

This is a potentially breaking change as it may conflict with rules put in place by users already running mixed clusters. I think there should be a variable, default false, to enable creation of these rules. We can then remove it in a future major release.

Agree @dpiddockcmp and done.

Error: Provider produced inconsistent final plan

When expanding the plan for
module.eks.aws_security_group_rule.cluster_primary_ingress_workers[0] to
include new values learned so far during apply, provider
"registry.terraform.io/-/aws" produced an invalid new value for
.security_group_id: was cty.StringVal(""), but now
cty.StringVal("sg-0892893c8410071f4").

This is a bug in the provider, which should be reported in the provider's own
issue tracker.

Means that module.eks.aws_security_group_rule.cluster_primary_ingress_workers[0].security_group_id was at "" during the plan phase and became sg-0892893c8410071f4 during the apply.
I think it's because of :
cluster_primary_security_group_id = var.cluster_version >= 1.14 ? element(concat(aws_eks_cluster.this[*].vpc_config[0].cluster_security_group_id, list("")), 0) : null

local.cluster_primary_security_group_id is empty for all clusters created before Kubernetes 1.14 and eks.3 during the plan phase. So during the apply phase, the cluster is updated and the primary security group is created by AWS. This is probably the inconsistency.
You can notice with a terraform plan, that there is no security_group_id for the cluster_primary_ingress_workers SG rule.
  # module.eks.aws_security_group_rule.cluster_primary_ingress_workers[0] will be created
  + resource "aws_security_group_rule" "cluster_primary_ingress_workers" {
      + description              = "Allow pods running on workers to send communication to cluster primary security group (e.g. Fargate pods)."
      + from_port                = 0
      + id                       = (known after apply)
      + protocol                 = "-1"
      + self                     = false
      + source_security_group_id = "sg-0d741dba799316523"
      + to_port                  = 65535
      + type                     = "ingress"
    }

So we have to ensure a correct ordering during resources creation to let Terraform wait for the SG creation or at least mark the security_group_id as unknown.
Here how I make it work (but I don't know if it brokes something else).
locals {
  cluster_primary_security_group_id = var.cluster_version >= 1.14 ? coalescelist(data.null_data_source.cluster[*].outputs["cluster_primary_security_group_id"], [""])[0] : null
}

data "null_data_source" "cluster" {
  count = var.create_eks ? 1 : 0

  inputs = {
    cluster_primary_security_group_id = aws_eks_cluster.this[0].vpc_config[0].cluster_security_group_id
  }
}

With the null_data_source, Terraform wait for the cluster_security_group_id attribute to be available.
You can see the difference with a plan (security_group_id = (known after apply))
  # module.eks.aws_security_group_rule.cluster_primary_ingress_workers[0] will be created
  + resource "aws_security_group_rule" "cluster_primary_ingress_workers" {
      + description              = "Allow pods running on workers to send communication to cluster primary security group (e.g. Fargate pods)."
      + from_port                = 0
      + id                       = (known after apply)
      + protocol                 = "-1"
      + security_group_id        = (known after apply)
      + self                     = false
      + source_security_group_id = "sg-01ee228ae43275c29"
      + to_port                  = 65535
      + type                     = "ingress"
    }

This will also make the output cluster_primary_security_group_id available only when the SG (cluster_security_group_id) is created by AWS.
@dpiddockcmp @itssimon Is this work for you ? Do you see any downside or side effects of doing this ?
@dpiddockcmp I assigned this PR to you, so you can squash and merge it when it's ready ^^

I don't think the added complexity of using a null_data_source here is necessary. I also tried removing the local so that the security groups were directly referencing aws_eks_cluster.this[0].vpc_config[0].cluster_security_group_id. Same result when updating a 1.13 cluster but only for the aws_security_group_rule.cluster_primary_ingress_workers resource. The other applied correctly.
I guess don't enable this on a 1.13 cluster 😉

I don't think the added complexity of using a null_data_source here is necessary. I also tried removing the local so that the security groups were directly referencing aws_eks_cluster.this[0].vpc_config[0].cluster_security_group_id. Same result when updating a 1.13 cluster but only for the aws_security_group_rule.cluster_primary_ingress_workers resource. The other applied correctly.

Ohh that was the first thing I've tried, but no luck. Maybe I didn't test it correctly ^^


Repackaging - ISC Bind

Required items, please complete
Change(s):

See guidance below

Reason for Change(s):

See guidance below

Version Updated:

Required only for Detections/Analytic Rule templates
See guidance below

Testing Completed:

See guidance below

/Package


Instruct users to clone specific branch of samples instead of master

Instruct users to clone specific branch of samples instead of master
Description
Replaced 'git clone https://github.com/dapr/samples.git' with 'git cl…one -b <dapr_version_tag> https://github.com/dapr/samples.git'. The change was made for all relevant samples except the "Dapr with Docker-Compose" sample as that one contains a Docker Compose file which refers to "daprio/daprd:edge". The edge image should always be compatible with the Daper samples master branch.
Issue reference
dapriosamples/hello-k8s-node:edge as used in sample "2.hello-kubernetes" fails with Dapr 0.8.0 as DAPR_HTTP_PORT and DAPR_GRPC_PORT isn't set #251
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] The sample code compiles correctly
[x] You've tested new builds of the sample if you changed sample code
[x] You've updated the sample's README if necessary

@HansK-p thanks for your contribution :)

@pruthvidhodda No problem:). This is one of the strengths of open source, and Dapr is an interesting open source project....


Error when import template community-templates/Storage_Devices/EMC/template_emc_unity/6.0/

Hello!
When i trying to import template template_emc_unity.yaml  to zabbix i have an error:
Invalid parameter "/1/request_method": value must be 0.

How can i resolve it?

The same problem with another template, downloaded from the same version of Zabbix 6.4.1.

Invalid parameter "/1/request_method": value must be 0.  same 6.4.1 for a palo alto 6.0 template. any fixes?

The same error message, even if importing other template files.
From the following link I found the answer, maybe helpful for you all.
https://www.zabbix.com/forum/zabbix-troubleshooting-and-problems/461658-error-importing-template

Template needs to be re-written.
Just open the Template YAML-file and comment "request_method" strings with #
Save it and import.

I deleted the whole line of "request_method" string and saved.
Finally imported successfully.


SSH test fails

Hi
I'm trying to test the ssh connection but it reports as "failed".
I see this in the logs:
[2016-12-13 22:51:04] production.DEBUG: ### Run script over SSH
ssh -o CheckHostIP=no \
    -o IdentitiesOnly=yes \
    -o StrictHostKeyChecking=no \
    -o PasswordAuthentication=no \
    -o IdentityFile=/var/www/deployer/storage/app/tmp/sshkeyFgEHq7 \
    -p 22<EMAIL_ADDRESS>'bash -s' << 'EOF'
        # Turn on quit on non-zero exit
        set -e
        ### Test server connection - 1
# Ensure the directory exists and can be written to
cd /tmp
ls

# Ensure it can be written to
touch<PHONE_NUMBER>_testing_deployer.txt
echo "testing" >><PHONE_NUMBER>_testing_deployer.txt
chmod +x<PHONE_NUMBER>_testing_deployer.txt
rm<PHONE_NUMBER>_testing_deployer.txt

# Ensure directories can be made
mkdir<PHONE_NUMBER>_testing_deployer_dir

touch<PHONE_NUMBER>_testing_deployer_dir/1481665864_testing_deployer.txt
echo "testing" >><PHONE_NUMBER>_testing_deployer_dir/1481665864_testing_deployer.txt
chmod +x<PHONE_NUMBER>_testing_deployer_dir/1481665864_testing_deployer.txt

ls<PHONE_NUMBER>_testing_deployer_dir/

rm -rf<PHONE_NUMBER>_testing_deployer_dir
EOF

I can't really see what's going wrong, maybe you can?
Thanks.

I had the similar problem.
At first check that you have added an public key into the .ssh/authorized_keys. If yes then check permissions for .ssh folder (should be 700) and authorized_keys file (should be 600).

I was just typing that very reply 😂

I did add the key that deployer generated. The other keys i have in the
authorized_keys file can connect without any problem :)
On Wed, Dec 14, 2016 at 5:14 PM ValerioOnGithub<EMAIL_ADDRESS>wrote:

I had the similar problem.
At first check that you have added an public key into the
.ssh/authorized_keys. If yes then check permissions for .ssh folder (should
be 700) and authorized_keys file (should be 600).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/REBELinBLUE/deployer/issues/294#issuecomment-267076817,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAmofKsqm4IaraMmPQrCslGRKR9Pad4Lks5rIBXRgaJpZM4LMRa1
.

It is a bit messy but could you edit app/Jobs/TestServerConnection.php and on line 62
change
$this->server->status = Server::FAILED;

to
$this->server->status = Server::FAILED;
\Illuminate\Support\Facades\Log::debug($error->getMessage());

then restart the queue listener, it should log the error message now

@REBELinBLUE I edited as followed:
try {
            $process = new Process('TestServerConnection', [
                'server_id'      => $this->server->id,
                'project_path'   => $this->server->clean_path,
                'test_file'      => time() . '_testing_deployer.txt',
                'test_directory' => time() . '_testing_deployer_dir',
            ]);

            $process->setServer($this->server, $key)
                    ->run();

            if (!$process->isSuccessful()) {
\Illuminate\Support\Facades\Log::debug("failung here");
                $this->server->status = Server::FAILED;
            } else {
                $this->server->status = Server::SUCCESSFUL;
            }
        } catch (\Exception $error) {
            $this->server->status = Server::FAILED;
\Illuminate\Support\Facades\Log::debug("yoyoyo");
\Illuminate\Support\Facades\Log::debug($error->getMessage());
        }

And got:
[2016-12-14 17:31:05] production.DEBUG: ### Run script over SSH
ssh -o CheckHostIP=no \
    -o IdentitiesOnly=yes \
    -o StrictHostKeyChecking=no \
    -o PasswordAuthentication=no \
    -o IdentityFile=/var/www/deployer/storage/app/tmp/sshkeyC6WJEk \
    -p 22<EMAIL_ADDRESS>'bash -s' << 'EOF'
        # Turn on quit on non-zero exit
        set -e
        ### Test server connection - 2
# Ensure the directory exists and can be written to
cd /var/www/test-chameleon-deployer
ls

# Ensure it can be written to
touch<PHONE_NUMBER>_testing_deployer.txt
echo "testing" >><PHONE_NUMBER>_testing_deployer.txt
chmod +x<PHONE_NUMBER>_testing_deployer.txt
rm<PHONE_NUMBER>_testing_deployer.txt

# Ensure directories can be made
mkdir<PHONE_NUMBER>_testing_deployer_dir

touch<PHONE_NUMBER>_testing_deployer_dir/1481733065_testing_deployer.txt
echo "testing" >><PHONE_NUMBER>_testing_deployer_dir/1481733065_testing_deployer.txt
chmod +x<PHONE_NUMBER>_testing_deployer_dir/1481733065_testing_deployer.txt

ls<PHONE_NUMBER>_testing_deployer_dir/

rm -rf<PHONE_NUMBER>_testing_deployer_dir
EOF

[2016-12-14 17:31:06] production.DEBUG: failung here

Does the process have a way to report what error it gets?

Hmm, "failung here" is really odd. Can't find anywhere that string would be coming from.
Try $process->getOutput(); I may have to change the server testing to log the output like it does during deployment, hadn't done it as there are normally only 2 or 3 possible causes at this strange, incorrect IP, incorrect username or missing SSH key but it doesn't seem to be any of those.

Never mind, I see where "failung here" is. Yeah try $process->getOutput() there

Unfortunately it's empty :/
[2016-12-16 17:09:47] production.DEBUG: ### Run script over SSH
ssh -o CheckHostIP=no \
    -o IdentitiesOnly=yes \
    -o StrictHostKeyChecking=no \
    -o PasswordAuthentication=no \
    -o IdentityFile=/var/www/deployer/storage/app/tmp/sshkeyZfBJql \
    -p 22<EMAIL_ADDRESS>'bash -s' << 'EOF'
        # Turn on quit on non-zero exit
        set -e
        ### Test server connection - 2
# Ensure the directory exists and can be written to
cd /var/www/test-chameleon-deployer
ls

# Ensure it can be written to
touch<PHONE_NUMBER>_testing_deployer.txt
echo "testing" >><PHONE_NUMBER>_testing_deployer.txt
chmod +x<PHONE_NUMBER>_testing_deployer.txt
rm<PHONE_NUMBER>_testing_deployer.txt

# Ensure directories can be made
mkdir<PHONE_NUMBER>_testing_deployer_dir

touch<PHONE_NUMBER>_testing_deployer_dir/1481904587_testing_deployer.txt
echo "testing" >><PHONE_NUMBER>_testing_deployer_dir/1481904587_testing_deployer.txt
chmod +x<PHONE_NUMBER>_testing_deployer_dir/1481904587_testing_deployer.txt

ls<PHONE_NUMBER>_testing_deployer_dir/

rm -rf<PHONE_NUMBER>_testing_deployer_dir
EOF

[2016-12-16 17:09:47] production.DEBUG: failung here
[2016-12-16 17:09:47] production.DEBUG:

try $process->getErrorOutput();

Ok, i get Permission denied (publickey).
Guess i need to do some more digging at make it not delete the keyfile so i can try it myself manually.

You can find the key in the DB in the projects table, but that sounds like one of the normal errors.... wrong username or SSH key not in ~/.ssh/authorized_keys for the user 😐

Any luck?

I am closing this for now, feel free to reopen


Stack buffer overflow crash in RLMHashData

Xcode 7 beta 4 & iOS 9 beta 4 on iPhone 5
When I try to init with Realm(), I get crash only if it's run by Xcode:

and

Some more informations: I get this error message with "Address Sanitizer" enabled. Otherwise it crash like this

Also, I found apparently this is not related to "run by Xcode", but the "All Exceptions" breakpoint enabled.

Hi @wangshengjia, we're looking into this, although we haven't been able to reproduce the issue in our first attempts.

I've run our unit tests in a number of configurations without being able to reproduce the issue. Tests were run on Xcode 7 beta 4 with an "All Exceptions" breakpoint and "Address Sanitizer" enabled.

iOS 9 beta 4 on an iPhone 6
iOS 7.0.3 on an iPhone 5S
iOS 8.1.3 on an iPhone 5

Given the circumstances in which Address Sanitizer detected a potential issue in combination with the "All Exceptions" breakpoint, I'd say this was probably a false positive and I wouldn't worry too much about it. As soon as you start catching exceptions in Objective-C code that isn't compiled to be safe with exceptions, you'll start seeing some funny behavior.
So in light of this, I'm closing the issue. If you can provide us a reproducible scenario affecting "normal" development (without Address Sanitizer or All Exceptions breakpoint), we'll reopen and address it.
Thanks!

I am totally agreed. Thank you all :+1:


[IA-2566] Fix Galaxy 401

We recently added a referrer check to all leo proxy routes (this feature is currently disabled in all environments until https://broadworkbench.atlassian.net/browse/IA-2566 is complete which is dependent on the AoU team). here is the PR for the referrer check: https://github.com/DataBiosphere/leonardo/pull/1854. we essentially check that every request to the leonardo proxy has a Referer header which is in our list of valid referrers to help prevent CSRF attacks
While doing some testing in dev with the referrer check enabled - realized that since Galaxy opens in a new tab (unlike jupyter and rstudio) that we were throwing a 401 since previously we were stripping the referrer in terra-ui when a new tab is opened:

This PR allows the referrer to be present in a new tab only when a user launches Galaxy. I tested this in my Fiab by pointing my local terra UI at my fiab and manually enabling the referrer check and verifying that Galaxy no longer fails with a 401:

Tested locally, looks good. For context, to anyone reading this who is curious, there is a lot of hidden stuff going on here. First of all, the snippet omitted by github that specifies these attributes are on a button that has an   tag  (i.e. a link) is important. Second of all, there are various attributes that can be specified within rel for these tags that can specify links. We are modifying one such attribute, which is detailed here: https://developer.mozilla.org/en-US/docs/Web/HTML/Link_types/noreferrer

Since newTabLinkPropsGalaxy is only used in the newGalaxyModal it seems like adding it to the general utils file would only make sense if it got renamed to something not Galaxy specific newTabLinkWithReferer with the assumption we might want to have a referer for other new tabs in the future.
Or possibly another more straightforward option would be to use the existing newTabLinkProps but overwrite the rel field when using it in the newGalaxy Modal and then you wouldn't have to add a new thing to utils at all

Since newTabLinkPropsGalaxy is only used in the galaxyLaunchButton it seems like adding it to the general utils file would make more sense if it was named to something not Galaxy specific ie newTabLinkWithReferer with the assumption we might want to include a referer for other new tabs in the future, otherwise another option would be to use the existing newTabLinkProps but overwrite the rel field when using it in the galaxyLaunchButton and then you wouldn't have to add a new thing to utils at all
i.e
 }, ...Utils.newTabLinkPropsGalaxy, rel: 'noopener',

I like the idea of renaming the method to something more general so that if anyone wants to do the same in the future they can just use this method :)


Sharing the split for partial training examples

Thank you for sharing the code,
Could you also share the data used for partial training examples (1%, 5% etc) for ytvis21?
The training script would also be appreciated!
thanks!

Hi @mkimhi, thanks for your attention.
We follow a similar way (MinVIS) to sample the partial video frames for ytvis21.
For example, for the 1% setting, we select 1% frames (at least one frame) from each video.
Since MinVIS did not provide the corresponding training dataset, it is also very difficult for us to align MinVIS. You can use similar methods to generate locally partial datasets that you want.


Exit after processing

Because rofi-polkit-agent spawns instances of itself from cmd-polkit-agent each time a request arrives, the agent only needs to read so long as a request is not fulfilled. New requests to cmd-polkit-agent will always spawn another rofi-polkit-agent. Over time this means users can end up with many rofi-polkit-agent processes stuck in a while loop waiting for messages.
Unfortunately this is not a true fix for the problem. There is an upstream bug with cmd-polkit-agent which causes it to not clear zombie processes properly. Over time users will still notice many zombie rofi-polkit-agent processes building up. However, the risk of running out of PID's is far lower than the risk of utilizing too many resources by running multiple agents perpetually.

I opened this issue with cmd-polkit to fix the zombie process problem.
Thanks for the great tool by the way. I realized I didn't say it before!

Thanks for the great tool by the way. I realized I didn't say it before!

Thank you!

I opened this issue with cmd-polkit to fix the zombie process problem.

Great! I've created a new https://github.com/czaplicki/rofi-polkit-agent/tree/dev branch while's resolving you issues/pull requests #3 & #2.
This commit 60b8e39 should resolve the exiting problems on my scripts side.

Just tested your commit on my local and it works with some minor tweaks. I had to remove the cat in terminate in order for the exit to be called. I also had to change the invalid json test to 5 for it to reach the terminate successfully. However, I re-added my code to remove the message since I still need that workaround anyway.


DiagLevel NullPointerExceptions

com/smartdevicelink/trace/DiagLevel

NullPointerException is thrown by the following methods int the DiagLevel test class.
try {
    // Invalid/Null Tests
    assertFalse(Test.FALSE, DiagLevel.isValidDetailLevel(testInvalidS));
    assertFalse(Test.FALSE, DiagLevel.isValidDetailLevel(null)); // Exception here
    assertNull(Test.NULL, DiagLevel.getLevel(null)); // Exception here
    DiagLevel.setLevel(null, null); // Exception here
    DiagLevel.setAllLevels(null); // Exception here
} catch (NullPointerException e) {
    fail("NullPointerException was thrown when attempting to set null values.");
}

Solution : The methods should verify that the parameters are not null before attempting to manipulate the data.

Fixed in this commit https://github.com/smartdevicelink/sdl_java_suite/commit/19363b7b89621d4cfd0b571734fd112fcfbd3af1


wine: Set WINELOADER in wrapper, wrap also wine64.

Fixes #63170. Note that this brings back wrapping of wine64 which was removed in #28486 because the underlying issue is now fixed.

Motivation for this change
Things done

Tested (on 19.03 branch) that with wineWowPackages.full:

wineboot populates the syswow64 directory in addition to system32 (before this fix syswow64 was not populated),

wine <32-bit-exe> works,

wine <64-bit-exe> works,

wine64 <32-bit-exe> works,

wine64 <64-bit-exe> works,

running a 32-bit exe from within a 64-bit program works,

running a 64-bit exe from within a 32-bit program works.

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)

Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"

[ ] Tested execution of all binary files (usually in ./result/bin/)

[ ] Determined the impact on package closure size (by running nix path-info -S before and after)

[ ] Assured whether relevant documentation is up to date

[x] Fits CONTRIBUTING.md.

Fixed two typos in comments (force push).

Ping. This fixes widespread Wine breakage, it's nice to have a working Wine.


adds startupMonitorPodConditionController for monitoring the lifecycle of the startup monitor pod

xref: openshift/enhancements#833

/lgtm
/approve


Draft: feat(color): Enable color output in GitLab CI

enables color output in CI.
Adds the option to disable color entirely with NO_COLOR (https://no-color.org/)

Codecov Report

Merging #636 (945a2be) into main (e631154) will increase coverage by 0.01%.
The diff coverage is 50.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main     #636      +/-   ##
==========================================
+ Coverage   93.98%   94.00%   +0.01%
==========================================
  Files         103      103
  Lines        4804     4819      +15
==========================================
+ Hits         4515     4530      +15
  Misses        289      289

Flag
Coverage Δ

unittests
94.00% <50.00%> (+0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

ggshield/cmd/main.py
87.17% <50.00%> (-2.01%)
:arrow_down:

... and 5 files with indirect coverage changes

@Walz This change is project wide and changes output behavior for secret detection as well.


Publish Page

Congratulations for this amazing work!
I would like to show you the Publish Page appearance, the profile selection isnt' visible, maybe needs some work here:

Thanks for fast answer @mantinband .
I've found this on Git Changes page:

I'm not sure if it's wrong, but I think I need to show you.

@HClausing I've updated the publish page to match the color theme:

I've found this on Git Changes page:

I'm not sure if it's wrong, but I think I need to show you.

You mean that the git container is darker than the background?

@HClausing I've updated the publish page to match the color theme:

I've found this on Git Changes page:

I'm not sure if it's wrong, but I think I need to show you.

You mean that the git container is darker than the background?

Sure, thanks for finding that bug!
Regarding the Git container, I think it looks good like this 😎


FIX: jenkins build

Add auto-rebuilding for the client with the dev docker-compose config.
Remove obsolete client Dockerfile.

Looks good to me 👍


macro calling twice a function to solve world age issues

In order to solve the world-age issue
julia> using LazyModules

julia> @lazy import ImageCore = "a09fc81d-aa75-5fe9-8630-4744c3626534"
LazyModule(ImageCore)

julia> function foo()
           c = ImageCore.RGB(0.0, 0.0, 0.0)
           return c .* 3
       end
foo (generic function with 1 method)

julia> foo()
ERROR: MethodError: no method matching length(::ColorTypes.RGB{Float64})
The applicable method may be too new: running in world age 31343, while current world is 31370.
...

julia> foo()
RGB{Float64}(0.0,0.0,0.0)

I wonder if we can introduce a macro to call the function twice:
@lazy ImageCore function foo()
    c = ImageCore.RGB(0.0, 0.0, 0.0)
    return c .* 3
end

expanding to
function foo()
    if is_not_imported_already(ImageCore)
        do_the_import(ImageCore)
        return foo()
    end
    c = ImageCore.RGB(0.0, 0.0, 0.0)
    return c .* 3
end

unfortunately, this approach doesn't really work:
julia> using LazyModules

julia> @lazy import ImageCore = "a09fc81d-aa75-5fe9-8630-4744c3626534"
LazyModule(ImageCore)

julia> function foo()
           if !ImageCore._lazy_loaded
               LazyModules.checked_import(ImageCore)
               return foo()
           end
           c = ImageCore.RGB(0.0, 0.0, 0.0)
           return c .* 3
       end
foo (generic function with 1 method)

julia> foo()
ERROR: MethodError: no method matching ColorTypes.RGB(::Float64, ::Float64, ::Float64)
The applicable method may be too new: running in world age 31334, while current world is 31363.
Closest candidates are:
  ColorTypes.RGB(::T, ::T, ::T) where T<:Union{AbstractFloat, FixedPointNumbers.FixedPoint} at ~/.julia/packages/ColorTypes/1dGw6/src/types.jl:108 (method too new to be called from this world context.)
  ColorTypes.RGB(::Real, ::Real, ::Real) at ~/.julia/packages/ColorTypes/1dGw6/src/types.jl:461 (method too new to be called from this world context.)
  ColorTypes.RGB(::Union{Real, ColorTypes.AbstractGray}, ::Union{Real, ColorTypes.AbstractGray}, ::Union{Real, ColorTypes.AbstractGray}) at ~/.julia/packages/ColorTypes/1dGw6/src/types.jl:603 (method too new to be called from this world context.)
  ...
Stacktrace:
 [1] foo()
   @ Main ./REPL[4]:6
 [2] foo()
   @ Main ./REPL[4]:4
 [3] top-level scope
   @ REPL[5]:1

Maybe @timholy has a solution?

We recently wrote some documentation to explain this, https://juliaio.github.io/FileIO.jl/dev/world_age_issue/
IIUC, there's no trivial solution that applies to all scenarios.


[Test] extract environment map as a class for testing

and try class mixin

It should consider more situations to support cross-file refactoring and jump to definition.

and try the hierarchy environment:
(struct env (binding* parent) #:transparent)
(define cur-env (make-parameter (make-env #f)))
(define (make-env [parent (cur-env)])
  (env (make-hash) parent))
;; bind, lookup

Notice this implementation need to check about concurrency problem since GUI could update it any moment. Each environment should be held by a semantic object, such as define, let, or a file.

close for now


Add .modal-open class to body if any modal is open

Hi, I am wondering if there are plans to support adding a .modal-open class to document.body whan any bootstrap modal is currently open?
We relied on this feature in vue 2 to apply additional styles to the body - changing the background color or bluring the content underneeth.
This is a hard thing to work around when you don't control the implementation of b-modal, so it would be nice to have this considered, or have an official alternative.
in vue 2:

in vue 3:

Relates https://github.com/bootstrap-vue/bootstrap-vue-next/issues/1030


Don't use arbitrary error message as the acknowledgement

Summary
currently, the transfer app uses whatever error message is returned as acknowledgment (code), which makes the error message a part of consensus state, so the error message change will be a consensus breaking change, which may not align with people's intuition, making it easier to bring in consensus breaking changes in minor upgrades.
Problem Definition

Proposal
Use static errors, suggested by @leejw51crypto

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged/assigned

Hi @yihuang, thanks for opening this issue. The code which you link is calling types.NewErrorAcknowledgement(err) from the transfer types pkg. See here.
This function extracts the abci error code and then includes a static ack error string (string constant).
// NewErrorAcknowledgement returns a deterministic error string which may be used in
// the packet acknowledgement.
func NewErrorAcknowledgement(err error) channeltypes.Acknowledgement {
	// the ABCI code is included in the abcitypes.ResponseDeliverTx hash
	// constructed in Tendermint and is therefore deterministic
	_, code, _ := sdkerrors.ABCIInfo(err, false) // discard non-determinstic codespace and log values

	errorString := fmt.Sprintf("ABCI code: %d: %s", code, ackErrorString)

	return channeltypes.NewErrorAcknowledgement(errorString)
}

I believe this is the desired behaviour which you propose. Please review this and feel free to close the issue or raise any other questions. Many thanks!

I see, I checked that in v2 and v3, it's indeed as you described, the issue I described is only in v1. I'll close this.


Multiple language support

Implement a language selector that will change all text in the app to the selected language.

screenshot ?

It's available here: https://create.dev.fairdatasociety.org

I see we will have to add whole language name, and extend button a bit, since it overflows in some cases

I've embedded Unicode characters for flags in the dropdown, but it seems that they can't be displayed on some systems/browsers. This is how it looks on my side:

Then maybe we should include svg images of flags.


Add authoring support for a @PageColor directive

Bug/issue #, if applicable: rdar://106153042
Summary
Adds a new @PageColor metadata directive that allows for customizing the color used to represent a given page.
@PageColor gives authors control over the color used when rendering a page – initially this will affect the background color Swift-DocC-Render uses in the page's introduction (hero) section.
Example
# What's New in SlothCreator

@Metadata {
    @PageColor(blue)
}

![A sloth on a tree wearing a fedora.](sloth-fedora)

Let's check out what's new in SlothCreator!

...

Details
Declaration
@PageColor(_ color: Color)

Overview
Use the PageColor directive to provide a hint to the Swift-DocC renderer as to what color should represent this page. The renderer will use this color, depending on the context, as a foundation for the other accent colors used in the page. For example, Swift-DocC-Render will use this color as the primary background color of a page’s introduction section and adjust the colors of the icon and text in the page’s introduction accordingly.
This directive is only valid within an @Metadata directive.
@Metadata {
    @PageColor(purple)
}

Parameters
@PageColor accepts the following parameters:

color: An unnamed parameter that accepts one of the following pre-defined colors. Each color is context-dependent.

blue
gray
green
orange
purple
red
yellow

Examples
The following screenshots show how Swift-DocC Render might interpret the given color inputs initially.

Forums post: https://forums.swift.org/t/support-for-customizing-a-page-s-accent-color-in-swift-docc/64093
Dependencies
Swift-DocC-Render PR TBD.
Testing
Add @PageColor to the @Metadata directive in a Swift-DocC article or extension file and confirm that the resulting RenderNode includes a color metadata property.
Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] Added tests
[x] Ran the ./bin/test script and it succeeded
[x] Updated documentation if necessary

@swift-ci please test

Not a blocker for this PR, but it would be awesome to update <doc:customizing-the-appearance-of-your-documentation-pages> article with an example of this new functionality.

Good point! I filed https://github.com/apple/swift-docc/issues/537 to track it.

@swift-ci please test

Does this work with Tutorials?

Does this work with Tutorials?

I am 99% sure it doesn't, Tutorial semantics are very different and this concept doesn't port cleanly over.


Mixed-content issues

Gitea version (or commit ref): 1.0.1
Git version: 2.7.4
Operating system: Ubuntu 16.04 LTS
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[x] SQLite

Can you reproduce the bug at https://try.gitea.io:

[x] Yes (provide example URL): https://try.gitea.io/gitea/gitea
[ ] No
[ ] Not relevant

Log gist:

Description
Gitea can still serve mixed-content pages, even with SSL. This is most pronounced with images being loaded over an insecure connection.
This can be solved by integrating a solution similar to camo (GitHub uses this) into Gitea.
In addition to security, it also provides a privacy benefit for users if the install is public.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

If we're embedding a camo server into gitea, I'd like to say go-camo is a thing and we could get most stuff from that: https://github.com/cactus/go-camo

go-camo makes sense, we don't want to reinvent the wheel

Anyone would like to send a PR to fix this?

Move this to v1.2 since no PR to send.

is there any progress?

Nobody are working on this.

can we do this onto 1.10.0?
i think it only afects the markdown parts of gitea?
which files are these so i'll have a look at it

to discust:
should we replace the src url of img nodes after blackfriday process the markdown
or
replace  with  ?

we ned in each case two config values

CAMO_URL
CAMO_KEY

for example:
nginx reverse proxy gitea.com/camo -> http://<IP_ADDRESS>:3456

CAMO_URL = https://gitea.com/camo
CAMO_KEY = 123secret

camo itself:
./go-camo -k jo --listen=<IP_ADDRESS>:3456 --stats --metrics &

I think the goal had been to embed it instead of relying on another service

We can use the routines of go-camo it shouldnt be that hard

I think the goal had been to embed it instead of relying on another service

and then https://github.com/go-gitea/gitea/issues/916#issuecomment-515718697 still remains

There has been a parallel discussion on this issue at Codeberg.org: https://codeberg.org/Codeberg/Community/issues/196