rcParams-like for styling to circumvent the gui

In the context of the workflow for snapping shots of isosurfaces: It is easy enough to save the canvas as a png when it is set up nice. Now say I have my set of defaults; the camera position, the field index, the material of choice for atoms (spheres) and the material of choice for surfaces, etc. These are not the defaults in the GUI and as such require manual setting each time a universe is loaded. This precludes automation of the process by requiring input each time. We should think about how to programmatically (sp?) set these values before the rendering of a universe.
Although I am not sure this solves the automation problem, it would certainly be useful regardless of this use case.

PS -- maybe this belongs in exa?

Possibly; but it is also application dependent. I would vote for developing the gui further - options for cam pos, material choices, etc. and then be able to specify the defaults via the _widget traits on the python side (with suitable defaults like we have now). With that infrastructure in place we can have params like options.
As an example, in my custom branch, I am using an attribute on the container called display which is a dictionary that has some options such as which coordinates I want to visualize (atom, visual_atom, projected_atom). This could be expanded to include default material choices, coordinate axis, etc.

Okay cool I will model my use case after your example. How does access an issue once it's closed?

You can filter issues by "is:issue is:closed". I don't know if that is the best idea, but from a pythonic perspective, following the dictionary model of rcParams makes sense (though I wouldn't call it rcparams). If you think of a better play post it.

Commit #804194c59f8e4bd0b09ea3b959e03fff763e3c60 has a template of how container display parameters may work.


Option to mute extension notifications

Issue Type: Feature Request
A lot of extension authors think it is a good idea to raise a notification for each operation we do on their extensions (for example: TT-Processor raises two notifications for every T4 template being processed, resulting in a huge spam of notifications that we're forced to dismiss manually).
It would be awesome if we could just silence all notifications from a certain extension (the notification already knows what extension invoked it, so it should not be hard).
Of course, we should be able to choose what kind of notification we want do dismiss: information, warning or error. In my case (TT-Processor), warning is used as a progress "I'm transforming your file", info is used for conclusion "I have transformed your file" and errors shows T4 compilation errors (that I would mantain, but the same output goes to OUTPUT window, so, potato potato =P)
VS Code version: Code 1.46.1 (cd9ea6488829f560dc949a8b2fb789f3cdc05f5d, 2020-06-17T21:13:20.174Z)
OS version: Windows_NT x64 10.0.19041

/duplicate of #41767


Home page: allow burger menu links to open to different tabs

Currently, clicking on a link from the burger menu to open a page from the Gitbook, and then going back to crDSR and clicking another link loads the Gitbook page in the same tab as the previous link. This should be opened in another tab.

Decided to keep it like this


is the estimate of Wasserstein distance negative?

In continuation with the discussion in issue #9, I'm still missing something about the critic objective.
Isn't Loss_D supposed to be the approximation of the Wasserstein distance? Then, if the critic is trained to optimalilty, the distance should be at least positive.
Related to this, the actual objective of the critic is going up when training. Does this mean that the estimate of the Wasserstein distance is not improving? Should the picture be the same if we train the critic to optimality at each iteration?

Loss_D is the Wasserstein estimate. You need to plot - Loss_D

Dang, github converted my minus sign into an item list. - Loss_D is the Wasserstein estimate and what you should plot. Considering what you said that curve will be positive and going down as expected.

Thanks, finally understood what actually happens.


ERROR:  tuple already updated by self

I getting after a fresh init again this errors (with 0.9.6 I had this already) when I reach the 700-800 Mio. record threshold  in the cont. view. After this the cont. view starts loosing massiv data during copy comamnds into the the stream. Before this threshold everything is fine.
2017-02-03 13:56:46 CET;;759;;2017-02-03 09:35:04 CET;16482STATEMENT:  raw_data_con_view
2017-02-03 13:57:04 CET;;860;;2017-02-03 09:35:04 CET;16481ERROR:  tuple already updated by self
2017-02-03 13:57:04 CET;;861;;2017-02-03 09:35:04 CET;16481STATEMENT:  raw_data_con_view
My cont. view looks like.
     Continuous view "public.raw_data_con_view"
Column     |  Type   | Modifiers | Storage | Description

---------------+---------+-----------+---------+-------------
day           | date    |           | plain   |
views         | bigint  |           | plain   |
user_id       | bigint  |           | plain   |
cid   | integer |           | plain   |
wid | integer |           | plain   |
View definition:
SELECT dt::date AS day,
count(*) AS views,
user_id,
cid,
wid
FROM ONLY raw_data_stream
GROUP BY (dt::date), user_id, cid, wid;
Indexes:
"raw_data_con_view_mrel_pkey" PRIMARY KEY, btree ("$pk")
"ix_raw_data_day" btree (day)
"ix_raw_data_user_id" btree (user_id)
"raw_data_con_view_mrel_expr_idx" btree (ls_hash_group(day, user_id, cid, wid))
                                                                    pipeline_version

PipelineDB 0.9.6 at revision 72b0c7fb25c2361399e17fde7eaf013b5f0dd7de on x86_64-pc-linux-gnu, compiled by gcc (Ubuntu 5.4.0-6ubuntu1~16.04.4) 5.4.0 20160609, 64-bit

Hi @nnfuzzy, a few questions:

Do you see this error if you (temporarily) drop the ix_raw_data_day and ix_raw_data_user_id indexes?
Can you share some example data along with the COPY statements you're using?

Hi @derekjn,

I take the index off , but it doesn't work.
update pg_index set indisvalid = false where indexrelid = 'ix_raw_data_day'::regclass;
update pg_index set indisvalid = false where indexrelid = 'ix_raw_data_user_id'::regclass;

Is there an more approriate method in your mind?

COPY raw_data_stream (cid,wid,user_id,dt,network) FROM STDIN DELIMITER;

Data looks like
1912779;3773502;0;2017-02-03 02:59:59;250
1916956;3774932;0;2017-02-03 02:59:59;250
1917575;3775158;6201504425456760844;2017-02-03 02:59:59;250
1917577;3775159;6382692426906534038;2017-02-03 02:59:59;250
1917577;3775159;6382691078283065368;2017-02-03 02:59:59;250
1917577;3775159;6192173372803973180;2017-02-03 02:59:59;250
1917577;3775159;0;2017-02-03 02:59:59;250
1917577;3775159;6376474508258768923;2017-02-03 02:59:59;250
1920414;3777595;6381774008284477462;2017-02-03 02:59:59;250

I find out that the con. view is stalled, doesn't aggregate more data, The db is throwing the errors and the 2 combiners hung on 100%.

First log entry with this error.
2017-02-03 09:50:11 CET;;2;;2017-02-03 09:35:04 CET;16482ERROR:  tuple already updated by self
And now
2017-02-03 17:05:49 CET;;1436;;2017-02-03 09:35:04 CET;16481ERROR:  tuple already updated by self

Thanks

Now I restarted the db, (btw. it  needs a hard kill -9 ) but again...
2017-02-03 17:27:39 CET;;10;;2017-02-03 17:24:41 CET;24724ERROR:  tuple already updated by self
2017-02-03 17:27:39 CET;;11;;2017-02-03 17:24:41 CET;24724STATEMENT:  raw_data_con_view
2017-02-03 17:27:40 CET;;10;;2017-02-03 17:24:41 CET;24723ERROR:  tuple already updated by self
2017-02-03 17:27:40 CET;;11;;2017-02-03 17:24:41 CET;24723STATEMENT:  raw_data_con_view
2017-02-03 17:27:41 CET;;12;;2017-02-03 17:24:41 CET;24724ERROR:  tuple already updated by self
2017-02-03 17:27:41 CET;;13;;2017-02-03 17:24:41 CET;24724STATEMENT:  raw_data_con_view

@nnfuzzy are you using a TTL?

@derekjn No not know. As I saw the problem occured around 900 Mio. rows in the CT. When I find some time next days I'll see if it is possible to reproduce this with randomly generated data.

@derekjn trying to replicate this behaviour with random data  I have until now no success. However after pushing random data  into the stream (docker version) I got suddenly after  > 100Mio in the CV this.
Session terminated, terminating shell...LOG:  received smart shutdown request
LOG:  autovacuum launcher shutting down
WARNING:  a background worker crashed while processing this batch
HINT:  Some of the tuples inserted in this batch might have been lost.
CONTEXT:  COPY raw_data_stream, line 8144000: "2017-01-10 11:55:10;6000000000010586703;1133921;153727"
WARNING:  a background worker crashed while processing this batch
HINT:  Some of the tuples inserted in this batch might have been lost.
CONTEXT:  COPY raw_data_stream, line 8187000: "2017-01-05 12:45:07;6000000000065059446;1867018;138760"
WARNING:  a background worker crashed while processing this batch

@nnfuzzy thanks for the update. Can you provide a dataset (or generation script) that reproduces the worker crash so we can look at it on our end? You can send me a private message on Gittr with a download link, or attach it here if it's not confidential.
Once we can reproduce this on our side (even if it takes hundreds of millions of events), we should be able to resolve this fairly quickly.


Documentation Incomplete - Sample Data

cmdlet documentation incomplete. Contains sample data {{ Fill User Description }} {{ Fill in the Synopsis }}.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: a27412b8-ee1c-f2f1-eaf9-3d86bf3f5dcc
Version Independent ID: e6c9ea70-3774-8fcc-ed2a-bb42c49531f1
Content: Set-CsUserTeamMembers (skype)
Content Source: skype/skype-ps/skype/Set-CsUserTeamMembers.md
Service: skypeforbusiness-powershell
GitHub Login: @hirenshah1
Microsoft Alias: hirshah

@officedocsbot assign @yogkumgit

Hello @tonywoodruff Thanks for noticing and reporting, created a pr to update the article with missing descriptions.

@get-itips  Thank you very much for the contribution and updating the documentation with PR. @tonywoodruff Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Feature Request: Add name to the events config

Currently the events config are shown with "Hue" and the IP address. Would be great if it could also have a name to distinguish them as I don't remember this IP address belongs to which Hue Hub. :)

When I remember right it should be possible to name the setup config.
If not, just use a comment node, write it down and place it on top:)

@bond2k Indeed it was missing I thought it was already possible. I've released v1.6.0 that adds the possibility to set a label on both node (PR #22).

The configs already could have different names, what in my opinion was finde, but thanks.

Great, thanks.


Gorouter should emit 404 on missing instance and 400 for bad instance id header

Issue
This seems to be a regression.
As described in https://github.com/cloudfoundry/gorouter/commit/a3583dbc8553d4a0902a42d7fa9ccddf03e54c56 gorouter should return 400 on malformed X-CF-APP-INSTANCE header and 404 if the instance denoted by said header does not exist.
This is also the described behavior in https://github.com/cloudfoundry/gorouter#headers (although the 400 response is missing)
Since we rolled out release 0.198.0 to production, our clients are complaining this behavior has changed. In both cases there is 400 bad request returned. This undocumented change has thrown off some customers as they had based their B/G deployment cycles on the status codes.
Context
This happens whenever you use the X-CF-APP-INSTANCE to address a specific instance of your app.
Steps to Reproduce

Deploy an app with only one instance
Do

curl -H "X-CF-APP-INSTANCE: <app-id>:1" https://my-app.com/

Expected result
404 Not Found: Requested instance ('1') with guid (<app-id>) does not exist for route ('my-app.com')

Current result
400 Bad Request: Requested instance ('1') with guid (<app-id>) does not exist for route ('my-app.com')

Possible Fix

Restore old behavior (400 on bad header, 404 on missing instance)
Update documentation

Slack thread

Relevant section in docs: https://docs.cloudfoundry.org/devguide/deploy-apps/routes-domains.html
"Routing Requests to a Specific App Instance" heading

Drive-by and perhaps vexatious comment, sorry

Relevant comment from Slack from @bruce-ricard:

The rationale was that a non existing instance in the header is in fact a malformed header. Our understanding is that 404s should only be used when the URL itself doesn't match any page, not when some post parameter or header is "not found". One argument for this is that some load balancers cache 404s from their backends. Please let us know if you have a different understanding.

I think the 400 is counter-intuitive and potential hostile to users, however the comment about load balancers caching 404s is useful to keep in mind.
There is probably additional context from the associated pivotal story with the commit (02af2cad0eef9966a118bdb4d24819f658e48c36) which changed this behaviour. However this story is private.

When gorouter handles missing routes, which is called either when:

the pool is empty (so nothing can handle the request) and gorouter is configured to not serve a 503
there is no pool for this route (so there are no registered backends)
the lookup has reported there is no instance with this instance ID

during this request path, it adds the header Cache-Control: no-cache,no-store, which means the client (or an intermediate load balancer) should never cache this response

With the above:

any load balancer which caches a 404 from gorouter, when any of the above 404 cases has been hit, is violating RFC 7234 (this RFC is only 5.5 years old though)
as a user, when I make a valid and well-formed request, to a resource which does not exist, getting a 400 instead of a 404 is counter-intuitive

This commit from my dev remote (404) can be cherry-picked to make this change (404), the change is very simple, and the tests already assert the headers are correct.

Alternatively, there is actually a HTTP status code specifically for this use case, as defined in RFC 7232: HTTP 412 Precondition Failed.

The 412 (Precondition Failed) status code indicates that one or more
conditions given in the request header fields evaluated to false when
tested on the server.

This commit from my dev remote (412) can be cherry-picked to make this change (412). As above, the change is simple, and the tests already assert the correct behavior.

Whatever we choose to respond with, it should be clearly communicated because a change in status codes is basically a change to the user API without deprecation notice. Such changes should always be part of release notes and doc updates.
If we picked @tlwr s commit at least the docs would match the behavior again :)
This would unblock my clients for now until a proper doc update + logic change is released.

Hi @domdom82,
First I'd like to apologize for the pain that this undocumented change caused you.
We have decided to stick with returning the 400. I updated the release note to include this reasoning:
Breaking change: Requests with nonexistent instance ID headers and with syntactically incorrect instance ID headers will now return a 400 status code instead of a 404 status code. A 400 status code indicates that the server cannot or will not process the request due to something that is perceived to be a client error (e.g., malformed request syntax, invalid request message framing, or deceptive request routing). We think this is better than a 404 which indicates page not found.

I also updated the following docs:

gorouter docs
CF Docs

We will try to do better at communicating these changes in the future. I'm not sure how this one slipped by.
I'm closing the issue. Feel free to reopen if you have any other questions or reach out to us in slack.
Best,
Amelia Downs, CF Networking Program Engineer

Hi @domdom82,
Thanks for bringing this up. I apologize for the lack of documentation on this change. We are working on updating the documentation to reflect the current behavior.
For visibility, here is a link to a slack conversation on this topic: https://cloudfoundry.slack.com/archives/CFX13JK7B/p1581323910042700
For now, we will retain what we have, but I believe the discussion is still ongoing as to what the appropriate error code is.

@ameowlia thanks for the quick fix. My clients will update their apps to the new behavior.


Remove digitalmarketplace-team

It posted quotes from our team charter to our Slack once a day. The Digital Marketplace is moving to CCS (and the CCS Slack) at the end of this month, however, so we no longer need this.
This PR removes everything Digital Marketplace-related that https://github.com/alphagov/seal/pull/248 does not.

Rebased to fix merge conflict. I'll merge later this month

I want to stop tracking this. Merging. Farewell, old friend.


mautech.edu.ng and mau.edu.ng point to the same university. mautech.edu.ng is slowly being migrated to mau.edu.ng

mautech.edu.ng and mau.edu.ng point to the same university. mautech.edu.ng is slowly being migrated to mau.edu.ng but all domains will remain active for the foreseeable future. The reason is  mautech is no longer only a technology related university but it is also now does other non-tech courses like medicine etc, therefore the name change. This university was established in 1981.

@CloudInnovator lib/domains/ng/edu/mau/Screenshot: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

Done! Here are added pictures for additional evidence.

@CloudInnovator Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Disable @typescript-eslint/restrict-template-expressions

I don't have an opinion here so suggest you guys discuss and come to a decision?


Add temporary node merge strategy to node post/patch permission methods

Not a fan of the ParamUtils class - couldn't really think of a better way to do it but open to suggestions.

Looks good to me, thanks :)
I'm happy with the ParamUtils class for now


feat: dynamic single tree select widget standardisation

Description

Passed the contentRef to SingleSelectTreeComponent from SingleSelectTreeComponent.
Wrapped the SingleSelectTreeComponent functional, component in the React.forwardRef.
Set the final ref from React.forwardRef to TreeSelectContainer which is a styled div.

Fixes #13057
Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Manually

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

/ok-to-test sha=1215134

@Tooluloope Could you review this?


[FEATURE] [Jacksonized] Convert javadoc into @JsonPropertyDescription

The goal is to automatically populate the description field in generated JSON Schemas from the javadoc by generating a @JsonPropertyDescription.
Lomboked:
@Jacksonized @Builder
Foo
{
    /**
     * This is the bar property.
     * This property is very important, and it has a very convoluted usecase which I'm about to describe here in depth.
     * ... more stuff with code snippets ...
     * @author John Smith
     */
    final int bar;
}

Plain Java:
Foo
{
    /**
     * This is the bar property.
     * This property is very important, and it has a very convoluted usecase which I'm about to describe here in depth.
     * ... more stuff with code snippets ...
     * @author John Smith
     */
    @JsonPropertyDescription ("This is the bar property.")
    final int bar;

   FooBuilder {

        @JsonPropertyDescription ("This is the bar property.")
        public int setBar(int bar) { }
   }
}

rzwitserloot has already commented:

That is probably technically possible, but I worry that there are context where javadoc disappears on us. Given that javadoc's already gone, the fact that we therefore cannot copy that javadoc around to the stuff we generate feels irrelevant, but when it changes the actual code we generate, I'm a little more skeptical. Some extra tests and a few caveats in the docs about such a feature would be warranted.

I don't know what are the contexts under which javadoc disappears that are being referred to.

Lombok works on ASTs, not raw source files. When lombok starts running, an AST is provided. There are many ways lombok can end up running; from the basic 'annotation processing round fired off by stock javactoeclipse agent has injected a call to lombok in the middle of eclipse's parsing process' to 'java -jar lombok.jar delombok, and many things in between. JavaDoc is an afterthought in these ASTs, usually stapled on using a hashmap that maps positions to javadoc nodes for example. Sometimes the map is empty or the 'javadoc' fields remain uninitialized, e.g. because it's a tooling run that thinks that documentation is irrelevant.
As a specific indication of how I'm really worried about this: The javac parser has code in it that will go through an entire doc comment to extract @deprecated out of there in order to set a flag on the node itself, and can otherwise be configured to entirely ignore documentation.
They fact that this was written is stringly indicative that transferring code-affecting things from javadoc to not-javadoc doesn't always work; if it did, that 'extract deprecated state explicitly' part wasn't neccessary. That part is not pluggable and thus lombok can't add something there.
The real question is: Is it still the case (I think yes), and the circumstances for which that deprecated flag was being extracted by hand, do those also apply here, and the answer is probably no, but I'm not sure. I don't really know where to start either, which is why this feature request feels so daunting (it requires a solution to this problem). At the very least:

plain jane javac run
javac -proc:only only run
eclipse, various versions and in various contexts. Scan every json/jackson relevant plugin from the marketplace.
big one: intellij plugin, I don't like introducing features that are extremely difficult for plushnikov to implement (although evidently now intellij itself has thrown its weight behind the plugin, so that's cool!)
delombok
any tool that directly looks at source files and the @JPD annotation, if they exist.

The @JsonPropertyDescription annotation is retention runtime, so all of the above is relevant; if there is a way to end up with class files or an AST that is then processed by another tool, they are supposed to be able to see that description. Will they - that's another matter.


Add tests about Date creation

Add tests demonstrating that a fake Date could be created with one argument as a String since this string is in a format recognized by the Date.parse() method.

Thanks! Tests are documentation too :-)


add more premade layouts

seems like some users made layouts with their hardware. i added that hardware to hardware selection dropdown :)

ill continue adding more layouts


⚠ Turnos 24x7 has degraded performance

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

HTTP code: 200
Response time: 958 ms

Resolved: Turnos 24x7 performance has improved in 90114a3.


Development Cluster Currently Broken

Cannot create a development cluster currently. I get the following error
$ gokp create-cluster development --cluster-name=chx-$RANDOM$$ --github-token=${GH_TOKEN}  --gitops-controller=fluxcd
INFO[0000] Running checks
INFO[0000] Creating temporary control plane
INFO[0020] Initializing Docker provider
FATA[0070] action failed after 10 attempts: failed to get cert-manager object /, Kind=, /: Object 'Kind' is missing in 'unstructured object has no kind

Looks like it's all broken...
INFO[0000] Running checks
INFO[0000] Creating temporary control plane
INFO[0024] Boostrapping Cloud Formation stack on AWS
I0207 14:27:01.800325  173076 service.go:59] AWS Cloudformation stack "cluster-api-provider-aws-sigs-k8s-io" already exists, updating
INFO[0026] Initializing AWS provider
FATA[0079] action failed after 10 attempts: failed to get cert-manager object /, Kind=, /: Object 'Kind' is missing in 'unstructured object has no kind'

Looks like the error comes at the Initializing Provider step

Release v0.0.7 addresses this bug


Create a webapp with search feature by keywords

and name SolutionsShowcaseApp

have an existing list of keywords e.g.
architecture, docker, stripe


Compile Boost

Hi,
I have no experience running .bat files, hence I have a question how to compile minkowski cpp files correctly.
I have downloaded Boost 1.68 version, added to environmental variables. I was using boost before and it works on visual studio.
What I am doing wrong in following procedure?

I open compile.bat using text editor and add path directory like this:
cl /Ox /Ob2 /Oi /Ot /Oy- /GL /GF /Gm- /MD /GS- /arch:AVX2 /fp:fast -I <C:\Libs\boost_1_68_0> /LD minkowski.cc

Then run visual studio command prompt (as administrator), navigate to correct directory and type compile. But I get this exception:

C:\Libs\DeepNestPort-master\DeepNestPort-master\Minkowski>cl /Ox /Ob2 /Oi /Ot /Oy- /GL /GF /Gm- /MD /GS- /arch:AVX2 /fp:fast -I  0<C:\Libs\boost_1_68_0 1>/LD
Access is denied.
I tried adding these directories and they all have the same issue:
C:\Libs\boost_1_68_0
C:\Libs\boost_1_68_0\boost
C:\Libs\boost_1_68_0\lib64-msvc-14.0

Hi,

it seems you wrote wrong path:

cl /Ox /Ob2 /Oi /Ot /Oy- /GL /GF /Gm- /MD /GS- /arch:AVX2 /fp:fast -I < C:\Libs\boost_1_68_0 > /LD minkowski.cc

should be
cl /Ox /Ob2 /Oi /Ot /Oy- /GL /GF /Gm- /MD /GS- /arch:AVX2 /fp:fast -I " C:\Libs\boost_1_68_0"  /LD minkowski.cc

for example my compile.bat :
cl /Ox /Ob2 /Oi /Ot /Oy- /GL /GF /Gm- /MD /GS- /arch:AVX2 /fp:fast -I "C:\Users\fel\Downloads\boost_1_62_0\boost_1_62_0" /LD minkowski.cc

You should use 1.62 Boost version

Access denied: i think, you should run Command Prompt as an Administrator

Dear @fel88 ,
Thank you for a reply, after changing to "path" it worked.
Now I will try to run visual studio project.
Do I need to reference generated .dll somehow specially or it can be referenced typically in visual studio project?

Minkowski.dll is unmanaged dll. Just put minkowski.dll in bin\Debug (or bin\Release) folder after building project. Then you can run DeepNestPort.exe

Dear @fel88 ,
Thank you it is working and nesting elements.
I would like to ask if I run your software on other computer, would I need to download boost and compile it again, or it would be just enough to have minkowski.dll together with .exe file?

I will try to set properties of sheets parts and settings, but if you can show an example of simple console application it would be super nice:)

Ok, no problem.

Thank you for console application.
Do you have an example of .xml file as an input to main function?
What would be the way to type  polygons manually by defining their coordinates instead of using .xml file?
For instance if I would like to nest 10 triangles (or other types of polylines) by specifying their coordinates.
The pseudo code I imagine:
for(int i = 0 ; i < 10; i++){
  var polygon = new SomePolygonClass();
  polygon.Add(0,0);
  polygon.Add(0,10);
  polygon.Add(10,10);

  sample.AddElementToNest(polygon );
}

I've added the example of .xml file.
You can run DeenNestConsole.exe with 'sample' argument (DeepNestConsole sample).
Look at LoadSampleData method. It is exactly what you want.

Thank you very much it works very well:)


Only works when format is iife or umd?

I experimented with vite-plugin-externals, but it didn't seem to have any effect.  Then I came across https://github.com/rollup/rollup/issues/2374, which notes that external and output.globals only work with iife or umd output formats.  Maybe there should be a disclaimer in the readme to explain that?

@IanVS vite-plugin-external or vite-plugin-externals? Would you mind if I got an example from you about this issue, actually I do not know this issue https://github.com/rollup/rollup/issues/2374 because I never do output.globals and esm output format together.

Sorry, vite-plugin-external.  I was trying to use this plugin along with vite in middleware mode to replace references to internal storybook packages with references from the global scope.  https://github.com/storybookjs/storybook/pull/19978 is the PR where I ended up using https://www.npmjs.com/package/rollup-plugin-external-globals instead.


TFRecordDataset causes seg fault with parallel reads and parallel map of an empty file list

== check python ===================================================
python version: 3.7.4
python branch:
python build version: ('default', 'Sep 7 2019 18:27:02')
python compiler version: Clang 10.0.1 (clang-1<IP_ADDRESS>)
python implementation: CPython
== check os platform ===============================================
os: Darwin
os kernel version: Darwin Kernel Version 19.0.0: Wed Sep 25 20:18:50 PDT 2019; root:xnu-6153.11.262/RELEASE_X86_64
os release version: 19.0.0
os platform: Darwin-19.0.0-x86_64-i386-64bit
linux distribution: ('', '', '')
linux os distribution: ('', '', '')
mac version: ('10.15', ('', '', ''), 'x86_64')
uname: uname_result(system='Darwin', node='Andrews-MacBook.local', release='19.0.0', version='Darwin Kernel Version 19.0.0: Wed Sep 25 20:18:50 PDT 2019; root:xnu-6153.11.262/RELEASE_X86_64', machine='x86_64', processor='i386')
architecture: ('64bit', '')
machine: x86_64
== are we in docker =============================================
No
== compiler =====================================================
xcrun: error: invalid active developer path (/Library/Developer/CommandLineTools), missing xcrun at: /Library/Developer/CommandLineTools/usr/bin/xcrun
== check pips ===================================================
numpy 1.17.4
protobuf 3.10.0
tensorflow 2.1.0
tensorflow-estimator 2.0.1
== check for virtualenv =========================================
True
== tensorflow import ============================================
tf.version.VERSION = 2.1.0
tf.version.GIT_VERSION = v2.0.0-rc2-26-g64c3d382ca
tf.version.COMPILER_VERSION = 4.2.1 Compatible Apple LLVM 10.0.0 (clang-1<IP_ADDRESS>)
== env ==========================================================
LD_LIBRARY_PATH is unset
DYLD_LIBRARY_PATH is unset
== nvidia-smi ===================================================
./tf_env_collect.sh: line 147: nvidia-smi: command not found
== cuda libs ===================================================
== tensorflow installed from info ==================
Name: tensorflow
Version: 2.1.0
Summary: TensorFlow is an open source machine learning framework for everyone.
Home-page: https://www.tensorflow.org/
Author-email<EMAIL_ADDRESS>License: Apache 2.0
Location: /Users/andrew/.local/share/virtualenvs/lib_andrew_scratch--yvJ8pLH/lib/python3.7/site-packages
Required-by:
== python version ==============================================
(major, minor, micro, releaselevel, serial)
(3, 7, 4, 'final', 0)
== bazel version ===============================================
Describe the current behavior
Segfault when next is called on the Itterable.
Describe the expected behavior
Crash/Error gracefully
Code to reproduce the issue
import tensorflow as tf

def _parse_function(example_proto):
    feature_description = {'features': tf.io.FixedLenFeature([], tf.string)}
    return tf.io.parse_single_example(example_proto, feature_description)

dataset = tf.data.TFRecordDataset([], num_parallel_reads = 2)
dataset = dataset.map(_parse_function, num_parallel_calls = tf.data.experimental.AUTOTUNE)

count = sum([1 for i in dataset])

I have reproduced this is Colab
https://colab.research.google.com/drive/1C6sJ0BVjbK1h78LojPxp1DAuILJC_9e8

@andrewstanfordjason I have executed the code shared by you on nightly and do not face any issues, please find the gist for the same.
The colab link shared by you does not have any error.

Clearly this is not an issue in the nightly. Thanks


accesswidth/regwidth limitations

SystemRDL-compiler is missing a check, standard section 10.6.1 Semantics, point c).
"The value of the accesswidth property shall not exceed the value of the regwidth property."
In my case access width was specified as a default of 64 and register width was 32, The tool did not report an error.
I initially misunderstood what is the purpose of accesswidth, and I thought it is the width of the data bus in RTL code.
It actually "Specifies the minimum software access width (power of two) operation that may be performed on the register.".
I tried to make a parallel with my experience, to be able to give a sensible example.
A 32-bit register with byte access would have:
accesswidth = 8
regwidth = 32
This is common for memories, but registers usually do not have byte access.
accesswidth = 32
regwidth = 32
I still have 2 questions remaining (I am asking since I noticed your errata for the standard, so I see you as an expert):

What would be the best way to describe/decide the bus address width? Is it one of the parameters, or should it be defined with a custom approach?
If I define a register with a width larger then 64-bit, for example 128 bits (quad precision floating point), how would I specify the reset value, since the data type used to specify reset values is only 64bit (longint unsigned). Even if regwidth is limited to 64-bit, would it be possible to have signalwidth=128 and somehow specify the concatenation of multiple registers into the signal?

I was actually looking into the same thing recently!
I agree that the check described in 10.6.1-c is missing. I had initially waived it in the errata, but after looking at this in more detail, I no longer agree with my rationale. I plan to enforce 10.6.1-c in the next release.

I initially misunderstood what is the purpose of accesswidth, and I thought it is the width of the data bus in RTL code.
It actually "Specifies the minimum software access width (power of two) operation that may be performed on the register.".

I agree this is a little strange. I had never noticed until your emphasis on "minimum". I think this is another example where poor phrasing in the RDL spec is causing confusion, and we're simply taking it too literally. This is a common issue I have noticed in other sections too.
Regardless of the phrasing of that clause, I believe accesswidth does indeed imply the bus width. The spec does not give too many examples, but it sure seems to imply it has something to do with the bus width in 10.6.1-f:

Any field that is software-writable or clear on read shall not span multiple software accessible sub-words (e.g., a 64-bit register with a 32-bit access width may not have a writable field with bits in both the upper and lower half of the register).

This presents an example where a hardware limitation prevents access to all 64-bits simultaneously. The use of "sub-words" implies this is referring to a hardware bus width (see https://en.wikipedia.org/wiki/Word_(computer_architecture))
Perhaps when they used the word "minimum", the spec authors were referring to register protocols like AXI4 or AHB where a transaction can perform a burst access. In the context of a burst protocol, the minimum access width would be identical to the bus width.
Even so, this phrasing is a little imprecise. Some protocols allow for write byte strobes which allow one to modify individual bytes, but I do not think this applies since it doesn't change the actual transaction's access width (strobes are meaningless for read transactions).
So to respond to your question:

What would be the best way to describe/decide the bus address width? Is it one of the parameters, or should it be defined with a custom approach?

I think accesswidth is still the correct property to use to infer the hardware bus width. It seems strange to me that this is a property of a reg component. It would have been more appropriate to be a property of addrmap. It would not make much sense if a register block had varying accesswidth.

If I define a register with a width larger then 64-bit, for example 128 bits (quad precision floating point), how would I specify the reset value, since the data type used to specify reset values is only 64bit (longint unsigned). Even if regwidth is limited to 64-bit, would it be possible to have signalwidth=128 and somehow specify the concatenation of multiple registers into the signal?

If you look at 9.5, Table 13, you'll see that the reset property is of type bit. The bit type is similar to Verilog's where it can contain large bit vector values. Specifying a large reset value will work fine:
reg my_reg {
    regwidth = 128;
    field {
        sw=rw; hw=r;
    } my_field[127:0] = 0x01234567_89abcdef_aaaabbbb_ccccdddd;
    // Or use verilog notation: 128'h01234567_89abcdef_aaaabbbb_ccccdddd
}

I will first write about the bit type, which the standard seems to use it as a vector, without explicitly saying bit vector.
As long as the SystemRDL-compiler can handle numbers larger then 64-bit, I can ignore the standard being vague about it.
Regarding accesswidth I found two short sections in the AMBA AXI5 standard discussing the relation between the full and lite standard that seem close to the accesswidth issue, this are sections B1.1.2 and B1.1.3.
I actually think accesswidth should be redefined to be more about byte access, and a custom parameter applied to addrmap should be used for bus datawidth. I do not think any mental gymnastics would shape this part of the standard into something sensible.


Old 'javax.ws.rs' library

I've come across some nasty dependency clashes by using eureka-client jar and some newer libraries. Eureka-Client jar does depend on jsr311 which is the old specification. Some of my dependecies do require javax.ws.rs-api which is the new one. For example javax.ws.core.Response is different in the new version and this causes some troubles for me. I've exluded the old one but this is just a ticking bomb.
Are there plans to migrate to the new api specification?

@Writtscher we have decided to take a different direction and will be making the client side of eureka independent of jaxrs completely. Work is being done internally, however we don't have any timelines that can be shared as of yet.

Well thanks for the quick answer. I'm fine for now pulling the new library and exclude the jsr311 dependecy as there is no failure at the moment.


Expand backend code to handle more generic layer

Currently the test dataset is hard-coded to MUR SST.  The task is to update the backend service to work with any layer in GIBS, in addition to MUR layer.
The work involves getting a query link from the frontend (worldview) to query datasets from GIBS, compute the statistics on the backend, and then send back the query to the worldview.

http://ec2-52-13-63-64.us-west-2.compute.amazonaws.com:8080/get_stats_oiip_dag?lon_min=-50.992&lat_min[…]on&analysisName=spatial_summary_statistics

@vaishnaviyathirajam to PR the updated code


Failed to load the dll - 0x80070057

Steps to reproduce
Just downloaded PS <IP_ADDRESS> Alpha x64 then install... when I tried to open it, got this error:
Failed to load the dll from [C:\Program Files\PowerShell\<IP_ADDRESS>\hostfxr.dll], HRESULT: 0x80070057
The library hostfxr.dll was found, but loading it from C:\Program Files\PowerShell\<IP_ADDRESS>\hostfxr.
dll failed

Expected behavior
Actual behavior
Environment data
x64 based system - Windows 7 64bit SP 1 (no updates) - PS <IP_ADDRESS> Alpha x64
Error
Failed to load the dll from [C:\Program Files\PowerShell\<IP_ADDRESS>\hostfxr.dll], HRESULT: 0x80070057
The library hostfxr.dll was found, but loading it from C:\Program Files\PowerShell\<IP_ADDRESS>\hostfxr.
dll failed

Hi @Yousha. I'm assuming that you are missing the necessary dependencies because you indicated "no updates" in the environment data section. Our downlevel packages have dependencies on components beyond what we ship. You can either install the
VC++ redistributable package directly or install the Universal C Runtime.
I recommend installing the Universal C Runtime to resolve this issue. It is packaged and distributed by Microsoft as an optional package via Windows Update, so you will pick it up automatically if you fully patch your Windows 7 system.

Hi
I have already installed VC++ Runtime libs 2008 to 2015, and both x64 and x86.
Also Universal C Runtime requires Windows updates, that I didn't install them
I'm using Windows 7 SP1 without any patch/update. (Just like many other people!)
[NOT A BUG]
Thanks.


Azure AD API

We (Juju) would like to automate the process of creating an application/service principal in Active Directory. Essentially we want to replicate the process described at https://azure.microsoft.com/en-us/documentation/articles/resource-group-authenticate-service-principal-cli/, in Go.

I've written code to create an AD App, AD ServicePrincipal and assign ARM RBAC Roles to the SP. If you want to take a look and/or borrow code: https://github.com/colemickens/azkube/blob/master/util/ad.go
The Azure SDK for Go can handle the role assignment, but the AD side interacts with the Graph API and we don't have an ADAL library for Go, so I implemented the Graph API calls "manually" using go-autorest.

@colemickens Thanks very much, I will take a look.

This is pretty essential, especially for tools trying to support a completely "Infrastructure as Code" approach. Is there any work being done on it?

@moritzheiber I ended up hand-crafting just the bits I needed. I've licensed it the same as the azure-sdk-for-go code, feel free to use it if it's useful: https://github.com/juju/juju/tree/staging/provider/azure/internal/ad. It was a while ago that I wrote it now, but IIRC the only reason why I couldn't just auto-generate it was because of the difference in result structures from the AD API (i.e. see refs to "odata.error" in https://github.com/juju/juju/blob/staging/provider/azure/internal/ad/client.go)

Howdy folks,
We're scheduled to add the Graph API in the next monthly release. We have a couple of related issues tracking the concerns brought up in this thread:

Azure/go-autorest#103
Azure/azure-sdk-for-go#455
Because there are more immediately actionable requests in those items, I'm going to close this issue in favor of the others. If folks think there is unresolved discussion unique to this thread, feel free to re-open.

Thanks for the update, @marstr.


Remove setting timestamp for existing MiqEvent instances during migration.

The class of EVMAlertEvent has been converted from EmsEvent to MiqEvent along with the timestamp column set.
Timeline which looks for events' timestamp checks for MiqEvent only for EVMAlertEvent.
So there is no need to set timestamp for existing MiqEvent instances that are not EVMAlertEvent.

@gmcculloug Please review.

@jerryk55 I think this is the issue you were running into, right?

Correct @Fryguy.  I believe that is why @lfu put in this PR. For some reason my appliance had over 5 million MiqEvent records.  Attempting to set the time on all of them was taking prohibitively long.


Feature: Change to another layout [src]

Follow up on the discussion in #488.
The MR suggested to change to a flat layout to activate typing easily (see #305).
@totallyzen  brought the argument:

Since I wrote the refactor to the current core and modules, I can chime in a bit.
There is a very simple reason why it is core and modules -> it is because all major testcontainers language flavors use these. I don't think I'd want to deviate from that as it allows the TC mainline documentation to converge and easily match support across languages (automation can be built for it, and issues tracked across language flavours)
I wish we had better automation and support from the TC staff (I'm only a volunteer myself like @alexanderankin) but this is what we've got for now.

However, after looking at the official .NET project, I find this argument weak. The .NET project doesn't follow this structure—there is no core or modules, only src and tests folders.
The current layout introduces several issues:

It is complex and hard to understand, as only with some Poetry-specific setup are the different folders combined.
It makes running poetry pytest to test the project nearly impossible due to "import file mismatches."
It suggests the existence of subprojects like testcontainers-core (which actually exists on PyPi, created by @tillahoffmann), adding confusion.
Properly type-checking the project is difficult because mypy expects the entire project (not only parts) to be typed when importing, and it checks for the existence of py.typed.

I propose adopting the src-layout, which has been embraced by many modern Python packages. This would:

Clarify the difference between tests for the final package and tests in "development mode."
Allow running pytest from the project root, reducing the need for a complex Makefile just to run tests. Tests for different modules can still be organized into subfolders in the tests directory.

Most importantly, this change would reduce complexity, enabling Python developers to contribute more easily with less cognitive overhead.
Lastly, it would mirror the structure of the official .NET project. :)

If the maintainers decide to switch to the src layout I could create a PR. But only if there is support for it and it will be merged in time.
Probably it would good to work off other PRs before creating it.

if you want to execute a subset of tests, you can use standard pytest features? can you explain which functions you wanted to target and i can try to craft a pytest command for you

hm, I supposed in a perfect world we would be able to run ci on all projects sometimes but it was a conscious decision to cut down and isolate ci runs of the individual modules (isolation helps also with dependency management)
in .NET the tooling situation is much better than python so I would say this is why the situation differs.
I have reproduced the problem and am going to fix it, but I will point out that we have a makefile that species how we run tests, so you can see there how we run the tests and it is not all in a row.

I might not understand the actions and poetry correctly but I don't see how the "isolation" is related to structure of the project.
The  suggested structure would be something like
src
├─ testcontainers
│ ├─ core
│ │ ├─ auth.py
│ │ ├─ config.py
│ │ ├─ …
│ ├─ compose
│ ├─ aws
│ ├─ …
tests
├─ conftest.py
├─ core
│ ├─ test_auth.py
│ ├─ test_conifg.py
├─ compose
│ ├─ test_compose.py
├─ aws
│ ├─ test_aws.py
├─ …

You only would need to adopt the paths inside the CI to keep the same functionality as now.
Running pytest tests/core instead of pytest core/tests.
Only difference is that in ci-community in modules=$(echo "${{ steps.changed-files.outputs.all_changed_files }}" |  jq '.[] | split("/") | first' | jq -s -c '. | unique') we might need to account for removing core to not test it twice.
Also in the end we build a monolithic package so I don't see that there are "different projects".
As also noted above: If you need a make file to run tests this is clearly an indicator that the project setup is too complex.
➡ So what is advantage of the current layout?
Sidenote
If you want to isolate the "core" from "community" modules it might be better to express this with a different package structure, something like
src
├─ testcontainers
│ ├─ auth.py
│ ├─ compose.py
│ ├─ config.py
├─ testcontainercontrib
│ ├─ aws
│ ├─ …

But as a monolithic package is released this make not a lot of sense.

the hope is that one day it is no longer a monolithic package i think - lowers the barrier to parallel community efforts that does not cost testcontainers org maintainer hours

FYI the commit you made failed in the pipeline.
I understand that goal but still even if the goal is not having a monolithic package changing the layout won't prevent it.
It even helps as indicated by the missing __init__.py in src/testcontainers that testcontainers is a PEP 420 namespace package.

yes, the pipeline failed because the cosmos emulator is not super reliable but i did not change any code, so if those tests passed before, its not something preventable by me today.
I'll leave this open for discussion in case I am wrong about this above, the situation may have changed, etc... I have yet to look at the dotnet implementation, etc.

Btw. make tests does not work for me:

For some reason it detects README.md as package?

Another reason why it might be nice to change to the src layout: It is easier to create patches.
For our local conda build of testcontainers I just was about to create a patch that includes the changes of #714.
In a source layout I could simply do git diff main src > testcontainers.patch   but here I need to be aware that only module files are part of the .tar.gz file, so I ended up in
git diff core/testcontainers > testcontainers.patch

If there would have been changes (not only to tests) also to modules I would have needed to include each module subfolder manually. Being very careful not include changes of tests or docs etc.

Hi @CarliJoy, thanks for initiating this discussion. I'd like to chime in from the perspective of the overarching Testcontainers Project leadership. However, I am not an experienced Python developer at all, so I can't really have a strong opinion on the details of the decision and what makes most sense for a Python project.
Please note that Testcontainers for .NET is currently in the same category of official/community as is Testcontainers for Python (even if the Testcontainers org README says otherwise), since it only has community maintainers (namely Andre Hofmeister) working on it. The difference here is, that it used to be commercially supported while we were still operating through AtomicJar. The situation is different after the Docker acquisition, which leaves us only with Testcontainers for Java and Testcontainers for Go maintainers on staff. Hence, it is not useful, to further use the .NET implementation as an example for a reference implementation, Java and Go are much better suited for this. And independent of this, there is always a historic context around past decisions and the status quo of the different languages, so a 1:1 comparison is not always helpful.
Ultimately, the goal for every Testcontainers project (which is based on our extensive experience developing and maintaining Testcontainers for Java) should be to have independently testable and releasable modules per technology, and in addition a minimal core module, that is free of these external technology dependencies. I have no opinion or guidance with regards to which folder structure is well suited to achieve this, since it depends on programming language and build system.
However, another critical aspect we want to ensure across languages, is minimizing CI load, since CI resources are shares across the GitHub organization. There can again different technical means and language dependent mechanisms to achieve this, but the desired goal is to avoid running CI for modules, that are not affected by changes.
Lastly, I leave the decision on how to proceed with this to the current maintainers and given the nature of a community supported project, I can entirely understand if there is a lack of bandwidth to currently support a bigger migration effort.


copy/paste plain text into Trumbowyg editor converts line break into paragraph break

Any time I copy paste text, doesn't matter where the source, even if I paste as plain text - even from notepad, this appears to be the default behaviour.   So then I have to go through removing all the extra lines.  Is there a way to fix this?  What am I missing?  Thx

same issue. Pasting leads to too much gaping as it converts all line breaks into


Implement modality of a JS dialog in app shell by using a new api.

This requires another pull request on brackets repo that makes calls to the new api in dialogs.js to have it working. https://github.com/adobe/brackets/pull/4341
A new API setModal is introduced so that Brackets can tell app shell to enforce modality when a JS modal dialog is showing. Before showing the JS dialog, we need to call this api with true argument; and after dismissing the dialog, we need to call it with false argument.
When the modality is on, all menu items will be disabled and users will not be able to click on any menu item or using its shortcuts. One exception to this on Mac is that some of the Edit menu items (Undo, Redo, Cut, Copy, Paste and Select All) will still be enabled so that users can use them in text fields in any JS dialog.
Modality is not only enforced on menus and shortcuts, but also on other UIs and shortcuts available on the window title bar. That is. X button and Alt+F4 shortcut will be disabled during the display of a JS dialog.
Although menus and commands won't be available in a JS dialog, dialog-specific shortcuts or key events will be available. For example, Cmd-D, Cmd-. and Escape key to dismiss the dialog.

Closing for now.


Rmd report: add software versions, samplesheet, workflow, database

right now some infos are included but not all, e.g. fasta database/ organism is missing

Closed with commit f9b5897.
This commit adds two more modules (rules) that collect:

software and env versions
module logs


cosmetic fix for index.html

use en-dash (U+2013) instead of hyphen-minus (U+002D) in index.html navigation

oh shit, I see: entities are required only for & < > etc


Wrong match Spat caillaudi ha Martens, 1866 Fuzzy to Cymatonautilus (Spath )

I think we can live with this for now, it is not very good that we fuzzy-matching a genus though.


Will we increase total number of licenses?

For now, it is

int8 internal licensesLimit = 52;

@igorbarinov

it's a limit for initial validators + all added?

yes: from ceremony (25) + from governance = 52

lets remove the limit

Let's set initial keys equal to 12 to match whitepaper. Validators will add more validators using governance dapp.

Let's set initial keys equal to 12 to match whitepaper. Validators will add more validators using governance dapp.

Done

Solution: lets remove the limit.
Rationale: we can't predict how many validators will be needed for mainnet

I agree with rationale, but doubt about solution. How about technical limit, let's say = 1000? IMO it is not a good idea to have dynamic sized array in solidity with possibility in contract to fill it with unlimited number of elements without any limits.

Tested that the technical limit is 2000. More validators in array will cause out of gas problem.


Rename Image-Exporter to Orca

I appear to have fixed the build, albeit by thinking way outside the box, relative to the task-at-hand: https://github.com/plotly/orca/commit/9aefae340488aac8d6767bc60e23c932c50f99d6

@scjody @etpinard

So if I understand correctly, docker-build-and-push has been broken since we changed the repo's name to orca.
Now, why does adding libgtk2.0 fix this issue?

Although I do not know, nor even feel like I have a "strong" theory, my hunch is that it relates to some difference in package-json.lock.
@etpinard:

But, please just get the tests to pass off one of those PRs w/o having to update quay.io things?

This PR meets your above criterion for merging, yes?

@etpinard

Can you confirm that Ubuntu versions differ in the built docker container before and after orca? To do so, I would click Rerun job with SSH on CircleCI and run lsb_release -a inside the built docker container.

Actually, I'm pretty sure lsb_release -a would look identical in both, i.e. that we are using Xenial. That command's output does not reflect the versions of any of the packages installed (not even the kernel). Ubuntu versions are only incremented exactly once every six months.

What is libgtk2.0?

It's part of GTK+, which is an old-school *nix toolkit for creating GUIs.

How did you think that adding it to our Dockerfile would fix the issue?

They call me the container whisperer… No, just kidding. The only thing that the CircleCI logs were telling me was that the smoke test was failing. So I SSH'd into the build box and ran a docker run command without the --detach or any other daemonizing arguments. Container creation failed, and part of the output of my attempt was

error while loading shared libraries: libgtk-x11-2.0.so.0: cannot open shared object file: No such file or directory

The above is 98% of what led me to my fix. The other 2% was me monkeying around, trying to determine which Aptitude package might have the shared object we needed.
===
All of that make sense? How shall we best proceed towards initial release of Orca?

Actually, I'm pretty sure lsb_release -a would look identical in both, i.e. that we are using Xenial.

Ok. You're probably right here. But can you try finding a way to confirm your hypothesis? There's a gotta a way to check whether libgtk2.0 is installed or not.

Moreover @JamesCropcho how big is libgtk2.0? We (probably) won't need it for our image server. So perhaps we could remove it from the Dockerfile and install only on CI?

@etpinard

Ok. You're probably right here. But can you try finding a way to confirm your hypothesis? There's a gotta a way to check whether libgtk2.0 is installed or not.

Let's say that I do as you propose—I somehow figure out the date of the last uncached build of that Docker Image layer, pull the corrresponding Xenial tag if it still exists, and build a cousin image without libgtk2.0 (we would never be able to get historically accurate Aptitude package versions, however). And let's say that this image works even without that library. But, what then? You already proved (thank you) on your dummy branch, that this fix in isolation differentiated between broken and fixed builds. I don't believe we ware fortunate enough that either hypothetical outcome of the proposed experiment would have any tangible strategic effect for us.
Given the above, would you still opt for me to proceed to see if we have the necessary historical fact(s) for me to perform the experiment?

Moreover @JamesCropcho how big is libgtk2.0? We (probably) won't need it for our image server. So perhaps we could remove it from the Dockerfile and install it only on CI?

Personally, that sounds scary to me. Wouldn't it defeat the purpose of the smoke test, to modify the Image after running it?

I somehow figure out the date of the last uncached build of that Docker Image layer, pull the corrresponding Xenial tag if it still exists,

No need to pull down anything. Just pick an old build on CircleCI and click on rebuild with SSH button on the top right corner.

Wouldn't it defeat the purpose of the smoke test

If the smoke test requires libgtk2.0 to run, we should be able to reproduce it locally. Have you been able to reproduce those error while loading shared libraries: libgtk-x11-2.0.so.0: cannot open shared object file: No such file or directory errors locally?

Personally, that sounds scary to me.

Maybe. But adding something in a Dockerfile we no apparent reason except it gets the test to pass on ci is even scarier.

Oh well, looks like you don't feel like investigating this hiccup further - which I can't really blame you for, it sounds pretty annoying. But can at least make a one-commit noise-less PR with just that one patch in the Dockerfile with a thorough explanation of why you did this to get our Docker expert @scjody to review the change? Thank you.

@etpinard

Oh well, looks like you don't feel like investigating this hiccup further - which I can't really blame you for, it sounds pretty annoying.

I do admit that it's annoying (ha). However, as explained above my reticence to further explore is not laziness nor feeling annoyed but rather that I question what if any practical change to would result in even a positive result from the proposed experiment.
I hope that makes sense, and that you find it reasonable.
I have created the precise pull request you seek, and requested Jody's (and your) diligent eye for its review.

@JamesCropcho would you be interested in making orca a single-entry-point app as discussed in https://github.com/plotly/orca/issues/21#issuecomment-384298071 ?

@etpinard

@JamesCropcho would you be interested in making orca a single-entry-point app as discussed in https://github.com/plotly/orca/issues/21#issuecomment-384298071 ?

If am the best person for the job, then yes, I am able to do so. However, I would need to commence on or after Wednesday, 9 May. (This does not mean I am AWOL for my current task(s) and any other communication until then of course!) What do you think of that date?
I knew that task was on the radar. Still, I have focused on making the Orca rename as being earlier in the pipeline, and have also make said renaming independent and separable from converting to single entry-point.

Ok. Thanks for the info.
I'll try to tackle it before May 9th, if not I'll ping you.

Thanks @JamesCropcho !


Autorest Regen Preview to 3.0.0-alpha.20230420.9 by Josh Love from refs/pull/3298/head

Triggered from https://github.com/Azure/autorest.csharp/pull/3298

API change check
API changes are not detected in this pull request.


kriti: add support for custom function

Description
This PR adds the support of custom functions.
Proposed runKritiWith function with the following type:
runKritiWith ::
    T.Text ->
    [(T.Text, J.Value)] ->
    [(T.Text, J.Value  -> Either CustomFunctionError J.Value)] ->
    Either KritiError J.Value

Here, (T.Text, J.Value  -> Either CustomFunctionError J.Value) is a tuple provided by the user, where the first element is the name of the function and second element is the function. For using this custom function, user would have to specify it in the template.
Click here for an example.
Template:
{
  "author": {
    "name": {{$.event.name}},
    "age": {{$.event.age}},
    "articles":
      {{ range _, x := $.event.author.articles }}
          {
              "id": {{x.id}},
              "title": {{customFunc x.title}}
          }
      {{ end }}
  }
}

Input:
{
  "event": {
    "name": "Freddie Jones",
    "age": 27,
    "author": {
      "articles": [
        { "id": 0, "title": "The Elements", "length": 150, "published": true},
        { "id": 1, "title": "ARRL Handbook", "length": 1000, "published": true},
        { "id": 2, "title": "The Mars Trilogy", "length": 500, "published": false}
      ]
    }
  }
}

Output:
{
  "author": {
    "age": 27,
    "articles": [
      {
        "id": 0,
        "title": "THE ELEMENTS"
      },
      {
        "id": 1,
        "title": "ARRL HANDBOOK"
      },
      {
        "id": 2,
        "title": "THE MARS TRILOGY"
      }
    ],
    "name": "Freddie Jones"
  }
}

Where customFunc is defined as:
uppercased :: J.Value -> Either CustomFunctionError J.Value
uppercased inp = case inp of
  J.String txt -> Right $ J.String (T.toUpper txt)
  _ -> Left $ CustomFunctionError "unexpected"

TODOs

[x] Add RFC: https://github.com/hasura/kriti-lang/pull/49
[x] Add support for user supplied custom function name
[x] Add example of a usecase of custom function.
[x] Add ability to use multiple user defined functions.

Future scope (can be taken up in separate PR)

Wrap the function inside the Function wrapper instead of passing to evaluator.
Don't split up the evaluator as eval and evalWith.
Add some more basic custom functions.
Extend this beyond unary functions.

Great work!


Considering all features in a feature including the features defined …

…inside conditionals
Continuation of https://github.com/apache/karaf/pull/880, but this time aiming to merge into master

Your change is not fully correct as the conditional feature might not have a version, and the resolution will fail.

@jbonofre I don't see how this can happens. In our validation example, we don't indicate any feature version - https://paste.ofcode.org/UjVKQENQCYDFrKFTwgARgX and it resolves the feature.
Can you please indicate a scenario where this can happen? So we can validated that. Ty

@jbonofre , can you please answer my question on latest comment? Ty

@jbonofre any update on this PR? Is missing something from my side?

@ricardosilva88 oops sorry I forgot your PR. Let me resume the review on it, but when I tested it breaks the resolution. Let me resume the review.

retest this please

Noticed that are some examples failing due to this change, I will be working to fix and probably adding some tests

@jbonofre just pushed my fix's for the tests that were failing

retest this please

retest this please

retest this please

@jbonofre I already rebase against master and on my local machine it does work. I don't see any valid reason or log on your Jenkins for this failure.

retest this please

"ERROR: Maven Home /home/jenkins/tools/maven/apache-maven-3.6.0 doesnt exist"

@ricardosilva88 yes, I'm fixing that. It's related to a change we did on Jenkins executors.

Jenkins configuration has been updated.

retest this please

retest this please

retest this please

retest this please


socket enricher: add socket earlier at tcp_connect

socket enricher: add socket earlier at tcp_connect
Before this patch, the first TCP-SYN packet could not be enriched
because the socket is added in the sockets map only at kretprobe-time
when tcp_v4_connect() and tcp_v6_connect() returns.
This patch changes this to update the sockets map at kprobe-time instead
of kretprobe. If the connection fails, the kretprobe detects it and
marks the socket for removal after the timeout.
Note that tcp_v4_connect() and tcp_v6_connect() both call tcp_connect(),
so the kprobe/kretprobe can be attached to tcp_connect directly.
The source port is updated to its final value by tcp_v4_connect() and
tcp_v6_connect() but before calling tcp_connect(), so attaching to
tcp_connect instead of the v4/v6 versions is necessary to get the
correct source port.
How to use
No changes
Testing done
The following tests are performed with https://github.com/inspektor-gadget/inspektor-gadget/pull/1630
$ docker run -ti --rm --name=test busybox wget wikipedia.org

No regression with the sni gadget:
$ go run -exec 'sudo' ./cmd/ig/... trace sni -r docker -c test
CONTAINER  PID        TID        COMM             NAME
test       1474265    1474265    wget             wikipedia.org
test       1474265    1474265    wget             www.wikipedia.org

The trace network gadget (with code from https://github.com/inspektor-gadget/inspektor-gadget/pull/1630) now supports enrichment:
$ go run -exec sudo ./cmd/ig/... trace network -r docker -c test
CONTAINER                                       PID        TID        COMM             TYPE      PROTO PORT  REMOTE
test                                            1474265    1474265    wget             OUTGOING  udp   53    <IP_ADDRESS>
test                                            1474265    1474265    wget             OUTGOING  udp   53    <IP_ADDRESS>
test                                            1474265    1474265    wget             OUTGOING  tcp   80    <IP_ADDRESS>
test                                            1474265    1474265    wget             OUTGOING  udp   53    <IP_ADDRESS>
test                                            1474265    1474265    wget             OUTGOING  udp   53    <IP_ADDRESS>
test                                            1474265    1474265    wget             OUTGOING  tcp   443   <IP_ADDRESS>
test                                            1474265    1474265    wget             OUTGOING  udp   53    <IP_ADDRESS>
test                                            1474265    1474265    wget             OUTGOING  udp   53    <IP_ADDRESS>
test                                            1474265    1474265    wget             OUTGOING  tcp   443   <IP_ADDRESS>

Yes, the tests will have to wait for #1630 because the current networking gadgets (dns or sni) are not sending events for the TCP SYN packet.


Get-DbaDatabaseSpace: improve [UnusableSpaceMB] for unlimited maxfilesize

[x] Bug fix (non-breaking change, fixes #)
[ ] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Pester test is included

Purpose
Calculations return 0 instead of negative numbers of UnusableSpaceMB when MaxSize is unlimited (-1).
Approach
Improved tSQL statement.

Thanks to you both! Here, Bun, a burrito 🌯


Feature/pending

Premise
This is a small pull request unrelated to what I was doing with the Log. They are just some fixes.
Changes

Added Interfaces for handling of pending operations, will be used later.
Added a search query to the SMSContactDao because getting the whole table just to then throw it away in SMSContactManager.contains() was a waste.

Side Changes

Removed EqualsVerifier dependency and just did all the relevant testing by hand.
Put SMSLogEvent fields back to non final.
Renamed LogEventType to just EventType. No big reason, sounded better.

No cute animals cause I already ran out of ideas.
On the other hand, this is the Looney Tunes episode where Bugs Bunny pretends to be an opera director. Notable is the fact that the video title is in Portuguese, where Bugs Bunny is called "Pernalonga".

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- app/src/main/java/ingsw/group1/findmyphone/event/EventType.java  3

Complexity decreasing per file
==============================
+ app/src/test/java/ingsw/group1/findmyphone/event/SMSLogEventTest.java  -1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- app/src/main/java/ingsw/group1/findmyphone/event/EventType.java  3

Complexity decreasing per file
==============================
+ app/src/test/java/ingsw/group1/findmyphone/event/SMSLogEventTest.java  -1

See the complete overview on Codacy


Can't set a non-English name for an automation

Home Assistant release (hass --version): 0.65.6
Python release (python3 --version): Whatever comes standard on the docker image
Component/platform: automation
Description of problem: I tried to create an automation with the name "おはよう", but it failed silently.
Expected: Either create the automation with my desired name (preferred), or show an error message telling me allowed characters.

Only ASCII lowercase letters, numbers and underscore are allowed. The docs could be clarified on this point:
https://home-assistant.io/docs/configuration/troubleshooting/#entity-names
If you can please provide a PR for the docs.

In that case, the bug is that there's no error message, and the automation is saved into the config (but not loaded) when it should fail properly.


build fails due to new yargs-parser dependency update

When you run yarn build it will throw the following error:
yarn run v1.22.17
$ yarn build:lib && yarn build:changelog
$ tsc
src/params.ts:16:14 - error TS2322: Type 'string | number' is not assignable to type 'string'.
  Type 'number' is not assignable to type 'string'.

16     return { generator, action, templates }
                ~~~~~~~~~

  src/types.ts:48:3
    48   generator: string
         ~~~~~~~~~
    The expected type comes from property 'generator' which is declared here on type 'ParamsResult'

src/params.ts:16:25 - error TS2322: Type 'string | number' is not assignable to type 'string'.
  Type 'number' is not assignable to type 'string'.

16     return { generator, action, templates }
                           ~~~~~~

  src/types.ts:49:3
    49   action: string
         ~~~~~~
    The expected type comes from property 'action' which is declared here on type 'ParamsResult'

src/params.ts:18:42 - error TS2339: Property 'split' does not exist on type 'string | number'.
  Property 'split' does not exist on type 'number'.

18   const [mainAction, subaction] = action.split(':')
                                            ~~~~~

src/params.ts:20:45 - error TS2345: Argument of type 'string | number' is not assignable to parameter of type 'string'.
  Type 'number' is not assignable to type 'string'.

20   const actionfolder = path.join(templates, generator, mainAction)
                                               ~~~~~~~~~

src/params.ts:43:3 - error TS2322: Type '{ templates: string; actionfolder: string; generator: string | number; action: string | number; subaction: any; } & { [argName: string]: any; '--'?: (string | number)[]; } & { name: string | number; }' is not assignable to type 'ParamsResult'.
  Type '{ templates: string; actionfolder: string; generator: string | number; action: string | number; subaction: any; } & { [argName: string]: any; '--'?: (string | number)[]; } & { name: string | number; }' is not assignable to type '{ templates: string; generator: string; action: string; subaction?: string; actionfolder?: string; name?: string; dry?: boolean; }'.
    Types of property 'generator' are incompatible.
      Type 'string | number' is not assignable to type 'string'.
        Type 'number' is not assignable to type 'string'.

43   return args
     ~~~~~~~~~~~

Found 5 errors in the same file, starting at: src/params.ts:16

This is caused by the new version of yargs-parser -> "^21.0.0"
This file should be changed  in order to comply with the new types.

This is strange, on CI build works, incl. my dev env.. any idea what's changed?

I've created a PR with the changes (604063d, it seems yargs-parser now can return string | number. Typescript is complaining about it.
To reproduce, try to delete node_modules package-lock.json and check if the error occur.

Are you using yarn or npm to install, because we have no pacakge-lock.json in the repo at all (if npm, can you try yarn?)

I get the same error running yarn build

you are right! package-lock.json is the cause of the problem!
Feel free to decline the PR if you want 😄

I mean, it would be great to have this fixed to work in both npm and yarn


New EC2 region "Asia Pacific (Seoul)"

Looks good.
Are you adding a spec test or perhaps somehow incorporating this region in thttps://github.com/ManageIQ/manageiq/blob/master/spec/models/manageiq/providers/amazon/cloud_manager_spec.rb ?

@jvlcek That's a good question.  We probably should have specs for the region stuff, but I don't think it belongs in cloud_manager_spec.  Probably some kind of spec/models/manageiq/providers/amazon/regions_spec.rb file, but I'm not sure exactly what we should test.


PowerToys Run default key shortcut

Description of the new feature / enhancement
Choose a different default keyboard shortcut for PowerToys Run.
Scenario when this would be used?
In all installations.
Supporting information
PowerToys Run defaults to alt+space as the keyboard command. This is a very poor choice of defaults, as it overrides the alt+space window control that has been part of Windows for decades.
Of course, people can change this combination but this default is like overriding ctrl+c. Why make things more difficult for users, who will obviously not expect a long-standing shortcut to be overridden?
The shortcut editor also does not display this combination as possible interference, which is kind of amazing.

/dup #3838


Adding var file for Red Hat 7 and example playbook

Running this example on a RHEL installs EPEL. Use at your own risk

Hi @as0bu, thanks for the pull request.
I notice that vars/redhat-7.yml is not any different from vars/centos-7.yml. Since RedHat would be the OS family for both CentOS and RedHat, could I have you either amend your commit or push a commit to remove the CentOS vars file so that there's no unneeded duplication?
Other than that, this PR looks fine. Test build errors seem to be transient. I'd restart them, but will wait for the above change.

I need to fix one other thing as well. The default PostgreSQL version in Red Hat 7 is too old for jsonb. So I'll be updating the example I provided in an earlier commit.

The other example uses a separate role to fulfill the 9.3 postgres dependency, actually. See https://github.com/lae/ansible-role-netbox/blob/master/examples/playbook_single_host_deploy.yml#L5 (and line 17)
While I don't mind having the method you have in the example, what are your thoughts? Keep as-is?

@lae I think we should keep that as is. The problem is that the default repos for Red Hat 7 only have PostgreSQL version 9.2, unless you enable SCL which I don't think will work for this. I think for anything other than Red Hat, that example works as expected.


Error on importing local package

Description
I downloaded the .yymp and imported it into a new project.

Expected Change
It shouldn't error at import
Steps To Reproduce

Start GameMaker
Import the .yymps under "Import Local Package"
Error

Output:
Feather finished analyzing project in 211ms
""
elapsed time 00:00:05.4746499s for command "C:\ProgramData\GameMakerStudio2\Cache\runtimes\runtime-2<IP_ADDRESS>/bin/projecttool/windows/x64/ProjectTool.exe" PROJECT SAVE SOURCE="C:\Users\samba\OneDrive\Desktop\Projects\Game Development\Packages\Twitch.Ext.v0.0.1.yymps" DESTINATION="C:\Users\samba\AppData\Local\GameMakerStudio2\GMS2TEMP\Twitch.Ext.v0.0.1_07459d67\extract\Twitch.yyp" started at 09/21/2024 06:25:25
""
elapsed time 00:00:01.6072577s for command "C:\ProgramData\GameMakerStudio2\Cache\runtimes\runtime-2<IP_ADDRESS>/bin/projecttool/windows/x64/ProjectTool.exe" PROJECT SAVE SOURCE="C:\Users\samba\OneDrive\Desktop\Projects\Game Development\Packages\Twitch.Ext.v0.0.1.yymps" DESTINATION="C:\Users\samba\AppData\Local\GameMakerStudio2\GMS2TEMP\Twitch.Ext.v0.0.1_7b07905c\extract\Twitch.yyp" started at 09/21/2024 06:25:42
How reliably can you recreate this issue using your steps above?
Always
Which version of GameMaker are you reporting this issue for?
Monthly
Which platform(s) are you seeing the problem on?
Windows
Compiler Log File

[X] I have attached my compiler log in a text file

Sample .yyz File

[ ] I have attached a small sample project

This should be fixed in new 1.0.0 release.
If this is still a problem please open a new issue.


Docs: Removed bs4 examples in favor of bs5

Hey, just wanted to let you know that I removed bootstrap4 from https://github.com/mandrasch/accessible-menu-bootstrap-examples.
I rewrote the project with laravel-mix compiler and can't include bs4 and bs5 at same time by now.  Guess people use bootstrap5 most of the time now anyways.
Best regards

@mandrasch Cool, thanks for letting me know!
I'll remove it from this particular repo and make sure its pushed up to the BS5 project.

Removed and merged.


Error when attempting use insert on  manually added cursors

Hey,
whenever I  add a cursor with the suggested mapping of the example config and afterwards try to insert/append anything:
      vim.keymap.set({ "n", "v" }, "<leader>cn", function()
        if mc.cursorsEnabled() then
          -- Stop other cursors from moving.
          -- This allows you to reposition the main cursor.
          mc.disableCursors()
        else
          mc.addCursor()
        end
      end)

I run into the following crash as soon as I hit esc:
Error detected while processing SafeState Autocommands for "*":
Error executing lua callback: ...multicursor.nvim/lua/multicursor-nvim/cursor-manager
.lua:1005: attempt to index field '_redoChangePos' (a nil value)
stack traceback:
        ...multicursor.nvim/lua/multicursor-nvim/cursor-manager.lua:1005: in function
 'packRedoCursors'
        ...multicursor.nvim/lua/multicursor-nvim/cursor-manager.lua:1289: in function
 'cursorContextUpdate'
        ...multicursor.nvim/lua/multicursor-nvim/cursor-manager.lua:1344: in function
 'update'
        .../multicursor.nvim/lua/multicursor-nvim/input-manager.lua:185: in function
'_onSafeState'
        .../multicursor.nvim/lua/multicursor-nvim/input-manager.lua:50: in function <
.../multicursor.nvim/lua/multicursor-nvim/input-manager.lua:49>

Please try :Lazy update and let me know if it is fixed.

Hey @jake-stewart, yes - this seems to have fixed the issue. Thanks :)
I once got another crash, however I can't quite reproduce it (yet). I'll let you know if I encounter it again

i found and fixed a crash & undo bug which was related to the changes i made to fix this. maybe it was the same one.


Write the documentation for the extension

As this extension is getting bigger, it would be a good idea to have a docs directory where everything is inside and people can read it.

I might want to setup a website using https://github.com/facebook/flow/tree/master/website


Fix possible crash when clicking on Skeleton2D or Sprite editor options

Partially fixes #16891

I'd prefer they're hidden than making an error message pops up when clicking on it.

@Noshyaar Ok, I hide these plugins by default, so they will likely be showed when appropriate sprites or skeletons is selected, and reduces the possibility of crash when checking for (!node)

Yeah, I rewrited it, thanks @akien-mga


Update for 0.8.9?

So, I love this mod and prepared tons of stuff in my upcoming campaign around it but when I started testing, it seems to me that a lot of stuff is broken:

First and foremost, looting works fine, but when a player tries to move an item to a chest (loot sheet token created to serve as a stash), it copies the item instead of moving it.
something weird keeps happening with currencies, looks like loot sheet currency system is different from the one used by a default character sheet? I think I've read that Foundry changed its' currency system but loot sheet stayed the same?
There seems to be no option to sell loot. I covered that a little in one of the other issue topics.

So, my question is, is there any hope to get updates on this mod? Because, again, I love it and don't want the campaign without it? ;)

Agreed. Currency seems to be glitched rn with the new 5e update today

I wouldn't suggest people attempt this change as I've hosed the coin in my loot sheet a few times doing this.
There were a lot of changes in dnd5e to allow for currency localization, I suspect this has caused some issues.
By editing Lines 121 and 153 in npc-sheet.html  from {{ lookup ../config.currencies i }}: to {{lookup (lookup ../config.currencies i) "label"}}: I was able to get the currency labels back into the sheet. However the split isn't working yet, it just doesn't display any values.
Getting into lootsheetnpc5e.js I changed 1429 - 1435 from:
        const conversionRates = {
            "pp": 1,
            "gp": CONFIG.DND5E.currencyConversion.gp.each,
            "ep": CONFIG.DND5E.currencyConversion.ep.each,
            "sp": CONFIG.DND5E.currencyConversion.sp.each,
            "cp": CONFIG.DND5E.currencyConversion.cp.each
        };

to
        const conversionRates = {
            "pp": 1,
            "gp": CONFIG.DND5E.currencies.gp.conversion.each,
            "ep": CONFIG.DND5E.currencies.ep.conversion.each,
            "sp": CONFIG.DND5E.currencies.sp.conversion.each,
            "cp": CONFIG.DND5E.currencies.cp.conversion.each
        };

This hasn't made a difference but I suspect it's required as the old methods seem to have been replaced in dnd5e. I don't have this working yet but I thought I'd share what I've found in case anyone else has any ideas.
I suspect the issue lies with how the .js files does the currencySplit and how it populates c.value on line 122 of the html, but I might not be a strong enough programmer to figure it out.

It turns out that the currencySplit that was loaded when the sheet is opened or changed is the only one not using currencysplit[c].value. I'm not sure why.
I've done some limited testing on my world and things seem OK, I'm not using all the features of the mod though so this should be checked before merging. I've created this PR.
#181
If you want to test yourself my fork is here: https://github.com/Cynicide/fvtt-loot-sheet-npc-5e Please please please backup your world before trying this. JS is not my primary language, I just wanted to get this up and going so I could prep for my game on Tuesday.

Also experiencing this same bug. Tanked my game mid-session when the party realized all their loot was bunked. Oops.


Jack detection on some devices

Some phones (Spreadtrum reference design for instance) uses the "T" pin to detect jack presence.
Considering it is left floating in the schematic, on those devices, the jack plug is not detected.
Wiring a 100-Ohms resistor on the pin fixed the issue.

Thanks! Added a resistor in the schematic & PCB. This should get into the next mfg run.


competition_name and competition_urls incorrectly configured

These are not defined in config.py and this was causing email.py to break here
This causes further issues when exceptions are thrown in logger.py here
PR #138 fixes this by adding this to the CTF settings and the Management tab of the admin configuration page

See https://github.com/picoCTF/picoCTF/pull/138


initial commit

This is the first Pull Request of this repo. It contains all the first version of the backend.


Use individual component sass

Update frontend to import the sass for only the components it is using, instead of all components, as documented here.
Filesize comparison
Before:

Total 900 KB
After:

Total 562 KB
Additional changes
Also update foreign travel advice to use the warning text component. This was previously not possible because the component did not allow for a heading but has been updated to allow this.
This is being done as part of this PR because this page used styles from the component but not the component itself - so I noticed it had broken when I made the Sass change (because the warning text component is not used anywhere, it was not part of the 'suggested sass' output. It now is).

Trello card: https://trello.com/c/GWttlBxs/228-upgrade-applications-to-use-individual-component-sass

@alex-ju is both of us switching to the warning component going to cause you a merge conflict, or are we doing exactly the same thing?

yeah, I'll have a conflict, but will rebase, don't worry.


Fixed cooperation buttons divider

[x] fixed cooperation buttons divider styles

In the process, I spotted that in the TabNavigation component, it was not possible to update default styles on line 51, so I fixed that and ensured that it wouldn't cause any problems on other pages where we use the TabNavigation component
screen-capture (4).webm

✅


Jason Bokesch

Created CodePen project and answered questions on the text file. Worked through the Lambda School Git Flow.

Could not add reviewer


[1.12] v4.0.1 Players disconnected from server with 'Fatally missing blocks and items' error

Issue Description:
After creating a valid crafttweaker zs recipe and restarting the server, when attempting to connect to the server, players are disconnected with a 'Fatally missing blocks and items' error. Mods being used include Actually Additions and Mystical Agriculture.
What happens:
I add the script to the scripts directory and restart the server. When I attempt to reconnect to the server, it kicks me out with "Fatally missing blocks and items".
What you expected to happen:
I expected to connect to the server successfully and see the new recipe in JEI / craft it.
Script used (Please Pastebin or gist your script, posting an unpasted or ungist'd script will automatically close this issue):
https://pastebin.com/8eSgcAD6
Minetweaker.log file (Please Pastebin or gist your file, posting an unpasted or ungist'd file will automatically close this issue):
https://pastebin.com/c9GZZzRg

Affected Versions (Do not use "latest"):

Minecraft: 1.12
Forge: forge-1.12-<IP_ADDRESS>3-universal
Crafttweaker: Crafttweaker2-1.12-4.0.1

Your most recent log file where the issue was present:
[pastebin/gist/etc link here]

Are the scripts on the client?

Addendum: I tested it with the script on the client and was able to successfully connect to the server.
Is that going to be a requirement with 1.12, or is there a way to bypass that? Because it's going to be a bit awkward to push a pack update for every single script...

No way to bypass that, sorry

Damn, alright. Thanks anyway.


Adding extra options to Kubernetes object generator for distributed example.

This enables multiple training clusters to be deployed in the same namespace.

Can one of the admins verify this patch?

No opinion about whether to merge, but I think these scripts should be used purely for testing and not a demonstration of how to run distributed TF :)

@jhseu Agreed. These scripts should be treated as they are, i.e., tests (they live in the folder called dist_test), not examples, even though some users might find bits and pieces of useful.

Closing for now as these changes aren't needed for testing. @caisq feel free to reopen if you disagree.

@caisq @jhseu The hardcoding of the objects into the generator script is definitely a shortcoming, and the data is baked into the containers, but aside from that what are the other shortcomings of this test?
I ask because I'm totally new to tensorflow, and was trying to prototype a kubernetes third party resource to help developers quickly create training clusters: https://github.com/elsonrodriguez/tensorsets
What's a better script/test to base this type of automation on?


update quickstart docs

found some changes in my terminal output, so I updated the quickstart with that output as well as the correct commands.
Signed-off-by: Matthew Fisher<EMAIL_ADDRESS>

the quickstart was removed and the tutorial has since been updated with new information. Closing.


87 add about page report to scheduler

Added Pangaea datasets to gleaner and nabu config
Added summary reports for all and 3 communities

This needs to merge into dev... and not dv_dev.


[Android] Expose embedding apis for video fullscreen.

The onShow/HideCustomView() was implemented in Crosswalk by default and
not exposed, it could not be overrided by customer to implement the video
fullscreen.
This patch is to expose onShow/HideCustomView().
BUG=XWALK-6939

@fujunwei @axinging @lincsoon @xzhan96 PTAL, thanks.

@wuhengzhi is this PR still vaild?

Yes, I will answer your questions later. Please wait a moment, thanks.

ping @lincsoon @axinging

Are the discussions above all settled?


Xtensa High priority interrupts cannot be masked during initialization

Describe the bug
High priority interrupts cannot be masked during initialization
Once the high-priority interrupt is enabled later before the interrupt stack is initialized and the interrupt occurs again, there will be issues.
XCHAL_EXCM_LEVEL cannot disable all interrupts, unless the architecture does not have high priority interrupts, we can leave only debug interrupts and NMI interrupts
file :crt1.s
# ifdef __XTENSA_CALL0_ABI__
	/*
	 * PS.WOE = 0
	 * PS.UM = 1
	 * PS.EXCM = 0
	 * PS.INTLEVEL = XCHAL_EXCM_LEVEL
	 */
	movi	a3, PS_UM|PS_INTLEVEL(XCHAL_EXCM_LEVEL)
# else
	/*
	 * PS.WOE = 1
	 * PS.UM = 1
	 * PS.EXCM = 0
	 * PS.INTLEVEL =XCHAL_EXCM_LEVEL
	 */
	movi	a3, PS_UM|PS_WOE|PS_INTLEVEL(XCHAL_EXCM_LEVEL)
# endif

My fix
#if XCHAL_HAVE_EXCEPTIONS
# ifdef __XTENSA_CALL0_ABI__
	/*
	 * PS.WOE = 0
	 * PS.UM = 1
	 * PS.EXCM = 0
	 * PS.INTLEVEL = XCHAL_DEBUGLEVEL-1
	 */
	movi	a3, PS_UM|PS_INTLEVEL(XCHAL_DEBUGLEVEL-1)
# else
	/*
	 * PS.WOE = 1
	 * PS.UM = 1
	 * PS.EXCM = 0
	 * PS.INTLEVEL = XCHAL_DEBUGLEVEL-1
	 */
	movi	a3, PS_UM|PS_WOE|PS_INTLEVEL(XCHAL_DEBUGLEVEL-1)
# endif

The same here uses XCHAL_EXCM_LEVEL. Some architectures also cannot disable high-priority interrupts.
I suggest using XCHAL_NUM_INTLEVELS -1 or XCHAL_DEBUGLEVEL-1, which may be better
static ALWAYS_INLINE unsigned int arch_irq_lock(void)
{
	unsigned int key;

	__asm__ volatile("rsil %0, %1"
			 : "=r"(key) : "i"(XCHAL_EXCM_LEVEL) : "memory");
	return key;
}

zephyr V2.7

This is the same comment from a different bug:  That's not a Zephyr interrupt.  It's not a Zephyr handler.  If you want to interrupt the OS at arbitrary times with an interrupt that it does not mask, you need to write a safe handler for that interrupt.
I don't know what you want the OS to do for you?

These are the xtensa arch code, it exists in zephyr, I think it should be reusable by most of the xtensa architecture, so I think this should be part of the zephyrrtos xtensa architecture. Of course I can change and PR

I'm going to close this one, as it's effectively a duplicate of #41039
Zephyr doesn't have a solution currently for applications registering interrupts at or beyond EXCM_LEVEL.  It would be a good feature if we had it, but we don't yet.


ODC-7398: add flag to check if TektonResult is installed

Story: ODC-7398
Descriptions: Add a flag to check if TektonResults is installed on the cluster.
Doc to install TektonResults on the cluster: https://gist.github.com/vikram-raj/257d672a38eb2159b0368eaed8f8970a

Tested locally. Working fine
/lgtm


pid 359 (EShop) does not have sandbox access for frZQaeyWLUvLjeuEK43hmg and IS NOT appropriately entitled

pid 359 (EShop) does not have sandbox access for frZQaeyWLUvLjeuEK43hmg and IS NOT appropriately entitled
how to fix this warnning

Sounds like you have an issue in your MacOS not having properly configured sandbox entitlements. This isn't a Starscream issue. You can read about App sandbox here:
https://developer.apple.com/library/content/documentation/Security/Conceptual/AppSandboxDesignGuide/AboutAppSandbox/AboutAppSandbox.html#//apple_ref/doc/uid/TP40011183-CH1-SW1


Add command for building contracts

What
Add command for building contracts:
soroban contract build

Why
The commands for building contracts are getting increasingly complex, e.g.:
cargo rustc --crate-type cdylib --target wasm32-unknown-unknown --release

A build command on soroban-cli has been requested many times. Most recently at: https://discord.com/channels/897514728459468821/1121218101489447054/1121219305133064284
Related to https://github.com/stellar/soroban-docs/pull/476

Should we check the rustc version to make sure the output is what we expect it to be ?

I don't think so because we're doing a few things to make sure that a change in rustc would be unlikely to break soroban-cli:

The cargo_metadata crate we're using passes --format-version 1 to the cargo metadata command, to ensure if the output is changed we continue to get the version 1 output.

The subcommand only uses stable cargo and rustc options, meaning that these options are highly unlikely to change.

There are no dependencies on the stdout or stderr output of the rustc command, other than assuming a non-zero error code is a failure.

The location of .wasm files is unlikely to change because there are many tools that assume the location of artifacts in cargo's target directory structure. Even if it was surprisingly changed on us (unlikely), it would only break the --out-dir option, which we could patch quickly. To my knowledge there's no way we could detect this ahead of time. We read the target directory location from the metadata, but nothing in the cargo metadata could tell us the location of the artifact.


AMQP and web support?

I know the motto says right there: "Just HTTP!", but what if I wanted to use this same "micro"-approach and use AMQP as the transport layer? Could this be done without much hassle? Is there any extension or plugin for this?
Also, if I'm tied to HTTP for transport: what's the benefit of using the framework compared to, say, plain http from Node, Express, Hapi, Koa, Restify and the like?
In fact, how does micro play along with web frameworks such as those? Is it possible to integrate them in some way? Could I, for instance, define API routes in Express and send messages over to another microservice?

@nfantone
The goal is to avoid the pattern of multiple route definitions per service. Each microservice performs a single function.
In terms of why not native Node, the main reason is that it's not designed around async / await, and you'll probably roll out an ad-hoc solution like micro every time you want to use those. You'll create Promise-friendly versions of body parsing like micro.json, you'll do exception handling, etc.
You can certainly use this same technique for AMPQ:
export default async function ampqHandler () {
  /* … */
}

and then run it micro-style with your own tool:
$ ampq my-handler.js

As far as aggregating the micro-services into paths, I recommend NginX upstreams. That way api.yourservice.com/woot can forward traffic to the woot microservice, and many different paths can co-exist under the same host. This will benefit browser lookups, will unify SSL termination, etc.


Major enhancements to SimpleStats and HitCounter

Hello, I would be very pleased if you would take my enhancements into your core product.
Current version
Simple Stats (data collection, Dashboard and separate Hit Counter)
Issues:

Count goes up on each page refresh
Collection of data fixed at 7 days needing a code change to alter.

Old Functionality:

Only collects daily results (fixed 7 days)
Hit counter only shows the current days hit rate.

New enhanced version
Simple Stats (data collection, Dashboard and separate Hit Counter)
Fixed:

A session for each page is started preventing constant page refreshing artificially increasing the count
Collection Periods are all parametrised and configurable in settings

New Functionality:

Enhanced – Daily stats retention now has variable data retention(able to independently turn off)
New –  Weekly stats captured for x weeks (able to independently turn off)
New – Monthly stats captured for x months  (able to independently turn off)
New – Page session with configurable timeout
New – Running total of all page hits captured
All existing Languages expanded for all changes plus some added
Dashboard - Can change the displayed chart type within plugin settings
New – Dashboard now also shows total page count
Total Page count can be set via a combination of admin settings and visiting a web page to trigger.
Enhanced - Hit Counter: sidebar independently displays all current collection data as configured in settings

Not done yet, but want to do

Display all charts within Tab Navigation without having to change settings
Individual rolling page count

Images:

Hi, I would like to keep as simple as it is.
You can create a plugin separated, like Simplestats+ or something like you want, don't feel I don't like what you did because for me is really cool, but I want to keep really simple the core then improve it with plugins.

Hi, that's a shame, especially as it keeps the simplicity of the original very simple stats plugin, but just enhanced. All the enhancementsbcan be turned off if people want it really simple, but it's your decision.
Thanks anyway .

@BlakesHeaven did you end up publishing this as an improved plugin?

Hi @DivineDomination
Yes I did, see https://plugins.bludit.com/plugin/simple-stats-plus

This pull request is now behind the plugin so I'll close it.