Page titles in "" don't show in sulu navigation

@wachterjohannes Is this still an issue?

i cannot reproduce it anymore ... but i will ask @BettinaKaljo for that issue!

still existing. see the attaches screenshot:


Add 'My pool balance' card to invest flow

Description
This PR adds a card to the right of the investment form that displays any existing pool balance the user has. The balances for individual pool tokens are derived from a proportional exit using their full BPT balance.
Again, this PR does not cover the mobile layout, that will be done in a later PR when all the page elements are included.
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Other

How should this be tested?
Please provide instructions so we can test. Please also list any relevant details for your test configuration.

[ ] Test correct pool balances and total are displayed for a pool you are invested in.
[ ] Test that 0s are displayed for a pool that you are not invested in but have a wallet connected.
[ ] Test the 0s are displayed when you don't have a wallet connected.

Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code where relevant, particularly in hard-to-understand areas
[x] My changes generate no new console warnings
[x] The base of this PR is master if hotfix, develop if not

@evgenyboxer

the previous design also showed how much % you have in each token, I guess it was dismissed?
It only showed the pool weights essentially, which we show in the main form. We could potentially add to this side module as well but I'm not sure it will add much.


Potential performance regressions in 3.0.0

Originally reported in https://github.com/sass/libsass/issues/1205.
Anecdotally I believe 3.0.0 has introduced a noticeable performance regression. Currently I suspect the recent binding updates foe custom functions and importers but I have no evidence as yet.

There is an old test called "Huge" (something) in spec repo, which vividly underscores this dramatic difference in performance between node-sass 1.x, 2.x and 3.x. If we target that test only, we can probably figure out the source of issue using some profiler tool (or just by tracing the call stack like how @mgreter do for source-map?).

Having tried to upgrade a project I work on to node-sass 3.0.0 to get the awesome of libsass 3.2.4 I ran into significant performance issues.
I have been using all the latest newness on much smaller project, less then 50 Sass files. But when I tried to upgrade a larger project with hundreds of files, I ran into the issues of;

there is no log output of the files created from the processing
there is a HUGE reduction in speed of processing

The following is a listing of versions that diff between the master branch and the upgrade branch.
master
/project/package.json "gulp-sass": "^1.3.3",
gulp-sass installed "version": "1.3.3",
/project/node_modules/gulp-sass/package.json "node-sass": "^2.0.1",
node-sass installed "version": "2.1.1", installed
/project/node_modules/gulp-sass/node_modules/node-sass "libsass": "3.1.0",
[07:44:21] Finished 'build:css' after 31 s

libsass upgrade branch
/project/package.json"gulp-sass": "^2.0.1",
gulp-sass installed "version": "2.0.1",
/project/node_modules/gulp-sass/package.json "node-sass": "^3.0.0",
node-sass installed "version": "3.1.2",
/project/node_modules/gulp-sass/node_modules/node-sass "libsass": "3.2.4",
[07:35:20] Finished 'build:css' after 3.15 min

In both cases, the Sass was expected to process 93 individual SCSS files due to our extended theme development process, not counting all the imported dependencies.

I am researching https://github.com/jtangelder/sass-loader/issues/100 now. This can be actually #857 but it needs to be verified.

@saper the regression isn't limited to sass-loader. The node-sass CLI is
also affected. My gut says something with the recent custom importer and
custom function binding is to blame.
On 4 Jun 2015 12:12, "Marcin Cieślak"<EMAIL_ADDRESS>wrote:

I am researching jtangelder/sass-loader#100
https://github.com/jtangelder/sass-loader/issues/100 now. This can be
actually #857 https://github.com/sass/node-sass/issues/857 but it needs
to be verified.
—
Reply to this email directly or view it on GitHub
https://github.com/sass/node-sass/issues/970#issuecomment-109015920.

@xzyfer yes, of course it has nothing to do with the sass-loader. Maybe some multi-file code triggers #857 ? #857 is relatively easy to detect if setting UV_THREADPOOL_SIZE to some large number (>number of opened files) fixes the issue.

Honestly I'm not convinced this is related to the thread pool. The larger
issue is that node-sass is slower at a base line. I ran some naive tests  a
while ago. Files that compile on 100ms in libsass takes 300ms via node-sass.
On 4 Jun 2015 13:39, "Marcin Cieślak"<EMAIL_ADDRESS>wrote:

@xzyfer https://github.com/xzyfer yes, of course it has nothing to do
with the sass-loader. Maybe some multi-file code triggers #857
https://github.com/sass/node-sass/issues/857 ? #857
https://github.com/sass/node-sass/issues/857 is relatively easy to
detect if setting UV_THREADPOOL_SIZE to some large number (>number of
opened files) fixes the issue.
—
Reply to this email directly or view it on GitHub
https://github.com/sass/node-sass/issues/970#issuecomment-109042463.

Is there any progree on this happening? Otherwise I'll have to instruct our team to possibly downgrade again to the previous version.
Our static compiles have gone down from 10 secs to 2 minutes.

We are also seeing performance differences between<EMAIL_ADDRESS>and<EMAIL_ADDRESS>Our compile times went up from 2.2s to 3.9s (an increase of 77%). Tested with<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

I got the same issue after upgrading to node-sass 3.2 (using gulp-sass). The compilation time increased from ~150-250ms up to 800-1.200ms. Will probably downgrade, as long as this issue isn't solved.

I've been noticing long compile times after upgrading gulp-sass to the most recent version (which bumped up to node-sass 3.0.0 -- I've managed to track the issue down to the upgrade of the node-sass module
After regressing to node-sass 2.1.1 the issue has gone away.
on 3.0.0 compilation takes ~5.5 seconds. Now it's taking ~2.5

Same issue here, the current performance makes the builds way too slow. Gonna downgrade until the issue is resolved.

Stuck to @2.1.1 too :

I've gone from subsecond compile times to 8s. Not acceptable. 2.1.1 looks like the only solution for the time being...

@Telemakhos  @quantix-studio I find these hard to believe, I'd be like to see what you're doing.
The regression at worst should make compilations twice as slow between 2.1.1 and 3.2.5. Which given the speed of node-sass isn't that bad.
I'd like more info about your respective projects i.e. asset pipelines grunt/gulp, libraries bourbon/susy/boostrap.. etc The more info the better.
Finally for those who are down grading I can't urge you enough not to do this. You're willing installing a buggy version of LibSass, and inheriting know installation issues with node-sass. If you want this fixed quickly we need your help in creating simple benchmarks that show the regression so we know how best to proceed.

With  every version we have new bugs :) First we need to isolate the problem - is this related to custom functions/importers or is it just libsass change.
A kind of minimal, dummy project that compiles slower would be perfect!

We've got a pretty massive Sass file with 80ish @imports. We use grunt to compile, and updating grunt-sass to version 1.0.0 (which in turn updated node-sass from 2.1.1 to 3.2.0) introduced some massive performance hits like are being discussed here. We were compiling in a couple seconds or less but now it's an unusable 20-30s. Compile time does go down when I remove the lengthiest @imported files (e.g., I can remove one 3000 line @import and gain about 2.5s back, remove another 2500 line @import and gain about 2s back, etc). Removing smaller imported files has less of an impact on compile time.

@levinelson can you ascertain whether it's the size of the file or the number of imports within the file you're removing.
i.e. if you remove a large with no imports do you see the speed up as a file with a large number of imports?

@xzyfer As near as I can tell, its just the sheer length of the imported Sass files that slow it down.
The baseline time with all imports as normal was 25.3s (same process that was sub 3s before update). Our file structure is pretty flat:
app.scss
|-- _settings.scss
|-- _app-main.scss
     |-- 80ish imported sass files (no further imports)

Out of the 80ish imports, I can remove the just the largest 7 imports (236 KB of the total 439 KB un-minified Sass files) and get the compile time down to 4.7s.
I put one of the large imports back in the pipeline (75KB file, 3338 lines), and the compile time went back to 9.7s. I tried taking the code that's in that large import and just putting it directly in our main app-main.scss stylesheet and compile time only went down slightly to 9.6s.
That large file (like most of the other files) is full of Bourbon mixins and other custom mixins. I tried removing those I could without completely breaking the compile process and reduced the file from 3338 lines to 2725 lines and got compile down to 7.4s.
The last thing I tried was removing one layer of imports (importing everything in app.scss instead of one layer deeper in app-main.scss). This resulted in no material change (25s just like the original).
Let me know if I can get you any more information. Appreciate the help!

I'm going to echo what @levinelson has said. Our scss codebases are medium-sized with about 70 scss files using a SMACSS architecture, only for custom sass. We also import bootstrap sass, bourbon, and other vendor libs.
We use gulp with swiip's gulp-angular as the base boilerplate. This boilerplate has also experienced an increase in twice the amount of compile time only with generated with the defaults, when you start adding stylesheets things go south.

I've done some quick investigation on this.
Setup
I created a simple boilerplate benchmark which consisted of 11,000 lines of
foo { bar: baz; }

Results

node-sass 2.1.1
node-sass 3.0.0
sassc 3.1.0
sassc 3.2.0
sassc 3.2.5
sassc master

0.5s
1.3s
0.5s
1.3.s
1.3s
0.35s

Summary
This really surprised me for a couple reasons.
We were are that LibSass 3.2.0 could be slightly slower due to some of the changes in that massive release. We had no idea of the extent of the regression.
Secondly, I had long suspected the Node Sass binding code was to for the speed regression, since we also heavily updated it in<EMAIL_ADDRESS>In fact this small test shows that there is very little over head in this layer which is very exciting!
Conclusion
This appears to be LibSass' fault, which puts it's out of our control.
We've been working on performance in LibSass and our next release will be significantly fast as the above benchmark shows.

Unfortunately I can reproduce the performance regression on windows with node-sass. I do not see it on linux where I get the expected speed, but on windows it runs about 10 times slower than expected. To reproduce it I simply created a pretty big file with some randomly nested selectors and properties:
https://github.com/mgreter/postcss-playground
# gulp sass
Using gulpfile postcss-playground\gulpfile.js
Starting 'sass'...
Finished 'sass' after 4.35 s

# gulp postcss
Using gulpfile postcss-playground\gulpfile.js
Starting 'postcss'...
Finished 'postcss' after 497 ms

# psass --benchmark test\nested.scss > psass\nested.css
 0 wallclock secs (0.4520 usr + 0.0310 sys = 0.4830 CPU)

@mgreter,
Setup:
On win10, I saved https://github.com/mgreter/postcss-playground/blob/master/test/nested.scss?raw=true as c:\temp\big.scss and then in powershell:
cd $env:UserProfile/Source/Repos
git clone https://github.com/sass/libsass ; cd libsass
git clone https://github.com/sass/sassc ; sassc/win/sassc.sln
# opened solution in vs2015

Auto diagnostics tool:
Then in vs2015 > Solution Explorer > Right clicked project > Properties > Debugging > and set Command Arguments to:

\temp\big.scss \temp\big.css

then changed configuration to Release and Alt+F2 (Debug > Start Diagnostics tools..) It eventually exported these reports: http://1drv.ms/1hJPdaG.
Manual benchmark:
After that, for the manual benchmarking, I used this function: http://stackoverflow.com/a/1861337, pasted it at the top of libsass/sassc/sass.c and applied the following patch:
diff --git a/sassc.c b/sassc.c
index a8dd220..23e5793 100644
--- a/sassc.c
+++ b/sassc.c
@@ -281,7 +331,12 @@ int main(int argc, char** argv) {
             strcat(source_map_file, extension);
             sass_option_set_source_map_file(options, source_map_file);
         }
+        uint64 t0 = GetTimeMs64();
+        printf("Started: %d\n", t0);
         result = compile_file(options, argv[optind], outfile);
+        uint64 t1 = GetTimeMs64();
+        printf("Ended: %d\n", t1);
+        printf("Difference: %d\n", t1-t0);
     } else {
         if (optind < argc) {
             outfile = argv[optind];
@@ -290,6 +345,6 @@ int main(int argc, char** argv) {
     }

     free(include_paths);
-
+    system("pause");
     return result;
 }

Ran 5 times and on average: with configuration Release:Win32, it gave 450-550ms and with Release:Win64, it was around 500-620ms.

It seems to be fixed with latest master! Did pretty much the same test and found out that producing the output is actually the slow partI don't know why it was triggering that regressions, but it mainly seemed to came from appending to strings. But I didn't look much further as using the lastest libsass master seemed to provide the expected speed. When I changed the output generating from using a string buffer to using ostringstream, I got decent results on MSVC but got a pretty big performance regression with gcc (as expected). Appending to a string should be faster than using ostringstream anyway!?
Long story short, lets wait for next release and try again!

Good news is that we have better performance than postcss, which is what I initially wanted to assess!

Does using string::append or operator+= fix the regression for both GCC and MSVCR?
Regarding the compiler bug, see https://gcc.gnu.org/bugzilla/show_bug.cgi?id=53221. Code involving certain string-related operations (including push_back()) compiled with GCC 4.6-4.9 will be much faster than if it is compiled with GCC5 (depending on which combination of operations you are using), because in 4x, it was avoiding the COW which the standard dictates and hence it was making a huge performance difference. In a perfect world, when "std::" will be more reliable and consistent cross-platform: "copy should take equal amount of time as move without no-except; while move with no-except should be much faster than copy and move without no-except". It turns out you can get very substantial performance improvements by not doing the right thing. 8-)

We are already using operator+=, but I also tried string::append and also tried to reserve the whole memory beforehand, but nothing gave the expected speed. Only changing it completely to use a stringstream solved the problem, and I fail to see why using strm << str vs buffer += str should have like 500ms vs 5s execution time (stringstream should in theory even be a bit slower)!? And I was not able to reproduce this with a very simple example.
The only reason it is slow seems to come from append_string in emitter.cpp:
wbuf.buffer += text

I found this article interesting: 4350% Performance Improvement with the StringBuilder for C++!. The class is built using the good parts of C++11 string functions stitched together. The code is redistributable commercially as per the code project license. I have added the extracted code here.
Do you think it is worth to give it a try, replacing OutputBuffer::buffer type from std::string to StringBuilder<wchar_t> and then use its .Append() and .AppendLine() functions?

As I said, it seems to be fixed in master, so I don't see a reason to change anything at that point ...

Yup, I can confirm that it is fixed when we compile sassc in Release configuration. With debug config, since the optimization flags are not set, it takes about ~10+ seconds to compile the big.scss file.
Since pangyp PR is pending the review: https://github.com/rvagg/pangyp/pull/9, I can't build node-sass locally with VS2015 at the moment. I requested CI build and tested with build artifact and it takes about half a seconds (347ms) to compile:
C:\Users\Adeel\source\Repos\node-sass [request-ci-artifacts]> node
> require('./').renderSync({file:'/temp/big.scss'})
{ stats:
   { entry: 'c:\\temp\\big.scss',
     start:<PHONE_NUMBER>033,
     includedFiles: [ 'c:/temp/big.scss' ],
     end:<PHONE_NUMBER>380,
     duration: 347 },
  css: <Buffer 2e 77 61 72 6e 69 6e 67 20 54 45 58 54 41 52 45 41 2e 6f 70 65 6e 20 23 63 61 72 6f 75 73 65 6c 2e 77 61
72 6e 69 6e 67 20 7b 0a 20 20 66 6f 6e 74 2d ... > }

Thanks to @mgreter and @xzyfer for the efforts. :+1:
This issue is fixed. Waiting for libsass vNext. :smiley:

this issue is resolved with the current version of node-sass?  or are you saying the issue is fixed in the latest version of libsass?  if it's fixed in libsass, has that version been updated in node-sass?

this issue is resolved with the current version of node-sass? or are
you saying the issue is fixed in the latest version of libsass? if
it's fixed in libsass, has that version been updated in node-sass?

You can try node-sass@beta, which includes a very recent libsass.
A lot of changed since then.

saw that, thank you!!!  works beautifully

Fixed in https://github.com/sass/node-sass/pull/1214. Will be in v3.4.0.


FieldNamesChecker thinks local vals are fields.

It thinks that ( is a field name and errors out.
object Foo {
  def foo(): Unit = {
    // Field name does not match the regular expression '[a-z][A-Za-z]*'
    val (thingOne, thingTwo) = (3, 4)
  }
}

Closed. Duplicate of #199


restore missing rule for active autocomplete entry

The CSS rule that highlights the active autocomplete list entry was missing in develop branch, but it exists both in master and on the live site, so I added it back.

Thanks


Feature: Update development tools to latest stable versions

Acceptance Criteria:

For each tool in ntools.json, update the tools to latest stable version

Resolved in #67


[FEATURE]: Integrate GaLore into Colossalai Optimizer(Gemini/Hybrid)

Describe the feature
A recent paper titled "GaLore: Memory-Efficient LLM Training by Gradient Low-Rank Projection" (https://arxiv.org/pdf/2403.03507.pdf) demonstrates a remarkable memory-efficient approach during the training of large language models (LLMs).
Can we integrate this memory-efficient technique into the Colossalai framework?
FYI
8bit-GaLora Adamw: https://github.com/jiaweizzhao/GaLore/blob/master/galore_torch/adamw8bit.py

Any ColossalAI-er could take a look?

Thanks! We will take a look.

I will take multiple looks

I see the MR, that's awesome, when can we use it?

I plan to release it next week


Merge v2 branch into master 🙌

Final merge before release of v2 🥳
To be merged after merge of #168.

I am just scrolling through the list of commits. Wow. So many memories! 🥲

Awesome! I'm out for lunch but will get v2 deployed when I'm home this afternoon! Thanks again for all your help!


Character animation with physijs

How would one apply physics on animated model such as collision, impulse etc.? I can't find anything on a subject in google.

@eddyuk,
The fact is that you probably have a very complex model. And Physijs does not have any real support for dynamic concave meshes (the documentation only said static if I remember correctly).
One solution, if you really wanted to keep the complexity of your mesh, would be to completely reconstruct your physical mesh (with Physijs.ConcaveMesh) so that it matches the one in your rendering window every frame. But that's a lot of work for a computer, not to mention all the wasted memory that will cause massive lag spikes during runtime. Plus, this would only work for collision detection and not response (unless, of course, you prefer making your custom forces).
Another way, which decreases physics accuracy but provides enough functionality, would be to encapsulate it into a Physijs.ConvexMesh. It's the closest thing to a concave mesh while supporting collision detection and response. You wouldn't even need to reconstruct it, just set it's position or velocity when you want to move it yourself (remember to use the .__dirtyPosition flag).
Finally, you could use a combination of THREE.js raycasting and some wrapper Physijs objects:
var raycaster = new THREE.Raycaster();

// Note: 1st parameter is three.js model (mesh),
// 2nd parameter is a physijs mesh that's going to collide with the model
function createModelCollider(model, obj){
  var crosses = [];
  var collider = new Physijs.SphereMesh(new THREE.SphereGeometry(1, 1, 1), fakeMaterial, 0);
  collider.visible = false;

  // Must find where to place collider object onto model using raycasting
  raycaster.set(obj.position.clone(), obj.getLinearVelocity().clone().normalize());
  crosses = raycaster.intersectObject(model, false);

  if(crosses.length){
    collider.position.copy(crosses[0].point);
  } else {
    return null;
  }

  // Object returned must be added to scene (can be removed after frame ends)
  return collider;
}

After that, all you need to do is wait for the collision event between the collider and the specified mesh. Tweak a few velocities from the callback, apply them to your model, and voila! Sure, this does sound like the 1st solution (because you are making your own forces) but you don't have to completely recreate your mesh. Plus, you can choose - even before it gets simulated - what collides with the model and what doesn't.

Thanks for the response! Actually, I am learning Three and Physi js and have in mind the aspect of game use case and trying logically to put it together with sample codes. So, was wondering if I have animated character, like running, jumping etc. I can use physijs for that in order to keep it on surface (terrain, floor or whatever) and not going through solid objects. I saw approach using raycast for that, but still don't entirely understand how to collaborate between physijs and threejs objects. probably will take time.

As I said before, you can't really use a reserved physics mesh for an entire character because that takes too many resources that could be spent elsewhere. You're gonna have to think what kind of obstacles are going to constrain your character and what kind of behaviour your avatar is going to have based on the size and shape of those obstacles, because trying to dump everything on the lap of a physics engine and waiting for results will end badly, even if it is running on a web worker.
The concept of raycasting is simple: throw a ray starting from said point in space in said direction (a normalized vector). The ray will go on forever (depending on implementation), testing for intersections with any objects it was given to evaluate. All you need to do is use the THREE.Raycaster object and configure it (set the origin to the object's position and the direction to its normalized linear velocity) to test for the objects you need. If it hits anything, its return value, an array, should be filled with 1 or more objects, describing the collision. All you need to do is create a Physijs object at wherever the collision with the model occured and add it to the scene. It will act as a "crash pad" for the object that will collide. The THREE.js documentation should help you out a little more.
I've also remembered another good way of implementing your model into the physical world: BVH (Bounding Volume Hierarchy). I haven't really looked into it, but the general idea is simple enough. At the base of your "tree", you have your model, with its bounding box. Now, you have 2 "branches" that split your model into 2 bounding volumes (a bounding box that encapsulates multiple objects). Then, each branch has 2 more branches that split into 4 branches that split into 8 branches... you get the idea. It reduces the number of checks onto your model via raycasting. After all, if the tree isn't hit, why check all the branches? The Wiki explains it here. There is also an example of it at work with an animated model and physics based from Oimo.js here.


fix(list): fix overlap in codebase list

fix the overlap in the codebase list due to adding a new element once a che workspace is created

Before Fix:

After Fix:

@mindreeper2420 snapshot fabric8-ui is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-1579-fabric8-ui.int.rdu2c.fabric8.io

@mindreeper2420 snapshot fabric8-ui is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-1579-fabric8-ui.int.rdu2c.fabric8.io


Drop stuck close execution transfer task

What changed?
Drop stuck close execution transfer task

Why?

How did you test it?

Potential risks

Coverage decreased (-0.02%) to 66.927% when pulling 2d89e9b97cd29ffe2b68cfdd823227752037ac9d on yycptt:drop-stuck-standby-tx-task into 01d2d1a475827257861b3e39846357d6f3e9def9 on uber:master.


Multi-atack items

Please add the ability to create objects with several separate attacks (as in pf1): this is a very necessary opportunity to implement, for example, an ax with a point on one side and a flat head on the other plus a blow with the handle. We've been waiting for this feature for a long time!

Yep, this feature is really important, make it please. Know it due to using this exact type of weapon)

Duplicate of https://github.com/foundryvtt/dnd5e/issues/461, which is more specifically going to be handled by some variant of this: https://github.com/foundryvtt/dnd5e/issues/1964


NPM install issue

npm i gives npm ERR! Could not install from "plugins/my-loaders" as it does not contain a package.json file.

👋 @EthanSK
Thanks for raising this issue.
I will close this issue since we do not struggle with this error when installing the website. Feel free to reopen the issue with more details to reproduce it if this error is still raised on your side.


add lavalink client

Hello.
I'd like to apply to add my own lavalink-client for v4 only.
In the past I've been maintaining my fork of erela.js, after some time, I decided to create a similar project with different branding and far more features + optimizations to not confess hate or other things.
I'm like 95% done for lavalink-features. and the only things that might change are: couple internal typing names. Or I'm gonna add features. In the docs it's state that i EST to finish it end of august, but I think i'll be done way earlier (in fact i already use it in production)
Also I'm currently writing excessive docs for it, so no worries.
It's both for beginners and profeciants. For beginners due to it's automated features like: Channel Delete detection, automated queue, simple typing style (almost the same like on lavalink) Validators and verificators (for example for links, plugins, sources). Which leads to less-errors during production. It's also suitable for advanced projects, hence it supports remote queue store(s) for stuff like redis, dragonflydb, psql, etc. etc. and included autoplay functions as well as high memory efficiency and modern code-style. That's why I chose to code it in typescript to support: old cjs, mjs, esm and ts code projects.
Additionally, if I get approved, I'd be greatful to be contacted on discord (chrissy8283) to retrieve my "own support channel" with Github integration webhook. Thank you very much.
If you want me to not name it "lavalink-client", I am flexible and can change it.
Note the contributer of the github project, didn't do anything of that lavalink client, only for an old project, who then made poru.
Also I applied sorting algorithmn to the Lavalink Clients:
1st Java Based Languages
then Languages sorted by alphabet
then Client-Names sorted by alphabet
Thanks in advance.

Thank you :3


As a user, I want to have a little avatar and see my and my opponent's avatar while playing, so that it is more fun

Acceptance Criteria

Users automatically get assigned an avatar, e.g. from DiceBear api
Users see their own avatar on the home page with their statistics
Users see their own avatar and their opponents avatar (as soon as available) in the lobby
Users see their own avatar and their opponents avatar (as soon as available) in the game
Users see their own avatar and their opponents avatar (as soon as available) in the end of round screen
Users see their own avatar and their opponents avatar (as soon as available) in the end of game screen

Development Tasks

[ ] when a new user is created, they are randomly assigned an Avatar (client or server)
[ ] display avatar on home page, lobby, game, end of round, end of game (client)
[ ] the server send the avatar together with statistics and together with game status updates

@trofej is this roughly what you had in mind for the avatar?


Apply shorten_and_uniqify to column table

Goal
This is stacked on PR https://github.com/ConservationMetrics/gc-scripts-hub/pull/44.
In a recent run for one of our users, I noticed that a very long table name resulted in an error when trying to create the columns mapping table. This is because the db_table_name had a length of 58 chars, and currently, we brute append __columns to the table name, thus violating Postgres's 63 char limit for table names.
This PR aims to solve that by applying the shorten_and_uniqify function (with a maxlen=54 argument) to the column table name before appending __columns.
What I changed

Tweaked the language in shorten_and_uniqify to reflect usage for many identifiers, not just keys.
At the very beginning of the handle_output method, create a columns_table_name string with the function applied with 54 maxlen and __column appended, and use that throughout.
Added a unit test to test the desired behavior for __column rewrites.

What I'm not doing here
Addressing a case when db_table_name itself is > 63 chars! Right now we use this argument as given and don't transform it in any way. But I thought to leave that for a different PR, because the same is true for alerts and comapeo connectors. Maybe there is a way to set a validation in the script.yaml file.

Would it be better to add a constraint on the Windmill input that you cannot type names > 54 chars in length?

Fair point! As I mentioned in the PR, we have to add a constraint for table names regardless.
I will still keep the changes so as to only write the __columns name once (with your suggestion to slice instead of using the uniqify function), but I'll open a new issue about char limit for table names across the scripts.


Document on Opacity that using a widget with RenderAnimatedOpacity is more efficient that building Opacity on animation

@xster should this ticket also be closed, like #15468 ?

This is on the Opacity widget. We can still add something in the doc that says if you're changing the opacity value in rapid succession, such as in an animation, while the child does not change, consider using [RenderAnimatedOpacity] such as via [FadeTransition]. We should also add [FadeTransition] and [AnimatedOpacity] in the 'see also' section.

Ah, I see, I was thinking AnimatedOpacity but we're talking about Opacity.  Alright I'll take a crack at updating Opacity's documentation.


SSV Normandy Auto Theming Not Working

When using a theme that is not Monika's the Normandy's secondary color remains themed for Monika, please fix!

 SSV Normandy Auto Theming Not Working


Remove unnessecary parentheses

(Feature Request)
The compiler provides warnings for unnecessary parentheses, so it should be possible to add a feature that removes them while formatting. This would probably need an option defined because not everyone would want to remove their parentheses.
Typically, the compiler reports a warning for if, for and while statements, as in each of these examples:
if (a == b) {
}
for (a in b) {
}
while (a < b) {
}

At first glance, this seems like a good first issue, and I would be interested in trying to implement this myself.

This seems like something better done with a rustfix suggestion from rustc, rather than via rustfmt.

What do you mean? rustc already suggests removing them (Using the 'Unnessecary Parathenthesesphrasing). I was hoping thatrustfmt` could automatically remove them. I would also expect the config option for this would default to off.

cargo fix can automatically apply the rustc suggestion already. There can be reasons to replicate the feature in rustfmt though (e.g. works for syntactically-valid but errneous codes).
/t/foo $ cat src/main.rs
fn main() {
    if (true) {
        println!("hello");
    }
}

/t/foo $ cargo fix
    Checking foo v0.1.0 (/tmp/foo)
      Fixing src/main.rs (1 fix)
    Finished dev [unoptimized + debuginfo] target(s) in 0.16s

/t/foo $ cat src/main.rs
fn main() {
    if true {
        println!("hello");
    }
}

I was not aware of Rust fix until now. Thanks for letting me know. I suspect that it doesn't need to be replicated into rustfmt. The only use case I can see (because the cases where rust fix doesn't work prevent rustfmt from working, as far as I know), is automatically applying the fix in the context of a Github hook or similar. I believe that automatically running cargo fix is sufficient, so it doesn't seem necessary.

I hope the Rust compiler would allow the "unnecessary parentheses", for some reason, it improves the readability.

You can put #![allow(unused_parens)] at your crate root.

In the meantime, I think we can safely close this issue, in favor of the recommendation to use cargo fix.

"cargo fix" dose not make sense to those compulsion :-), it removes the desired parenthesis!
The extra parenthesis is not harmful and it improves the readability, maybe we have to fork a Rust compiler and apply the fix?
diff --git a/src/librustc_lint/unused.rs b/src/librustc_lint/unused.rs
index 9f293bdaa10..ded027d4603 100644
--- a/src/librustc_lint/unused.rs
+++ b/src/librustc_lint/unused.rs
@@ -331,7 +331,7 @@ impl<'a, 'tcx> LateLintPass<'a, 'tcx> for UnusedAttributes {

 declare_lint! {
     pub(super) UNUSED_PARENS,
-    Warn,
+    Allow,
     "`if`, `match`, `while` and `return` do not need parentheses"
 }

Hi all, I forked the compiler and made the change, it works! let's have more optimizations to Rust.
https://github.com/weisunding/rust (master)

Sorry for continuing off-topic, but I'm curious why you need to change the default. With lint-levels externally changed, your code still generates warnings for others. What is wrong with #![allow] attribute?
(By the way, it may be a code smell that ifs look better with parenthesis (i.e. too long expression). I'd rather factor out the condition.)

Sorry for off-topic.
unused parentheses are definitely no side-effect, so it should be the compiler's default. why not "#![allow(unused_parens)]" attribute?  since the extra code is unnecessary.
I think the Rustacean are minimalist, you use fn instead of function, pub and not bar, so I have more to do, such as
statement
let mut i = 100;
will be changed to
mut i = 100;
and more to do, thanks for discussion, you hearted people! 😀

Closing https://github.com/rust-lang/rustfmt/issues/3823#issuecomment-558756746.

In the meantime, I think we can safely close this issue, in favor of the recommendation to use cargo fix.

As a data point, I'm currently working on a project where I'm generating Rust code. This is part of Android and rustfmt is readily available to me in the build environment — but Cargo is not since Android doesn't use Cargo to build its Rust code.
I realize this is non-standard in the Rust ecosystem, but all large companies end up using rustc directly instead of relying on Cargo. So from that point of view, it's attractive to put features like this into tools other than Cargo.

Thanks for sharing your perspective @mgeisler. Understand where you're coming from, but that's still an insufficient argument to justify duplicating functionality in another tool. rustfix is definitely the best place for this type of feature, and as noted, is already available there.
I recognize that rustfix isn't available as an executable binary outside a cargo context (at least AFAIK), but if there's a need/desire for being able to use rustfix separate from cargo (which would be fair IMO) then I think that's best taken up as a feature request for the rustfix team at https://github.com/rust-lang/rustfix/

Understand where you're coming from,

Thanks! The extra parenthesis is not a showstopper for me — we'll either disable the lints for the auto-generated files, or we'll  put in the extra effort to not generate unnecessary parenthesis. I only found this issue since it occurred to me that perhaps rustfmt could simplify things for us.

but that's still an insufficient argument to justify duplicating functionality in another tool.

I would not suggest duplicating anything — I would move the code from cargo fix to rustfmt.

rustfix is definitely the best place for this type of feature, and as noted, is already available there.

Actually, that's not at all clear to me :smile: When I think of rustfmt, I think of a tool which modifies and normalizes the AST. This often means adding/removeing braces and commas, so I would not at all be surprised to see it also add/remove  parenthesis.
On the other hand, my conceptual model of cargo fix is that it does much more complex and semantic operations on the AST. As an example, it has knowledge about differences between the editions and it can thus edit the code to move from one edition to the next. That is the kind of operation that I would not expect rustfmt to do since it's way outside the scope of "formatting".

if there's a need/desire for being able to use rustfix separate from cargo (which would be fair IMO) then I think that's best taken up as a feature request for the rustfix team at https://github.com/rust-lang/rustfix/

Good point, I'll open an issue there to at least register this potential problem. I thing the problem will only grow in the future as more large companies integrate rustc into their build tools.

Actually, that's not at all clear to me smile When I think of rustfmt, I think of a tool which modifies and normalizes the AST. This often means adding/removeing braces and commas, so I would not at all be surprised to see it also add/remove parenthesis.
On the other hand, my conceptual model of cargo fix is that it does much more complex and semantic operations on the AST.
I would not suggest duplicating anything — I would move the code from cargo fix to rustfmt.

A good way to think about it is that rustfix is typically the "right" home for autofixing something the compiler complains about.
Yes, both operate on the AST, but we wouldn't want to remove rustfix's existing ability to fix a compiler lint and move that into a different tool (e.g. rustfmt) just because it's feasible to implement that feature in said tool, and that tool can be invoked independently of cargo


feat(observability): trace Database methods

Adds trace spans for Database methods, as well as tests
for methods:

getSession
getSnapshot
run
runStream
runTransaction

tracing of other methods shall come in follow-up PRs.
Updates #2079
Built from PR #2087
Updates #2114

@surbhigarg92 kindly please re-review!

Hello @surbhigarg92 kindly please take a look. Thank you!

@surbhigarg92 @harshachinta kindly please help trigger the bot runs. Thank you.

@surbhigarg92 thank you very much for the reviews! I've addressed all feedback and gotten the PR ready for merge by squashing commits into one. Kindly help me re-run the bots.


Mf incident test parameter update

Corrected parameter order for Incident Analysis tools: Cluster Analysis and Count Incidents By LOC. Tests needed to be updated to match the changes.
Description
For Cluster Analysis moved the optional Cluster Distance parameter from 2nd position to third (last) and updated test.
For Count Incidents By LOC moved the optional Search Distance parameter to the last position and updated test.
Related Issue
N/A
Motivation and Context
Following parameter order convention that makes it easier for script users to only use minimum number of parameters and not be forced to write out optional parameters.
How Has This Been Tested?
Ran automated (unittest) tests for Incident Analysis tools to verify tests run successfully. The modified test are running, however there are errors in other tests.
Python 3.4 Tests ===============================
Debug messaging is OFF
Logging results to: SGT_2017-April-03_14-55-51_seq0.log
adding test: test_cluster_analysis_pro
adding test: test_count_incidents_pro
adding test: test_incident_density_pro
adding test: test_hot_spots_by_area_pro
adding test: test_incident_hot_spots_pro
adding test: test_incident_table_to_point_pro
adding test: test_percent_change_pro
running 7 tests...
...
...
...
Test success: False
RESULTS =================================================

Number of tests run: 7
Number of errors: 0
Number of failures: 3

FAILURES ===============================================

test_incident_density_pro (patterns_tests.IncidentDensityTestCase.IncidentDensityTestCase)Traceback (most recent call last):
  File "D:\GitHub\solutions-geoprocessing-toolbox\utils\test\patterns_tests\IncidentDensityTestCase.py", line 78, in test_incident_density_pro
    self.assertTrue(arcpy.Exists(outputDensity))
AssertionError: False is not true

test_incident_hot_spots_pro (patterns_tests.IncidentHotSpotsTestCase.IncidentHotSpotsTestCase)Traceback (most recent call last):
  File "D:\GitHub\solutions-geoprocessing-toolbox\utils\test\patterns_tests\IncidentHotSpotsTestCase.py", line 75, in test_incident_hot_spots_pro
    self.assertEqual(featureCount, int(7302))
AssertionError: 7286 != 7302

test_incident_table_to_point_pro (patterns_tests.IncidentTableToPointTestCase.IncidentTableToPointTestCase)Traceback (most recent call last):
  File "D:\GitHub\solutions-geoprocessing-toolbox\utils\test\patterns_tests\IncidentTableToPointTestCase.py", line 73, in test_incident_table_to_point_pro
    arcpy.IncidentTableToPoint_iaTools(self.inputTable, coordFormat, xField, yField, outputTable)
arcgisscripting.ExecuteError: ERROR 000576: Script associated with this tool does not exist.
Failed to execute (Add Unique Row ID).
Failed to execute (IncidentTableToPoint).

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "D:\GitHub\solutions-geoprocessing-toolbox\utils\test\patterns_tests\IncidentTableToPointTestCase.py", line 76, in test_incident_table_to_point_pro
    self.fail('Exception in IncidentTableToPoint_iaTools for Pro toolbox \n' + msg)
AssertionError: Exception in IncidentTableToPoint_iaTools for Pro toolbox
ERROR 000576: Script associated with this tool does not exist.
Failed to execute (Add Unique Row ID).
Failed to execute (IncidentTableToPoint).

END OF TEST =========================================

Python 2.7 Tests ===============================
Debug messaging is OFF
Logging results to: SGT_2017-April-03_15-09-23_seq0.log
adding test: test_cluster_analysis_desktop
adding test: test_count_incidents_desktop
adding test: test_incident_density_desktop
adding test: test_hot_spots_by_area_desktop
adding test: test_incident_hot_spots_desktop
adding test: test_incident_table_to_point_desktop
adding test: test_percent_change_desktop
running 7 tests...
...
...
...
Test success: False
RESULTS =================================================

Number of tests run: 7
Number of errors: 0
Number of failures: 3

FAILURES ===============================================

test_hot_spots_by_area_desktop (patterns_tests.HotSpotsByAreaTestCase.HotSpotsByAreaTestCase)Traceback (most recent call last):
  File "D:\GitHub\solutions-geoprocessing-toolbox\utils\test\patterns_tests\HotSpotsByAreaTestCase.py", line 95, in test_hot_spots_by_area_desktop
    self.fail('Exception in HotSpotsByArea_iaTools for Desktop toolbox \n' + msg)
AssertionError: Exception in HotSpotsByArea_iaTools for Desktop toolbox

test_incident_table_to_point_desktop (patterns_tests.IncidentTableToPointTestCase.IncidentTableToPointTestCase)Traceback (most recent call last):
  File "D:\GitHub\solutions-geoprocessing-toolbox\utils\test\patterns_tests\IncidentTableToPointTestCase.py", line 96, in test_incident_table_to_point_desktop
    self.fail('Exception in IncidentTableToPoint_iaTools for Desktop toolbox \n' + msg)
AssertionError: Exception in IncidentTableToPoint_iaTools for Desktop toolbox
Failed to execute. Parameters are not valid.
ERROR 000816: The tool is not valid.
Failed to execute (IncidentTableToPoint).

test_percent_change_desktop (patterns_tests.FindPercentChangeTestCase.FindPercentChangeTestCase)Traceback (most recent call last):
  File "D:\GitHub\solutions-geoprocessing-toolbox\utils\test\patterns_tests\FindPercentChangeTestCase.py", line 94, in test_percent_change_desktop
    self.fail('Exception in FindPercentChange_iaTools for Desktop toolbox \n' + msg)
AssertionError: Exception in FindPercentChange_iaTools for Desktop toolbox
Failed to execute. Parameters are not valid.
ERROR 000725: Output Percent Change: Dataset C:\Users\<user>\AppData\Local\Temp\scratch.gdb\percentChange already exists.
Failed to execute (FindPercentChange).

END OF TEST =========================================

Error in atexit._run_exitfuncs:
Traceback (most recent call last):
  File "C:\Python27\ArcGIS10.3\lib\atexit.py", line 24, in _run_exitfuncs
    func(*targs, **kargs)
  File "C:\Python27\ArcGIS10.3\lib\site-packages\matplotlib\_pylab_helpers.py", line 84, in destroy_all
    for manager in Gcf.figs.values():
AttributeError: 'NoneType' object has no attribute 'figs'
Error in sys.exitfunc:
Traceback (most recent call last):
  File "C:\Python27\ArcGIS10.3\lib\atexit.py", line 24, in _run_exitfuncs
    func(*targs, **kargs)
  File "C:\Python27\ArcGIS10.3\lib\site-packages\matplotlib\_pylab_helpers.py", line 84, in destroy_all
    for manager in Gcf.figs.values():
AttributeError: 'NoneType' object has no attribute 'figs'
Fatal Python error: PyEval_RestoreThread: NULL tstate

This application has requested the Runtime to terminate it in an unusual way.
Please contact the application's support team for more information.

new problem tests are...
Pro:

test_incident_density_pro
test_incident_hot_spots_pro
test_incident_table_to_point
Desktop:
test_hot_spots_by_area_desktop
test_incident_table_to_point_desktop
test_percent_change_desktop

Screenshots (if appropriate):
N/A
Types of changes

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[X ] My code follows the code style of this project.
[X] My change requires a change to the documentation.
[X] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING document.

Changes look good but most/all the unit test failed (logs attached in case they help) - just let me know if this is a concern.
Failures seem related to scratch = None issue mentioned here: https://github.com/Esri/military-tools-geoprocessing-toolbox/pull/206
SGT_2017-April-04_07-59-30_seq0.zip
Also, this step/test takes 10 minutes which didn't seem quite right:
Running tool (Count Incidents By LOC - Desktop)

Looking at those logs you've got some really strange errors going on there. First any ideas why everything in is running from AppData\Local? Is this on your desktop? Also seeing a lot of errors pointing to output files like  ..\path\scratch.gdb\weirdname.shp.shp.shp which is a bad combo: File GDB + Shapefile.

No idea - except they are all set to None here:
https://github.com/Esri/solutions-geoprocessing-toolbox/blob/mf-incident-test-parameter-update/utils/test/Configuration.py#L61
I can debug further if you are also testing with this identical branch.

So I'm a little lost. do we merge or not? I want to investigate your issues further, but don't want to keep these changes hanging. Suggestions?

Merging - unit test failures can be new issue if desired


DecoratorConstructors.java:33-38: The first attempt to...

The puzzle 252-0e01e3fe from #252 has to be resolved:
https://github.com/objectionary/jeo-maven-plugin/blob/4b6582f4b0da82207a7aa4f3f9363eef59d0321b/src/main/java/org/eolang/jeo/improvement/DecoratorConstructors.java#L33-L38
The puzzle was created by @rultor on 17-Nov-23.
Estimate: 90 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

The puzzle 252-0e01e3fe has disappeared from the source code, that's why I closed this issue.


Update README example.

The code example from the documentation's latest version contains some important updates, such as the usage of NO_PARAMS. The README should be updated to reflect this.

Thanks.


Data Processing Lock Manager

RW Lock differs from lectures.

Waiting confirmation from lecturer on correctness of lecture content here
Need to fix and improve current Lock Manager implementation in transactions/code

Decided to leave a diagram, code will be complex and harder to understand.

Interesting problem to solve efficiently: scanning for range intersections, take advantage of read overlap, write exclusivity to reduce scan work


Be more format agnostic

In order to be successful, this website should be more accessible to people who are not computer scientists. Therefore, it should not matter which kind of video someone Uploads. gocast should always try its best to convert input to a format that is recognised by popular browsers.

@joschahenningsen Do you mean for example converting a lecture.avi file into lecture.mp4 or some other format? Just want to be sure what is the desired outcome here.

In essence yes, I'm more concerned about Codec than Container though. It shouldn't matter, which Codec the input video and audio are, gocast should try its best to always present the stream and vod in h.264, aac.

I checked the code and it seems we already check the video codec. What we still need to check is that the audio codec is aac.


[Identity] Adjust IMDS retry backoff

This better aligns with other languages in terms of IMDS retry behavior.

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


Adding function for searching by field

Expose swift function similar to the original ts_node_child_by_field_name
Cheers

Sorry I didn't check that #6 was doing the same thing, feel free to close it if it doesn't make sense then.

That's completely my fault for not merging it more quickly! I'm going to do that now. Sorry for wasting your time here.


Create blueprint using S3 mountpoint for external jar files to run Spark workloads

Community Note
For Spark workloads requiring additional JARs, it's unnecessary to include these JARs in the base Docker image. These JARs can be developed and maintained by multiple teams and stored in an S3 bucket. Spark Operator or Spark jobs can reference these JARs from S3 using S3 mountpoint.

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

What is the outcome that you are trying to reach?
Problem Statement
Image Size: The Docker images are getting larger with the increasing number of JARs added.
Container Start Time: The startup time of containers is slowing down due to the loading of multiple JARs.
Continuous Updates: Updating these JARs requires recreating the images, which is time-consuming. Managing versions for each JAR is also challenging.

Describe the solution you would like
Developers will continue to generate the latest JARs and store them with tags or versions in the S3 bucket.
The Spark application will specify the appropriate tag of the JAR in the job specification.
The S3 mountpoint will be dynamically configured by the CSI driver to mount the S3 bucket to each node, specifying the path to the JARs location.
Spark jobs submitted to EKS will read the JARs from S3, as it is mounted as a local drive in the pods via S3 mountpoint (e.g., /opt/spark/jars/...).
Enable caching with the S3 mountpoint to store the JARs on the node. This allows new executors to load the JARs faster than reading directly from the S3 bucket each time.

Describe alternatives you have considered

Additional context

@ratnopamc let's do this in the existing spark blueprint.

@vara-bonthu, an we close this since the PR is merged?

Fixed in #606


Potential bug in map->spare usage (used both as struct bucket and stored item)

In the initialize function map->spare is initialized to have sizeof(hashmap) + 2 * bucketsz
But it is used both as struct bucket and as the items stored in the buckets in the setter and getter.
https://github.com/tidwall/hashmap.c/blob/84d0d3bf75f297443a5069b801ea13d93ee05e0a/hashmap.c#L280-L290
https://github.com/tidwall/hashmap.c/blob/84d0d3bf75f297443a5069b801ea13d93ee05e0a/hashmap.c#L361
As long as elsize is smaller than bucketsz this shouldn't be a problem.

This is by design. It's not a bug, nor even potentially a bug.
The spare field is large enough to hold a bucket or an item, and a bucket size is always larger than an item size.
https://github.com/tidwall/hashmap.c/blob/84d0d3bf75f297443a5069b801ea13d93ee05e0a/hashmap.c#L105

Thanks for the explanation. I wasn't completely sure prior to this explanation.


Can not get EnvoyFilter to work

Istio version: 1.0
We would like to validate client certificate's Common Name with use of EnvoyFilter.
As a begining I wanted to create some simple filter that responds with 403 on every request:
apiVersion: networking.istio.io/v1alpha3
kind: EnvoyFilter
metadata:
  name: forbidden-lua
  namespace: istio-system
spec:
  workloadLabels:
    app: my-service
  filters:
  - filterConfig:
      inlineCode: |
        function envoy_on_request(request_handle)
          request_handle:respond({[":status"] = "403"},"nope")
        end
    filterName: envoy.lua
    filterType: HTTP
    listenerMatch:
      listenerType: ANY

Although I could not get it to work. The service was working the same as without the filter. Only thing I have found related to the filter in proxy logs was: [2018-08-27 07:19:26.140][19][info][lua] external/envoy/source/extensions/filters/http/lua/lua_filter.cc:449] envoy_on_response() function not found. Lua filter will not hook responses. so I guess the filter was detected.
I have also tried some simple examples from Envoy's documentation but the outcome was the same.
I am not sure if I am missing something important or is that some kind of bug. Also the documentation on EnvoyFilters seems a bit poor, could you provide some step by step example of how to get it to work?

The message in the logs is as expected as you havn't defined an "envoy_on_response" function (which is optional).
Can you also post your deployment?
You have to add "app: my-service" in your depoyment at spec->template->metadata->labels

Yes, I have mentioned that log because it told me that the EnvoyFilter is visible to the proxy in the pod with desired application.
The label is also set in the deployment:
apiVersion: extensions/v1beta1
kind: Deployment
metadata:
  labels:
    app: my-service
    heritage: Tiller
    release: my-service
  name: my-service
  namespace: integration
spec:
  replicas: 1
  selector:
    matchLabels:
      app: my-service
      release: my-service
  strategy:
    rollingUpdate:
      maxUnavailable: 0
    type: RollingUpdate
  template:
    metadata:
      annotations:
        sidecar.istio.io/inject: "true"
        traffic.sidecar.istio.io/includeInboundPorts: "8081"
      labels:
        app: my-service
        release: my-service
    spec:
      containers:
      - args:
        - /myservice
        image: my-image
        imagePullPolicy: IfNotPresent
        name: my-service
        ports:
        - containerPort: 8081
          name: my-port

And I also have a service:
apiVersion: v1
kind: Service
metadata:
  annotations:
    "auth.istio.io/8081": NONE
  name: my-service-api
  namespace: integration
  labels:
    app: my-service
    release: my-service
    heritage: my-service
spec:
  type: ClusterIP
  ports:
    - port: 8081
      protocol: TCP
      name: my-port
  selector:
    app: my-service
    release: my-service

Looks good for me. You could try to debug it using request_handle:logInfo("Hello World"); and looking for it in the logs: kubectl logs  -c istio-proxy -n

Alright, I have tried deploying it on a clean cluster, only with fresh Istio 1.0 and it is working, so I guess the problem might be with some configuration on our cluster.
Thanks for help.

Might be related to https://github.com/istio/istio/issues/8616 ?


Condition disappear for Idefics activation

Hi,
since release version 4.0.3 it seems there is condition disappeared for Idefics Interface activation or not.
You can compare this latest version with previous one like v4.0.3 at the same line :
https://github.com/aws-samples/aws-genai-llm-chatbot/blob/3c710bf28bd74eb3b576b863a4b678ed8ef06e0b/lib/aws-genai-llm-chatbot-stack.ts#L110
is it normal or a mistake ? In my case I cannot not deploy anymore due this new resource not available before and it need a default security group ID for private VPC endpoint.
This happened when VPC already exist, it seems the method "ec2.Vpc.fromLookup" does not detect a default security group.
Here the impact :
https://github.com/aws-samples/aws-genai-llm-chatbot/blob/main/lib/model-interfaces/idefics/index.ts#L41
This property "props.shared.vpc.vpcDefaultSecurityGroup" should contains a default security group ID.
For an existing VPC this value seems to be blank.
This condition impact also ResourcePath for NagSuppressions.
Best regards,

Hi,
anyone could check this issue ?

Hello
@hghandri i had the same error when using the existing VPC with  Idefics Interface can not find the default security group ID.
Do you find the solution about this problem ?

I've just opened a PR : #498


Feature Request: Large Icons

I feel the current icons are too small. If it's meant to be like the Mac app switcher, the icons should be a bit larger. Or maybe a option to change the icon size would be cool, too.
btw, thank you for making this! I've been looking for something like this for a while. While I'm currently unable to support the project, if I continue using it in the future, I'll definitely donate. Best of luck, buddy! ❤

+1 for this.
Thanks for making it!

Thanks for the feedback! This is a good idea, I'll add a size option 🙂

This should be in the last update. The default size is 1.5 times Windows default icon size. A "scale" option lets you tweak it from the config file.
Some icons still lack definition. It is not trivial to get high res. icons for some apps, but I will try to improve this.

Thank you! ❤
This is great, exactly what I was looking for. Love that it auto-resizes when there are too many apps.


Always exits 0

When unsatisfied rules are detected, the Difflint CLI tool is expected to exit 1 signifying that the check has failed. However, the tool currently always exits 0.

Context: https://github.com/acmcsufoss/acmOpportunities/actions/runs/5340797294/jobs/9680997051?pr=17#step:4:14.

The experienced unexpected behavior described in this issue is caused by other bugs resolved in #11.


Added personal sponsorships for Karl Tynan

Sponsorships add by Karl Tynan for Lee and Soren.

Hi @karltynan
Thanks for adding your sponsorship sections, that is awesome!
Have merged  🥳
Thanks
Emma


Add feed rate to gcode generation

Should I try to fix the failing check? It is indirectly covered by the modified tests @voneiden

That's alright, bump me once you've got those minor fixes in place and I'll approve & merge.

And the codecov/patch fail is fine in this case.

On the note of simplifying the Command class I was thinking of following the following approach:

Internally represent commands using Command class. Each command will hold all the information that it needs to be converted into a G-code block
The post-processors will read that information and generate the G-code. Post processor will be especially important when generating StartSequence, StopSequence, ToolChange, SafetyBlock and HomePosition. Some machines cannot use Canned Cycle commands, in that case the post-processor will simulated the Canned Cycle using G00, G01, and G04 commands.
Unlike the current gcode generation, the post processors will produce a verbose gcode test:

G0 X0.0 Y0.0 Z0.0
G0 X0.0 Y0.0 Z5.0

Introduce optimizer.py. It will take the verbose representation from the post-processor and transform it based on the options below:

remove unnecessary modal addresses

G0 X0.0 Y0.0 Z0.0
G0 X0.0 Y0.0 Z5.0

G0 X0.0 Y0.0 Z0.0
X0.0 Y0.0 Z5.0

remove unnecessary axis addresses

G0 X0.0 Y0.0 Z0.0
G0 X0.0 Y0.0 Z5.0

G0 X0.0 Y0.0 Z0.0
G0 Z5.0

remove spaces

G0 X0.0 Y0.0 Z0.0
G0 X0.0 Y0.0 Z5.0

G0X0.0Y0.0Z0.0
G0X0.0Y0.0Z5.0

Thank you, merged in 66a635be8d3c641256be70768c4deb6b60196fd4


Imported package functions completion inside a defer and/or go func(){...}() fails.

go version go1.12.5 darwin/amd64

Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?

Click here to see

GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/simonblixenkrone/Library/Caches/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/simonblixenkrone/dev/go"
GOPROXY=""
GORACE=""
GOROOT="/usr/local/Cellar/go/1.12.5/libexec"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.12.5/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/simonblixenkrone/dev/gocode/basic-server/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/b1/53jz_b9j1rb0nhhzmv4kwbk80000gn/T/go-build791443395=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
I.e: I have an imported package "log", that returns different types of logging (Error, Info, Print etc). Inside a defer or go func() function, the autocompletion is completely broken. This applies to any imported package function, but not own functions defined in the workspace. Outside go func it works perfect.
What did you expect to see?
Function auto-completion.
What did you see instead?
Global variables. See images 1 and 2.

Duplicate of https://github.com/golang/go/issues/29313

Duplicate of https://github.com/golang/go/issues/29313


fix(idempotency): idempotent_function should support standalone falsy values

Issue number: #1663
Summary
When using idempotent_function for any synchronous Python function (different from Lambda Handler), customers set which keyword argument we should use to hash data as an idempotency key.
We raise RuntimeError for cases where the keyword argument wasn't used correctly to prevent hashing the incorrect data. For example, function was called without the keyword argument (function(), instead of function(data=data)).
However, the current logic incorrectly checks the value available in the keyword argument, rather than the keyword argument was used. The following code leads to a RuntimeError when we should be serializing None->null just fine.
...

import random

@idemp_f(data_keyword_argument="key")
def echo(key: str = None):
    return random.randint(1, 1000000)

echo(key=None)

Changes

Please provide a summary of what's being changed

This PR corrects the logic and only raises RuntimeError if the keyword argument specified for hashing isn't in the list of keyword arguments available at the function call.
User experience

Please share what the user experience looks like before and after this change

Checklist
If your change doesn't seem to apply, please leave them unchecked.

[x] Meet tenets criteria
[x] I have performed a self-review of this change
[x] Changes have been tested
[x] Changes are documented
[x] PR title follows conventional commit semantics

Is this a breaking change?
RFC issue number:
Checklist:

[ ] Migration process documented
[ ] Implement warnings (if it can live side by side)

Acknowledgment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

Codecov Report
Base: 99.35% // Head: 99.35% // No change to project coverage :thumbsup:

Coverage data is based on head (f6da33f) compared to base (87de70c).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #1669   +/-   ##
========================================
  Coverage    99.35%   99.35%
========================================
  Files          128      128
  Lines         5908     5908
  Branches       373      373
========================================
  Hits          5870     5870
  Misses          18       18
  Partials        20       20

Impacted Files
Coverage Δ

...da_powertools/utilities/idempotency/idempotency.py
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Implement the update last location use case

Survivors must have the ability to update their last location. When added to the database, a survivor will receive a unique identification from the system. The survivors must be able to search for themselves and effectively update the last location using that identification.

The implementation should follow this component diagram:


Use llvm.dbg.value instead of llvm.dbg.declare for SSA values

Technically these are not addresses in memory, so it's important to use dbg.value instead of dbg.declare
Also insert the debug instrinsic just after the instruction that defines it.
Without this change, almost none of the local SSA values are available in gdb (it might be just the arguments that are) but with it I can very often print the arguments of the instruction execution is halted on:

This doesn't seem to work for PHINodes for some reason (they don't even show up in info locals), but it's a big improvement for other values based on my local tests


HTTP solver caches permanent redirects

Describe the bug:
Where the target domain is initially pointed to a server which does an HTTP redirect to force upgrade HTTP to HTTPS, the http solver's HTTP client appears to cache the redirect (as typically expected). When the config for the domain is then updated to point to somewhere else, this cached redirect prevents the solver from reaching the non-HTTPS endpoint required.
Expected behaviour:
This caching should not take place. Once the domain is pointed at the 'right place', it should be able to visit the well-known solve endpoint over HTTP.
Steps to reproduce the bug:
Scenario: imagine you're moving domain example.com from legacy infrastructure where it is currently configured to new infrastructure, upon which we're using cert-manager for certificate provisioning.

Have domain example.com pointed to a server (legacy infrastructure) which upgrades HTTP to HTTPS with a 301. Assume this server has a valid certificate etc and 'works'.
Use cert-manager to attempt to provision a certificate using the http01 challenge type (new infrastructure).
Observe in the cert-manager controller logs as the HTTP solver fails to verify the endpoint (as the domain is still pointed at the legacy infrastructure).
Point example.com at new infrastructure.
After propagation etc, verify manually that the challenge endpoint is available via HTTP.
Observe in the cert-manager controller logs as the HTTP solver continues to follow the now non-existent redirect to HTTPS and fails (exact failure will depend on the setup).

Anything else we need to know?:

Workaround is to restart the controller pod.
In my case this is the error on the controller, just for reference ‒ domain and token replaced with dummy values:

I0115 12:51:37.890944       1 http.go:110] could not reach 'http://example.com/.well-known/acme-challenge/example_token': failed to GET 'http://example.com/.well-known/acme-challenge/example_token': Get https://example.com/.well-known/acme-challenge/example_token: x509: certificate is valid for ingress.local, not example.com

Environment details::

Kubernetes version (e.g. v1.10.2): 1.13.1
cert-manager version (e.g. v0.4.0): v0.6.0-alpha.1

/kind bug

Ping on the above 😄 if you're no longer able to reproduce this with the v0.6.1 release, please close this issue! 😄

Sorry, forgot about your initial reply! That is strange. It's a bit of a faffy repro so it'll probably be a little while before I can find time to arrange all the pieces to do it again?

I'm going to close this issue now as we've not had a response for a while, but please re-open and update with more information if you've got more to add 😄


Add Python version 3.13 to test matrix.

Fixes #1372
Add Python version 3.13 to matrix.

@JE-Chen Thanks for checking. It looks like Py3O is already at 3.13: https://github.com/PyO3/pyo3/issues/4636 I was hoping that maybe bumping the Poetry build might fix it https://github.com/apache/iceberg-python/pull/1379. Could you rebase? Otherwise we have to track which dependency is still locked at this older version of Py3O.

Done

Same error.
error: the configured Python interpreter version (3.13) is newer than PyO3's maximum supported version (3.12)

Currently, we are using pyo3 v0.20.3. I think only versions of pyo3 v0.21 or newer support Python 3.13.

@JE-Chen I think it pulls pyo3 in when it can't find pre-build wheels for a combination of the Python version and architecture.

  at ~/.local/lib/python3.10/site-packages/poetry/installation/chef.py:164 in _prepare
      160│
      161│                 error = ChefBuildError("\n\n".join(message_parts))
      162│
      163│             if error is not None:
    → 164│                 raise error from None
      165│
      166│             return path
      167│
      168│     def _prepare_sdist(self, archive: Path, destination: Path | None = None) -> Path:

Note: This error originates from the build backend, and is likely not a problem with poetry but with cramjam (2.8.3) not supporting PEP 517 builds. You can verify this by running 'pip wheel --no-cache-dir --use-pep517 "cramjam (==2.8.3)"'.

Looks like we're still pinning on an older version of cramjam, as 2.9.1 is out already: https://pypi.org/project/cramjam/


MV Query Rewrite for JDBC Table

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

Describe the solution you'd like

Describe alternatives you've considered

Additional context

closed by #28171


Authentication issue after updating to Module 1.3.2

Hi,
i used to use  Moduleversion 1.2.1 before  and upgraded to 1.3.2.
Now I get the following error Message:
Connect-MSIntuneGraph -TenantID $tenantID
"WARNING: An error occurred while attempting to retrieve or refresh access token. Error message: The property 'Authority' cannot be found on this object. Verify that the property exists."
It is the same user which I used for connection with 1.2.1.
Do you have any hints what the reason could be?

FYI: It is still not working under 1.3.3 with the newest MSAL.PS Module installed.
However when I manually downgrade MSAL.PS to <IP_ADDRESS> it is working.
With <IP_ADDRESS> it is failing, when I login zu AzureAD before to grab the tenant information I need before connecting against the intunegraph.

I am having a similar issue. I am trying to connect using an registrated app from my azure ad, so i assume i use the arguments TenantID, ClientID and ClientSecret, but afterwards i only get the error chibs89 already mentionend.
It seems like it is erroring out somewhere in this try/catch https://github.com/MSEndpointMgr/IntuneWin32App/blob/644deff377ed9e1527f0c26dc99cfb137ee4fd6e/Public/Connect-MSIntuneGraph.ps1#L145

Ok, i got it fixed quite easily by adding "-AzureCloudInstance 1" at the end of this line https://github.com/MSEndpointMgr/IntuneWin32App/blob/644deff377ed9e1527f0c26dc99cfb137ee4fd6e/Public/Connect-MSIntuneGraph.ps1#L147 as suggested here https://github.com/AzureAD/MSAL.PS/issues/45#issuecomment-1279193657

I can't reproduce this issue unfortunately. I just tried to authenticate using the client authentication flow (client id/client secret) using MSAL.PS <IP_ADDRESS> and it worked.

I was able to reproduce the problem if I install module "Microsoft.Graph.Authentication" and use "Connect-MgGraph" for a different part of my script before using "Connect-MSIntuneGraph".
I do the following after the install module part to fix the issue for my scripts:
Import-Module IntuneWin32App -ErrorAction SilentlyContinue $moduleItem = Get-Module IntuneWin32App if ($moduleItem){     		Get-ChildItem -Path ($moduleItem.ModuleBase) -Recurse -filter 'Connect-MSIntuneGraph.ps1' | ForEach-Object { 		$stringToReplace = 'Get-MsalToken @AccessTokenArguments' 		$stringToReplaceWith = 'Get-MsalToken @AccessTokenArguments -AzureCloudInstance AzurePublic' # <- This is the fix to avoid a problem with MSAL.PS 		# Lets check if we replaced the string already         		if (-NOT(select-string -Path $_.FullName -Pattern $stringToReplaceWith)){ 			# Lets replace the string             			$scriptContents = Get-Content $_.FullName             			$scriptContents = $scriptContents -replace $stringToReplace, $stringToReplaceWith 			$scriptContents | Out-File -FilePath $_.FullName -Encoding utf8 -Force         		}       	}     	# we neeed to re-load the module in order for the new code to work     	Remove-Module IntuneWin32App     	Import-Module IntuneWin32App }


How can I got the UUID without -?

I need the UUID like this 5CC31D610AC1160C4C28744C52C73A96,what should I do？

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/replace

I got one way to
const uuid = require('uuid/v4');
let hi = uuid().replace(/-/g,'');

@broofa thanks

@broofa , in the cli, how to do it ?


Move scala sources to src/main/scala from src/main/java

It's just a convention.  Gradle doesn't care.  IntelliJ doesn't care.  Maybe some tool somewhere cares.  My low-grade OCD cares.  :-)

thanks!


Modes for torch.nn.functional.pad and tf.pad

PyTorch has constant, reflect, replicate, and circular padding modes while TensorFlow has CONSTANT, REFLECT, and SYMMETRIC.
PyTorch’s reflect and TensorFlow’s REFLECT are similar.
PyTorch’s replicate is similar to TensorFlow’s SYMMETRIC.
TensorFlow does not have an equivalent for PyTorch’s circular mode.
When I try to convert a replicate pad to tf, it seems that nobuco sends REPLICATE to tf.pad and it raise an error that it doesn't have such a mode.
ValueError: Exception encountered when calling layer "tf.pad" (type TFOpLambda).

Value of argument `mode` expected to be one of "CONSTANT", "REFLECT", or "SYMMETRIC". Received `mode` = REPLICATE

Thank you for your work! This issue should be closed.


Implement POST /e-entity/freme-ner/documents using FREME cloud

For now this is implemented using the proxy service. FREME cloud is delayed for unknown time and this ticket is outdated


Merge changes from release 2.2

@chiptus, @mcpacino can I please ask you two to review the resolution of the merge conflict: https://github.com/portainer/portainer/pull/4930/commits/a7d5f9f2c972533a562a23d97f872a923b64ba6a
The files were changed by both of you, one in CE2.2 another in develop branch.

@yi-portainer LGTM


[QUAD] Enhancement: Enforce thumbnail for Houdini publish

Interesting approach :) But I think it's not the way for AYON. Preferably we define which are reviewables and from reviewable representations a thumbnail may be taken if it's not defined yet and I believe it does that already in AYON too.
@iLLiCiTiT @MustafaJafar thoughts?

Interesting approach :) But I think it's not the way for AYON. Preferably we define which are reviewables and from reviewable representations a thumbnail may be taken if it's not defined yet and I believe it does that already in AYON too.
@iLLiCiTiT @MustafaJafar thoughts?

In my opinion, I think it'd be much better to have that for all product types.
I can think of the following approach:
Thumbnail:
1. Artists can use the publisher to capture a thumbnail.
2. If no thumbnail and `create_thumbnail == true`
  a. For products with a review representation: get the first frame of the representation.
  b. For render products: get the first frame. *we may convert it to a different extension first.*
  c. Fall to the default: Take a screengrab of the scene view.

I think this can be implemented in ExtractActiveViewThumbnail plugin and we can provide some settings to allow admins to control how the plugin should work.
It's also worth mentioning that we already create thumbnails for workfile instance. and we can re-use it with other product types.
Maybe we can create an issue on ayon-houdini and link to this PR for reference.


combo chart with line with equal values (bug)

I actually need a goal line in bar chart.
I created for this combo chart with line than has equals values. But, for example, if you will change code in ComboLineColumnChartActivity class like this:
private LineChartData generateLineData() {
        List<Line> lines = new ArrayList<Line>();
        for (int i = 0; i < numberOfLines; ++i) {

            List<PointValue> values = new ArrayList<PointValue>();
            for (int j = 0; j < numberOfPoints-1; ++j) {
                //values.add(new PointValue(j, randomNumbersTab[i][j]));
                values.add(new PointValue(numberOfPoints-1, 100));
            }

...
where value is greater than bars values.
you will not see line on this chart at all. Only bars.

Confirmed. A LineGraph can't draw a line of two identical points.

hi,
I am using ComboLineColumnChartView , and different set value for Line and column with same size. But when all the column data set is 0, line is starts from beginning of the viewport  and covers  full view port, and when column data is set , i mean not 0, it squeeze with left and right padding of 5dp approx.
How to solve this any idea?


Failed to access http://<IP_ADDRESS>:3000 with default docker-compose.yml

Note: for support questions, please join our Discord server

I'm submitting a ...
[* ] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Action taken (what you did)
git clone the latest repo, and then run
docker-compose up

Expected result (what you hoped would happen)
I can access <IP_ADDRESS>:3000

Actual result (unexpected outcome)
I got the following error:
Something happened to either Gekko or the connection. Please check the terminal where Gekko is running or your network connection.

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, etc)
The terminal does not show anything error:

gekko_1  |      I'm gonna make you rich, Bud Fox.
gekko_1  |
gekko_1  |
gekko_1  | TAlib is enabled
gekko_1  | TULIP indicators is enabled
gekko_1  | Serving Gekko UI on http://localhost:3000/
gekko_1  |
I also tried another branch  "stable", it also did not work.

Output of docker --version and docker inspect gekko_gekko assuming that is the name of the docker container you created.

[docker --version]
Docker version 17.09.0-ce, build afdb6d4
[docker inspect gekko_gekko]
[
{
"Id": "sha256:99ae23e39cce782981e55720165fb83d85bec63e50f603068c6805db266caa41",
"RepoTags": [
"gekko_gekko:latest"
],
"RepoDigests": [],
"Parent": "sha256:9f4a1ceadfcf7e2fa2db08fcde43c16dcd11b717e79999febb01afba99519ecb",
"Comment": "",
"Created": "2017-11-28T22:52:53.790570647Z",
"Container": "4ec25403b63ee4bb0eac4b9cc279d3167eb34c1cb3fe2e8d2aeca7ed3833f39b",
"ContainerConfig": {
"Hostname": "2da0903ff372",
"Domainname": "",
"User": "",
"AttachStdin": false,
"AttachStdout": false,
"AttachStderr": false,
"ExposedPorts": {
"3000/tcp": {}
},
"Tty": false,
"OpenStdin": false,
"StdinOnce": false,
"Env": [
"PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin",
"NPM_CONFIG_LOGLEVEL=info",
"NODE_VERSION=6.3.1",
"HOST=localhost",
"PORT=3000"
],
"Cmd": [
"/bin/sh",
"-c",
"#(nop) ",
"CMD ["npm" "start"]"
],
"ArgsEscaped": true,
"Image": "sha256:9f4a1ceadfcf7e2fa2db08fcde43c16dcd11b717e79999febb01afba99519ecb",
"Volumes": null,
"WorkingDir": "/usr/src/app",
"Entrypoint": [
"/usr/src/app/docker-entrypoint.sh"
],
"OnBuild": [],
"Labels": {}
},
"DockerVersion": "17.09.0-ce",
"Author": "",
"Config": {
"Hostname": "2da0903ff372",
"Domainname": "",
"User": "",
"AttachStdin": false,
"AttachStdout": false,
"AttachStderr": false,
"ExposedPorts": {
"3000/tcp": {}
},
"Tty": false,
"OpenStdin": false,
"StdinOnce": false,
"Env": [
"PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin",
"NPM_CONFIG_LOGLEVEL=info",
"NODE_VERSION=6.3.1",
"HOST=localhost",
"PORT=3000"
],
"Cmd": [
"npm",
"start"
],
"ArgsEscaped": true,
"Image": "sha256:9f4a1ceadfcf7e2fa2db08fcde43c16dcd11b717e79999febb01afba99519ecb",
"Volumes": null,
"WorkingDir": "/usr/src/app",
"Entrypoint": [
"/usr/src/app/docker-entrypoint.sh"
],
"OnBuild": [],
"Labels": {}
},
"Architecture": "amd64",
"Os": "linux",
"Size": 768077716,
"VirtualSize": 768077716,
"GraphDriver": {
"Data": {
"DeviceId": "3965",
"DeviceName": "docker-253:6-2684573752-fbf2ab6192e485ce12abcf24fcae52d5af69e5996b4549928f174d7fe7d64c08",
"DeviceSize": "10737418240"
},
"Name": "devicemapper"
},
"RootFS": {
"Type": "layers",
"Layers": [
"sha256:2f71b45e4e254ddceb187b1467f5471f0e14d7124ac2dd7fdd7ddbc76e13f0e5",
"sha256:66d8e5ee400cb0660c8c0ad51d30d822365a995eb90d35750a1e641cae7c887c",
"sha256:1050aff7cfff31f37ff91f26ebe8583521a9329da5a7d179400228551b1506cf",
"sha256:04dc8c446a384974754388c0f906520e6f332d6d7afe0d3734a8feed7472a831",
"sha256:80c332ac5101def4d5e0be589704ae16ddb5f3a2bf783c995e837ca9bfe44a08",
"sha256:7156e1b76a3e8bf5757d53cbf409a09fc031d8d41462bfe7730b3d99cdb448e6",
"sha256:d42304ba6267ebe61f66288406c018dfe70df818f60967711e196d5a858575be",
"sha256:94acd9641c3f6d37f3582206a808c6902b87f40f402507bf194e5c15845426f6",
"sha256:6da8615cce98665b7464a593f9d26ea42d4f9d89c043f9ca163afe3dd533fdfe",
"sha256:78b772a0369ff84bc9fbf69f1c9fbedfdb4deb60d0a9766b3bcdd200673ddfac",
"sha256:b72e2d807e1586c75a1e4c231b5672f5d542ec510ff468837ac31e032f56e79a",
"sha256:173573a5e25da162ba1dba388915a128b9ce3162c17d34a27b558d514f8c13ea",
"sha256:f1c0ce5fae3b5d49e279ecf59860d3db7aa675e444dfe2c39d6c8481b85aaf20",
"sha256:81ad52c55bc0b4748d2aa52bb56ccc6a199ef00579664095bfe339f9c9a55331"
]
},
"Metadata": {
"LastTagTime": "2017-11-29T06:52:57.946416477+08:00"
}
}
]

It looks to be configured correctly, I think this is going to come down to a configuration issue on your PC that is potentially blocking to docker container from binding or being accessible on that port.

The port is accessible because:

I opened the port using the command:
sudo firewall-cmd --zone=public --add-port=3000/tcp --permanent
I have other tens of dockers using other ports, all of those dockers ran OK.

Besides, if the port was blocked, I should not be able to see the that page showing "Something happened to either Gekko or the connection. Please check the terminal where Gekko is running or your network connection."

Besides, if the port was blocked, I should not be able to see that page showing "Something happened to either Gekko or the connection.

You are right.

Something happened to either Gekko or the connection. Please check the terminal where Gekko is running or your network connection.

This means Gekko is unable to setup a websocket connection, can you try navigating to http://localhost:3000/api/gekkos in your browser? If you see [] that means your browser can connect to the rest API. If you see something else please let us know.

I got the following page with several buttons:
JSON RawData Headers Save Copy,
When I clicked RawData, I got []
When I clicked Headers, I got:

Access-Control-Allow-Methods
GET,HEAD,PUT,POST,DELETE

Access-Control-Allow-Origin
*

Connection
keep-alive

Content-Length
2

Content-Type
application/json; charset=utf-8

Date
Fri, 08 Dec 2017 11:11:28 GMT

have same problems
i have try it with external IP Adress but, no success because cant reach Error: listen EADDRNOTAVAIL:
const CONFIG = {   headless: true,   api: {     host: 'IPADDRESS', ....   ui: {     ssl: false,     host: 'IPADDRESS', ... }
with Emtpy String I got full response, but white page ...
const CONFIG = {   headless: true,   api: {     host: '', ....   ui: {     ssl: false,     host: '', ... }
with <IP_ADDRESS>, i got html page but no access to /api and BAM disconnected
const CONFIG = {   headless: true,   api: {     host: '<IP_ADDRESS>', ....   ui: {     ssl: false,     host: '<IP_ADDRESS>', ... }

@gdomod did you find a solution to your problem?


#150 allow icon overlap

Set "icon-allow-overlap": true for physical_stores layer to close #150.
Additional improvements:

Changed map language to German
Adapted map style to app design

Looks very nice!

Looks very nice!

Why are there that many reformatting changes? makes it pretty much impossible to review

Why are there that many reformatting changes? makes it pretty much impossible to review

Why are there that many reformatting changes? makes it pretty much impossible to review

I used the maputnik GUI editor to edit it, and it exported it like that 😕

Why are there that many reformatting changes? makes it pretty much impossible to review

I used the maputnik GUI editor to edit it, and it exported it like that 😕


244 add alembic utils

Fix #157, #244, #253.
Adds four types of triggers which fire when

any table in hame schema is updated (or a new row is inserted), modified_at field gets a new timestamp,
any row on plan, plan_regulation or plan_proposition table gets new lifecycle_status --> new row is added into lifecycle_date table.
lifecycle_status on any row in table plan is updated, all plan objects, plan regulations and plan propositions associated with it get the same lifecycle_status.
a new plan object is created, it gets plan_id from the most recent plan (newest by created_at)

@Rikuoja I made the requested changes.


check lang detect, which seems to be setting lang = NULL for some obviously english cases

https://mail.google.com/mail/u/0/#inbox/FMfcgxwDrvDbsjsXSDHBlJVfKfCvGfMQ

Washington (CNN) Senate Republicans voted to advance to floor debate on their efforts to repeal and replace Obamacare.
Vice President Mike Pence cast the tiebreaking vote.
In a dramatic moment, Sen. John McCain returned from Arizona to applause from fellow senators. He cast a necessary Republican vote for the motion after two GOP senators -- Susan Collins and Lisa Murkowski -- sided with all Democrats in opposition.
As the vote began, protesters in the Senate gallery shouted "kill the bill" and "shame, shame, shame!"
The vote came as Senate Majority Leader Mitch McConnell and President Donald Trump dared their fellow Republicans to block their seven-year campaign promise to repeal and replace Obamacare.
The next step is floor debate on the legislation to overhaul the Affordable Care Act even though there aren't any guarantees the votes are there to eventually pass it -- and it's unclear what a final bill will look like.
The vote was up in the air until the last moments, when several Republican holdouts announced their support, including Sens. Rand Paul, Dean Heller, Rob Portman and Shelley Moore Capito.
Trump, who has repeatedly said he's ready to sign any repeal legislation, celebrated the vote, which creates a path to give him the major congressional victory that's eluded the White House thus far.
"I'm very happy to announce that with zero of the Democrats' votes, the motion to proceed on health care has moved past and now we move forward toward truly great health care for the American people. We look forward to that. This was a big step," Trump said at a White House news conference.
"I want to thank Sen. John McCain," he added. "A very brave man. He made a tough trip to get here."
But while Trump was happy with the result, he also acknowledged there's a long road ahead for the legislation and criticized the two GOP senators who opposed the motion.
"We had two Republicans that went against us, which is very sad, I think," Trump said. "It's very, very sad -- for them. But I'm very, very happy with the result. I believe now we will, over the next week or two, come up with a plan that's going to be really, really wonderful for the American people."
Democrats are united against the bill, saying it would end health care coverage for millions of Americans.
"Anyone who thinks this is over is sadly mistaken," said Minority Leader Chuck Schumer, D-New York. "There are many, many Republicans who don't like this bill, who don't want to vote for it. They are under enormous pressure to vote for it. ... Anyone who voted to move to proceed and certainly anyone who votes to send this bill to conference is virtually, certainly, voting to kick millions off health care, to make it much harder to get coverage if you have a pre-existing condition, to slash Medicaid and give a huge amount of tax cuts to the rich."
Sen. Bernie Sanders Monday called the bill the "cruelest, most destructive and irresponsible piece of legislation ever brought to the United States Senate in the modern history of this country."
In a speech at the NAACP national convention, highlighted the possible effects of the bill's provisions, which include cutting Medicaid, defunding Planned Parenthood and roadblocks for those with pre-existing conditions.
JUST WATCHED McCain returns to Senate floor (full speech) Replay More Videos ... MUST WATCH McCain returns to Senate floor (full speech) 15:25
Paul, Heller back motion
McConnell won over key holdouts on the procedural vote, but those senators haven't said they will back the final measure.
Paul, a Kentucky Republican, said he would support the procedural motion to open debate on the health care bill, so long as leadership guarantees a vote on a full repeal of Obamacare.
"If this is indeed the plan, I will vote to proceed and I will vote for any all measures that are clean repeal," Paul tweeted. Such an amendment would be expected to fail, however.
Heller, who has complained that Obamacare repeal efforts could hurt Nevada residents dependent on Medicaid, nevertheless said he'd vote to move forward. Heller is facing a tough re-election campaign in Nevada next year.
"Obamacare isn't the answer, but doing nothing to try to solve the problems it has created isn't the answer either," Heller said in a statement. "If the final product isn't improved for the state of Nevada, then I will not vote for it; if it is improved, I will support it."
Republican leaders Tuesday worked thread any needle they can to appease other holdouts.
"The only goal is to get onto the bill. Nothing happens until we do, so that's the only goal," a Republican aide said.
"These are the moments legislatively when you get creative. We're getting creative."
JUST WATCHED Trump: Senate GOP can end Obamacare nightmare Replay More Videos ... MUST WATCH Trump: Senate GOP can end Obamacare nightmare 02:28
Now what?
Leadership was floating a strategy Tuesday morning that lays out a series of proposals that attempt to give everyone something of what they want, even though nearly every element is destined for failure on the floor during the amendment process.
That strategy is as follows, in terms of amendment order:
The full repeal bill that Paul backs.
A bill including the "consumer freedom amendment" that Sens. Ted Cruz and Mike Lee originally advocated for, which includes a request from Sen. Rob Portman for $100 billion in additional Medicaid funding. That would require 60 votes to pass.
The GOP bill proposed earlier this month that McConnell pulled when too many Republicans opposed it.
Finally, a "skinnier" repeal bill that repeals Obamacare's individual and employer mandates.
There are also several parts of any legislation that could end up stripped from the final bill thanks to Senate rules that will require 60 votes for passage rather than 50.
The Senate parliamentarian says a provision that would allow insurers to charge higher premiums to older enrollees, as compared to younger ones, would be subject to the 60-vote requirement. Obamacare limited insurers to charging those in their 50s and 60s up to three times the rates of younger consumers. The Senate bill would widen that to five times, hiking the premiums that older enrollees must pay.
This provision has drawn sharp criticism from the influential AARP, which has slammed it as an "age tax."
The parliamentarian is also reviewing whether to allow several key changes Republicans want to make to Obamacare's insurance regulations. Obamacare mandated that insurers cover 10 essential health benefits, including maternity, substance abuse and prescription drugs. The Senate bill would allow states to apply for a waiver from this requirement so insurers could offer skimpier polices with lower premiums.
Conservatives will not be pleased if this waiver section is removed.
This story has been updated.

To help with debugging, here is a list of 62K row IDs where lang is NULL but langdetect outputs 'en'
lang_null_langdetected_en.zip

The issue is due to logical problem in https://github.com/fhamborg/news-please/blob/3b93656a87305a2c896d8d5bba1acbca6144f21d/newsplease/pipeline/extractor/comparer/comparer_Language.py#L43
Trying some things to fix

fixed by 23cffaf5df407cddcd0f941dec87a4619cf6f476
(not available as pypi yet)


Iqm plugin for Blender 3.7x

Will there be an update for blender 3.7x veesion?

The latest one works perfectly fine with 2.73a, I use it regularly.

You sure?  (Maybe something changed between 2.73 and 2.74 that wrecked it)

As far as I can tell, f91c459d8b6ed9a51b4d52e94fff7a3c2f21e360 broke
normals. I've just been using a version from before that.

Colby

On Mon, Feb 8, 2016 at 6:46 PM, FuzzyQuills<EMAIL_ADDRESS>wrote:

You sure?  (Maybe
something changed between 2.73 and 2.74 that wrecked it)
—
Reply to this email directly or view it on GitHub
https://github.com/lsalzman/iqm/issues/13#issuecomment-181677298.

Broke how? It would help if there was more description here.
On Tue, Feb 9, 2016 at 7:24 AM, Colby Klein<EMAIL_ADDRESS>wrote:

As far as I can tell, f91c459d8b6ed9a51b4d52e94fff7a3c2f21e360 broke
normals. I've just been using a version from before that.

Colby

On Mon, Feb 8, 2016 at 6:46 PM, FuzzyQuills<EMAIL_ADDRESS>wrote:

You sure?
(Maybe
something changed between 2.73 and 2.74 that wrecked it)
—
Reply to this email directly or view it on GitHub
https://github.com/lsalzman/iqm/issues/13#issuecomment-181677298.

—
Reply to this email directly or view it on GitHub
https://github.com/lsalzman/iqm/issues/13#issuecomment-181844329.

Okay, I tried fixing up the blender-2.74 exporter. Can you try the latest
one in the git repo now and see if it works?
On Tue, Feb 9, 2016 at 7:33 AM, Lee Salzman<EMAIL_ADDRESS>wrote:

Broke how? It would help if there was more description here.
On Tue, Feb 9, 2016 at 7:24 AM, Colby Klein<EMAIL_ADDRESS>wrote:

As far as I can tell, f91c459d8b6ed9a51b4d52e94fff7a3c2f21e360 broke
normals. I've just been using a version from before that.

Colby

On Mon, Feb 8, 2016 at 6:46 PM, FuzzyQuills<EMAIL_ADDRESS>wrote:

You sure?
(Maybe
something changed between 2.73 and 2.74 that wrecked it)
—
Reply to this email directly or view it on GitHub
https://github.com/lsalzman/iqm/issues/13#issuecomment-181677298.

—
Reply to this email directly or view it on GitHub
https://github.com/lsalzman/iqm/issues/13#issuecomment-181844329.

Yea sure, sorry for the late reply BTW. :)
If that update doesn't fix it, I will post screenshots of the problem.


Update bad-words.txt

Added some bad words

hi! thanks for this, but i just updated this to use the badwords package!


Relative Copy Class should be CURIEs

Any children under https://www.ebi.ac.uk/ols/ontologies/efo/terms?iri=http%3A%2F%2Fwww.ebi.ac.uk%2Fefo%2FEFO_0030066&lang=en&viewMode=All&siblings=false are allowed

Should make a branch off of v0.0.11 tag

Remove focal genome amplification


Fixing linting errors on align-after-merge

@zmek in case its helpful, this branch fixes all the checks set up as pre-commit hooks on top of the changes on your align-after-merge branch so should hopefully mean the linting workflow passes. A lot of the changes are automatic reformatting, with the remaining changes mainly adding in missing imports, having the dependencies pulled in to pyproject.toml from requirements.txt and some other minor sundry changes.
EDIT: I've added comments to the main bits there was a decision point.

@matt-graham Thank you so much for taking the initiative here!

No problem. I'd made most of these changes while replying to your message yesterday to try and diagnose the linter errors and realised this morning it would have probably been easier / clearer to just submit a PR 😁


Updates Mongoose and replaces callbacks with promises

What's this PR do?

Updates Mongoose to 4.6.1
Replaces findOne calls with findById
Switches create, findById, save callbacks with promises ref
Abstracts the dbUsers.findOne call into a loadUserAndSignup function
Renames continueReportbackSubmission as loadReportbackSubmission
Renames startReportbackSubmission as createReportbackSubmission
Removes self = this noise

How should this be reviewed?
Test CampaignBot locally, on staging upon deploy.
Any background context you want to provide?
Next up, adding relevant Phoenix JS and Northstar JS networking calls.
Checklist

[x] Tested on staging.

@sergii-tkachenko Thanks so much for the helpful review -- I think this branch should be good to go! Tested on staging.