[Feature]: generate a pdf with PDF/A-3 format

Feature description
purpose: generate a pdf in PDF/A-3 format containing informations in xml format.
use case: in order to meet future accounting and invoicing requirements, pdf files must include informations in xml format (https://fnfe-mpe.org/factur-x/factur-x_en/).
https://stackoverflow.com/questions/74489918/does-puppeteer-support-pdf-a-3
https://stackoverflow.com/questions/57186252/does-puppeteer-support-creating-pdf-a

Puppeteer uses Chrome's PDF printing so this feature request should be on Chromium bug tracker
Please file a bug at https://bugs.chromium.org/p/chromium/issues/entry (Component: Internals>Skia>PDF)
Here is a bug that mentions that Chrome should produce at least PDF/A-3u crbug/1484941.


Getting specific currency

Hi , im tring to connect tebex api to my website which written in nextjs
how can i get the packages in the different currencies then the store default currency ?
and how can i get the packages tax based on customer ip address

Hi Larevio,
Thank you for bringing this to my attention. I reviewed the official documentation on Postman and found some information regarding this issue.
Tebex requires basic authentication using both your public and private keys to forward IPs.
I have prepared a brief example illustrating how to set up these keys, which you can find here: https://github.com/grp-gg/tebex_headless?tab=readme-ov-file#ip-forwarding
Please take a moment to review the latest commit to see the changes I made: https://github.com/grp-gg/tebex_headless/commit/930f7276374e7d7c805eb3f77b1b8807080c5f99
Make sure to use the latest package version (1.4.0) available on npm to access these features.
Once you have set up your keys, you can use the IP parameter in the following functions:
GetCategories
GetCategory
GetPackage
GetPackages
CreateBasket

Unfortunately, I cannot tell you to what extent Tebex determines the taxes or currency itself. I have not found anything about the currency itself in the Postman documentation.

so what we should do about currencies ?
the documents said this website  using headless api
and it support currencies

I think you have to implement it on your own. The example website displays the amount in the other currency but automatically switches it back (I think automatically based on your IP) when you're on the checkout page.

Hi @larevio,
In the past few days, I've been in communication with Tebex and their developers, and we've implemented some fixes.
Taxes and Country are now successfully determined based on the provided IP address in a basket, as you can see in this integration test.
However, there are still some issues:

The currency in packages and baskets is still determined by your currency or webstore settings.
When you request all packages with the GetCategories or GetPackages functions (but it will work when you provide a basket identifier), the taxes are still based on your IP address.


gulp-rename 1.4.0 does not signal async completion

I recently ran yarn upgrade and my gulp builds for vscode started failing:
[01:43:25] The following tasks did not complete: compile-extensions-build
[01:43:25] Did you forget to signal async completion?

I bisected it by checking each dependency update with git bisect and traced it back to upgrading gulp-rename from 1.2.2 to 1.4.0. It appears that it fires an async task that it never completes, per the error message - and CHANGELOG.md doesn't have any records pre-2.0. Were there any breaking changes between the two releases that I need to account for?

Can you include a full code sample?

Hey there - sorry about the delay on a response; I'm currently working through my final exams at university, so time is limited!
I'll write a reproduction script over this weekend/early next week.

This became a difficult problem to reproduce. I tried to reproduce this as a simple case in a different project than the real project:
Node Version: 16.14.0
gulpfile.js
const gulp = require('gulp')
const glob = require('glob')
const rename = require('gulp-rename')
const source = require('vinyl-source-stream')
const print = require('gulp-print').default
const es = require('event-stream')
const browserify = require('browserify')

gulp.task('default', function (end) {
  glob('./app*.js', function (err, files) {
    if (err) end(err)

    var tasks = files.map(function (entry) {
      console.log('entry', entry)
      var bundleStream = browserify(entry).bundle()

      return bundleStream
        .pipe(source(entry))
        .pipe(rename({
          dirname: '',
          extname: '.js'
        }))
        .pipe(gulp.dest('./build'))
        .pipe(print(function (filepath) {
          return console.log('Completed JS: ' + filepath)
        }))
    })

    console.log('Waiting for merge')
    es.merge(tasks).on('end', () => {
      console.log('The end')
      end()
    })
  })
})

package.json
{
  "name": "gulp-error",
  "version": "1.0.0",
  "description": "",
  "main": "app1.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "author": "",
  "license": "ISC",
  "devDependencies": {
    "browserify": "^16.5.2",
    "event-stream": "3.3.4",
    "gulp": "^4.0.2",
    "gulp-print": "^5.0.2",
    "gulp-rename": "^1.4.0",
    "vinyl-source-stream": "^2.0.0"
  },
  "dependencies": {}
}

app1.js
console.log('App 1')

app2.js
console.log('App 2')

the issue is not reproducible in this simple project but if it doesn't work in the real project even if all dependencies are exactly same and of same version.
I did a fresh install of node_modules and checked it on node 14.18.3 too but this didn't help.

@riteshjagga Can you use the finished function instead of listening for the end event on the es.merge?
Also - usage of event-stream is not recommended, it's 3 years out of date and has a lot of issues dealing with newer streams - use a different library to merge the streams. Likely this is the root issue.

@yocontra Thank you for the suggestion, listening the finished instead of end throws the error for 1.2.2. too. I'm not entirely sure though that if it is event stream issue. Trying more...
I inspected in detail and found that commenting this line of code fixes the issue.
Just in case it is helpful, this line of code was added on 25 Aug 2015 few months after the npm publish of 1.2.2 on 02 Apr 2015.
Trying to use any merge stream now as it is recommended in a note by the event stream to work with Gulp 4...

For the complete gulpfile, please see this comment.
I don't completely understand streams and buffers but I think this has to do with the file size which has to be renamed. When I tried to reproduce this, the file size could have been very small and rename didn't throw the error but it throws the error in the real project where file size is large.
Adding buffer (as explained in the gulp-buffer package documentation before the gulp-rename fixed the issue.
Also repeating here that it was the cloning of file in the gulp-rename package which led to this error.

Got a similar issue:
I have a folder with subfolders and tens of scss files that needs to be processed into similar folders without combining.
e.g.
src/
-folder_1/
--file_1.scss
-folder_2/
--file_2.scss
to
build/
-folder_1/
--file_1.css
--file_1-rtl.css
-folder_2/
--file_2.css
--file_2-rtl.css
    $.gulp.task("styles:page", () => {         let fileBase = "src/";         return (             $.gulp                 .src([                     fileBase + "**/*.{scss,map}",                  ])                 .pipe(                     scss({                         outputStyle: "compressed",                     })                 )                 .pipe(                     autoprefixer({                         overrideBrowserslist: ["last 3 versions"],                     })                 )                 .pipe(                     rename(function (path) {                         // Returns a completely new object, make sure you return all keys needed!                         return {                             dirname: path.dirname,                             basename: path.basename + "-rtl",                             extname: ".css",                         };                     })                 )                 .pipe($.gulp.dest("./build"))                 .pipe($.browserSync.stream())         );     });
I tried

1) gulp-buffer before the rename
2) all ways described in npm readme - mutation, map
3) different version of package

i noted that providing a fixed value to dirname - works well.
Providing a same variable e.g. path.dirname = path.dirname, or omitting it - results in the above mentioned error.

this only happens with dirname. basename works well and extname as well.

so basically, I can achieve that:
build/
-file_1.css
-file_1-rtl.css
-file_2.css
-file_2-rtl.css
but I can't achive the above mentioned structure of folders.

So basically this happens when buffer: false on the gulp stream. Smells like something that can be fixed.


refactor: simplify char handling in to-lua

In https://github.com/NixNeovim/NixNeovim/pull/50, we hard-coded lowercase and uppercase chars as lists to the codebase. While working on another project, I observed that this exact definition is already provided by the nixpkgs lib: https://github.com/NixOS/nixpkgs/blob/3055d6a051cea2928d73dabd7c4f1fd807c0a245/lib/strings.nix#L551

Thanks for noticing and reporting. 👍


Changing hard code adapter count limit from 8 to variable

Hello,
new to vagrant development and looking for advice on how to change the line in find_empty that sets the adapter count limit to 8. Don't see where libvirt or qemu sets this limit, if I'm wrong please point me to this information. I hacked the line by changing it to 100 , and I can create a 96 nic linux VM using libvirt tcp tunnel interfaces,  and it seems to work so far. So was thinking of changing this into a variable and having it set to a default of 8. Then define a host specific parameter like
node.vm.adapter_count => 32 which would change that variable to 32 so I can configure 32 NICs. This is needed for the work I'm doing with linux based switch VMs.
What's the vagrant recommended way to make this change and I will do a Pull Request with the advice provided?

hello
there is not limits, it is just trick for not huge amount of iterations.
so you can increase it.
I think 100 will not too complicated for ruby

setting it to 100 was just as an experiment. creating a PR to add this as a variable, with a default setting of 8, and  recommendation of 37 if using  cumulus linux VMs in a multi-machine vagrant setup or single machine setup. This matches the capabilities of vagrant virtualbox provider which has a limit of 37 nics, and should hopeful not be too complicated for ruby as far as iterations go, as mentioned earlier.

commits referenced here are from a deleted repo. I deleted my fork of vagrant-libvirt and regenerated again. still learning my way around git. if someone has a suggestion on how to delete the entries completely so it doesn't show up in this issue, please let me know.

merged correct.


Problem with downloading

Hello,
I saw that your fork is still active. Can I report a problem?
I get this error with your fork:
05-13 21:19:12: ERROR : Error for aaaaaaaaaaaaaaaaaaaaaaaaaaaaa.aaaaaaaaaaaa.: Nothing to download .. Filtered out or bad torrent/nzb ?
05-13 21:19:12: WARNING : Retrying failed download in 10 minutes for: aaaaaaaaaaaaaaaaaaaaaaaaaaaaa.aaaaaaaaaaaa.
Any Idea why?
Thank you

Settings - global - logging / Debug mode
If you enabled this and restart you might see more information what's happening.
Filtered out or bad torrent/nzb:
Settings / categories
There you could whitelist certain file extensions that would cause nothing to be downloaded.
WARNING : Retrying failed download:
Should not happen if there is nothing to download but I suspect something went wrong, else this would be a program with the Downloader

Added job "aaaa.aaaaa" to job store "downloads"
05-13 21:48:42       apscheduler.executors.default            : INFO     : Job "update (trigger: interval[0:00:03], next run at: 2024-05-13 21:48:45 CEST)" executed successfully
05-13 21:48:42       apscheduler.scheduler                    : DEBUG    : Looking for jobs to run
05-13 21:48:42       apscheduler.scheduler                    : WARNING  : Execution of job "aaaa.aaaaaa (trigger: date[2024-05-13 21:48:42 CEST], next run at: 2024-05-13 21:48:42 CEST)" skipped: maximum number of running instances reached (1)
This is what the debug says and for the extensions i just have exe and txt as blocked.
For information I was using the original till 5 days ago and just tried yours because the other is "archived"

This was my error before with the original package

skipped: maximum number of running instances reached (1)
This doesn't matter.
That handshake error looks like it's from pySmartDL then I assume haven't tested that in a while.

just tested the pySmartDL (default) downloader and working here however blacklist is not working and keeps working like a whitelist

These are my settings and did not change it since the last year. Maybe need to recreate the settings with a new folder.

I fixed the blacklisting in new commit, docker build should be available in few minutes.
at the moment it would have only downloaded an exe or a txt file for you, so if it only had an mkv it would give the 2 errors from your first post :)
those ssl handshake errors however don't know where they are from

Hello,
thank you very much. I recreated the whole settings and container. No wit is working again.


Locking for shares part 1 #11

Locking for concurrently edited shared files. There was a need for a large refactoring because of duplicate code and not working tests.

I'm sorry for the last time, I pushed incomplete commit. This time everything should be fine.

Can you please rebase? I'm seeing tons of commits that should not be here.

Can you please rebase? I'm seeing tons of commits that should not be here.

done

Can you please rebase? I'm seeing tons of commits that should not be here.

Actually wait, I didn't do the rebase correctly

@dagl the pipeline doesn't work because the extension is not ok. You have 2 folders (logic and utils) that are not treated as modules because they don't have the __init__.py file inside.
If you pip install the extension and do the command that fails in the pipeline, you see that it fails while trying to import.

Ok thanks, it worked.
From: Diogo Castro @.>
Sent: Monday, January 24, 2022 2:40 PM
To: sciencemesh/cs3api4lab @.>
Cc: Dawid Gołosz @.>; Mention @.>
Subject: Re: [sciencemesh/cs3api4lab] Locking for shares part 1 #11 (#74)
@daglhttps://github.com/dagl the pipeline doesn't work because the extension is not ok. You have 2 folders (logic and utils) that are not treated as modules because they don't have the init.py file inside.
If you pip install the extension and do the command that fails in the pipeline, you see that it fails while trying to import.
—
Reply to this email directly, view it on GitHubhttps://github.com/sciencemesh/cs3api4lab/pull/74#issuecomment-1020111489, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APLNZABN5SEJIFKJFXMYZWDUXVJCDANCNFSM5EEPVEVA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

@dagl the pipeline doesn't work because the extension is not ok. You have 2 folders (logic and utils) that are not treated as modules because they don't have the __init__.py file inside. If you pip install the extension and do the command that fails in the pipeline, you see that it fails while trying to import.

Ok thanks, it worked.


Add tests for #182/#183

#182 was a regression, but we should add a test for that so we don't regress again in the future. Opening this issue to track the status of that.

Yeah, having those ^^ changes minimal/granular as possible still hit the regressions. I'm afraid to think what would happen with multiple changes coming in https://github.com/StackStorm/hubot-stackstorm/pull/168.
Minimally, something along the lines would be useful to test:

Nock the failing HTTP response from ST2 API. Ensure that function doesn't throw and that hubot doesn't exit.
Additionally need to ensure that the warning message is sent back to chat as well:

@armab Yep, I think we all made the right call (thanks to you).
If we need to check the output we should simply make it an end-to-end test in st2tests. I’ll do that next week; should be pretty straightforward.

Thanks, I'll take care of the first part then: Nock/HTTP unit test in this repo to ensure hubot doesn't exit.


Series constructor ignores dtype

Have you tried latest version of polars?
yes
What version of polars are you using?
0.9.0
What operating system are you using polars on?
NixOS unstable
What node version are you using?
node 20.11.0
Describe your bug.
 pl.Series("index", [1, 2, 3], pl.Int64)

yields:
 shape: (3,)
 Series: 'index' [f64]
 [
   1.0
   2.0
   3.0
]

Hmm, this is an interesting one, b/c it works fine in bun:
shape: (3,)
Series: 'index' [i64]
[
	1
	2
	3
]

But it produces wrong results in node v20.11.1

@richard-vock can you provide a full MRE? The JS ecosystem has a lot of different module systems, so I wonder if that has some effect here.


Fix 'token request' authorization header

This endpoint must use the Basic prefix, not the Bearer prefix.
Notion docs
Issue ref

Great job!
Thanks a lot 🔥


web sql is broken

specifically query is undefined here because the short circuits in these 2 functions will always trip.

Ah shoot - i'm going through the tickets now and cutting a new 0.8 release by the end of the day, I'll look to get this working

not a super high priority especially as safari (sorta) supports indexedDB


Define the packages which should be added to the SDK

There are some packages which will be useful/required to use the SDK library:

Config - for configuration file management.
Authentication
Authorization
Logging

iThe biggest one would be to see if we need i18n (and hope we do not ;) )


refactor: switch from turnkey ethers to turnkey viem (lighter & faster)

Remove the dependency to TurnKey ethers
Add the required dependencies to use turnkey viem turnkey/(http | stamper | viem)
Move a bit the logic inside the getCustodialOwner method to be more readable.
Add a few comments.

Thanks for this PR @KONFeature ! I have merged it in 👍


Getting Tenant In Service Layer/Repository Layer

Hello, I've been struggling with this so I thought I'd ask here. I can easily resolve Tenants in a Web API using a Host or a HTTP Header information. I can stuff this tenant in the Request.Items["Tenant"] using an Web API Actor Filter. Now I would like to pass this Tenant into a Service and the Repository layer easily without having to modify each method to accept the Tenant.
The Service and Repository classes are built up using Unity. Can I pass the Http Request to the Service (or Repository) so I can access the tenant there?
I want to avoid writing calls like _service.WithTenant(tenant).DoSomething();
Instead I would ideally like to automatically inject the tenant in the service when it is needed so I can continue doing _service.DoSomething();
The DoSomething would then be able to access a tenant inside its implementation and use it accordingly.
Would this be done using an OwinContext given that I don't want to rely on HTTP Context/Request only?
I would love to hear some thoughts. I've spent quite a bit of time searching online and it looks like this maybe the best place to ask because I'm sure someone else has had a similar issue after resolving the tenant.
@benfoster - thank you for this wonderful library. Wondering if you had this problem you had to solve when building Fabrik?
Anup

Hello All,
I was able to figure this out. I'm going to try my best to explain this here just in case someone else needs it. The problem statement was simple, I was using Web API 2.x and wanted to use SaaSKit in it to resolve tenants (based on Claims embedded in Tokens and/or Hostname resolvers). Once a Tenant was identified, I would pass this tenant to my Service Layer and/or Repo Layers etc. In doing all this, I wanted to avoid adding the Tenant as a Service method argument.
Much of this is very straightforward with ASP.NET Core. So if you are using .NET Core, please ignore this.
The answer to injecting a Tenant lies with Dependency Injection (Unity in my case), obviously. What I couldn't figure out was how I could get the Tenant on each request. After doing some research, I found out that Unity has Lifetime managers. So I started digging in on how to inject the Tenant or an Http Context if needed.
I ended up building both approaches, still not sure which one I prefer more, but one allows you to Inject the HttpContext directly into a Service, the other allows you to inject the Tenant. Once you have the Http Context, you can actually infer the Tenant from the Owin Pipeline if you want.
My Tenant is called 'TsTenant'. If you just want to inject the Tenant Directly, you can do so quite simply by registering it like so:
container.RegisterType<TsTenant>(new TenantLifetimeManager());
Once this is registered, you can pretty much declare this Tenant as an argument in any constructor and it will resolve it using the "TenantLifetimeManager". The Tenant Lifetime manager is where everything fun happens:
`public class TenantLifetimeManager : LifetimeManager
{
public override object GetValue()
{
if (HttpContext.Current == null) return null;
        var owinEnv = HttpContext.Current.GetOwinContext().Environment;
        return owinEnv?.GetTenant<TsTenant>();
    }

    public override void RemoveValue()
    {
        if (HttpContext.Current == null) return;
        var owinEnv = HttpContext.Current.GetOwinContext().Environment;
        //uh-oh, how to I remove?
    }

    public override void SetValue(object newValue)
    {
        if (HttpContext.Current == null) return;

        var owinEnv = HttpContext.Current.GetOwinContext().Environment;
        owinEnv?.SetTenantContext(new TenantContext<TsTenant>((TsTenant) newValue));
    }
}`

As you can see, if the HttpContext isn't null, I simply get the Owin Context and its Environment and then use the extension method GetTenant.
This gives me what I need. However, I was wondering, what if the HttpContext is null. So I ended up building an HttpContext registration that would either give me the real Http Context, or a fake one. This kind of stuff might be useful for Unit Testing scenarios. I am still marinating with this and haven't really finalized it.
But here's how you register the Http Context resolution:
container.RegisterType<HttpContextBase>(new HttpContextBaseManager());
The HttpContextBaseManager is responsible for managing the lifetime/creation:
`public class HttpContextBaseManager : LifetimeManager
{
public override object GetValue()
{
return HttpContext.Current != null
? (HttpContextBase) new HttpContextWrapper(HttpContext.Current)
: new FakeHttpContextBase();
}
    public override void RemoveValue()
    {
        throw new NotImplementedException();
    }

    public override void SetValue(object newValue)
    {
        throw new NotImplementedException();
    }
}`

Notice how it creates a FakeHttpContextBase. This fake class simple does this:
`public class FakeHttpContextBase : HttpContextBase
{
private IPrincipal _principal;
    public FakeHttpContextBase()
    {
    }

    public FakeHttpContextBase(IPrincipal principal)
    {
        _principal = principal;
    }

    #region OVERRIDES

    public override IPrincipal User
    {
        get { return _principal; }
        set { _principal = value; }
    }

    #endregion

}`

Now I can inject the HttpContext directly, and still get to the Tenant using Owin if needed. If I wanted to unit test this stuff, I could build a fake Owin Environment (i'm sure I can) to resolve a fake tenant.
I wanted to take this one step further. Instead of passing around Tenants, I thought it might be much better to create a TenantConnectionFactory. This factory would implement an ITenantConnectionFactory and expose the various Connections (Sql Connection for OLTP, Data Warehouse Connection, MongoDB connection, Azure blob storage connection, queue connections etc). Then I could hydrate this factory using the DI and pass it around in a Work Context.
So I now have:
container.RegisterType<ITenantContextFactory, TenantContextFactory>(); container.RegisterType<IWorkContext, ApiWorkContext>();
And my Api Work Context constructor is:
`public class ApiWorkContext : IWorkContext
{
private readonly HttpContextBase _httpContext;
    public ApiWorkContext(HttpContextBase httpContext, ITenantContextFactory tenantContextFactory)
    {
        _httpContext = httpContext;
        TenantContextFactory = tenantContextFactory;
    }`

Now I can use this Work context to get to Http Context, and the Tenant Context Factory. Here's an example:
  public SeedDataService(ISeedDataRepository repo, IWorkContext workContext)         {             _repo = repo;             _workContext = workContext;         }
A few challenges that still remain:

I still haven't been able to figure out Child Containers per Tenant. I would hate to create my Mongo connection again and again. In fact, the Mongo Driver I currently have for a single tenant environment is a Singleton. Mongo internally manages the connection pooling and recommends using a Singleton. I've seen some crazy errors if we create multiple instances of the Mongo connection. So the idea is to build a Singleton per Tenant and use it for that tenant dynamically. I have read Ben Foster's post on Per Container Tenant Lifetimes. Now I just need to implement this in Unity and in Web API 2.0.

Any thoughts here would be greatly appreciated.

I'm still trying to figure out and hopefully minimize the impact of this change when I migrate from Web API2.0 to .Net Core. Right now, this isn't an option given the complexity of my app (which was built as a semi-monolith).

Alright, I feel like I can pretty much build on this now that I have the core figured out. The rest of the stuff like building the various connections based on the Tenant seem very straightforward.
Cheerz,
Anup


add clarifying message for failed jobs with no logs

Summary
closes #852

adding functionality (check_logs) to enhance the logs of a failed job with no logs with an error message that acknowledges the failure.
adding respective unit tests and testing the expected behavior when the job status is updated.

Details and comments

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@david-alber Thanks for your contribution.  It looks good to me!

yey Thanks @IceKhan13 @akihikokuroda This was fun!!!
Looking forward to the next one


ui-multiple model summary

mostly done by https://github.com/rai-project/carml/commit/acc7b1c648ee99ae21822c5e53e1d6a6275f1e6b  , https://github.com/rai-project/carml/commit/00d9532bd59cc77ee98ffbb4fdddb57ae6a89739 , and https://github.com/rai-project/carml/commit/34ff95edda0f0c9485b50affb5926ff9a7a3273c


Delete Command issue

I have tried copy and pasting the event name into the delete command. Also tried the format as follows: !delete day time
No matter what, I receive an error message "Couldn't find event with that name - make sure you use exactly what the event is named!"

!delete has the following restrictions

within date threshold
matches stored event summary
cannot match by date/time
case sensitive

relevant code https://github.com/niles-bot/niles/blob/db8ef87fdace758a52c888283f5a752118c5b3c5/handlers/commands.js#L684-L703

That still doesn't help me figure out why it's not working.
If I have an event that was created with the command:
!create Test Event on Sept 25 at 4PM EDT
that then gives me an event with the name/summary "Test Event", shouldn't I be able to delete it with the following command:
!delete Test Event

trying with debug, there was an issue a lot time ago with having spaces but that was resolved a while ago

race condition, node isn't running it synchronously so I'll have to force an await

So, this means it likely won't work for the time being and manually delete will be required?


✨: CC-app-usage report

Expected outcome
Get the report out in test and prod.
Ensure the correct access control.
Set the correct refresh frequency
Business Value / Developer Experience
Get insight in the usage of CC-apps
Acceptance Criteria
Should be out in test and prod

@ken-mellem Denne kan vel lukkes?

yes - this is to be covered by Fusion admin usage app.


More mistakes in Brazil

First of all, you forgot to rename the incorrect Brasilia to something else in localisation.
I suggest the name "Poços de Caldas". Province id is 2446, btw.
Because of Rio Grande do Sul's cores inside Brazil, other countries will declare war on Brazil a lot to liberate the cores. Since the idea of a independent RGS died in the regencial period of the Empire of Brazil, it makes no sense to keep the cores in the late 40s and up
A few suggestions now.
Vic2 uses empire-era names for the brazilian provinces, and many of these name have changed.
Desterro (2461) should be Florianópolis, named after president Floriano Peixoto in 1891.
Sabará (2442) should be Belo Horizonte. Sabará is a small town and BH is one of the largest cities in the country and the state capital.
Dourados (2416) should be Campo Grande.  Campo Grande is much bigger and is the the state capital, also closer to the ingame province.
Tarauacá (2549) should be Acre. Acre is the name of the state where the province is. Tarauacá is the 4th most populous city in the state of Acre.
And finally, Forte São Joaquim (2532) should be Boa Vista. It is the state capital of Roraima, while Forte São Joaquim is an old fort which once existed in the region and now lies in ruins.
Thanks for the mod.

Thanks for the bug report :+1:
2446 renamed to Poços de Caldas.
Will keep the Rio Grande do Sul corese for "vic2 historic" reasons.
Will change the names.


⚠ Turnos 24x7 has degraded performance

In d26a9ee, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 891 ms

Resolved: Turnos 24x7 performance has improved in 9e3f31b.


[docs] Add NCCL environment variable docs

What does this PR do?
Closes #7179
Adds docs for parameters that can be adjusted when using custom cluster setups for speed improvements.
cc @ananthsub
Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
[ ] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)
[x] Did you list all the breaking changes introduced by this pull request?

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃

Glad to see this change, which can potentially give some good performance wins for a wide range of use cases that have relatively high communication cost.
Did you get a chance to evaluate the speedup on some benchmarks or real workloads?


CODAS Perspectives updated - need to temper with more perspectives

Updated the model with the CODAS team with 'rode colored glasses' and 'cloud 9' vision
Must be updated with more perspectives.

Several interviews later it is ok now


Problems with config

Hello,
i need help with installation.
My config is as followed:
wbecpro - heidelberg energy control - evcc on windows - while installation i get following message while configure wallbox:
cannot create charger 'heidelberg': dial tcp <IP_ADDRESS>:502: connectex: No connection could be made because the target machine actively refused it.
can someone help me?

Hello,
the configuration is described in the wiki: https://github.com/steff393/wbec/wiki/Configuration-Example:-EVCC-via-MQTT
Important: It's not necessary to configure wbec in evcc as 'heidelberg'.


update:logo & heading

Description
Added logo and redesigned heading
Please describe the changes you made to the Projectshut Open Source website. What features did you add or modify? What bug did you fix?
Screenshots

Related Issue
Please list any related issues that your PR addresses. For example, if you fixed a bug, please reference the issue number that describes the bug.
Checklist
Please review and check off the following items before submitting your PR:

[x] I have tested my changes to ensure they work as expected
[x] I have included documentation for any new features or changes
[x] I have formatted my code according to the project's style guidelines
[x] I have written tests for any new features or changes
[x] I have run the project's test suite and verified that all tests pass
[ ] I have updated the README and any other relevant documentation
[x] I have checked for any spelling or grammatical errors

Additional Information
Is there anything else you would like to include about your PR? This could include information about how to test your changes or any potential issues that may arise.
Thank you for contributing to the Projectshut Open Source website!

This branch has conflicts please resolve them.


task(files): display % while downloading a file

STR:

go to chat
send a file
receive a file

specially when a file is big example 100mb, theres no indication UI wise how many % regarding the download

This can be resolved by performing the checks while polling the stream
https://github.com/Satellite-im/Uplink/blob/03cd0782de4612f3df3e69a43549c0a032ce3224/ui/src/utils/async_task_queue.rs#L150-L164


Add AB test code to HMRC's ready reckoner page

What
Add AB testing logic to one page of interest following a departmental request
Related PR:  https://github.com/alphagov/govuk-fastly/pull/35
Why
Trello
Screenshots

Variant A and Z
Variant B

@anatron @davidtrussler
I've drafted this PR as second line was quiet today - but I'm without the full context (see below), so I hope it's helpful! Please feel free to close and start fresh if not!
I noted in the content doc that there should be a placeholder string of {{ab_test_sa_video_ready_reckoner}}  but as the text is intended to go underneath the button, and mainstream publisher doesn't currently support a text block in that location, it's easier to implement without depending on the presence of the placeholder string. But perhaps that is not the desired behaviour - I wasn't sure.


Fix #94 Timeout getReportedCursorPosition

On Unix-like operating systems, stop the use of getChar in System.Console.ANSI.Unix.getReportedCursorPosition blocking after 500 ms, if no information is forthcoming on the console input stream.

Not yet tested on a Unix-like operating system. For consistency with the Windows equivalent, is intended to return "" if the expected information is not available from the console input stream within the timeout window.

For consistency with the Windows equivalent, is intended to return "" if the expected information is not available from the console input stream within the timeout window.

Makes sense — could you please also add this as a comment in the code?

Tested on macOS High Sierra 10.13.6 with the same test used in #95 (compiled as executable tsTest). Same results, except, as expected:
stack exec tsTest -- >/dev/null
Nothing
Nothing


Improved query for resolving domain

The tenant relation is instantly called 100% of the times this method is called.
Including the tenant relation, we are saving an additional database query.

What's the performance benefit of this?
I understand the reasoning, since we're always accessing the tenant, but with() makes sense when there are multiple records. In this case, it just executes another query to get the tenant, just like the $domain->tenant would.

My mistake. I thought the with() method had some inner join magic.
You can safely just deny this PR. Sorry for wasting your time with this :)

No worries, appreciate the effort to help!


arm32v7/buildpack-deps:bionic-* images don't seem correct

I am confused at why the base layers of the arm32v7/buildpack-deps:bionic-* images are different than the arm32v7/ubuntu:bionic images.
docker history --no-trunc arm32v7/buildpack-deps:bionic-curl
IMAGE                                                                     CREATED             CREATED BY                                                                                SIZE                COMMENT
sha256:20754fee7abf041e9fd7fbbe677447d84ec5bdc015c6ea8e0c883a1c9e34997f   44 hours ago        /bin/sh -c set -ex;  if ! command -v gpg > /dev/null; then   apt-get update;   apt-get install -y --no-install-recommends    gnupg    dirmngr   ;   rm -rf /var/lib/apt/lists/*;  fi   0B
<missing>                                                                 44 hours ago        /bin/sh -c apt-get update && apt-get install -y --no-install-recommends   ca-certificates   curl   wget  && rm -rf /var/lib/apt/lists/*                                                13MB
<missing>                                                                 2 weeks ago         /bin/sh -c #(nop)  CMD ["bash"]                                                                                0B
<missing>                                                                 2 weeks ago         /bin/sh -c #(nop) ADD file:1fdf1946f816fc31d9b70835fe5cc0346daccdfa3c06a8420b30437b9df12c84 in /                                                                                80.6MB

 docker history --no-trunc arm32v7/ubuntu:bionic
IMAGE                                                                     CREATED             CREATED BY                                                                                SIZE                COMMENT
sha256:b47afe3525fa5431d9adb152197e2c0156303b7c7d6f5a4769b41ef29dbb05ac   3 weeks ago         /bin/sh -c #(nop)  CMD ["/bin/bash"]                                                                                0B
<missing>                                                                 3 weeks ago         /bin/sh -c mkdir -p /run/systemd && echo 'docker' > /run/systemd/container                                                                                7B
<missing>                                                                 3 weeks ago         /bin/sh -c sed -i 's/^#\s*\(deb.*universe\)$/\1/g' /etc/apt/sources.list                                                                                2.83kB
<missing>                                                                 3 weeks ago         /bin/sh -c rm -rf /var/lib/apt/lists/*                                                                                0B
<missing>                                                                 3 weeks ago         /bin/sh -c set -xe   && echo '#!/bin/sh' > /usr/sbin/policy-rc.d  && echo 'exit 101' >> /usr/sbin/policy-rc.d  && chmod +x /usr/sbin/policy-rc.d   && dpkg-divert --local --rename --add /sbin/initctl  && cp -a /usr/sbin/policy-rc.d /sbin/initctl  && sed -i 's/^exit.*/exit 0/' /sbin/initctl   && echo 'force-unsafe-io' > /etc/dpkg/dpkg.cfg.d/docker-apt-speedup   && echo 'DPkg::Post-Invoke { "rm -f /var/cache/apt/archives/*.deb /var/cache/apt/archives/partial/*.deb /var/cache/apt/*.bin || true"; };' > /etc/apt/apt.conf.d/docker-clean  && echo 'APT::Update::Post-Invoke { "rm -f /var/cache/apt/archives/*.deb /var/cache/apt/archives/partial/*.deb /var/cache/apt/*.bin || true"; };' >> /etc/apt/apt.conf.d/docker-clean  && echo 'Dir::Cache::pkgcache ""; Dir::Cache::srcpkgcache "";' >> /etc/apt/apt.conf.d/docker-clean   && echo 'Acquire::Languages "none";' > /etc/apt/apt.conf.d/docker-no-languages   && echo 'Acquire::GzipIndexes "true"; Acquire::CompressionTypes::Order:: "gz";' > /etc/apt/apt.conf.d/docker-gzip-indexes   && echo 'Apt::AutoRemove::SuggestsImportant "false";' > /etc/apt/apt.conf.d/docker-autoremove-suggests   745B
<missing>                                                                 3 weeks ago         /bin/sh -c #(nop) ADD file:886ddcd1b8d1c90b4eb406530752924e28c90f8252af281e708532d67def3c2c in /                                                                                68MB

I am running into things that appear to be missing from the buildpack-deps image for example apt is missing, the registered package feeds are different therefore packages available on arm32v7/ubuntu:bionic are unavailable "out of the box" on arm32v7/buildpack-deps:bionic-*
 docker run -it --rm  arm32v7/buildpack-deps:bionic-scm grep -h ^deb /etc/apt/sources.list /etc/apt/sources.list.d/*
deb http://deb.debian.org/debian wheezy main
deb http://deb.debian.org/debian wheezy-updates main
deb http://security.debian.org wheezy/updates main

docker run -it --rm  arm32v7/ubuntu:bionic grep -h ^deb /etc/apt/sources.list /etc/apt/sources.list.d/*
deb http://ports.ubuntu.com/ubuntu-ports/ bionic main restricted
deb http://ports.ubuntu.com/ubuntu-ports/ bionic-updates main restricted
deb http://ports.ubuntu.com/ubuntu-ports/ bionic universe
deb-src http://ports.ubuntu.com/ubuntu-ports/ bionic universe
deb http://ports.ubuntu.com/ubuntu-ports/ bionic-updates universe
deb-src http://ports.ubuntu.com/ubuntu-ports/ bionic-updates universe
deb http://ports.ubuntu.com/ubuntu-ports/ bionic multiverse
deb http://ports.ubuntu.com/ubuntu-ports/ bionic-updates multiverse
deb http://ports.ubuntu.com/ubuntu-ports/ bionic-backports main restricted universe multiverse
deb http://ports.ubuntu.com/ubuntu-ports/ bionic-security main restricted
deb http://ports.ubuntu.com/ubuntu-ports/ bionic-security universe
deb-src http://ports.ubuntu.com/ubuntu-ports/ bionic-security universe
deb http://ports.ubuntu.com/ubuntu-ports/ bionic-security multiverse

Docker version
Client:
 Version:       18.02.0-ce
 API version:   1.29 (downgraded from 1.36)
 Go version:    go1.9.3
 Git commit:    fc4de44
 Built: Wed Feb  7 21:16:33 2018
 OS/Arch:       linux/amd64
 Experimental:  false
 Orchestrator:  swarm

Server:
 Engine:
  Version:      17.05.0-ce
  API version:  1.29 (minimum version 1.12)
  Go version:   go1.7.5
  Git commit:   89658be
  Built:        Thu May  4 22:30:54 2017
  OS/Arch:      linux/arm
  Experimental: false

Docker info
Containers: 9
 Running: 0
 Paused: 0
 Stopped: 9
Images: 60
Server Version: 17.05.0-ce
Storage Driver: overlay2
 Backing Filesystem: extfs
 Supports d_type: true
 Native Overlay Diff: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge host macvlan null overlay
 Log:
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Init Binary: docker-init
containerd version: 9048e5e50717ea4497b757314bad98ea3763c145
runc version: 9c2d8d184e5da67c95d601382adf14862e4f2228
init version: 949e6fa
Kernel Version: 4.9.35-v7+
Operating System: Raspbian GNU/Linux 8 (jessie)
OSType: linux
Architecture: armv7l
CPUs: 4
Total Memory: 923.4MiB
Name: raspberrypi
ID: 3UO3:FS2Q:BZB2:GFD5:5AVY:R4FH:OCFJ:IQ4C:7MCH:TGAH:RCFG:CAWQ
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
Experimental: false
Insecure Registries:
 <IP_ADDRESS>/8
Live Restore Enabled: false

WARNING: No swap limit support
WARNING: No cpu cfs quota support
WARNING: No cpu cfs period support
WARNING: No cpuset support

Forgot to include the digest for the images I am using - they are the latest as of today.

arm32v7/buildpack-deps@sha256:b9f623c5cf2411863ab32774098aad2fca7ccb71aa328d35a123af0a71abf8c4
arm32v7/ubuntu@sha256:0152d89c7a79c83bd89a0757419674ee3d14c565d514e0c97447b058b4d8fa3a

This appears to be another instance of: https://github.com/docker-library/openjdk/issues/180. (It looks like arm32v7/buildpack-deps:bionic-curl got built from a debian base that was mistagged).

I have triggered the build jobs so they should be updated shortly


CLRC663

We are using the custom board with the Reader ic is CLRC663 and SPI is used for the interfacing with microcontroller. SPI is configured as mentioned in the Datasheet of the clrc663 reader.The issue is there is no response from the reader.SiPI is working FIne.
**need help for the communication

If it's a custom board and the SPI communication looks fine with a logic analyzer it's likely a hardware problem. It can be a lot of things, from incorrect interface selection to the crystal not providing a stable clock. There's little I can do besides saying double-check the circuitry, check if the pins are at the appropriate logic levels. In particular check that the IFSEL pins are set up correctly for SPI, check all the power pins and the PDOWN pin.
I used the mfrc630_read_fifo, mfrc630_fifo_length and mfrc630_write_fifo functions to confirm that the chip was responding to my SPI signals when I worked on this. If you can write data and read the same back you've confirmed the chip works and your communication works.

Thanks for replaying me,i am sending the 0x7f command for the getting
version of the clrc663 reader it is not responsing ,what are the commands
need to send to get the product information.
On Sat, Apr 13, 2019 at 6:36 AM Ivor Wanders<EMAIL_ADDRESS>wrote:

If it's a custom board and the SPI communication looks fine with a logic
analyzer it's likely a hardware problem. It can be a lot of things, from
incorrect interface selection to the crystal not providing a stable clock.
There's little I can do besides saying double-check the circuitry, check if
the pins are at the appropriate logic levels. In particular check that the
IFSEL pins are set up correctly for SPI, check all the power pins and the
PDOWN pin.
I used the mfrc630_read_fifo, mfrc630_fifo_length and mfrc630_write_fifo
functions to confirm that the chip was responding to my SPI signals when I
worked on this. If you can write data and read the same back you've
confirmed the chip works and your communication works.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/iwanders/MFRC630/issues/7#issuecomment-482809892, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AZrU8IOgBY20HdGGByvPxp1jI5epIhCdks5vgd1jgaJpZM4crxTS
.

I think 0x7f is correct, you should be able to rely on the functions from this library and do:
const uint8_t ic_version = mfrc630_read_reg(MFRC630_REG_VERSION);

If the SPI communication is setup correctly this should be all you need to do.

Sir,i am done the following method to get the data about version and sub version register,

i am taken Temp as variable in code,i am attaching the code
spi_cmd_send(0x7f);//sending the command 7f
Temp=0;//restting the variable
while (!SPI_GetFlagStatus(HT_SPI0, SPI_FLAG_RXBNE));
Temp=SPI_ReceiveData(HT_SPI0);//receving the data through the spi recieve function
spi_cmd_send(0x01);
Temp=0;
while (!SPI_GetFlagStatus(HT_SPI0, SPI_FLAG_RXBNE));
Temp=SPI_ReceiveData(HT_SPI0);
spi_cmd_send(0x00);
Temp=0;
while (!SPI_GetFlagStatus(HT_SPI0, SPI_FLAG_RXBNE));
Temp=SPI_ReceiveData(HT_SPI0);

each time i am checking the temp variable.

Still not getting the version and subversion ID through the SPI, please provide any examples.

i am making any wrong in  the code????please guide me sir

i am checked following method also sir,still i am oxff only
spi_active();
spi_cmd_send(0xFF);
while (!SPI_GetFlagStatus(HT_SPI0, SPI_FLAG_RXBNE));
Temp=SPI_ReceiveData(HT_SPI0);
spi_inactive();
spi_active();
spi_cmd_send(0x00);
while (!SPI_GetFlagStatus(HT_SPI0, SPI_FLAG_RXBNE));
Temp=SPI_ReceiveData(HT_SPI0);
spi_inactive();

I presume spi_cmd_send is a raw SPI command, in that case it is wrong. You have to set the bit to read the register. See the read_reg implementation. Read the datasheet to understand how to talk to this chip.
You are not using this library, neither do you take the effort to format your posts such that they are conveniently readable. I'm closing this issue as it is off-topic as it does not pertain to this library. I don't want to provide general help in how to talk to an IC over SPI.


Fix heap corruption in manual tests on macOS 64-bit build

Log

[----------] 4 tests from APICSolver3Tests
[ RUN      ] APICSolver3Tests.WaterDrop
[       OK ] APICSolver3Tests.WaterDrop (33582 ms)
[ RUN      ] APICSolver3Tests.DamBreakingWithCollider
ManualTests(4353,0x700008db7000) malloc: * error for object 0x7ff9a84197b0: double free
* set a breakpoint in malloc_error_break to debug
[1]    4353 abort      ./ManualTests

[----------] 4 tests from FLIPSolver3Tests
[ RUN      ] FLIPSolver3Tests.WaterDrop
[       OK ] FLIPSolver3Tests.WaterDrop (32374 ms)
[ RUN      ] FLIPSolver3Tests.DamBreakingWithCollider
ManualTests(2837,0x70000209b000) malloc: * error for object 0x7ff84f900038: incorrect checksum for freed object - object was probably modified after being freed.
* set a breakpoint in malloc_error_break to debug
[1]    2837 abort      ./ManualTests

Can you reproduce this error with the original test from Jet Framework as well? If not, what would be the delta between two codebase?

@doyubkim I'll check it.

@doyubkim I think Jet Framework seems to have no problems. In the case of CubbyFlow, there is only a problem with manual tests starts with DamBreaking. 🤔

@doyubkim It seems to be an intermittent problem. Occasionally, errors such as segmentation fault, bus error, and double free occur, but sometimes they work fine. (I'll check another Macbook.)

You mean from this project (CubbyFlow), right?
Usually that happens when you’re writing something outside a malloced region. If lucky, it may not crash, but it’s definitely a heap memory access issue. I’m pretty sure there is a loop that iterates greater range than it supposed to run. Or maybe there’s an unsigned integer overflow where the integer is used for array indexing. Print out some messages for each computation step and see where it crashes.

@doyubkim I'll check it. Thanks. :+1:

@doyubkim I ran manual tests of fluid-engine-dev several times. Eventually, I noticed that I found the same problem in the Jet. Can you test it several times? Thanks.

This might be related to the MatrixCsr code. I recently fixed some assertion issues so please try again with the latest master from Jet.

@doyubkim Okay. I'll test again later. Thanks.

PR for the fix is posted on Jet's repo. That should solve this problem.

@doyubkim It was a race condition problem. I was thinking that each time I ran the test, there was a problem in different places and it was a thread related problem. I thought about using a mutex, but it was not too efficient and I was trying to find another way. Thank you for editing. I'll test it. :^)


Implementation helpers for datalayout abstraction

The diff of
https://github.com/hfp/libxsmm/blob/numa_gxm_batch_reduce_gemm_generic_conv/src/template/libxsmm_dnn_convolve_st_fwd_custom_custom_generic.tpl.c
and
https://github.com/hfp/libxsmm/blob/numa_gxm_batch_reduce_gemm_generic_conv/src/template/libxsmm_dnn_convolve_st_fwd_nhwc_custom-rsck_generic.tpl.c
is only LIBXSMM_VLA_DECL / LIBXSMM_VLA_ACCESS . The rest of the files is identical.
Ideally, a set of macros (on top of LIBXSMM_VLA_DECL/LIBXSMM_VLA_ACCESS) unifies the implementations.

In addition to LIBXSMM_VLA_DECL/LIBXSMM_VLA_ACCESS, we need an entity to express linear increment for two or more loop-dimensions. The mentioned source code also increments some pointer involving two dimension which are impacted by the permutation.
Since for the given case the permutation of an N-dimension access is not just a pair or single interchange of two indexes, we need a gather- or  scatter-set. It should be an optional argument or separate top-level primitive (default: 0..N-1 i.e., non-permuted access).

With the current accessors (no code change in user code), a layout permutation cannot involve the prime-dimension, i.e., the first index must stay in front. This is because the extent associated with the prime-index is omitted and not part of the access-expression.

An example of what's possible is incorporated into the VLA test (IJK and IKJ index permuation). As noted above, the first/prime dimension cannot be involved into permutation.


Custom name attributes

Currently, the name attribute of a field is defined from the field name. This means that if I have the same form in different languages, there's no way of mapping the fields between languages. Is there a way to define a custom name attribute value for each field?

Hi @lmanzurv, sorry for a late response.
What to you mean by same form but different language? 2 identical forms in the same page but in different language, that are displayed at the same time? And what to mean by mapping? Is this for some custom javascript? Please can you give an example?

Sorry if I wasn't clear. I have 1 page that exists in 5 different languages. Each of the languages has one form (in the corresponding language). All the forms are the same. When the form is submitted, I execute a function that is triggered by the form_submission signal you send. I have to obtain all the values and send them to an external system. But the values come with different field names (i.e. in english I get field 'name' but in spanish I get field 'nombre'). This happens because the fields' names are set from the label that is specified on the form creation.
I suggest adding a property for a field where we can define the value that the HTML attribute name is going to have. That way I can define the same name for the fields that are "a match" between languages, thus avoiding having to make workarounds to "map" them on the signal-triggered function.
Right now, what I do, is pass an object on the description with different properties, one of which is the name attribute, and on the template, I process the object and set the values accordingly. That way, on the signal I process the fields received and instantiate a FormBuilder object to rename the fields with the information of the name set by the form and the name received on the object (specified on the description).
Don't know if I was clear, and can include code snippets if you wish.

Wow. That's a lot of workaround for something that could be solved by adding an extra field. I'll update the package this weekend to include optional 'field_name' field.

Yes. I know, but unfortunately I didn't have the time to make the changes myself because I needed the solution ASAP.
I was actually thinking that there could be a dict-like field to include HTML attributes (like data-attributes) instead of just a field for the name. I could make that myself and send a pull request.

Laura Cristina Manzur Villalobos, M.Sc
Ingeniera de Sistemas y Computación
Cel.: [57]<PHONE_NUMBER>
Bogotá, Colombia

On 2/06/2016, at 04:43, Mishbah Razzaque<EMAIL_ADDRESS>wrote:
Wow. That's a lot of workaround for something that could be solved by adding an extra field. I'll update the package this weekend to include optional 'field_name' field.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Hey @lmanzurv, Please add DJANGOCMS_FORMS_ALLOW_CUSTOM_FIELD_NAME=True to your settings.py.
This will add extra field to the form field inlines, see screenshot below.

If you find any issue, please feel free to reopen this issue.
Otherwise, leave me 👍 and I'll do a new release to PyPi.
Don't forget to 🌟 this repo 😛
Many Thanks

It is perfect. Thank you very much!

Hi, is the modification already up in PyPi? I checked and still see version 0.1.17

Sorry haven't had a chance to do that :-( .. I'll do it as soon as I can!

No problem. Please let me know as soon as you do so I can update my project ;)

Done! https://pypi.python.org/pypi/djangocms-forms/0.2


chore(alacritty template) fix window opacity

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Naming issue for the transcriptome file when running ./simpleaf index

Dear Authors,
I'm having the follow issue when running ./simpleaf index:
The file [custome_index/ref//transcriptome_splici_fl86.fa] provided for the transcriptome does not appear to exist.
The cause seems to be that the default value for the filename_prefix parameter of the make_splici_txome function is "splici", whereas the line cmd="$salmon index -t $outref/transcriptome_splici_fl$fl.fa -i $outidx -p $threads $sparse_flag" in the usefulaf/bash/simpleaf file has an extra prefix of "transcriptome". Moreoever, ./simpleaf index doesn't provide a --filename-prefix parameter to get around this. Additionally, since outref="$output/ref/", there is an extra "/" in the path.
Could you please help look into this?
Thanks very much!

Cc @DongzeHE --- also, can you please do a full run through to ensure everything works as expected end-to-end? We'd done this but something must have changed. We need a regression test for this.

sure! will test it now.

Hi @naity2,
I have fixed the bug and given it an end-to-end run using a real dataset. Please do git pull to pull my latest changes.
I am sorry for the inconvenience and thank you so much for bringing this issue up.
-Dongze

Thank you, @DongzeHE, for the prompt response!
Would you please kindly update the docker container as well? Thanks a lot!

The docker image is updated. Enjoy. 😉

The docker image is updated. Enjoy. 😉

Awesome, thank you so much!

Hi @naity2,
I just wanted to mention that we just released pyroe 0.6.0.  This version adds some new commands including id-to-name that can convert a gene_id to gene_name mapping from a GTF or GFF3 file, as well as a convert command that can convert an alevin-fry quantification directory to e.g. an AnnData or loom file.  The convert command can take an id to name mapping and apply the renaming during conversion.  In general, we've greatly improved the documentation for pyroe, which you can find here.  Let us know if you have any questions.
Best,
Rob

Hi @naity2,
I just wanted to mention that we just released pyroe 0.6.0. This version adds some new commands including id-to-name that can convert a gene_id to gene_name mapping from a GTF or GFF3 file, as well as a convert command that can convert an alevin-fry quantification directory to e.g. an AnnData or loom file. The convert command can take an id to name mapping and apply the renaming during conversion. In general, we've greatly improved the documentation for pyroe, which you can find here. Let us know if you have any questions.
Best, Rob

Awesome, thank you so much for the very useful updates!


Support anonymous accounts

So that people's read states / stars are allowed (don't allow commenting).
Merging will be the main challenge

Merging will be highly non-trivial. The normal cases of anonymous account morphing into a non-anonymous one isn't hard (the uid stys the same). The bigger problem is when you do have a real accouunt but on this particular device didn't realize you weren't logged in and so have two accounts with real data.
There isn't that much of a need to do it; in the meantime it's fine to just change the buttons like comment/message that would normally be disabled if not log in to instead prompt and ask for sign in instead.

[ ] Make sure we don't have a pop up blocked for sign in (just use redirect?)

https://firebase.google.com/docs/auth/web/account-linking


Is it possible to redirect admin on the basis of their roles. Like each role has it's own CMS? And how we can publish the controller files of superadmin to app/controllers so that we can customize them according to our needs?

Describe the bug
A clear and concise description of what the bug is.
Laravel Version
Which laravel version you are using
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]

Additional context
Add any other context about the problem here.

Hey, it is possible to control admin behavior, you can publish routes file and then create your own controller and link that in published route file.
Try to extend package controller for each new controller you make.


feature(search): hide invoice type from search LHN

Explanation of Change

Fixed Issues

$ https://github.com/Expensify/App/issues/51620
PROPOSAL: https://github.com/Expensify/App/issues/51620#issuecomment-2443146043
Tests

Login with user A that does not have any workspace with invoices enabled
Navigate to search from bottom tab
Verify that invoices category is not visible in the Search LHN
Enable invoices in an existing workspace or create new workspace and enable invoices
Navigate to search from bottom tab
Verify that invoices category is visible in search LHN
In a new window, Login with a new account B that is not member of any workspace with invoices and has never release fil
Verify that invoices category is not visible in the Search LHN
Go back to User A and send an invoice to user B, make sure that user B is not present in workspace
Veriy that invoices category is not visible in the Search LHN

[x] Verify that no errors appear in the JS console

Offline tests

Login with user A that does not have any workspace with invoices enabled
Navigate to search from bottom tab
Verify that invoices category is not visible in the Search LHN
Enable invoices in an existing workspace or create new workspace and enable invoices
Navigate to search from bottom tab
Verify that invoices category is visible in search LHN
In a new window, Login with a new account B that is not member of any workspace with invoices and has never release fil
Verify that invoices category is not visible in the Search LHN
Go back to User A and send an invoice to user B, make sure that user B is not present in workspace
Veriy that invoices category is not visible in the Search LHN

QA Steps

All steps are same as Offline Tests

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/2ea51b06-f984-4afc-bb66-ae4e6447ce0d
https://github.com/user-attachments/assets/8e177303-c1bf-4742-b0bc-943c1ef8cd37

MacOS: Desktop

@luacmartins @eVoloshchak Please review the PR

All yours @eVoloshchak. Please prioritize this review as it's a critical one.

@rushatgabhane I think that's expected since the user has an invoice in that case and should be able to look for those.

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

This PR is failing because of issue #52379


Better support for comparing compressed storage in unit tests

I am unsure abut the CompressedStorage and make_compressed_storage names. Any suggestion?

If we are talking about the present state, SortedCrsGraph. If we consider it could be used in the future for other formats comparison, maybe SortedGraphStorage.

Another place I wanted feedback on is that currently I am printing the "rows" after reordering but in principle I could compare them sorted and print them in their original order.

I have not seen the output on an error, so it's hard to judge. What we would want is to a) print the offending row in the original order, and b) point out the element that offends.

<ARBORX_DIR>/test/tstUnitTestingSupport.cpp:51: error: in "Dummy": check make_compressed_storage(u, v) == make_compressed_storage(std::vector<int>{0, 3, 3}, std::vector<int>{0, 0, 0}) has failed
  - mismatch at position 0: [( 0 ) == ( 0 0 0 )] is false
  - mismatch at position 1: [( 1 2 ) == ( )] is false

Hmm, if it prints something like [( 3 1 2 ) == ( 1 2 4 )] is false, it's a bit tricky to remember how to treat that, or which element is wrong. I guess if that's the case, printing out sorted ones would be slightly better, as we'll immediately see what element is off.

How about FlattenedVectorOfVectors?


Publish to Bower

… and add a manifest.

v2.0.1 seems to be tagged on components/es6-promise repo, I guess this can be closed.


feat: New Header styles for Beam tables

Creates new GridTable story for showing new header styles

@stephenh - after @jeffreyqua went through with QA it was determined design wants the sort icon to show up on the left of the text for right aligned columns. I've updated the PR in this commit to support that: https://github.com/homebound-team/beam/pull/432/commits/95dd70d9e3e69d1c252075d6d9b392c5989146cd
Would you mind re-reviewing. Would like to get your blessing on this approach.

@bdow thanks for the ping! Yep, that looks great!


Service Broker Policy uses Human-Readable Names

By default the service broker only receives the GUID-based identifiers for org, space, and name. It may be desirable to use the human-readable names instead for policy to allow users to more easily read and understand policy and entitlements.
This will require the service broker to be able to access the CF Cloud Controller API with at least Auditor level access.

Wanted to note here, this is easier as of service broker api version 2.15, organization_name and space_name are part of the context object now. But with that said, this would be a huge change and needs to be rolled out with a "feature flag" so to speak. And since names can be edited, would require some work to handle those cases.

I think with the changes in #238 we can close this older feature request as "wontfix" now :)


smartcrop: bad extract area

In very specific case resizing with cropping set to sharp.strategy.attention or sharp.strategy.entropy will result with error smartcrop: bad extract area
Repository to reproduce - https://github.com/pieh/sharp-smartcrop-bad-extract-area-repro
This happens when width is set to 1110 and height is unspecified using image from above repository (it is 1600 x 1200)
In readme of repository I placed more outputs where it works fine:

with crop set to sharp.gravity.center
with slightly higher and lower widths (1120 and 1100 respectively)
with specified height (both calculated using source image aspect ratio and full image height)

This was originally reported in https://github.com/gatsbyjs/gatsby/issues/4142 (just to keep paper trail - reproduction repository should be enough)

Hi Michal, thank you for this detailed report and accompanying test repository.
The "attention" and "entropy" crop strategies rely on the resize() operation being provided with both a width and height but this isn't checked or enforced until after passing control to libvips.
I think the best fix here is for sharp to ignore the strategy choice should either width or height be missing as that situation will result in no crop taking place anyway. I'll take a look at this.

When resizing with just width (or height) crop strategy/gravity shouldn't have any effect - it's just scaling without cropping, right? What I find weird is that it works fine (doesn't throw error) with some width (f.e. 1120px or 1100px in my example) and not with others (1110px).
I will add conditional check to not use attention or entropy crop strategy in gatsby for now if it's just scaling and not cropping.

This error occurs when cumulative rounding during the resize operation occasionally produces an image 1px smaller along one edge than the expected target dimensions. The bug here is that there's no conditional guard to prevent a "smartcrop" operation taking place when the image is already small enough.
I've added your example dimensions as a new unit test locally, which currently fails, and expect to push the fix that will get this test to pass within the next 24 hours. Thanks for reporting!

v0.19.1 now available with the fix for this.

Hi, I am getting the same error with a 1024x1034px  image on version 0.20.0. I am also using gatsbyjs
I have a project with quite a few images (about 100) and this is the only image that's causing problems:

@ralrom Please can you open a new issue with a minimal code sample (using only sharp) that exhibits this behaviour.


vFile beta task list

Testing (Perf/scale/feature stress)

[x] Modify deploy script to deploy vFile plugin on docker host: #1684
[ ] Have CI testbed for vFile, and enable existing e2e test for vFile  to run in CI (Need Anup or Nirdesh to bring up the testbed in CI) #1776
[x] Review current tests we have for vFile, and figure out new tests that need to be added:  #1806
[ ] Configure CI testbed setup with multi manager swarm cluster setup for failover testing #1682

Address correctness issues

[ ] Failed to mount vFile volume. Error: exit status 255 on Swarm master #1816
[ ] Populate client list on mount and depopulate on unmount #1761
[ ] Handle swarm node role change in vFile plugin https://github.com/vmware/docker-volume-vsphere/issues/1732

Document

[ ] FAQs : Document why multi writer mode does not solve shared volume use case for all applications.  #1681
[ ] Customer configuration recommendation, based on the performance/stress testing result

Misc(Fit and finish)

[ ] vFile: Complete Docker certification to enable listing on Docker Store https://github.com/vmware/docker-volume-vsphere/issues/1777
[ ] upgrade guide #1877
[ ] Containerize our own Samba server https://github.com/vmware/docker-volume-vsphere/issues/1729

This is Epic so adding mustfix


Hi, which python version should we install exactly? Can you post a link for that version?

Hi, which python version should we install exactly? Can you post a link for that version?
最初由 @Recklessxq 在 https://github.com/KSCHdsc/BlackCap-Grabber/issues/20 发布

Our supported CPython version range is 3.7 through 3.11, inclusive. The current supported range is available on our pypi page.


Possibility to add labels

When you have a lot of files in a project, harpoon helps tremendously, but I still sometimes forget why some file is there.
It would be nice to add a label in front of the filename. it could be separated from the filename by - E.g.
sends onclick event - src/services/analytics.tsx

Came here to suggest something similar. I would want the harpoon buffer to show which of my keybindings corresponds to each file. Perhaps this can be done with an ftplugin vimscript that adds diagnostics (so, not actually buffer contents?).


[Bug]: Watch fails if SL (Steam Locomotive) is installed

Version
29.5.0
Steps to reproduce

Install SL (Steam Locomotive); e.g. brew install sl
Clone my repo https://github.com/dvanoni/jest-watch-sl-bug
npm install
npx jest --watch

Expected behavior
npx jest --watch runs and prints the following message:
No tests found related to files changed since last commit.
Press `a` to run all tests, or run Jest with `--watchAll`.

Watch Usage
 › Press a to run all tests.
 › Press f to run only failed tests.
 › Press p to filter by a filename regex pattern.
 › Press t to filter by a test name regex pattern.
 › Press q to quit watch mode.
 › Press Enter to trigger a test run.

Actual behavior
npx jest --watch fails with the following message:
Determining test suites to run...

  ● Test suite failed to run

thrown: [Error]

Additional context
#13941 introduced support for Sapling which provides a command named sl. This conflicts with the sl command provided by SL (Steam Locomotive). When you have SL installed on your machine rather than Sapling, jest-changed-files fails to run properly.
Environment
System:
  OS: macOS 13.2.1
  CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
Binaries:
  Node: 18.15.0 - ~/.asdf/installs/nodejs/18.15.0/bin/node
  npm: 9.5.0 - ~/.asdf/plugins/nodejs/shims/npm
npmPackages:
  jest: ^29.5.0 => 29.5.0

Good catch. I was looking at Sapling documentation, might be there is one more related issue on PowerShell:

Note that the name of the Sapling CLI sl.exe conflicts with the sl shell built-in in PowerShell (sl is an alias for Set-Location, which is equivalent to cd).

Seems to be sensible to check that sl is really Sapling. Perhaps sl --version could be the command for this check? It prints: Sapling [version number].

I think that would work. If you run sl --version with SL installed, you'll get the steam locomotive yet again. 😄

On my system, having sl installed didn't cause the tests to fail to run; rather, determining which tests to run takes a long time now with the --watch command, about 6 seconds. The tests execute instantly when running normally. Removing it via pacman -R sl makes --watch behave as before.
Environment:
$ npx envinfo --preset jest
  System:
    OS: Linux 6.2 Arch Linux
    CPU: (8) x64 11th Gen Intel(R) Core(TM) i5-1135G7 @ 2.40GHz
  Binaries:
    Node: 18.1.0 - ~/.asdf/installs/nodejs/18.1.0/bin/node
    Yarn: 1.22.19 - ~/.asdf/shims/yarn
    npm: 9.6.2 - ~/.asdf/plugins/nodejs/shims/npm
  npmPackages:
    jest: ^29.5.0 => 29.5.0

I would like to suggest that fixing this issue should include displaying a more descriptive error in this case than the one it does:
  ● Test suite failed to run

thrown: [Error]

/cc @vegerot

We might need to revert the sapling support and figure out a better approach. The timeout suggested in https://github.com/facebook/jest/pull/14061 seems icky to me. My own suggestion of grepping man pages or crawling the FS doesn't seem good either.

We can take a similar approach to ripgrep and walk up the directory tree until we find a .git, .sl, or .hg directory which determines which program to use.
  What if someone install both Steam Locomotive and Sapling, but renames Sapling from sl to sap?  What do we do then?  I guess it would be the same problem if someone renames the git exe 🤷♀

@SimenB I called in the pros

I would like to suggest that fixing this issue should include displaying a more descriptive error

The conclusion in https://github.com/facebook/sapling/issues/597 is similar, but what should that error say? "Jest detected that sl binary is not Sapling. Please leave sl exclusively for Sapling"?
Many users will not know what Sapling is (I did not know few weeks ago). Also, why Jest needs sl to run the tests? Why a Git user should not be able to have Steam Locomotive installed? Tricky.
This brings an idea of some opt-out mechanism (leave my sl alone, please). Wait... Why not opt-in?! This is how I came to the idea of adding an scm configuration option (see https://github.com/facebook/jest/pull/14061#issuecomment-1500259969).
Just wanted to explain what was the thinking path. Not a fan of adding more and more options.
Obviously the problem should be solved on Jest side. Its main task is to run tests. Having an opt-in scm option there will be no need to spent time guessing which is SCM is used in a repo. That is performance gain for test running. Users like performance and users like Steam Locomotive too (;

I'm liking the suggestion here to ship a sapling command that could be used in this scenario instead of sl. It's certainly not the most immediate solution because it would require Sapling users to upgrade to a version that includes it (whenever that happens), but it does feel like the most robust suggestion I've seen.
I think it would also avoid the issue of having to add some sort of scm option to Jest, and I think it also avoids the issue with any error message weirdness.
Of course, this all assumes there isn't already some other tool that provides a sapling command. 😅

but what should that error say?

I think that it should pass through the existing error, something like 'Command failed with ENAMETOOLONG: sl status -amnu /Users/nokel81/repos/lens/packages/core\n' +     'spawn ENAMETOOLONG',
This case doesn't need a special error, because the underlying issue should be fixed by one of the several proposals (a sapling command, only running sl if the scm configuration option is set to sapling, etc).
Just having that error show up would unblock people, because you can see "something is up with sl".
I'm asking for this also because this is not the only Jest error I've seen that failed with thrown: [Error] (the last ended up being something with watchman) and I'm hoping to get some insight into the next one of those I see, as well.

This is certainly still an issue

@rmartine-ias I agree with you. If I would have seen that error I would have been able to find this bug report much faster. I only found it because I also started digging in node_modules for answers. Showing the error would help more people finding a faster solution.
I uninstalled sl as my solution, I had even forgotten that I installed it just for fun ages ago.

I uninstalled sl as my solution

This is the obviously correct solution.  The problem is that the error message is so bad people don't realize they need to do this.  I think we can check the size of the sl binary to provide a better error message.
@sTRAGER also had another idea to improve the error reporting, but needs to elaborate it more

This is the obviously correct solution.

Unlinking/uninstalling sl is a correct temporary workaround for this issue. (You could also use direnv to selectively alias it to false when working on jest projects, or wrap it in something that detects if jest is calling it, maybe...)
Homebrew has > 20k people who have installed sl in the past year, debian popcon has > 1% of submitted reports with an install. I think that is more than enough usage to not declare incompatibility. It'd make me sad, too -- there is far too little whimsy on my computer and I do not like the idea that since one tool I must use has mistaken assumptions, I should have to reduce that further.

Still an issue

Not stale, I like the idea of a config that defaults to git

:+1:

Not stale, would be great if this was a configuration option instead

@rmartine-ias I fully agree that uninstalling is a decent temporary workaround.  What do you think a more permanent solution to the jest --watch incompatibility with everyone's favorite steam locomotive ❤?
A config option? or something better possibly?

@rmartine-ias I fully agree that uninstalling is a good temporary workaround. What do you think a more permanent solution to the jest --watch incompatibility with everyone's favorite steam locomotive ❤?
A config option? or something better possibly?

I am not familiar with the project's internals, or what tradeoffs the maintainers make, so I don't think I am the person to ask. I am probably missing some context or misreading some code.
That said, I would:

Fix the error message system
Only find roots once, when starting the watch process
Store SCM source at that time on a new Root object
Find sl roots through directory traversal, instead of binary calling (note: not sure if you'd need to look for sapling-specific files because of the git compatibility)
AND/OR: Only find sl roots if any of the sapling configuration files exist

I think this would get:

No errors, because sl isn't called unless a sapling root is found, and/or a sapling config file exists
No need for additional configuration options, which would break existing workflows when mercurial/sapling users have to add the new option across all machines (including CI)

It would make it so that adding or removing SCM roots while running watch does not work as prior, though. I am sure somebody is doing this. The "only find roots once" thing can be dropped; that is a performance enhancement of questionable value.

gosh I feel super frustrated when the worlds collide and you can't understand why that's happening.
the error message desperately need fixing. luckily, we do know the culprit in this example, but next time it could be some other piece of software causing interference and as-is there's not a single clue to tell the reason and debug.

Why not use $(npm bin)/sl to call sapling?

A framework whose purpose is to support quality approaches, with an error message like this ("thrown: [Error]"), with no fix in the 10 months since the bug was reported - it is shameful.  I just lost hours to this.
By merely providing a clearer error message this would be greatly remediated, at least as a stopgap measure until a more sophisticated improvement can be made.

I just ran into the same issue; this needs to be fixed.

For a hacky solution to get yourself unblocked:

go into node_modules/jest-changed-files/build/sl.js
inside getRoot return null

or wrap it in something that detects if jest is calling it, maybe...)

I created a workaround using that approach, since jest checks for sapling with sl root
#!/usr/bin/env zsh

if [[ $1 == 'root' ]]; then
  exit 1
fi

exec /opt/homebrew/bin/sl "$@"

Wasted a few hours on this.

@rmartine-ias #14046 (comment)

or wrap it in something that detects if jest is calling it, maybe...)

I created a workaround using that approach, since jest checks for sapling with sl root
#!/usr/bin/env zsh

if [[ $1 == 'root' ]]; then
  exit 1
fi

exec /opt/homebrew/bin/sl "$@"

This could be patched into the sl binary either upstream or just in homebrew and other distributions' builds.

Why not just uninstall SL? :D

@vegerot

Why not just uninstall SL? :D

it's critical to my workflow :P

@rmartine-ias do you think we can call this fixed after https://github.com/jestjs/jest/pull/15053?

@SimenB yes, I think this should do it :)

Yay 😁!  Thanks so much @rmartine-ias for fixing my shitty code


Cluster is recreated with every apply

This is linked to https://github.com/hashicorp/terraform/issues/16724 and might be fixed by https://github.com/cloudposse/terraform-aws-rds-cluster/issues/35
Currently a terraform plan shows that the RDS cluster is recreated with every apply, the cause seems to be a wonky availability-zones attribute.
Somehow this seems to trigger a new resource. See plan output.
-/+ module.rds_cluster_aurora_mysql.aws_rds_cluster.default (new resource required)
      id:                                  "namespace-stage-project" => <computed> (forces new resource)
      apply_immediately:                   "true" => "true"
      arn:                                 "arn:aws:rds:eu-central-1:123456789:cluster:namespace-stage-project" => <computed>
      availability_zones.#:                "3" => "2" (forces new resource)
      availability_zones.1126047633:       "eu-central-1a" => "eu-central-1a"
      availability_zones.2903539389:       "eu-central-1c" => "" (forces new resource)
      availability_zones.3658960427:       "eu-central-1b" => "eu-central-1b"
      backup_retention_period:             "5" => "5"
      cluster_identifier:                  "namespace-stage-project" => "namespace-stage-project"
      cluster_identifier_prefix:           "" => <computed>
      cluster_members.#:                   "2" => <computed>
      cluster_resource_id:                 "cluster-AAAXXXX" => <computed>
      database_name:                       "project" => "project"
      db_cluster_parameter_group_name:     "namespace-stage-project" => "namespace-stage-project"
      db_subnet_group_name:                "namespace-stage-project" => "namespace-stage-project"
      endpoint:                            "namespace-stage-project.cluster-sensitive.eu-central-1.rds.amazonaws.com" => <computed>
      engine:                              "aurora-mysql" => "aurora-mysql"
      engine_mode:                         "provisioned" => "provisioned"
      engine_version:                      "5.7.12" => <computed>
      final_snapshot_identifier:           "namespace-stage-project" => "namespace-stage-project"
      hosted_zone_id:                      "Z1RLSENSITIVE" => <computed>
      iam_database_authentication_enabled: "false" => "false"
      kms_key_id:                          "arn:aws:kms:eu-central-1:123456789:key/xxx" => <computed>
      master_password:                     <sensitive> => <sensitive> (attribute changed)
      master_username:                     "project" => "project"
      port:                                "3306" => <computed>
      preferred_backup_window:             "07:00-09:00" => "07:00-09:00"
      preferred_maintenance_window:        "wed:03:00-wed:04:00" => "wed:03:00-wed:04:00"
      reader_endpoint:                     "namespace-stage-project.cluster-ro-sensitive.eu-central-1.rds.amazonaws.com" => <computed>
      skip_final_snapshot:                 "false" => "false"
      storage_encrypted:                   "true" => "true"
      tags.%:                              "3" => "3"
      tags.Name:                           "namespace-stage-project" => "namespace-stage-project"
      tags.Namespace:                      "namespace" => "namespace"
      tags.Stage:                          "stage" => "stage"
      vpc_security_group_ids.#:            "1" => "1"
      vpc_security_group_ids.636648702:    "sg-080d3cfa4609edea8" => "sg-080d3cfa4609edea8"

-/+ module.rds_cluster_aurora_mysql.aws_rds_cluster_instance.default[0] (new resource required)
      id:                                  "namespace-stage-project-1" => <computed> (forces new resource)
      apply_immediately:                   "" => <computed>
      arn:                                 "arn:aws:rds:eu-central-1:123456789:db:namespace-stage-project-1" => <computed>
      auto_minor_version_upgrade:          "true" => "true"
      availability_zone:                   "eu-central-1a" => <computed>
      cluster_identifier:                  "namespace-stage-project" => "${aws_rds_cluster.default.id}" (forces new resource)
      db_parameter_group_name:             "namespace-stage-project" => "namespace-stage-project"
      db_subnet_group_name:                "namespace-stage-project" => "namespace-stage-project"
      dbi_resource_id:                     "db-SENSITIVE0" => <computed>
      endpoint:                            "namespace-stage-project-1.sensitive.eu-central-1.rds.amazonaws.com" => <computed>
      engine:                              "aurora-mysql" => "aurora-mysql"
      engine_version:                      "5.7.12" => <computed>
      identifier:                          "namespace-stage-project-1" => "namespace-stage-project-1"
      identifier_prefix:                   "" => <computed>
      instance_class:                      "db.t2.small" => "db.t2.small"
      kms_key_id:                          "arn:aws:kms:eu-central-1:123456789:key/xxx" => <computed>
      monitoring_interval:                 "0" => "0"
      monitoring_role_arn:                 "" => <computed>
      performance_insights_enabled:        "false" => <computed>
      performance_insights_kms_key_id:     "" => <computed>
      port:                                "3306" => <computed>
      preferred_backup_window:             "07:00-09:00" => <computed>
      preferred_maintenance_window:        "mon:04:25-mon:04:55" => <computed>
      promotion_tier:                      "0" => "0"
      publicly_accessible:                 "false" => "false"
      storage_encrypted:                   "true" => <computed>
      tags.%:                              "3" => "3"
      tags.Name:                           "namespace-stage-project" => "namespace-stage-project"
      tags.Namespace:                      "namespace" => "namespace"
      tags.Stage:                          "stage" => "stage"
      writer:                              "false" => <computed>

-/+ module.rds_cluster_aurora_mysql.aws_rds_cluster_instance.default[1] (new resource required)
      id:                                  "namespace-stage-project-2" => <computed> (forces new resource)
      apply_immediately:                   "" => <computed>
      arn:                                 "arn:aws:rds:eu-central-1:123456789:db:namespace-stage-project-2" => <computed>
      auto_minor_version_upgrade:          "true" => "true"
      availability_zone:                   "eu-central-1b" => <computed>
      cluster_identifier:                  "namespace-stage-project" => "${aws_rds_cluster.default.id}" (forces new resource)
      db_parameter_group_name:             "namespace-stage-project" => "namespace-stage-project"
      db_subnet_group_name:                "namespace-stage-project" => "namespace-stage-project"
      dbi_resource_id:                     "db-SENSITIVE1" => <computed>
      endpoint:                            "namespace-stage-project-2.sensitive.eu-central-1.rds.amazonaws.com" => <computed>
      engine:                              "aurora-mysql" => "aurora-mysql"
      engine_version:                      "5.7.12" => <computed>
      identifier:                          "namespace-stage-project-2" => "namespace-stage-project-2"
      identifier_prefix:                   "" => <computed>
      instance_class:                      "db.t2.small" => "db.t2.small"
      kms_key_id:                          "arn:aws:kms:eu-central-1:123456789:key/xxx" => <computed>
      monitoring_interval:                 "0" => "0"
      monitoring_role_arn:                 "" => <computed>
      performance_insights_enabled:        "false" => <computed>
      performance_insights_kms_key_id:     "" => <computed>
      port:                                "3306" => <computed>
      preferred_backup_window:             "07:00-09:00" => <computed>
      preferred_maintenance_window:        "sun:03:45-sun:04:15" => <computed>
      promotion_tier:                      "0" => "0"
      publicly_accessible:                 "false" => "false"
      storage_encrypted:                   "true" => <computed>
      tags.%:                              "3" => "3"
      tags.Name:                           "namespace-stage-project" => "namespace-stage-project"
      tags.Namespace:                      "namespace" => "namespace"
      tags.Stage:                          "stage" => "stage"
      writer:                              "true" => <computed>

Config:
module "rds_cluster_aurora_mysql" {
  source             = "git::https://github.com/cloudposse/terraform-aws-rds-cluster.git?ref=master"
  engine             = "aurora-mysql"
  cluster_family     = "aurora-mysql5.7"
  cluster_size       = "${var.rds_cluster_size}"
  namespace          = "dc"
  stage              = "${element(split("-", var.name), 1)}"
  name               = "${element(split("-", var.name), 0)}"
  admin_user         = "${element(split("-", var.name), 0)}"
  admin_password     = "${random_string.password.result}"
  db_name            = "${element(split("-", var.name), 0)}"
  instance_type      = "${var.rds_instance_type}"
  vpc_id             = "${aws_vpc.this.id}"
  availability_zones = ["${var.azs}"]
  security_groups    = ["${module.security_group_webapp.this_security_group_id}", "${module.security_group_bastion.this_security_group_id}"]
  subnets            = ["${aws_subnet.private.*.id}"]
  # zone_id            = "${aws_route53_zone.internal.zone_id}"
  storage_encrypted  = true
  maintenance_window = "wed:03:00-wed:04:00"
  skip_final_snapshot = false
}

I assume that it would work fine with the availability_zones variable dropped.

See also https://github.com/hashicorp/terraform/issues/16724#issuecomment-420752345

thanks @nodomain
we'll review it promptly

One more report on the very same issue as reported. And again it seems to be related to AZs in a different region this time. My config specifies 2a and 2b. us-east-2 has 3 AZs.

Thanks for the fix! This appears to be corrected now in my environment. Appreciate the quick response.


1.8.2 doesn't install into program files directory

After running the EXE, it doesn't seem to install but rather just run from the local download directory.  Not sure if that was intentional or not.

Oops you're right. I'll replace it with the installer shortly ☺

Fixed.