Typetura NPM issue

My keyframes works when I use a script tag but not via the npm. Why would that be?

Related project: https://github.com/SteamTrainCreative/Boilerplate/blob/master/package.json

@SteamTrainCreative Thank you for creating this issue! Are you running this with a node server?

I downloaded your project and the script is in the node_modules folder, but you need to be running a node server in order to use the js import. I can work to make that more clear in the documentation if that will be helpful.

We are creating static websites hosted on netlify...this means that we can't use typetura?
That's would be sad...

Typetura will work here, and there are a few approaches you can take. Before getting into those here is what is going on. It looks like you are using Node to manage dependencies and compile Sass, but you’re not using it for markup or script compilation. This means you don’t have any system in place to route the typetura script in your node modules to the pages you are working with. So here’s how you might be able to do this:

Link directly to the file in node_modules. You will need to make sure this exists when deploying to Netlify or you can make some system to pull whatever version you are using into your main directory.  This will let you use the common js line from the readme. Pinging @clickclickonsal to weigh in on this as he is more of a node expert than me. All this stuff is built into modern JS frameworks like React and Vue.

Thanks for the support. We will go on option #2 for now. How often do you think we would have to update this file?

@SteamTrainCreative Not often at all. There isn’t much to it so not much to change. I have been playing around with when to write the initial styles so I may change that soon. Currently it’s on load but I may change it back do document ready.
In about a year or so, when CSS register properties lands in browsers, I plan on another release. This will include some modular scale additions but the core that exists currently will not change much.

This is very exciting stuff @scottkellum, all the best in your efforts with this project. I heard about you on ShopTalk show and immediately knew I wanted to use Typetura for our clients. All the best and hope more people will pay attention your project.

@SteamTrainCreative Thank you! I appreciate it 😄
If you have any other questions, comments, thoughts, or ideas don’t hesitate to reach out!


Add CocoaPods in your releases

Almost all iOS developers use CocoaPods, but it is bad idea to keep such big project in cocoapods - it takes forever to download an configure on CI-servers.
In our project we use j2objc-local cocoapod for referencing locally installed j2objc distribution: https://github.com/actorapp/j2objc-local
You just need to define environment variable J2OBJC_HOME with path to J2OBJC.
Hope it helps someone.

@ex3ndr The j2objc-gradle plugin provides a task for cocoapods see: https://github.com/brunobowden/j2objc-gradle/blob/master/j2objc.gradle#L1153

I have same file in my sources, but it will be better to have something official.

Having official CocoaPods support would be fantastic. Thanks!

+1

I really believe that cocoapods is a bad way of integration of j2objc, i tried may different ways, this is just doesn't work well. Best type is writing makefile and compile java from sources to static library, link it with app and include all required headers.
Cocoapods always do some awful things.

@ex3ndr That is not true. Integration with Cocoapods works great. See: https://github.com/brunobowden/j2objc-gradle

@cdibona ok, when i will add/remove any file or update them i need to run something? Can't do this in better way? Recreating/updating xcode projects when add/remove files? Correct me if i wrong. Also your layout doesn't work well with GWT or something. I also tried to do something with gradle, but found it a bit unusable and switched to makefile. Very easy install and you can manage everything.

Cocoapods support available in https://github.com/brunobowden/j2objc-gradle.


Add Next.js 14 support for App router

Is your feature request related to a problem? Please describe.
My team already uses Next.js with Newrelic integratin, recently we moved to Next.js 14.1 and I started to see in the logs Next.js middleware instrumentation only supported on >=12.2.0 <=13.4.12, got 14.1.0.
Feature Description
It would be nice to have Newrelic fully supported in latest version of Next.js
Describe Alternatives
If middleware instrumentation is not supported it would be nice to turn off by passing props.
Additional context
I'd like to integrate Newrelic using the AMP, integrating Browser monitoring using getBrowserTimingHeader and get support for middleware.
Priority
Really Want

@abrahamgr our testing shows Next.js 14 works.  During research in the fall of 2023 we also found that app router works.  We are doing more research soon around officially supporting app router which means perhaps introducing capture more segments/spans.  The error message you mention is around middleware.  Next.js has change significantly since we added our instrumentation and the middleware function should only be a small part of the observability.

@bizob2828, thanks for you response. I also faced the same issue when setting up NewRelic with Next AppRouter structure with both server and client side (I only tested in Next 13). It is really nice to hear that you were able to make it work in Next 14.
Can you share the details to make it work in Next 14 or let us know once you have an official release with support to newest versions of NextJs?

@agwermann we didn't do anything. we just never had tests against app router.  Looking at it more closely we will only name transactions and keep the appropriate async context.  getServerSideProps is no longer a thing so we will not capture spans for it. Also we no longer instrument middleware as it runs in a worker thread and our agent cannot propagate to child threads, however that's unrelated to app router.  Also api routes no longer seem relevant as you make calls to functions within your page components so we will not track API calls as spans.  Is there something you expect to see that you're not currently?

@bizob2828 thanks, good to know.
I have one more question, reading the Next.js docs I see they suggest to register the instrumentation on the instrumentation.ts file, something like:
export async function register() {
  if (process.env.NEXT_RUNTIME === 'nodejs') {
    await import('@newrelic/next')
  }

Is this now the best place to setup New Relic? or is better setting Node.js to pre-load @newrelic/next as this guide mentions?


Nested tab with conditional rendering cause endless looping of condition function

Link to reproduction
https://github.com/slavanossar/payload/tree/repro/nested-conditional-tab-loop
To Reproduce

Go to the Menu global
Observe tab field isn't rendering
Meet condition by typing abc into globalText field.
Check console for endless console.log('loop')

Describe the Bug
When creating a nest tabs field, if admin.condition is set the function is run in an endless loop and the field never display, even if the result of the function is true.
Just started happening in 1.10.0, maybe related to https://github.com/payloadcms/payload/commit/967f21734600de1fec8c1227a354ef5a417e54c5
Payload Version
1.10.0

Thanks for reporting @slavanossar!
This should be fixed with https://github.com/payloadcms/payload/pull/2890. Please update to https://github.com/payloadcms/payload/releases/tag/v1.10.1 and reopen if you still have trouble.


Bail if targets require nonexistent feature.

Bin-, bench-, test- and example-targets may specify a set of required
features. Before this commit, it was possible to specify features not
declared in [features], which could be confusing in case of typos. This
is now a hard error during manifest-parsing.
Fixes #4854

r? @alexcrichton
(rust_highfive has picked a reviewer for you, use r? to override)

Well I just realized this is simply wrong. Features can be given as foo_dep/foo_feature, which is not in the manifest's feature list at all.


Use OpenID Connect to Deploy

See https://docs.github.com/en/actions/security-for-github-actions/security-hardening-your-deployments/configuring-openid-connect-in-amazon-web-services

all done.


Fixed Spline_Coefficent_500.txt not found

spline_coefficient_500.txt was not put onto the repo. This is critical to the running the methodology.

Fixed.


Remove rdxml scaffolding

Remove the rdxml scaffolding to simplify our code and reduce maintenance load.
Triage: this is a TODO task to do after RC2.

I wonder if we should also try and block native compilation if the updated Win10 Tools aren't installed.

I think we should do this in RC2 (not after) since updated tooling will be available.

@natemcmaster you can go ahead and do this now. We are already broken on the current UWP tooling.


[doc] Fix minor grammar mistake

Hi! Thanks for the PR but I'm afraid I must turn it down. We're transitioning to Discourse later this week, and our FAQ along with it, so faq.org will be deleted soon, so I'm not accepting PRs to the docs for the time being. Thanks for the help in any case!

Hi! Thanks for the PR but I'm afraid I must turn it down. We're transitioning to Discourse later this week, and our FAQ along with it, so faq.org will be deleted soon, so I'm not accepting PRs to the docs for the time being. Thanks for the help in any case!


The process cannot access the file because it is being used by another process

Environment： Windows10 operating system
Quesion:  npm download failed, Rolling Back, Rollback failed. Remove npm-v3.10.10.zip: The process cannot access the file because it is being used by another process.

Tried solutions: Computer shutdown and restart, still not solve the problem, is still the same error.

Cannot recreate:

Things to look for

Make sure you've uninstalled any prior versions of Node that were installed without NVM4W. Same goes for npm.
Make sure a firewall isn't blocking the download.
Make sure your user has the appropriate permissions (this is unlikely to be the problem in this particular case, since you can get a partial file)

This is most commonly caused by a network failure. You can also download npm and install it manually if needed.

We see this frequently in our shop.  The message is generated in web.go->func  Download(...).  The top of this function contains:
 output, err := os.Create(target)
 if err != nil {
   fmt.Println("Error while creating", target, "-", err)
 }
 **defer output.Close()**
...
**  defer response.Body.Close()**

Later, there are 2 different error handling stanzas.  The one for ctrl-c includes:
	output.Close()
	response.Body.Close()

The one for an HTTP failure does not include these closes.  So, in the event of an HTTP failure, the code attempts to delete the folder containing the failed download, but cannot because the output file is still open.  This results in the "rollback failed" message.
If this analysis is correct, one solution would be to add the closes that are found in the ctrl-c handler to the http error handler.

I modify the nvm mirror(npm&node) and  met the same issues，delete the mirror saved the problem.

The issue is still present on v1.1.9, I just tried to use nvm install 13.7.0 and cannot use that version

Same I am also getting this issue:
C:\WINDOWS\system32>nvm install 16.13.0Downloading node.js version 16.13.0 (64-bit)... CompleteCreating C:\Users\n0311764\AppData\Roaming\nvm\temp Downloading npm version 8.1.0... Download failed. Rolling Back.Rollback failed. remove C:\Users\n0311764\AppData\Roaming\nvm\temp\npm-v8.1.0.zip: The process cannot access the file because it is being used by another process.Could not download npm for node v16.13.0.Please visit https://github.com/npm/npm/releases/tag/v8.1.0 to download npm.It should be extracted to C:\Users\n0311764\AppData\Roaming\nvm\v16.13.0
--

Turn on "Developer Mode" by searching for the developer settings.

Turn on "Developer Mode" by searching for the developer settings.

Developer settings for NVM?

Developer Mode is part of Windows. See https://github.com/coreybutler/nvm-windows/wiki/Common-Issues#permissions-exit-1-exit-5-access-denied-exit-145

Developer Mode is part of Windows. See https://github.com/coreybutler/nvm-windows/wiki/Common-Issues#permissions-exit-1-exit-5-access-denied-exit-145

This worked perfectly,  thanks

I get this error only if trying to install node 14.19.2
I can install 10.23.0 and 12.23.0 without issues and use them.
Maybe this can help identify the cause?

I get this error only if trying to install node 14.19.2 I can install 10.23.0 and 12.23.0 without issues and use them. Maybe this can help identify the cause?

Confirmed. I could even install 14.9.1 (npm 6.14.16) without problems, while 14.9.2 (npm 6.14.17) gives "download failed".

I see the same behaviour as @johnvt. Older versions such as 14.17.5 and 14.9.1 work just fine. But 14.19.2 doesn't. Toggling the developer mode setting on or off doesn't seem to matter.
The command is ran in an elevated Git Bash Windows Terminal instance.
$ nvm install 14.19.2
Downloading node.js version 14.19.2 (64-bit)...
Complete
Downloading npm version 6.14.17... Download failed. Rolling Back.
Rollback failed. remove C:\Users\xxxxx\AppData\Roaming\nvm\temp\npm-v6.14.17.zip: The process cannot access the file because it is being used by another process.
Could not download npm for node v14.19.2.
Please visit https://github.com/npm/cli/releases/tag/v6.14.17 to download npm.
It should be extracted to C:\Users\xxxxx\AppData\Roaming\nvm\v14.19.2

$ nvm version
1.1.9

Output of winver

Can also confirm this happens for 14.19.3 for me.

nodejs 14.19.3，14.19.2 is failed.

https://github.com/coreybutler/nvm-windows/issues/373#issuecomment-529991743
https://github.com/npm/cli/issues/4890
The above helped me, but the specific reason is still unknown, maybe npm official problem

I have the same problem, it only installs which nvm list available shows in the list, rest fails.


Could you please update build configuration to allow hledger-lib 1.22?

Fixed in git, release will follow shortly


About Scape Dataset Ground-truth

Hello,
Thanks, I have got my matches on Scape dataset and now I want to calculate the geodetic error, but I don't know how to use the file of [.vts]. Just like you write in the Readme, the file of .vts is the ground truth maps, but I don't know how to get maps around any pairs shapes, just like the map mesh060.off to mesh061 et al. Looking forward to your reply, thanks.
Best.

Hi,
The .vts files are ground truth map to a template mesh, so shape_1.vts is the p2p from the template to shape_1.
Given a shape_1 and shape_2, and a predicted p2p map from shape_2 to shape_1 T21, the geodesic error is:
mean_geod_error = geodesic_matrix_shape1[T21[vts_shape2], vts_shape1].mean()

closing the issue, feel free to re-open it if you have further questions!


initial implementation of "functional" interface

This is an initial attempt at a "functional" interface; those of us who prefer using arrow functions will hopefully appreciate this.  I've called it the func UI; here's how it looks:
Usage
'use strict';

describe('behavior', suite => {
  // just like this.timeout()
  suite.timeout(200);

  it('should work synchronously', test => {
    expect(1 + 1, 'to be', 2);
    expect(2 + 2, 'to be', 4);
    // test.done() must be called unless a Promise is returned
    test.done();
  });

  it('should work asynchronously', test => {
    expect(1 - 1, 'to be', 0);
    expect(2 - 1, 'to be', 1);
    process.nextTick(function() {
      test.done();
    });
  });

  it('should work with a Promise', async () => {
    expect(1 - 1, 'to be', 0);
    expect(2 - 1, 'to be', 1);
    // test.done() needn't be called
  });

  it('should work with context methods', async test => {
    test.timeout(400);
    expect(1 - 1, 'to be', 0);
    expect(2 - 1, 'to be', 1);
  });

  afterEach(hook => {
    // hooks have same behavior
    hook.timeout(200);
    hook.done();
  });

  afterEach(async () {
    // and also support Promises
  });
});

Implementation Details
Mocha makes assumptions about how Runnables (Tests, Hooks) are executed; what function context they are called with, and what parameters.  To address this, I needed to decouple some parts of Mocha's core.
So, I've introduced a new concept: the Behavior.  We can think of this as a collection of mixins which may apply (currently) to Suites and Runnables.  These mixins introduce alternative behavior for certain methods within those objects.  This is not the mythical plugin API, but it could be a step in that direction.
There are currently a select few places where the Behaviors are delegated to, and I've broken out the default functionality into a Behavior (the Default one, fwiw).
Due to the way Mocha's Suites configure their children (Suites, Hooks, and Tests), in which all children "inherit" the configuration of their parent Suite, an Interface (bdd, tdd, qunit, etc.) only need configure the Behavior on the Root Suite.  This configuration propagates throughout all Suites and children thereof.
This means an Interface can provide an alternate Behavior.  It's not the only way to provide an alternate Behavior, but seems like a reasonable one; see the simplicity of lib/interfaces/func.js.  (Maybe the interface is where the Functional Behavior should live?)
A Behavior may have one or more methods and needn't implement all of them.  The default methods will always be present, and if multiple behaviors are configured, then the last one wins (think of how Object.assign() works).
Ultimately, I'm not married to this API.  I'm open to considering an event-driven model, as long as the scope doesn't blow up.  But once it's merged, we are married to it, so it behooves us to be confident about what we're doing.
See #1856, #2657, probably others.

lol, guess I can't use Object.assign... I'll fix that.

this needs

[ ] docs
[ ] an integration test to see what happens when e.g. test.done() is called in a Promise-returning Test
[ ] more unit tests, probably

imo this is exactly what i would expect an arrow-func-centric Mocha to look like!

Coverage decreased (-0.6%) to 89.381% when pulling b96abc99cb241f8f5805ed2c2e2966e33bba7a2d on experimental/bdd2 into 0d95e3f41a0859be4e7225541575e78b6d8926ee on master.

Is it possible to omit explicit ctx.done() for sync tests/hooks? Suppose that the test is sync if the return value is not a promise? If ctx.done() is required for sync tests, all existed tests have to be modified to switch from bdd to func:
describe("...", () => {
-  it("...", () => {
+  it("...", t => {
    // some asserts...
+    t.done();
  });
});

if we didn’t require it then how would we know a test is async? if we supported a test, done signature, then every async test that didn’t need to use methods on test would have test anyway.
probably just as effective to useasync keyword instead of test.done() ..

Something like this?
describe("...", () => {
  it("...", t => {
    t.async(true);
    setTimeout(t.done, 200);
  });
});

I wonder if I can use this feature to pass a client to the functions (wd client for native testing in my projects).

@wavebeem We do check for a Promise return value, but not all async behavior is Promise-based.
@Bamieh I'm not entirely sure what you're asking, but if I had to guess, it's here.  We call shouldRunWithCallback to determine which of the two methods (run or runWithCallback) we call.  Mind you, this is not applicable to the func interface, which (currently) will always pass a single parameter.
@eight04 Is there a precedent for that--does another test framework do it that way?  I'm basing my implementation on stuff like tape, which requires t.end() (or t.plan(), which we can't support) to be called unless a Promise is returned.
This is why I mentioned it'd actually be easier to use the async keyword for otherwise-synchronous tests:
it('should synchronously assert', t => {
  assert(true);
  t.done(); // might as well alias t.end() to this if we do it
});

is effectively:
it('should synchronously assert', async t => {
  assert(true);
});

FWIW, the Jasmine/Jest strategy seems to actually make timers Promise-returning or synchronous (not sure which) by monkeypatching them. I don't want that to be a choice Mocha makes for its users; libraries like Sinon can be added to make timers synchronous, if needed.

Is there a precedent for that--does another test framework do it that way?

No. Actually, I think (test, done) is better. It matches the old interface nicely.
The original purpose of passing the context as the first argument is to solve the problem that users can't access .skip(), .timeout() when using arrow functions. I don't think people would like to change all tests into async function just to gain the ability to access .skip(), .timeout() for maybe one or two sub-tests.
I prefer (test, done). Reasons:

All existed sync tests don't have to be modified.
All existed promise-based async tests don't have to be modified.
All existed callback-based async tests just need to change the signature. (done -> (t, done))
arrow-mocha users can migrate to the new UI without modification.

If we still want to keep async as the default, it is probably worth adding an autoend option that would automatically done() the test when the return value is not a promise, just like current mocha.

@boneskull do you think there is a chance you work on this approach again? Could you explain why it was postponed / what's currently blocking it? I would be happy to help if I can.

🤖 Closing out old PRs to keep the review queue small. Cheers!


Campaign | Teams: List of all team badges

The link probably links to a page showing all available team badges that teams can win.

Prototype: https://www.figma.com/proto/7wE0Iveas3GGb8u4oL0714/App-Design-Mitglieder-App-(1)-(Copy)?node-id=2743-11128&t=b8swD5IOAFfZ0OSo-1&starting-point-node-id=2425%3A17429
Pollion reference: https://github.com/pollion/sunflower-app/issues/110

UX questions at @leagerndt @JuliaFreitagBGSt:

further details needed on this page
need artwork for all team badges


[AMBARI-22914] Oracle DDL is broken at trunk

What changes were proposed in this pull request?
The Oracle DDL is broken
ERROR at line 1:
ORA-00904: "CREATE_TIMESTAMP": invalid identifier

insert into user_authentication(user_authentication_id, user_id, authentication_type, authentication_key, create_time, update_time)
*
ERROR at line 1:
ORA-02291: integrity constraint (AMBARI.FK_USER_AUTHENTICATION_USERS) violated
- parent key not found

The error is in the following insert statement found at https://github.com/apache/ambari/blob/ef801acbb290657760ced9917055fbb6250c95d4/ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql#L1179-L1180
This is fixed by using the correct column name for the create timestamp, create_time, rather than create_timestamp.
insert into users(user_id, principal_id, user_name, display_name, local_username, create_time)
  SELECT 1, 1, 'admin', 'Administrator', 'admin', CURRENT_TIMESTAMP from dual;

Please review Ambari Contributing Guide before opening a pull request.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/436/
Test PASSed.


Repo.aggregate/3 does not typecast the result

Environment

Elixir version: 1.9.2 and 1.10.0-dev
Database and version (PostgreSQL 9.4, MongoDB 3.2, etc.): Postgres 9.6.3
Ecto version (mix deps): 3.1.x, 3.2.0
Database adapter and version (mix deps): Postgres 0.15.1
Operating system: MacOS X Mojave

Current behavior
Repo.aggregate/3 does not support type casting the result.  In my example, the ex_money_sql package supports serialising a Money type to Postgres as a composite type. For example these are returned as {"USD",#Decimal<100>} from Postgrex.
It also includes a custom aggregate function to sum money types. The following example illustrates the error:
# Correct value is returned but it is not typecast event though `:payroll` is defined as a `:money_with_currency` type
iex> Repo.aggregate(Organization, :sum, :payroll)
{"USD", #Decimal<300>}

# Now try forced typecasting
iex> Repo.aggregate(Organization, :sum, type(:payroll, :money_with_currency))
== Compilation error in file test/db_test.exs ==
** (CompileError) test/db_test.exs:17: undefined function type/2

Expected behavior
Ecto.aggregate/3 to support typecasting the result as the other ecto_sql macros/function do. This is preferably done automatically since the type is known at compile time. Otherwise support the type/2 macro.

@kipcole9 does it work if you write the aggregate query by hand (without the type)? I want to figure out it is an Repo.aggregate issue or a general query issue.

Btw could you please provide a sample app that reproduces the error?

@jose, the aggregate function works correctly and there is a test for that in the repo below.

Clone https://github.com/kipcole9/money_sql
mix deps.get
Update the postgres connection information in config/test.exs
mix test

The particular tests are in test/db_test.exs. The final test is commented out since it creates a compile error. Its the test where I try to use the type/2 macro in Repo.aggregate/3.

@josevalim If I'm not wrong then @kipcole9 missed it in documentation:
aggregate(
  queryable :: Ecto.Queryable.t(),
  aggregate :: :avg | :count | :max | :min | :sum,
  field :: atom(),
  opts :: Keyword.t()
) :: term() | nil

Source: https://hexdocs.pm/ecto/Ecto.Repo.html#c:aggregate/4
This means that ecto does not support type() casting in this place.
Since we have an opts here we can add [type: :money_with_currency] support if you don't mind.
If everything I wrote is correct then I may take a look and create PR for that as it does not looks really hard.

@Eiji7, I think that Repo.aggregate/3 should typecast the result automatically since it can already determine the type of the field based upon the schema (for schema queries). Doing so would be in conformance to the typespec.
The option to force a typecast is, in my opinion, a less satisfactory option that I offered as only as an alternative idea that may be required for schemaless queries.

@kipcole9 I'm not 100% sure about this. What you wrote is generally correct, but only in case ecto function would return a struct containing such key. This function is just a shortcut for often used queries and not replacement for complex Ecto.Query.API with all its features.
Just for example how about summing by 2 different currencies? Current implementation is a really simple and not over complicated solution. If you would go to work with sum for more than one currency then you most likely would need group_by (money by currency) and therefore you would not be able to call Repo.aggregate/4 (as it does not allows queries with group_by field set).
Look that such change may be considered as breaking change and affect other projects as well. My proposition however allows enhance this function without causing any other problems.
What do you think about it?

I recognise that Repo.aggregate/3 isn't a full replacement for Ecto.Query's api. But it does allow an queryable as its first argument so I do think it's a fully fledged member of the family. Its also clear in the guides that it has a special place in the api:

At first, it looks like the implementation of aggregate/4 is quite straight-forward. You could even start to wonder why it was added to Ecto in the first place. However, complexities start to arise on queries that rely on limit, offset or distinct clauses.

I also think its more a surprise that a result isn't typecast than to find that it is not since the rest of the query API does cast (whenever it can detect the the type).

Just for example how about summing by 2 different currencies? ....

My custom postgres aggregate function raises an exception if you try to sum monies that have different currencies :-). As it should - its a domain issue. Not an issue related to Repo.aggregate/3 in my view. I'm not asking for a change in API, or a change in semantics.

I also think its more a surprise that a result isn't typecast than to find that it is not since the rest of the query API does cast (whenever it can detect the the type).

Ok, that's the 2nd time you most probably misunderstood documentation. :smiley:
Ecto.Repo.aggregate/4 is of course in Ecto.Repo module and therefore it's not a part of Ecto.Query.API. Personally I was more surprised seeing type/2 macro call in your example.
Of course there is no sense to limit Ecto.Repo.aggregate/4 that much to not support Ecto.Query, but I believe that the most used feature of it is where and type casting is probably a really edge case here.

I'm not asking for a change in API, or a change in semantics.

That depends … There were cases when (probably in Elixir core) API allows to do something that its maintainers not expected and even if that was a breaking change it was considered as bug fix, because such behaviour should never happen.
However this does not mean that if someone expect from some API then it always need to be a bug. I was a bit confused seeing this issue for first time, because again personally I was used to integer (like count and sum) and float (like avg and sum) results and I never have a use case in which I even had in mind type casting at this place. Also after quickly looking at documentation I did not found an explicit example or mention that it may return custom types. There is only any in typespec which does not say much.
Also I found this part:

Any preload or select in the query will be ignored in favor of the column being aggregated.

If I remember correctly the type casting you expecting for Ecto.Query happens exactly on select field. Again I may be wrong, but just seeing documentation I do not see a behaviour that in some way does not much documentation.
In short everything depends how @josevalim and others would see this. If this would be considered as a bug then it would be fixed as you said. However in other case my proposition is probably better, because as said somebody else in some project may already be in similar use case and did not expect such behaviour. If you would ask for "fix" when this issue won't be considered as a "bug" then it means that it becomes a breaking change as now test for other developers may fail in favor of your tests.

Hmmm, i don't think I'm misunderstanding the documentation. I didn't say "part of Ecto.Query", because I know it's not there. But most definitely Repo.aggregate/3 is part of the query API otherwise it wouldn't actually do anything! Its take a queryable as an argument, and it generates a queryable under the covers so I think that qualifies.
I do agree that this can be interpreted as a bug or as a feature request. My position is clear I hope - I believe the user expects, and the typespecs allow, for the returned result to be typecast where possible. Like the rest of the query API (small 'q' query, not Ecto.Query).
As you say, lets see what @josevalim and co think.

But most definitely Repo.aggregate/3 is part of the query API otherwise it wouldn't actually do anything! Its take a queryable as an argument, and it generates a queryable under the covers so I think that qualifies.

Yes, it takes Ecto.Query and to be honest I was also really confused on any type in typespec.

Its take a queryable as an argument, and it generates a queryable under the covers so I think that qualifies.

Oh, that's 100% way too generic. There are lots of things you may want to do with Ecto.Query struct. You may want to work with just one field (like select) in order to for example extract some information from query and forget about all other fields. In such case accepting Ecto.Query is easier for few main reasons:

There is no need to define new struct and typespec
Making API developer-friendly i.e. there is no need to explicitly call query.some_field - simply pass query and function will just have one extra pattern matching.
As there is passed already verified struct there is less things to check for exmaple in tests

and having all of that in mind please see again that select with type casting are even omitted at that point as well as group_by field which matches what I just said.
I would like to be well understood, so let's take an example completely outside of it. If there is a conflict of interpretations of specific law paragraph then judge things for what cases such law was introduced. Again go back to documentation and take a look at examples. Of course everything depends to maintainers, but in my opinion it's clear for what purposes it was created at least at first. I just want to let you know the way I think and they way why by default I expect integer and float as "typical" result. Hope you now better understand my point of view.

My position is clear I hope - I believe the user expects, and the typespecs allow, for the returned result to be typecast where possible.

Just to point that. If we would add type option then result would also match typespec and code change would not affect possible other projects.
Anyway I have hope for you as well. I also remember when few times I reported something which I was not sure if such thing is a bug or not. :smiley:
In any case I can make a PR and I just need to know "what and where".


Refactor the handling of nested types.

While building this with clang static analysis, it was complaining that the 'ret' value in the nested type handling was being ignored, which is correct.  But while reviewing the code, I realized that it would be much easier to read, and hence to review for these kinds of violations, if this was split into a separate function.  So I've gone ahead and done that here, and then fixed the bug that caused us to ignore the return value.

CI:

Linux
Linux-aarch64
Windows


Optimise CMS page loading by simplifying model query

This PR adds the method CmsObject::listInThemeArray which can be used to return an Collection of arrays containing filenames and url patterns.
Each element of the collection will look something like this:
[
    'fileName' => 'example.htm',
    'pattern' => '/example/url'
]

The router has been changed to call this method instead of listInTheme.
In short this means that instead of returning a collection of page objects and storing only filename & url, we can just retrieve the filename and url and directly store them as the url map in the router.
The speed results with these changes are as follows:

Tests have been done isolating the the time it takes for $page = $this->router->findByUrl($url); within Cms\Classes\Controller to execute. All tests have been done with 1000 CMS pages.

@jaxwilko when this is ready for review, feel free to take it out of draft :)

@bennothommo my bad, didn't realise it was still in draft lol

@jaxwilko it wasn't - I put it in draft. It still had [WIP] in the title though :P

Bump

I've had a test of this - it does work, but I only see a very small performance benefit. I imagine it's more pronounced when there's more pages to read, but when there's only a handful of pages, it's minuscule.
I just had an idea though - given that for this (and our routing in general), we only want the filename and URL to be retrieved, could we be clever and format the URL to be the first line and just read the first line of pages to determine the URL? It would save us having to parse the sections on each page in order to retrieve the URL, and might make a larger performance boost.
We could maintain backwards compatibility by storing the URL as a "hashbang" of sorts - if the first line doesn't match the format, then assume it's still the old format and would need to be parsed for each of the sections, but we could perhaps save it in this new format on save, going forward?
Eg.
#</test-page
title = "Test Page"
layout = "home"
==
Content

As opposed to the current:
title = "Test Page"
url = "/test-page"
layout = "home"
==
Content

Or even better, could we create a URL cache file and have all CMS objects register a URL in this cache along with a reference to the file name (and perhaps object class) and just use this for routing? Saves us reading all the files altogether.

While I was testing I had 1000 CMS pages which each has a img tag with a base64 encoded image to simulate medium sized html content (in terms of bytes). With less page sizes this issue isn't massively apparent.
It was a while ago i looked at this, but the parser is actually super efficent and just opening the file and reading doing something like:
$fp = fopen($file, 'r');
while ($line = fgets($fp) && !feof($fp) && str_starts_with($line, 'url =')) {
    fclose($fp);
    return trim(explode('=', $line)[0]);
}

Was not really even quicker than just calling the parser.
The issue comes from creating a Halcyon model for each page (Cms\Classes\CmsObject::listInTheme), then discarding the model straight away (Cms\Classes\Router::getUrlMap) which means there was no point to wasting time and memory initing the models.

@bennothommo for multilingual sites, the translated url in the viewBag needs to be read though...

How does this compare to just using the route cache on the next couple of lines? If we have problems with cache invalidation then that's a simple fix, just trigger cache clears whenever the cache would be invalidated


Update library to use new architecture.

This adds breaking changes to the library and would necessitate a major version update.
Among other things, this branch:

Changes the name.
Adopts an OOP architecture.
Adds caching.
Adds the country_name, ip_address, latitude, and longitude attributes to the response object.
Updates documentation to match current IPinfo specs.
Modifies the request headers.
Extends tests.

I know the repl style examples with >>> are popular for python, but is that also the case for ruby? I looked at a couple of gems and they didn't seem to.

Sorry to cheap in, just wanted to give some feedback before you merge this.
While there is on PEP-8 alternative in Ruby, there is still some community style guides that most of teams that use Ruby respect. You can get some taste of it here
https://github.com/rubocop-hq/ruby-style-guide
Code I see here would be normal for Java, but it makes me cringe as rubyist. Namely:

prepending get_ and set_ to method names  -- https://github.com/rubocop-hq/ruby-style-guide#accessor_mutator_method_names
CamelCase for methods -- https://github.com/rubocop-hq/ruby-style-guide#snake-case-symbols-methods-vars
Interface class for cache seems like a leaky abstraction -- it brings no value to code, used once and gets immediately redefined. I would suggest dropping it, but if you plan on leaving it -- at least make it useful.

module IPinfo
  class CacheInterface
    class InterfaceNotImplemented < StandardError; end
    def get(key)
        raise InterfaceNotImplemented
    end
    def set(key, value)
        raise InterfaceNotImplemented
    end
    def contains(key)
        raise InterfaceNotImplemented
    end
  end
end

But again, API for this interface just feels like a person is writing Java :)

Thanks so much @skatkov, this is super helpful and I really appreciate the feedback. In this case I've opted to keep the interface, but that's mostly bc that's a design we've been using across libraries. Thanks again!


Cannot locate Apex Type for ID  CaseComment

I get this error when trying to rollup from CaseComment

Error: Invalid Data.
Review all error messages below to correct your data.
Apex trigger dlrs_CaseCommentTrigger caused an unexpected exception, contact your administrator: dlrs_CaseCommentTrigger: execution of AfterInsert caused by: System.SObjectException: Cannot locate Apex Type for ID: 00aS00000027nijIAA: (dlrs)
You can reproduce this error in Anonymous Apex:
Id myId = '00aS00000027nijIAA';

SObjectType childObjectType = myId.getSObjectType();
System.debug(childObjectType);

I'm not sure what the solution is as this seems to be a Salesforce bug. I couldn't find this reported in any known issues, but I'm generally new to Salesforce and could just not know where to look.
I'm going to pursue this through other channels, and will update this issue as I gather information.
Any short-term solutions short of very specific workarounds would be appreciated. Worst case I might write a small patch to my version just to meet my business objectives until something else presents itself.

hey @afawcett, is there any movement on this one?

I'll take a look at this next time i get a burst of dev time on the tool.

Thanks for looking into this guys. I'm dealing with an issue unrelated to DLRS and this helped me out. Just in case anyone else sees this I ran into this issue with the a CustomShare object, which shares its id prefix with any other CustomShare object. SALESFORCE OYYY.

Thanks for sharing @skamensky 👍

v2.9

After some digging, and experimenting, this appeared to work for me:
(called from after insert, after update, and after delete sections in trigger class)
`  private void declarativeRollupSummary (map<Id,CaseComment> mapOldRecs,map<Id,CaseComment> mapNewRecs) {
dlrs.RollupService.rollup(mapOldRecs,mapNewRecs,CaseComment.SObjectType);

}
`


Dark theme loading problems on Home screen

OS: MacOS 10.12
Issue Descriptions: When using the Dark theme, elements of the theme will often fail to load correctly on the Home page. This tends to occur not on initial use, but when browsing back to the Home page after an indeterminate period of use.
Issues tend to include the dark background missing, or the dark background present but dark text for the hover text on the Recent Activity items.
Steps to Reproduce:

enable Dark theme
play music for an period of time (a few songs)
return to the Home page

Examples:

Please raise over at the theming repo, I noticed this the other day but haven't had time to look into it 👍


RSA signature using SHA-256 hashing algorithm

Is it possible to create an RSA signature using SHA-256 hashing algorithm?
As this is the only one supported by the Google App Engine for authentication.

Obviously I suck and haven't added this - working on a book and hope to be done end of the month - I'll try to add the missing hash algorithms shortly.

Any progress on this? I also would love the addition of RSA since it would let me better use private keys for signing and public keys for verification, if you need some help I'd be up to pitch in.

It's not done - it should be fairly trivial. If you want to do it, fork and make a pull request - I'll review it and accept it. I'm pretty busy until maybe mid july but it really should be trivial to implement. I'm leaving myself a post-it to remind me to do it after hours one day. It would be awesome if you can do it - just let me know if you start work so we don't duplicate efforts.

I can start tonight probably, I just want to double check I'm understanding things correctly so here is my use case:

I generate a JWT and instead of using HMAC I use JWT, and instead a passphrase like "password" I will be able to pass in a RSA private key
Then on whatever other server is attempting to verify the JWT, they can do so using a matching public key?

For coding, its really just an additional algorithm, the rest is already in place or would be done by the user implementing authentikat-jwt.

Hey, you should just be able to implement the algorithm fairly simply:
The JWS file here has the existing algorithms:
https://github.com/jasongoodwin/authentikat-jwt/blob/master/src/main/scala/authentikat/jwt/JsonWebSignature.scala
Add the new case classes for the algorithms, the new pattern matching for the different RSA generators, and then the private key is still just a text string I believe. So you'd make the method to do the hashing. That should be more or less it. Validation is just done by hashing and comparing. If you don't get it done I'll try to tackle it over the weekend as well - I'm just really behind on writing this damn book.

Any progress? I'll look at it tonight if not.

Hey I had a stab at it tonight but was not able to complete it. This probably requires an API change - the signature is produced with a private key, and then verified with the public key. Not a terribly complex problem to solve but the APIs aren't perfectly straight forward. There Nimbus Connect API has examples here of usage: http://connect2id.com/products/nimbus-jose-jwt/examples/jwt-with-rsa-signature
Ideally we'll have something somewhat similar but with extractors able to bump out the values in the claims set still. It's worth doing for the nicer scala API I believe - shouldn't be too too hard to get a nice API in place. I'll keep working on it.

Example here _ I could be wrong in terms of jwt though. I'll read through the spec and libs. http://www.java2s.com/Tutorial/Java/0490__Security/RSASignatureGeneration.htm

Any progress updates?

@spry-rproud none from me but thanks for the poke :) @jasongoodwin if you don't get to this during the week I'll take another stab at it over the weekend, I have a hectic week coming up so I can't do this any sooner unfortunately. I'm making a note now for myself to try to resolve this on the weekend :+1:

Hey obihann has a PR for this - I'll hang out w/ it this week and hopefully release it by next weekend.
Thanks for your patience and sorry for the delay in getting this feature in. I'm hoping by next weekend I can have it validated and released.

I'm Working on this over the weekend. The fact that eg RSA works a bit differently with the public and private keys has me rethinking the API. It looks a bit messy in inside too - I want to switch it up a bit. I have more time so I should be able to finish this up over the weekend. Thanks for your patience!!

K I have a bunch of the refactoring done for RSA signing and validation (finally). Sorry for the wait. I'm going to change how it's implemented to use something like bouncy castle for better memory usage as the Java Security pieces used are not thread safe so there is a bunch of extra memory overhead. I should be able to release it this week.
It's a major revision - API breaking change - so I need to produce a migration guide to go along with it.
Thanks for waiting!

hey my book is published etc so I have some more time now. That killed me. http://www.amazon.com/Learning-Akka-Jason-Goodwin/dp/1784393002
I gave this another run over today and tested it a bit. I think it needs some peer review and a bit of focus before I formally release and publish. I pushed the change to the develop branch as 1.0.0-SNAPSHOT. if you need it publish it locally and drop some feedback here. I'll think about it a bit but would be nice to have someone to bounce ideas off of about the use of the api. I should write a few integration tests to see how it works all together and give examples. The docs will be out of whack now too as the API is broken from previous versions.
If you want to contribute, try testing the other rsa algorithms etc. Also changing to bouncy castle from the java security libs should net performance improvements as the java abstractions are not thread safe so require lots of object creation.
Because it's a fairly significant change to the api, it would really help if someone publishes it local and tries to upgrade their project. I'm wondering if there should still be some sort of token verify abstraction but I'm starting to think not:
One of the concerns there is that someone trying to break it could pass algorithm=none and so the check would pass verification so there would need to be allowed algorithms. It might be fine to ship without this though - it moves the concern to the application to ensure that the token is signed with an expected algorithm - I'm not sure if the JWT library should handle too much of this but it makes it a touch harder to use without it.
Right now the RSA implementation expects PrivateKey and PublicKey to be provided, so again, how those are obtained is up to the user.

Added ES and PS algorithms as well. I'll release it soon but it needs some additional verification. Can anyone try 1.0.0-M1 RSA on the develop branch to see if it works in their use cases? The API is a bit different in that you explicitly have to provide the signer or verifier.

1.0.0-M1 published to central. I want to add some more features and test this better w/ integrations before calling it a real release but all encryption algorithms are in place.

Have you been able to test/verify the behavior?

I tried it out this weekend and found some bugs:
https://github.com/jasongoodwin/authentikat-jwt/pull/34

Thank you so much for looking at this!!
On Sun, Jul 16, 2017 3:07 PM, Philippus Baalman<EMAIL_ADDRESS>wrote:
I tried it out this weekend and found some bugs:
#34
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or mute the thread.


Fix CollectionPropertyInternal

Signed-off-by: Dan Sanduleac<EMAIL_ADDRESS>
Fixes #8943
Context
I want to use addAllLater on a configuration's dependencies or constraints.
However, I can't, because no matter what I pass in, it fails with a ClassCastException as the internal code from DefaultDomainObjectCollection.addAllLater expects a CollectionProviderInternal, of which there is only a single internal implementation.
https://github.com/gradle/gradle/blob/1fb12afd4f34951c9c2bd8a592fda8ef5da18dd1/subprojects/core/src/main/java/org/gradle/api/internal/DefaultDomainObjectCollection.java#L296
Solution
Allow ListProperty, SetProperty to be passed to addAllLater, by making CollectionPropertyInternal (an interface they implement) extend CollectionProviderInternal.
Contributor Checklist

[x] Review Contribution Guidelines
[x] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[x] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[ ] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[ ] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[ ] Ensure that tests pass locally: ./gradlew <changed-subproject>:check

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

I don't understand why these tests keep failing, it seems like a flake every time :(
* What went wrong:

Execution failed for task ':distributions:checkBinaryCompatibility'.
> Could not resolve all files for configuration ':distributions:japicmp-baseline-Antlr'.
   > Could not download gradle.zip (gradle:gradle:5.6)
      > Could not get resource 'https://services.gradle.org/distributions/gradle-5.6-bin.zip'.
         > Could not GET 'https://downloads.gradle-dn.com/distributions/gradle-5.6-bin.zip'.
            > peer not authenticated

I'm happy to add tests here too if you think it's worth it, currently all the applicable tests mock a CollectionProviderInternal so they never expose the fact that nothing public actually implements it..

Just to properly understand the issue. Here, we are fixing the issue where you would have something like the following:
def c = project.container(String)
def l = project.objects.listProperty(String)
c.addAllLater(l)  // This fails because of the uncheck cast?

// Somewhere else:
l.add('some-string-later')

Currently adding the ListProperty to the container via addAllLater fails because of the uncheck cast? If that is the case, then we can look into adding tests to AbstractDomainObjectCollectionSpec to verify all domain object collection behave the same. This PR shouldn't try to solve issues around custom implementation of CollectionProviderInternal as this is an internal API and unlikely to be made public or even allow custom implementation by the users.

We are currently tracking down the peer not authenticated issue, you can safely ignore for now.

Currently adding the ListProperty to the container via addAllLater fails because of the uncheck cast
yes that's right

This PR shouldn't try to solve issues around custom implementation of CollectionProviderInternal as this is an internal API and unlikely to be made public or even allow custom implementation by the users.
Not sure what you mean. This is just the most convenient way, I thought, to make existing collections all pass the unchecked cast.
It also makes sense that CollectionPropertyInternal would implement CollectionProviderInternal, since PropertyInternal implements ProviderInternal.
The fact that it's an internal API is even better, as it's not a break to make CollectionPropertyInternal implement another class.

I added the tests to https://github.com/gradle/gradle/pull/10426 which will supersede this PR. As soon as CI pass, we will merge it. Thanks for your contribution.


🐛 Delete 3PID association upon deactivating user.

Have some initial linting and unit tests failing here for some reason 🥲 Will take a look tomorrow.


Making addons persistent when using the storage addon

Currently addons such as prometheus will deploy several deployments, such as Grafana for example. However, the deployments are not persistent as the important volume mounts don't have a underlying persistent volume and use emptyDirs for example. When the pod is restarted, the data is gone.
As an enhancement, I would suggest that when the addon is enabled with the microk8s enable command, one checks if there is a possibility to create and mount persistent volumes and uses them. Otherwise, one can fallback to the classical solution with emptyDirs. As an quick idea, I would simply check in the script if the hostpath-provisioner deployment is up and running and then deploy a persistent volume claim with a deployment that uses that claim, aka use a slightly modified deployment for the grafana example for instance.
Problems that one could be facing are inproper write/read permissions when creating the persistent volume, however, we could add some troubleshooting hints for users that would face such issues.
Alternatively, we could also make a seperate addon called prometheus-persistent for example, and deploy the hostpath-provisoner with the modified deployment files there.
I would give this a try if there is interest in having it in a future release :)

Hi @Pflegusch this is a great suggestion. My recommendation is to extend the Prometheus addon [1] (possibly rewrite it in python) and extend it with a --storage-class argument that you could tell prometheus how/if to persist its data. Of course this would require some editing in the manifests before applying them. I believe the default behavior ("emptyDis") should stay as is so we do not surprise users. What do you think?
[1] https://github.com/canonical/microk8s-core-addons/blob/main/addons/prometheus/enable


Impossible to change the admin's password

I have setup a PocketBase database a while ago and I wanted to go back to it and connect to the admin dashboard. Unfortunately I forgot the password, I have absolutely no clue what it could be. I tried this:
cd ~/myproject/pb_data
sqlite3 data.db 'UPDATE _admins SET passwordHash="$2a$10$xWbkSks7OiTmrri2DU6QSe6OAekcdqkOrkcI1wi4uiv9SU2HTlHoq" where<EMAIL_ADDRESS>
From: https://www.programonaut.com/how-to-reset-the-admin-account-in-pocketbase/#db
The command above should change the password to 123456.
When I do a "select" SQL command, i can see the password hash has changed, but when I try and connect in the admin dashboard, it doesn't work.
When I try and reset the password from the UI by receiving an email, it doesn't work. I'm not receiving the mail.
How can I reset the password please ?

The above command should work as long as you are using the specified email is correct.
But as mentioned in your earlier commented issue, with PocketBase v0.16+ there is a dedicated command to change the admin password:
./pocketbase admin update<EMAIL_ADDRESS>1234567890

When I try and reset the password from the UI by receiving an email, it doesn't work. I'm not receiving the mail.

This is because most likely you haven't enable the SMTP configuration.
More than this I don't think I can help you because this neither a but with PocketBase, nor I have access to your environment.

I forgot to mention it @ganigeorgiev but the ./pocketbase admin command doesn't work either. Well, the command is successful, but i still cannot connect to my pocketbase account.

Okayyy so I fixed my issue. Basically, I was trying the sqlite3 command in the wrong folder... If someone does the same mistake than me someday, then here's what I did to fix it :
# assuming the pocketbase executable is in /opt/pocketbase:
cd /opt/pocketbase/myproject/pb_data
# if sqlite3 is not already installed:
sudo apt-get install sqlite3
# sets the password to 123456 for a given email
sudo sqlite3 data.db 'UPDATE _admins SET passwordHash="$2a$10$xWbkSks7OiTmrri2DU6QSe6OAekcdqkOrkcI1wi4uiv9SU2HTlHoq" where<EMAIL_ADDRESS>

Again, the raw setting a hash is not needed with PocketBase v0.16+.
The admin command works because I just verified it locally and we also have automated tests for it - https://github.com/pocketbase/pocketbase/blob/master/cmd/admin_test.go#L91.
It doesn't work in your case because most likely you didn't run the executable against the correct pb_data directory.


feat(interview-prep): Converting and Transferring Rosetta problems

Checklist:

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #XXXXX

@scissorsneedfoodtoo Hey, this is the next set of problems. Can you please review this? Thank you :)

@NitronR, thank you for your patience and for porting over more Rosetta Code problems. Also, thank you @thecodingaviator for your thorough review.
I agree with a lot of the review here, and left a couple of comments with some minor suggestions. Please let us know when you're finished updating this PR.

One more note -- I think it's fine to have Wikipedia links here. A number of challenges throughout the curriculum include links to Wikipedia articles. However, we should add target="_blank" to them so they open in a new tab in case the camper is in the middle of coding.

@scissorsneedfoodtoo @thecodingaviator I have done most of the changes and added some minor suggestions. Please take a look and let me know your views.

Hi @NitronR, thank you making those changes. I hope you don't mind, but I pushed a commit to clean up some very minor spacing issues. Otherwise, the content is the same.
Everything LGTM locally. @thecodingaviator, could you take another look at this when you get the chance?

Awesome, thank you for taking another look at this @thecodingaviator and for your detailed review. Also thanks to @NitronR for all of your work porting these Rosetta Code challenges over. Looking forward to your next PR :+1: :+1:

@thecodingaviator @scissorsneedfoodtoo Thank you for your hard work. I will get the next set of problems ready :)


User/Passwd

HI community,
What is de default username and password to log into the control center ??.
BR,
Edgar

I have already seen that with the OS user I can log in !!
Thanks in advance

You can login to serviced with OS user root or a user which belongs to serviced group.
The playbook is configuring serviced user, but the password is not set by default.
You can do it by uncommenting the line 16 at https://github.com/sergevs/ansible-serviced-zenoss/blob/master/group_vars/serviced, recommend to provide your own password hash.


Patch fivemat progress to use chalk

mocha has switched to use 'chalk' for colors instead of color codes however our project reporter was not updated meaning that our test execution was interrupted by internal errors somewhere at the very beginning (which explains why test execution time on the CI server suddenly dropped from 10 to 2 minutes). PR to 'fivemat-progress-reporter' is already submitted however as a temporary measure we can apply this PR

Better option will be to fix mocha version to the last known good


Errors in Observable.create arg not caught

RxJS version:
5.0 beta 10
Code to reproduce:
Observable.create(() => { throw new Error('oops'); }).subscribe({error: _ => console.log('got it')});

Expected behavior:
> got it
Actual behavior:
Error: oops
Additional information:
Rx 4 exhibits the expected behavior here (catching the error and sending it to the observer).
cc @hansonw

@matthewwithanm hm, good catch. I think I can get a PR for this ready soon.

hi! I can still reproduce this with the latest stable RxJS as of now (5.4.3). Is d4a9aac7fa151695fe5567192f370164b001501b a breaking change targeted at 6.0 only or will this patch land in v5 at some point?

@wbinnssmith I would expect this should go out in a patch release. @benlesh thoughts?


Hello, i have a problem when run: cordova run browser -- --lr

$ cordova run browser -- --lr
Before deploy hook started...
Checking is node modules installed...
Node modules already installed.
Error happened on main chain:
{ Error: Cannot find module 'lodash/assign'
at Function.Module._resolveFilename (module.js:555:15)
at Function.Module._load (module.js:482:25)
at Module.require (module.js:604:17)
at require (internal/module.js:11:18)
at Object. (D:\work\mobile\soccer\node_modules\cheerio\lib\options.js:1:76)
at Module._compile (module.js:660:30)
at Object.Module._extensions..js (module.js:671:10)
at Module.load (module.js:573:32)
at tryModuleLoad (module.js:513:12)
at Function.Module._load (module.js:505:3) code: 'MODULE_NOT_FOUND' }
(node:3152) UnhandledPromiseRejectionWarning: Error: Cannot find module 'lodash/assign'
at Function.Module._resolveFilename (module.js:555:15)
at Function.Module._load (module.js:482:25)
at Module.require (module.js:604:17)
at require (internal/module.js:11:18)
at Object. (D:\work\mobile\soccer\node_modules\cheerio\lib\options.js:1:76)
at Module._compile (module.js:660:30)
at Object.Module._extensions..js (module.js:671:10)
at Module.load (module.js:573:32)
at tryModuleLoad (module.js:513:12)
at Function.Module._load (module.js:505:3)
(node:3152) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:3152) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Slightly off track from your issue but I too am seeing DeprecationWarning: Unhandled promise rejections are deprecated whenever I have a JavaScript compilation error
Are we missing a catch somewhere that is now needed for newer Node?

Something similar here, after run cordova run browser -- --lr i can see and use the web at http://localhost:8081 but i have that two issues at the end of the loading.

node: v8.9.4
cordova: 8.0.0

$ cordova run browser -- --lr
[...]
webpack: Compiled successfully.
Cordova hook completed. Resuming to run your cordova command...
Unable to load PlatformApi from platform. Error: Cannot find module 'C:\app_phonegap\platforms\browser\cordova\Api.js'
(node:17660) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Error: Unhandled "error" event. (The platform "browser" does not appear to be a valid cordova platform. It is missing API.js. browser not supported.)
(node:17660) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Hey guys before running could you check your npm, node and cordova versions?
Npm version: 5.6.0
Node version: 8.9.4
Cordova version: 8.0.0
cordova-android version: 7.0.0
cordova-browser version: 5.0.3

Actually i have this versions:
Npm:     5.6.0
Node:    8.9.4
Cordova:    8.0.0
Cordova-android:    7.0.0
Cordova-browser:   4.1.0

I try to update my Cordova-browser version and get the same error
C:\project\cordova platform update browser
Using cordova-fetch for<EMAIL_ADDRESS>Updating browser project...
Unable to load PlatformApi from platform. Error: Cannot find module 'C:\project\platforms\browser\cordova\Api.js'
(node:9852) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Error: Unhandled "error" event. (The platform "browser" does not appear to be a valid cordova platform. It is missing API.js. browser not supported.)
(node:9852) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

hey, @ralbear try this:
Remove the platform which you try to run on it (browser, ios, android, and etc.)
cordova platform rm browser
Add the platform again
cordova platform add browser
You can run
cordova run browser -- --lr

Yep, i try that but it gives some related errors, i come back with more info when i have it working!

Finally i start from scratch, create a fresh new cordova project based in this repo.
The cordova run browser -- --lr works fine.
This method still droping this error but all looks fine.
(node:52024) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
To make the android emulator works, finally i must run the emulator from android studio and when i have it up. I run cordova run android and then the app gets packed, the apk generated and then launched in the emulator.
This is not the perfect solution because i have not the live reload in android but is ok for my actual project.

Hey @ralbear some people had the same problem. We fixed that changing our webpack.config.js
From ->
{test: /\.vue$/, loader: 'vue-loader', options: {loaders: {js: {loader: 'babel-loader', options: {presets: ['env'], plugins: ['transform-object-rest-spread']}}}}}, {test: /\.js$/, exclude: /node_modules\/(?!(framework7|framework7-vue|template7|dom7)\/).*/, use: {loader: 'babel-loader', options: {presets: ['env'], plugins: [ 'transform-runtime', 'transform-object-rest-spread' ]}}}
To ->
{test: /\.vue$/, exclude: /node_modules/, loader: 'vue-loader', options: {loaders: {js: {loader: 'babel-loader', options: {presets: ['env'], plugins: ['transform-object-rest-spread']}}}}}, {test: /\.js$/, exclude: /node_modules(\/|\\)(?!(framework7|framework7-vue|template7|dom7)(\/|\\)).*/, use: {loader: 'babel-loader', options: {presets: ['env'], plugins: [ 'transform-runtime', 'transform-object-rest-spread' ]}}}

With this i must find solved the (node:52024) [DEP0018] DeprecationWarning: [...] or the android enviroment run? After updating my webpack.config.js i have the same problems.

Just to make your android enviroment running because like others you are using windows, could you show me the error when you are using android?

When i run cordova run android -- --lr i only get this extra error, at the middle of the log
[...]
Starting a Gradle Daemon (subsequent builds will be faster)
publishNonDefault is deprecated and has no effect anymore. All variants are now published.
Configuration 'compile' in project ':app' is deprecated. Use 'implementation' instead.
The Task.leftShift(Closure) method has been deprecated and is scheduled to be removed in Gradle 5.0. Please use Task.doLast(Action) instead.
        at build_esumd79ylzhrq8b8aw89zqt7i.run(M:\app_beta\platforms\android\app\build.gradle:144)
:CordovaLib:preBuild UP-TO-DATE
[...]

when the process finish the app-debug.apk package was correctly created but the android enviroment don´t start by itself.
[...]
:app:packageDebug
:app:assembleDebug
:app:cdvBuildDebug

BUILD SUCCESSFUL in 37s
47 actionable tasks: 3 executed, 44 up-to-date
Built the following apk(s):
        M:\app_beta\platforms\android\app\build\outputs\apk\debug\app-debug.apk
Copying platform cordova files...
(node:17772) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Error: EISDIR: illegal operation on a directory, link 'M:\app_beta\platforms\browser\www\favicon.ico' -> 'M:\app_beta\www\platform_cordova_files\browser\favicon.ico'
(node:17772) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Also i can move the app-debug.apk to the android emulator and it works fine.

I got a similar error when my android paths were wrong could you check your android paths? I don't know how to use that on windows platform.

another thing you could try starting from scratch using my last release:
https://github.com/caiobiodere/cordova-template-framework7-vue-webpack/releases/tag/2.0.5

The android paths are ok, i have issues with this when i install it. I´m not sure if can be a incompatibility with the new Android Studio version.
I come back with my feedback when i try the new release :)

hey @ralbear how was it?

I have the same error with @ralbear too.
When I run cordova build android:
`
Before deploy hook started...
Checking is node modules installed...
Node modules already installed.
Error happened on main chain:
{ Error: Cannot find module 'lodash/assign'
at Function.Module._resolveFilename (module.js:536:15)
at Function.Module._load (module.js:466:25)
at Module.require (module.js:579:17)
at require (internal/module.js:11:18)
at Object. (/Users/ouxuan/CODE/sea_code/nfcDev/node_modules/cheerio/lib/options.js:1:76)
at Module._compile (module.js:635:30)
at Object.Module._extensions..js (module.js:646:10)
at Module.load (module.js:554:32)
at tryModuleLoad (module.js:497:12)
at Function.Module._load (module.js:489:3) code: 'MODULE_NOT_FOUND' }
Error: Cannot find module 'lodash/assign'
`
When I run cordova platform rm android:
`
Removing platform android from config.xml file...
Error: npm: Command failed with exit code 1 Error output:
Cannot download "https://github.com/sass/node-sass/releases/download/v4.8.3/darwin-x64-57_binding.node":
ETIMEDOUT
Timed out attemping to establish a remote connection
`

And when I create a new project, and do it again, it works! This time , there is no errors anymore!

I think that's closed now


Firefox xpi request

Hi! I use your app on chrome and it works great for me on that browser.
Is it possible to get the extension verified so it can be installed properly on FIrefox too?

the screen goes black and freeze in the miniplayer mode in Edge when clicking on a new video

@44ron
Verified Extension: https://addons.mozilla.org/en-GB/firefox/addon/youtube-adblocker-free/

@44ron
Verified Extension: https://addons.mozilla.org/en-GB/firefox/addon/youtube-adblocker-free/

Thank you so much!


Not load plugins

Problem:
Cant load use: [
'plugin'
]
Solved:
I just remove line saying... delete md.use
I hope this people resolve this problem in code, bye!

There is indeed a<EMAIL_ADDRESS>However, it has some strange syntax, I guess it's JSP. Do you know how to use it?
I tried to put it in the plugins for NuxtJS, but it refused the syntax.


docs(samples): included snippet for unmute finding

Added snippet to unmute individual findings.

I have only reviewed it as a shadow-reviewer. We would still need a review from @averikitsch to ensure its still good to go


Fast feedback (chat)

Is posible create chatroom for fast feedback, like on gitter.im or discrod? For "all" projects.
Example for fast report:
Missing translation string:
- `We could not {action} {package}. Please try again later. Click on \"{showDetails}\" to get the logs from the installer.`

please email me at<EMAIL_ADDRESS>to discuss that

Missing string is relevant issue, sorry, if it was not clear.
We could not {action} {package}. Please try again later. Click on \"{showDetails}\" to get the logs from the installer.

@marticliment

ah that was real, I thought it was an example

please email me so. I can give you my telegram, I don't want to post it here publicly

please email me so.  Thanks


Question

Although I created my page using your example, I can't select item by mouse only using keyboard. Is documentation correct?

Thanks to @njdowdy for the short-term workaround. This is related to Command component in shadcn-ui.
Adjusting the line 120 in shadcn's Command.tsx (which is CommandItem's className) to:
"relative flex cursor-default select-none items-center rounded-sm px-2 py-1.5 text-sm outline-none aria-[selected='true']:bg-accent aria-[selected='true']:text-accent-foreground data-[disabled='true']:pointer-events-none data-[disabled='true']:opacity-50",
This is the solution


CHIP writes should be queued for sleepy devices on the iOS framework

Problem
Sleepy devices cannot accept writes, we should queue these up.
Proposed Solution
Come up with a queue for reads/writes

Stale?
Or potentially a duplicate of #11554 (read queue) and #11555 (write queue).


fix: use fixed version for rspack in template

Summary
Use fixed version instead of latest in template of create-rspack

Test Plan

Require Documentation?

[x] No
[ ] Yes, the corresponding rspack-website PR is __

!canary

Maybe the Rspack release CI should update the version in template automatically.


feat: dynamic flow control for batcher part 2

Implementation of go/veneer-dynamic-flow-control part 2.
Adding a DynamicFlowControlSetting and new aPIs in FlowController so the flow control limits in the FlowController can be adjusted.

@igorbernstein2 The second part is ready for review. This part adds the DynamicFlowControlSettings and makes flow control limits in the FlowController adjustable. Thanks! :)

@vam-google I'm going to merge this, if you still have concerns after @mutianf responses, we can open follow up PRs.


(ios) Update README.md

Apple complaints on missing NSLocationAlwaysUsageDescription key in "*-Info.plist" file.

Platforms affected
ios
Motivation and Context

Apple won't approve the app otherwise

Description

Added info and example about how to add an additional key with string value to config.xml that is added to the app's Info.plist file upon build.
Testing

When the key is not present in the config.xml, Apple sends an email with info about a missing key in the app's Info.plist file when you upload the archive file to Apple Store.
When including the key NSLocationAlwaysUsageDescription according to my description, Apple does not complain anymore.
Checklist

[ ] I've run the tests to see all new and existing tests pass
[ ] I added automated test coverage as appropriate for this change
[x] Commit is prefixed with (platform) if this change only applies to one platform (e.g. (android))
[ ] If this Pull Request resolves an issue, I linked to the issue in the text above (and used the correct keyword to close issues using keywords)
[x] I've updated the documentation if necessary

Since iOS 13 was released, Apple has rejected two of my non Cordova apps for requesting Allow Always permission when it was not needed.  In both cases I was using geofencing so Apple approved the apps after I explained my use case.
In this plugin, the Allow Always permission is a fallback for when the NSLocationWhenInUseUsageDescription is missing in the plist file.  A better solution would be to get rid of the fallback and just ask users to ensure they specify the NSLocationWhenInUseUsageDescription permission string.
This is the code I am referring to in CDVLocation.m starting at Line 119:
#ifdef __IPHONE_8_0
    NSUInteger code = [CLLocationManager authorizationStatus];
    if (code == kCLAuthorizationStatusNotDetermined && ([self.locationManager respondsToSelector:@selector(requestAlwaysAuthorization)] || [self.locationManager respondsToSelector:@selector(requestWhenInUseAuthorization)])) { //iOS8+
        __highAccuracyEnabled = enableHighAccuracy;
        if([[NSBundle mainBundle] objectForInfoDictionaryKey:@"NSLocationWhenInUseUsageDescription"]){
            [self.locationManager requestWhenInUseAuthorization];
        } else if([[NSBundle mainBundle] objectForInfoDictionaryKey:@"NSLocationAlwaysUsageDescription"]) {
            [self.locationManager  requestAlwaysAuthorization];
        } else {
            NSLog(@"[Warning] No NSLocationAlwaysUsageDescription or NSLocationWhenInUseUsageDescription key is defined in the Info.plist file.");
        }
        return;
    }
#endif

Getting rid of the call to requestAlwaysAuthorization will save developers unnecessary rejections from Apple.

I think this might be conflicted because of https://github.com/apache/cordova-plugin-geolocation/pull/181 which I believe adds the same documentation notice. @yze77 are you able to confirm?

@breautek It seems so, and it also introduces a third flag NSLocationAlwaysAndWhenInUseUsageDescription so I'm closing this pull request.

Anyways, thanks for your contribution!


Bot starts breifly, then stops abruptly.

Hi, I had hosted this bot roughly a year ago but lost the time to continue to manage it. Today I bought a new VPS with the aim of hosting the bot again. After doing the necessary installations, which I had to remind myself how to do, I've come into a problem. When I start the bot, it says that the bot starts well and asks for me to invite it to a Discord server and all, but I can see the bot go online and then offline almost instantly.
I'm not sure what is causing this.

The bot is offline until you start casting to it. Did you try joining a voice channel and casting from Spotify?

Oh my god, you're correct. The bot does work now. I was oblivious to that. Sorry, and thank you!

Glad it's working now, partly my fault because it's non-obvious behavior. Will update the readme.


partititioning error

I'm getting an error
MFEM_VERIFY((*offsets[i])[0] >= 0 && (*offsets[i])[1] >= 0,"overflow in offsets")
at line 1260 in pmesh.cpp
What does this mean?

Are you solving a big problem with >2G unknowns? Maybe you need to compile
hypre with --enable-big-int.
Jakub
On Thu, Apr 12, 2018 at 7:37 PM, Dan2997925<EMAIL_ADDRESS>wrote:

I'm getting an error
MFEM_VERIFY((*offsets[i])[0] >= 0 && (*offsets[i])[1] >= 0,"overflow in
offsets")
at line 1260 in pmesh.cpp
What does this mean?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mfem/mfem/issues/473, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABSW-2cQwT43qLSdUQgGpN6F7R9pWSyxks5tn5D0gaJpZM4TSNXv
.

Yes it’s a big problem. My mesh is smaller than 2G elements. But it is an elasticity problem so the displacement field could perhaps be exceeding 2G.
When is this function being called? Have I successfully partitioned and distributed and refined the mesh before I get to this point?
I don’t know if we are using –enable-big-int or not I’ll look in to it.
From: Jakub Červený<EMAIL_ADDRESS>Sent: Thursday, April 12, 2018 10:53 AM
To: mfem/mfem<EMAIL_ADDRESS>Cc: White, Dan<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [mfem/mfem] partititioning error (#473)
Are you solving a big problem with >2G unknowns? Maybe you need to compile
hypre with --enable-big-int.
Jakub
On Thu, Apr 12, 2018 at 7:37 PM, Dan2997925 <notifications<EMAIL_ADDRESS>wrote:

I'm getting an error
MFEM_VERIFY((*offsets[i])[0] >= 0 && (*offsets[i])[1] >= 0,"overflow in
offsets")
at line 1260 in pmesh.cpp
What does this mean?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mfem/mfem/issues/473, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABSW-2cQwT43qLSdUQgGpN6F7R9pWSyxks5tn5D0gaJpZM4TSNXv
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/mfem/mfem/issues/473#issuecomment-380890687, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AXZLEu1y4MrL6CbX1iP618BD-ThvUoA5ks5tn5RegaJpZM4TSNXv.

This code is usually called when constructing the FE space, to determine
the DOF partitioning. I would guess the mesh is OK, the space typically
overflows first.
Jakub
On Thu, Apr 12, 2018 at 8:02 PM, Dan2997925<EMAIL_ADDRESS>wrote:

Yes it’s a big problem. My mesh is smaller than 2G elements. But it is an
elasticity problem so the displacement field could perhaps be exceeding 2G.
When is this function being called? Have I successfully partitioned and
distributed and refined the mesh before I get to this point?
I don’t know if we are using –enable-big-int or not I’ll look in to it.
From: Jakub Červený<EMAIL_ADDRESS>Sent: Thursday, April 12, 2018 10:53 AM
To: mfem/mfem<EMAIL_ADDRESS>Cc: White, Dan<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [mfem/mfem] partititioning error (#473)
Are you solving a big problem with >2G unknowns? Maybe you need to compile
hypre with --enable-big-int.
Jakub
On Thu, Apr 12, 2018 at 7:37 PM, Dan2997925<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:

I'm getting an error
MFEM_VERIFY((*offsets[i])[0] >= 0 && (*offsets[i])[1] >= 0,"overflow in
offsets")
at line 1260 in pmesh.cpp
What does this mean?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mfem/mfem/issues/473, or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABSW-
2cQwT43qLSdUQgGpN6F7R9pWSyxks5tn5D0gaJpZM4TSNXv>
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<https://github.com/
mfem/mfem/issues/473#issuecomment-380890687>, or mute the thread<
https://github.com/notifications/unsubscribe-
auth/AXZLEu1y4MrL6CbX1iP618BD-ThvUoA5ks5tn5RegaJpZM4TSNXv>.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/mfem/mfem/issues/473#issuecomment-380893746, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABSW-yhgA788rB6nKtJnVylTBhHEM-kcks5tn5bFgaJpZM4TSNXv
.

OK this makes sense. So I would need to rebuild hypre, and mfem, right?

That's right.
On Thu, Apr 12, 2018 at 8:11 PM, Dan2997925<EMAIL_ADDRESS>wrote:

OK this makes sense. So I would need to rebuild hypre, and mfem, right?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/mfem/mfem/issues/473#issuecomment-380896426, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABSW-0p2K9Y_QzPPDCIxvz8PZLaBqhVPks5tn5jrgaJpZM4TSNXv
.


Add variant methods to SingletonContext and Context

Add VisitAllModuleVariants, PrimaryModule, and FinalModule to
SingletonContext so singletons and tools can deal with modules with
multiple variants.

LGTM


Add Lomap conda package

@richardjgowers I need to make a release/tag for conda-forge, should I start at 0.1 or start somewhere else?

I'd vote 0.1 (or even 0.0.1). We're not promising a stable API in Lomap at this point, are we? If we want the first "fully-ready" version to be 2.0, we can always skip ahead to that.

I like 0.0.1 since it says "here be dragons" and I agree, we can jump versions once we are happy with it

I guess the question is "how much development of Lomap are we offering"?
As far as I understand our brief was "get Lomap working" and that was it. If that's the case then we've hit our brief and this is version 1.0.0.

There's also the fact that versions existed before us, so this is sort of v2-beta (we've broke stuff relative to v1 and this isn't everything we're breaking for v2)

I guess it is @richardjgowers call, are we at 1.0.0 or do we have some dev to do? I just need a tag/release to build the conda-forge package, so if we can decide on something else later if we want. 2.0.0-pre maybe? or something?

This issue will be resolved once https://github.com/conda-forge/staged-recipes/pull/18154 is merged

Added!
https://github.com/conda-forge/staged-recipes/pull/18154


Volumes-from does not work with network shares

docker version:
Client:
 Version:      1.10.0
 API version:  1.22
 Go version:   go1.5.3
 Git commit:   590d5108
 Built:        Thu Feb  4 18:41:30 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.10.0
 API version:  1.22
 Go version:   go1.5.3
 Git commit:   590d5108
 Built:        Thu Feb  4 18:41:30 2016
 OS/Arch:      linux/amd64

docker info:
Containers: 11
 Running: 10
 Paused: 0
 Stopped: 1
Images: 28
Server Version: 1.10.0
Storage Driver: btrfs
 Build Version: Btrfs v4.0
 Library Version: 101
Execution Driver: native-0.2
Logging Driver: json-file
Plugins:
 Volume: local
 Network: bridge null host
Kernel Version: 4.2.0-27-generic
Operating System: Ubuntu 15.10
OSType: linux
Architecture: x86_64
CPUs: 8
Total Memory: 15.56 GiB
Name: isoly
ID: NNYS:HTN6:3KVB:6XRX:MZD5:SJFM:QVEB:CEVX:BBMT:EZSL:3PB7:VENT
Username: olymk2
Registry: https://index.docker.io/v1/
WARNING: No swap limit support

uname -a
Linux isoly 4.2.0-27-generic #32-Ubuntu SMP Fri Jan 22 04:49:08 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
I have setup two container, one is setup to be a data volume inside that container I have mounted two network shares,
/etc/fstab
ipaddress:/srv/export/filestore/shared_app_data /srv/shared_app_data  nfs defaults 0 0
ipaddess:/srv/export/filestore/shared_app_data /mnt/shared_app_data  nfs defaults 0 0

If I jump inside the container with exec I can see the contents of the container.
In a second container I have mounted the volumes from the container above which should pull in /mnt/shared_app_data and /srv/shared_app_data and in fact it does mount them in the new container how ever these folders are completely blank this has been the case with the current and previous versions
I have created and launched both container with the --priviledged flag, I am sure this used to work on an older version but i have been unable to resolve this issue is it a bug ?

This is because mounts created in a container are not seen outside the container unless you do something like -v /foo:/foo:shared.  This allows the mounts to propagate to the host and as such be shared with other containers.

@cpuguy83 thanks for the suggestion can you elaborate a little, if I have my data_container with the above mounts mounted inside it how do I then expose it to the second container ?
I though -v was for mapping folder on the host to inside the container not for mapping a folder inside one container to another ? I thought thats what --volumes-from was for ?
so how do i map /srv/shared_app_data mounted inside the data_volume to a new container ?

in further investigation, I think this needs to be reopened this has nothing to do with folders on the host so using -v /foo:/foo:shared will not solve the issue.
The docks say that -v is to mount from the host into the container

-v|--volume[=[[HOST-DIR:]CONTAINER-DIR[:OPTIONS]]]
Create a bind mount. If you specify, -v /HOST-DIR:/CONTAINER-DIR, Docker
bind mounts /HOST-DIR in the host to /CONTAINER-DIR in the Docker
container. If 'HOST-DIR' is omitted,  Docker automatically creates the new
volume on the host.  The OPTIONS are a comma delimited list and can be:

the volumes-from does not take a shared parameter, as I believe this is a container to container issue can we reopen the issue @cpuguy83 or shall i create a new one ?

I think you are misunderstanding.
You cannot share mounts between two containers unless the volume the mounts are in uses the "shared" flag.
A mount created in one container will not propagate otherwise.

@cpuguy83 I think I get you so my data volume has
VOLUME /srv/shared_app_data
VOLUME /mnt/shared_app_data

can I make that shared some how ? the format above assumes I am mounting from my host or is this only possible with a volume that lives outside the container ?
Basically what I wanted was a container which has my mounts which I can then make available to all other containers because the apps in the other container all rely on these file on the network share.

@olymk2 This is only available on host-mounted volumes.
Alternatively, you may want to use a plugin.

@cpuguy83 thanks perhaps i should put it in as a feature request, I was hoping to make it so every developer would not have to manually setup and mount the shares and then expose them into the container.
Thanks for clarifying


False positive for LC0033

I've just updated my app platform and application from 23.0 to 24.0, and I've updated the runtime to 13.0 accordingly.
I get the following info LC0033:

The specified runtime version in app.json is falling behind. The project targets application version 24.0 and the current runtime is 13.0. Update the runtime to 13.1 for compatibility with the latest runtime features. AL LC0033

Why does it suggest 13.1?

Under the hood I'm depending on the same logic as the altool, which suggest with the GetLatestSupportedRuntimeVersion that Platform version 24.0 supports runtime version up to 13.1.

If i look into the LatestSupportedRuntimeVersions  of the Microsoft.Dynamics.Nav.CodeAnalysis.dll, it could be that there's been a misstake with the latest 13.1 release of the AL Language?

public static readonly Version Spring2024 = new Version(24, 0);
public static readonly Version Spring2024CU1 = new Version(24, 1);
public static readonly Version Spring2024CU2 = new Version(24, 2);
public static readonly Version Spring2024CU3 = new Version(24, 3);
public static readonly Version Spring2024CU4 = new Version(24, 4);
public static readonly Version Spring2024CU5 = new Version(24, 5);

public static readonly Version RuntimeVersion.Spring2024 = new Version(13, 0);
public static readonly Version RuntimeVersion.Spring2024RV1 = new Version(13, 1);

I've created an issue on the AL repo to investigate: https://github.com/microsoft/AL/issues/7786

Good and bad news: The good news is that it's not a false positive if you're targeting SaaS, but that's also kinda the bad news.
The current implementation of this rule has now become 'SaaS only' and I'm not sure if this rule can cover both targeting On-Prem and SaaS anymore. There was once a "target":"OnPrem" property in the app.json, but with the Universal Code Initiative I believe this isn't a thing anymore. So I don't think I have any way of knowing if the extension targets a SaaS or On-Prem environment.
I'm thinking of re-branding this rule that it's only SaaS-compatible, where I don't have a solution for On-Prem environment at the moment. This isn't ideally, because during development it shouldn't matter what the target environment is.
An other approach would be to create a own mapping inside the LinterCop itself, where this is sensitive to maintenance.

Thanks Arthur. I've followed Microsoft's explanation on that, and to be honest, I still don't understand their explanation 😁
Anyway ... The apps I'm writing are written for the cloud, but can (and will be) installed on-premises as well. As I understood, the correct platform then must be 13.0 due to on-premises. From my perspective, LC0033 should be inactive as long as Microsoft does not provide a function for on-premises.

For now I'm reverting back to the previous logic of this rule, where the version of the platform and the runtime should be a offset of eleven (with some more logic behind it, then simply subtracting the value of 11).

The version v0.30.28 of the LinterCop is now the latest release. Could you verify if this is now working as expected? If this is the case, you may close this issue.

Thanks Arthur! I no longer receive LC0033 on runtime 13.0. Reducing it to 12.0 will raise it again - it works as expected :-)


[Feature Request] New commands

Is your feature request related to a problem? Please describe.

maestro upgrade

it would automatically update the new version: curl -Ls "https://get.maestro.mobile.dev" | bash
In case your version is updated to the latest.
Congrats! You're already on the latest version of Maestro (which is v1.34.1)

maestro update
It should tell you if there is a more up-to-date version than yours.

Hey, thanks for creating an issue. This sounds like something that should be a responsibility of a package manager. So I'll consider it a duplicate of #1513


WIP: Update test app

Goal
Update the test app with iOS 6.x changes

Closing for now due to the conflicts with main, it may be easier to remove this in favour of easier templating to create new testing apps under integration_testing/


Avoid re-flattening in jit() when no donate_argnums are present.

Following the same special-casing of static_argnums, this should provide a speedup specially when the number of arguments provided is large.
Resubmission where donate_tuple is the right name in pmap's case so that it is a proper tuple.

I have checked that the internal tests pass.

This is a 2nd attempt for #3945.


Multiple versions of adb can fight each other, breaking the service protocol

Splitting off from:  https://github.com/flutter/flutter/issues/8876

We should probably detect this and report it, at least.


WSL will not start after recent update due to network error

Windows Version
Microsoft Windows [Version 10.0.22621.2792]
WSL Version
WSL version: <IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>-1
Distro Version
Ubuntu (unsure of version, cannot start WSL to verify)
Other Software
No response
Repro Steps
This error happens when trying to start WSL. No additional steps are required. I have none of the Hyper-V Windows features enabled, but did confirm that Virtual Machine Platform is enabled.
Expected Behavior
I expect WSL start up without an error.
Actual Behavior
Failed to configure network (networkingMode Nat). To disable networking, set wsl2.networkingMode=None in C:\Users\brett.wslconfig
Error code: Wsl/Service/CreateInstance/CreateVm/ConfigureNetworking/HNS/ERROR_UNHANDLED_EXCEPTION
Diagnostic Logs
I have a zip of the networking logs that I collected using the steps here  but the file is too large (cap is 25MB and my zip is 27MB). Is there another place I can upload? Or can you share which of the files would be OK to remove? Looks like the largest file in there is logs.etl and the next largest is pktmon.etl.

@brettw1234: If the .zip file is too big, you can try to upload it OneDrive / Gdrive

@brettw1234: If the .zip file is too big, you can try to upload it OneDrive / Gdrive

Ok thanks - didn't know if you guys had issues using an external hosting platform. Here is a link to the logs for my issue.

Hello, have you resolved this issue? I have also encountered the same problem.

Hello, have you resolved this issue? I have also encountered the same problem.

Nope. I've just disabled networking in order to continue using my WSL2 instances, but I would love to have the issue resolved!

Just disable all related features other than "Virtual Machine Platform".

I'm not sure if it this is related, but just a few weeks after reporting this issue the networking on my entire Windows install broke. The Windows DnsClient service refused to start due to an "Access Denied" error, and no amount of debugging got that working. I now have a new Windows install and everything seems to be working again.

I'm not sure if it this is related, but just a few weeks after reporting this issue the networking on my entire Windows install broke. The Windows DnsClient service refused to start due to an "Access Denied" error, and no amount of debugging got that working. I now have a new Windows install and everything seems to be working again.

I don't think so, the original problem was that the virtual switch cannot be created, which would not affect the host's network status. And reinstallation will always work for such problems.

Sorry - I missed this report. This is failng deep in virtual switch. As of the information we have I don't know what is causing the failure :(
I would recommend 2 things:

check to see if you have 'ghost' adapters and remove them
---> from Device Manager, click "View" --> "Show hidden device" --> refresh and expand "Network adapters" --> if there are any 'ghosted'/'hidden' adapters, uninstall them.
I would just remove vswitch completely and try reinstalling. this would mean removing all "HyperV" optional components, "Virtual Machine Platform" optional component, "Windows Sandbox" optional component, and "MDAG" or "Microsoft Device Application Guard" (all of these require vswitch)
---> if you do this, you'll of course need to backup any virtual machines that you have.

Hope this helps.

I delete the file .wslconfig in C:\Users{USER_NAME} and WSL back into working state.