Incrase max ops 10x

In practice there are actually messages with over 500 attachments. We've seen this at Nylas while running our sync engine that provides an email API.
You can simulate the failure with this script:
from flanker import mime
import uuid

msg = mime.create.multipart('alternative')
msg.append(
    mime.create.text('plain', 'Hello'),
    mime.create.text('html', '<html>Hello</html>'))

text_msg = msg
msg = mime.create.multipart('mixed')

msg.append(text_msg)

for a in range(5000):
    # Disposition should be inline if we add Content-ID
    msg.append(mime.create.attachment(
        'text/plain',
        str(uuid.uuid4())))

msg.headers['Subject'] = 'Breaks Flanker'
msg.headers['To'] =<EMAIL_ADDRESS>msg.headers['From'] =<EMAIL_ADDRESS>
rfcmsg = msg.to_string()

break_msg = mime.from_string(rfcmsg)

Throws
Traceback (most recent call last):
  File "too_many_attachments.py", line 27, in <module>
    break_msg = mime.from_string(rfcmsg)
  File "/Users/mg/workspace/sync-engine/flanker/flanker/mime/create.py", line 82, in from_string
    return scanner.scan(string)
  File "/Users/mg/workspace/sync-engine/flanker/flanker/mime/message/scanner.py", line 24, in scan
    return traverse(Start(), TokensIterator(tokens, string))
  File "/Users/mg/workspace/sync-engine/flanker/flanker/mime/message/scanner.py", line 96, in traverse
    parts.append(traverse(token, iterator, content_type))
  File "/Users/mg/workspace/sync-engine/flanker/flanker/mime/message/scanner.py", line 34, in traverse
    iterator.check()
  File "/Users/mg/workspace/sync-engine/flanker/flanker/mime/message/scanner.py", line 270, in check
    self.opcount, _MAX_OPS))
flanker.mime.message.errors.DecodingError: Too many parts: 501, max is 500

Obviously one can create an email message with a nearly arbitrary number of attachments, but with our entire mail corpus, we haven't seen any that hit the limit in this commit. It's also worth noting that we have not experienced any "runaway" parsing that would require this limit check in the first place. (Perhaps it was only used when Flanker was being developed?)

We also met this problem recently with our corpus. Is it planned to include this adjustment to a next published release?

This PR fails its tests with the following:
FAIL: tests.mime.message.scanner_test.bad_messages_test
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/mg/jenkins/shiningpanda/jobs/1cc186e5/virtualenvs/d41d8cd9/lib/python2.7/site-packages/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/var/mailgun/jenkins/workspace/flanker-pr/tests/mime/message/scanner_test.py", line 161, in bad_messages_test
    assert_raises(DecodingError, scan, ENCLOSED_ENDLESS)
AssertionError: DecodingError not raised

Given that master passes this test, and the branch is quite old, @grinich could you maybe rebase it?

Just rebased and I'm not sure why the builds are failing. The branches have literally a 1 character difference... https://github.com/mailgun/flanker/compare/master...nylas:master

Can one of the admins verify this patch?

Hey @mhahnenberg do you still want to merge this changes to the master? Looks like you collected several assorted commits here. Some of them are not relevant anymore. So if you do then let me know and I can work with you on making this happen.

Huh, didn't realize this was updating every time we pushed something. We'd really like Python 3 support, but it could be nice for us to get back onto upstream flanker in the interim.

@mhahnenberg I hear you. I'll see what I can do.

I think I successfully rebased onto upstream/master. I'll try opening another PR.

@mhahnenberg I guess this one can be closed?

Yeah! 👍


range/location support?

Hi!
I notice that CommonMark's AST has been implemented location info(but it is unstable).
{ t: 'Document',
  start_line: 1,
  start_column: 1,
  end_line: 20,
  children: []
}

An example is azu/commonmark-ast-sandbox.
Do you have any plans to support range or location on AST(like Esprima)?

/subscribe
Yes, this would be an awesome feature to have.

I've began work, but found that the initial setup (from marked), and the whitespace-sentsitive language, pose problems. It probably requires more work than expected.

Great work :bangbang:
I'll try it.


test: Increase timeout of boot VM stage to 45 minutes

We have seen a massive amount of CI builds timing out. Some of them are close
to completing the stage but then fail because the download of the vagrant
images takes longer than usual. Bump the timeout until this is resolved.

This change is

test-me-please

Coverage increased (+0.01%) to 41.552% when pulling 5d0a9f977e19227d85e22c1938c24869a2e7ce20 on pr/tgraf/increase-boot-vm-timeout into 6ecfff82c2314dd9d847645361b57e2646eed64b on master.


Generic Value Typed Gaussian Processes

General Value Type:
Introduced Gaussian Processes with general type Value as output instead of Vector[DO].
OLD: GaussianProcess[D, DO].apply(Point[D]): Vector[DO]
NEW: GaussianProcess[D, Value].apply(Point[D]): Value
Vectorizer:
The introduced change needed the concept of a Vectorizer. The vectorizer is passed as impiclit parameter. The functionality should remain the same where the implicit VectorVectorizer of the Vector companion object can be found. The DiscreteField companion object offers methods to vectorizer and unvectorize a DiscreteField or an IndexedSeq[Value].
Kernels:
The kernels return now simply a DenseMatrix[Double]. They have an additional field outputDim instead of the dimension [_ DO].
UseCase:
As an example the scalismo.statisticalmodel.StatisticalMeshModel is a working example where a Gaussian Processes from Point[_3D] to Vector[_3D] is used.

@Andreas-Forster Thanks a lot for. This looks excellent to me (except the two very minor changes mentioned above).
I am looking forward to use it to build my first intensity model with scalismo.

I also agree on the general structure of the changes. This will add a lot of potential applications with GPs.
Thanks a lot!

As the only possible objection seems to be that we are loosing the type safety in the output dimension (for which there is currently no obvious fix), I am going to merge the pull request in order not to block further developments. As soon as we have a fix to make the output dimension type-safe, we can always fix that "problem" in  a separate PR.


Add rewards pools to liquidity rewards and penalties page

I think on this page https://docs.vega.xyz/testnet/concepts/liquidity/rewards-penalties we just need maybe another subheading on "Additional liquidity provider rewards" or words to that effect.
It would just say something like:
"In addition to fees, anyone can fund reward pools that will pay out to LPs at the end of each epoch based on the proportion of fees they have received.  To learn more about this, and trading rewards in general see https://docs.vega.xyz/testnet/concepts/trading-on-vega/fees-rewards."

Closing this as it's in the wrong repo. I've recreated it here https://github.com/vegaprotocol/documentation/issues/470


nix-shell failing to parse nix that nix repl doesn't fail

Describe the bug
A nix-shell derivation I use daily broke after a system upgrade. nix-shell complains with syntax errors, however when loading it in nix repl it has no problems.
Steps To Reproduce
Steps to reproduce the behavior:

Download this file
Run nix-shell -p shell.nix (assuming shell.nix is in $PWD)
It complains that line 29 (the in statement) is an unexpected token despite nix repl having no issue parsing the same file.

Expected behavior
nix-shell derivation should work as intended and the environment should be created.
Screenshots

Additional context
Notify maintainers

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.4.132, NixOS, 21.11pre302615.967d40bec14 (Porcupine)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.12`
 - channels(root): `"nixos-21.11pre304626.8ecc61c91a5"`
 - channels(infinity): `""`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module:

nix-shell -p foo is roughly equivalent to with import <nixpkgs> {}; runCommand { buildInputs = [ foo ]; }
https://github.com/NixOS/nix/blob/47e96bb533f8cacc171bec9b688b134de31a48a9/src/nix-build/nix-build.cc#L272-L275
There is also a feature of stdenv where standalone files in buildInputs are interpreted as shell hooks, so in your case you are making it parse shell.nix with bash.

nix-shell -p foo is roughly equivalent to with import <nixpkgs> {}; runCommand { buildInputs = [ foo ]; }
https://github.com/NixOS/nix/blob/47e96bb533f8cacc171bec9b688b134de31a48a9/src/nix-build/nix-build.cc#L272-L275

I'm exhausted, I just realized I was adding the -p option the entire time when I didn't mean to, thanks for pointing that one out.

There is also a feature of stdenv where standalone files in buildInputs are interpreted as shell hooks, so in your case you are making it parse shell.nix with bash.

That's actually pretty nice to know, thanks for that one.
Closing this as it was user error.


Suggestions for Game Studio UI

Unfortunately, this topic no attention is paid, I want to try to present convincing facts -  what we must change in that direction.
As you know, the engine received a new dark user interface and this is excellent, but comparing the color scheme with the old version, I realized that the new color scheme is very unpleasant in harmony, all the elements (panels and menu sections) not having color differences and merge into each other, the places where they are bordered by ribs visible weakly. The old version of the interface in this regard is much better, its interface is clearly divided into blocks with different gradients that are visually well different from each other and look like separate blocks. For clarity, I made a comparison of several engines, you need to understand what I mean.

Basis of the foregoing, I can suggest the following steps:

Improve the current color scheme by delimiting individual elements with other gradients plus ... I would still make it a little lighter in general
It makes a lot of sense to leave the old version of the light interface version, but with new icons
It would be nice to systematize new icons by color, that is, to make them in a uniform color scheme, it doesn’t matter what tint they should be, blue as in Unreal or gray as in Unity, just now there is too much variety of icon colors in the engine, it looks a bit chaotically ... yellow, red, green blue, black, etc.

I would welcome any suggestions and comments.

Very nice comparison. Looking at the comparison picture I have to say that I find Unity's layout the best. Unreals icon set is far too big in my opinion. I personally haven't checked it but is it possible to switch to another theme or define a custom one?

As far as I know, in Unity it is possible

I think this solved long ago but in case you made an issue about it, let me explain again.
Xenko's new design is influenced by Visual Studio Blend, which is built for designers and designed by designers. Also, the icons are coming from the geometric paths of Visual Studio Image Gallery, and changing the colors of them (which is fairly easy to do, as they are paths) makes them inconsistent, prolly incompatible with Luminosity Converter, and makes it hard to build multi-theming features in future. The only way to have a uni-color consistent set of icon, is to hire some icon designers to design icons dedicated for Xenko (which I'm not sure if you/anyone is currently volunteer to spend money for it); That's why I think we need to stick with multi-color icons of VS for now (they are very well designed tho, but I understand this is subjective).
Also, the solution is not "Let's remove the entire black theme which people done tones of work for it, and which is actually based on a design philosophy", the solution is to "Implement multi-theming features, so people can add their own themes based on some config files".
Thus, trying to change the colors is useless. Next guy comes, creates an issue and changes the theme to what he/she likes.
There is already a color converter, try to implement a theme config file and load color resources from it, then people can even make light themes for the editor, and we will all live happy ever after.

by the way, #000 masterrase :D
just kidding

influenced by Visual Studio Blend, which is built for designers and designed by designers.

@0xaryan
you have a lot to learn, young man. :) taking microsoft applications as a design reference guide made me laugh a bit. I am working in the creative coding and design field since 20 years and micosoft is never a good design reference. so I agree with the author of this thread, the color contrast is a bit too harsh and not so pleasant to work with for long hours.
brightening the dark backgound would indeed release some strain to the eye. especially with new high quality screens it's not necessary to have such a high contrast.
is there a central place to change the background color? or do we have to go thru all xaml files separately?

@tebjan

taking microsoft applications as a design reference guide made me laugh a bit.

I don't see this anything more than a "microsoft bad" meme.
"Microsoft Application Design" makes 0 sense. MS got tones of designers that you can't even count. Different apps have different designs based on context and platform, and for a popular product like VS, MS actually has a dedicated team of UI/UX designers, and some other people who bring customers to MS camp to get live feedback from them.
One should already know these if he/she isn't even working in the design field.

micosoft is never a good design reference. so I agree with the author

Wait what? You agree with the author because Microsoft is not a good design reference? Maybe your idols in the design field are not referring VS as a product with a bad design but arguing about some other app (assuming the people who you're talking about are actually good designers).
Don't agree with people because some other people have some opinions. Think for yourself.

brightening the dark backgound would indeed release some strain to the eye.

This is true for some people, but also there are other people who prefer to have it darker. All these subjective opinions are fine.
As I already mentioned in the comments above, if you really care about contributing in this area, just implement a theming feature and then there would be themes available for everyone, even who disagree with both me and you, like those who use everything with Light theme. That's the correct way of doing this.

is there a central place to change the background color? or do we have to go thru all xaml files separately?

There is no need to go through all Xaml files, but there should be multiple files which hold primary and secondary colors if I recall correctly.

you have a lot to learn, young man. :)

That's true. Not only for young ones but everyone. Learning never ends. (:

Thus, trying to change the colors is useless. Next guy comes, creates an issue and changes the theme to what he/she likes.

I can’t agree with you, all world-renowned game engines adhere to the exact color philosophy that I mentioned above. In companies such as Unity or Unreal work certainly not fools, the color scheme that they have now appeared is not random, it didn't appear due to the subjective opinion of the developers, they did a huge titanic work for the optimal color ratio. But you stubbornly continue to defend your subjective opinion of the bliss of black ...

Can we stay on the subject at hand please, let's not start a flame war in here, that's not what issues are for.
Implementing a way to swap themes seems like the better idea here as it would allow other people to have their own while we could also adopt one of those as the official theme if necessary, let's continue from there.
What would be feasible given the current (xaml based) interface ? How could we swap colors, icons, layout and such of our ui elements ? How to load it when it isn't bundled with the engine ?

Can we stay on the subject at hand please, let's not start a flame war in here, that's not what issues are for.
Implementing a way to swap themes seems like the better idea here as it would allow other people to have their own while we could also adopt one of those as the official theme if necessary, let's continue from there.
What would be feasible given the current (xaml based) interface ? How could we swap colors, icons, layout and such of our ui elements ? How to load it when it isn't bundled with the engine ?

Totally agree with you! I was waiting for constructive criticism and not a categorical protest and a complete rejection

you stubbornly continue to defend your subjective opinion of the bliss of black ...

Dude. You are asking the community to turn the theme to what you like. I am asking the community for a theme infrastructure so everyone work in the editor with the color they like, and you say I am the one who is defending a subjective opinion? Think before you talk...

In companies such as Unity or Unreal work certainly not fools

Of course they are not. Unity actually sells its dark theme. It's a source of income for them to force free licensed people buy the product to gain a darker theme...
Also, just don't do whatever others do. Be initiative. A multi-theme feature for Xenko makes it better than other engines in case of personalization.

Following MS design is bad, they make bloated, ulgy, enterprise like, and non consistent UI, better to take inspiration from blender 2.8 and the new unity

What would be feasible given the current (xaml based) interface ? How could we swap colors, icons, layout and such of our ui elements ? How to load it when it isn't bundled with the engine ?

I have implemented a Luminosity transform for the Icons, that should do it (of course If you want to use VS Icons). And for Xaml I'm thinking about FsXaml Type Provider. It is a generative type provider, that means it can create actuall .NET types from Xaml in compile time, so it can be used in another .NET assembly and will be easy to load and modify resources from code. I'm not 100% sure if it can help in this case, or if the maintainers are ok with using F# tho.

I'm also thinking about a redesign and reimplementing the whole editor in Avalonia which is cross-platform. That would be a huge work tho.

https://devblogs.microsoft.com/dotnet/calling-all-net-desktop-and-mobile-developers/

Changes have been made towards this issue, see #637 and #667.
We also had some discussion around the property grid on discord, the gist of it is that its layout could be better by chaning the size and spacing of its elements.
@RayKoopa

the font size and expander glyph are too big aswell, given its already highlighted by color too
ignore the colors and that the font size is still wrong in this quick mock. i just wanted to test small margins

A quick mockup I did

I can also add that I'm currently working on a followup PR to #667 , updating the AvalonDock theme to the most recent official one being the core motivation. It provides some stronger contrasts and some style to aid in separating content. Partly also because blurry elements are fixed in #667 . I also add a VS2019 Dark-like theme (brighter than Blend Black), and an experimental VS2019 Blue-like theme for more colorful contrasts right now.
However, it is still hard to recognize which tool window is currently active in the Studio, as even inactive tool windows have all their list selections at the same bright blue. If you check VS2019, only the active tool window is allowed to also have blue selection highlights, selections in inactive tool windows are grayish. I wanted to try to fix it but currently cannot get the focus detection to work properly yet...


θ in Greek book font rendered as swash form

OCR result: ϑεοὶ γὰρ οὔποτ᾽,
This is an ordinary book font used by editions of classical texts. Because the design of its theta, however, this letter is frequently OCR’ed as a swash form and requires manual correction as it stands out from the rest of the text when rendered in other (esp. sans) Greek fonts.

So improved training is necessary. Do you know a freely available computer font which emulates that design? Or is there a ground truth data set which can be used to train recognition of that font?


Floating point postal codes in 9 US sources

Hi all,
Looks like several US sources currently have zip codes that have somehow been converted to floating point. I haven't taken a look yet to see if this is an upstream issue, but because it's affecting several different sources it might be an OA machine problem.
We first noticed this after a failed acceptance test from one of our Geocode Earth Pelias builds identified an incorrect zipcode in San Francisco. All the zipcodes in us/ca/san_francisco.csv are floating point numbers.
head -n 10 us/ca/san_francisco.csv
LON,LAT,NUMBER,STREET,UNIT,CITY,DISTRICT,REGION,POSTCODE,ID,HASH
-122.4163828,37.784297,609,ELLIS ST,,,,,94109.0,497814-789864-0,6b6a58ebcb6f10dd
-122.4164064,37.7842939,611,ELLIS ST,,,,,94109.0,497815-789865-0,940ebaffd7f740c5
-122.4016514,37.7811548,829,FOLSOM ST,216,,,,94107.0,470860-793855-0,c10795d400b385b1
-122.401629,37.7982992,868,SANSOME ST,,,,,94111.0,476244-709543-0,3ff504903f2ffe6b
-122.390987,37.7870388,400,BEALE ST,2306,,,,94105.0,483007-768145-0,1ea1e3950b2952db
-122.4273408,37.7766936,565,IVY ST,,,,,94102.0,499404-811386-0,37a51616e9eec91d
-122.3978927,37.7396926,2025,NEWCOMB AVE,,,,,94124.0,500794-829852-0,ca334d8c281e68ab
-122.4094763,37.7528848,2860,24TH ST,,,,,94110.0,494477-754373-0,7cdb018df267af51
-122.4542507,37.7228655,267,LEE AVE,,,,,94112.0,499919-820033-0,ea071809ad239b37

Taking a look with trusty old grep, I was able to confirm this is more widespread than just SF.
$ grep -E ',[0-9]{5}\.0,' */*.csv */*/*.csv --color=auto | awk -F ':' '{print $1}' | uniq -c
    176 au/act/statewide.csv
 384134 us/ca/san_francisco.csv
   9992 us/co/pitkin.csv
 215432 us/fl/seminole.csv
   7663 us/ga/city_of_marietta.csv
   5418 us/ks/coffey_county.csv
   4330 us/mn/mcleod.csv
  11529 us/ok/city_of_muskogee.csv
  30090 us/or/coos.csv
   8784 us/or/umatilla.csv

Let me know if anyone has a suspicion as to where this sort of issue might be cropping up, or where a fix could be put in, and I'll try to take a stab at it if I can.

This happens when we consume a data source with a float data type. When it gets stringified it gets the decimal. For example, the San Francisco data lists the datatype for zip_code as "Number":

My guess is that this happens inside OGR conversion of Shapefiles and GDB source:
https://github.com/openaddresses/machine/blob/1b76b192a474210c28e5c9ed9b1f09f4bf203e12/openaddr/conform.py#L724-L729
We could special-case removing .0 from the postcode output, I suppose.


Limit attachments files and folders with special characters in name

: ; , = /
On Chat UI - We should show error message when attachment has special characters in name
On core API - reject such files with special characters in name (@avavilau to discuss with @Maxim-Gadalov)

@IlyaBondar returning the ticket.
We need to show error on "Upload from device" pop-up if user renames the files using ;:,/= symbols.
Now we show error on Attach files pop-up and for the file with : Also file with / in name isn't shown at all. Files with ;,= is uploaded without any error

Verified on staging successfully


Modified documentation and README

Corrected some typos and inconsistencies with regards to running
ibutsu containerized. README should now align with docs.
Resolves #360

Hey @EliadCohen thanks for your contribution! There seems to be a linting issue, if you can fix that up then we can get this merged.

Hey @EliadCohen thanks for your contribution! There seems to be a linting issue, if you can fix that up then we can get this merged.

On it


testing: Add -v flag when running individual test

Thanks for the help here. Love the improvements you have been making. Especially the improved test discovery, result streaming, and test debugging support!
Just a possible UX improvement I would love. When I run an individual test with the UI:

I get this in the testing panel:
        first_few = list(
            result[["record_id_l", "record_id_r"]]
            .head(len(first_few_expected))
            .to_pandas()
            .itertuples(index=False, name=None)
        )
>       assert first_few_expected == list(first_few)
E       assert [(0, 1), (0, ..., (1, 3), ...] == [(0, 0), (0, ..., (1, 0), ...]
E
E         At index 0 diff: (0, 1) != (0, 0)
E         Use -v to get more diff

mismo/block/tests/test_block_array.py:92: AssertionError
=========================== short test summary info ============================
FAILED mismo/block/tests/test_block_array.py::test_benchmark_array_blocker[n=10000,k=5,npairs=39,990]
============================== 1 failed in 0.24s ===============================
Finished running tests!

It would be awesome if the extension added the -v flag for me, so I would by default get the full diff. When I select a single test, that is probably because that individual case is failing, and so I want as much info as possible.
When running multiple tests, I think it makes sense to leave the behavior as is, keep it a bit more succinct, and leave the -v flag off.

Thank you for submitting your feature request and everyone who considered it! Unfortunately, this issue did not receive enough votes over the allotted time, and so we are closing the issue.


Using pool without destroying connection makes system to hang

var settings = require('./settings');
var mysql = require('mysql');
var pool = mysql.createPool({
    host: settings.host,
    database: settings.database,
    user: settings.user,
    password: settings.password
});
pool.query('select 1', null, function (error, result) {
    console.log(result);
});

This code prints result but never ends.
If I deploy this code to Amazon Lambda - endpoint finishes with timeout error.
If I add connection.destroy - this code finishes successfully:
pool.getConnection(function (error, connection) {
    connection.query('select 1', null, function (error, result) {
        connection.destroy();
        console.log(result);
    });
});

But I suppose destroying connection is not a good practice.
If I use connection.release() instead of connection.destroy - this doesn't help, hangs as in first example.

Node.js will hang open in your example because there is still an active network connection. You just need to use poo.end to close the pool when you want everything to shutdown: https://github.com/mysqljs/mysql#closing-all-the-connections-in-a-pool

@dougwilson This solution is OK for this one-time command line program.
But how can I close the pool when I have some server that should serve requests for indefinite time?
In my case I am using Amazon Lambda + API Gateway.
Request fails with timeout error if I don't destroy all connections.

Hi @bfanyuk the purpose of the pool is that the connections do not close, and remain open between use. If leaving them open doesn't work for your use-case, then you would need to create a brand new connection each time.
I am not familiar with Lambda, so have no specific information regarding the use on Lambda. The reason I suggested pool.end is because that is the solution for the reproduction case you provided. It may not be the issue in Lambda, I just have no idea how Lamba works.

@bfanyuk , you can try context.callbackWaitsForEmptyEventLoop = false; http://docs.aws.amazon.com/lambda/latest/dg/nodejs-prog-model-context.html.
Worked for me. But I didn't test it on a huge amount of requests (don't understand how this "freeze" behavior works on connections). I will do it later and put the results here when I finish

Thanks, @ayankousky !
Yes, since I posted this issue I found the solution you proposed and actively use it.


line buffering (buffering=1) warning on Python 3.8

Since the recent change made in #1014, I got a warning like below when launching images based on Python 3.8 (python=3.8 in environment.yml).
/srv/conda/envs/notebook/lib/python3.8/subprocess.py:848: RuntimeWarning: line buffering (buffering=1) isn't supported in binary mode, the default buffer size will be used

I believe this issue is related to the change in Python 3.8 where it started complaining whereas previous versions silently ignored buffering=1 for binary mode. I guess the related code is below:
https://github.com/jupyterhub/repo2docker/blob/a5f5bbbb75a9945d1f8fe8f8ff4844dfd4481742/repo2docker/buildpacks/repo2docker-entrypoint#L40-L46
Not sure if related, I also noticed texts on the console (mostly) lost colors after this recent change; Jupyter log messages used to have colored headings for warning/info/etc, but they are now all monochrome. Yet some texts are still printed in colors (i.e. Julia banner lost colors, but its prompt still has a color).

@manics what do you think about this issue?
I think the buffering=1 in the comment is a typo and should be bufsize=1 (like in the linked code sample). If you agree that it is a typo in the comment: I think this is an easy thing to fix.
The fact that colours have stopped working is annoying and we should try and get that fixed. @tomyun do you have any idea why "colours stop working" when you read in and output the output off a process? Do you have time to investigate that a bit?

Somehow I failed to include a complete warning message.

/srv/conda/envs/notebook/lib/python3.8/subprocess.py:848: RuntimeWarning: line buffering (buffering=1) isn't supported in binary mode, the default buffer size will be used
  self.stdout = io.open(c2pread, 'rb', bufsize)

The warning about buffering=1 is coming from io.open() accepts buffering which is set to bufsize when io.open() is called in subprocess.Popen(). AFAICT, it's not about a typo between buffering and bufsize, but more about the value itself.

Regarding color output, I guess it's about the difference between pipe and tty.
For example, Julia running on a regular terminal clearly shows it's on TTY.

julia> Sys.stdout
Base.TTY(RawFD(14) open, 0 bytes waiting)

On the other hand, Julia running on an image created by the latest repo2docker has stdout mapped to pipe which is as expected with the recent change.
julia> Sys.stdout
Base.PipeEndpoint(RawFD(12) open, 0 bytes waiting)

It turns out Julia initializes colored stdout for non-TTY only when it's given an explicit option (color=yes, not default auto) that explains my experience.
It's likely Jupyter (and IPython) works in a similar way that they ended up getting a non-color output here. Seems like this kind of issue is not uncommon as Python has a package like ptyprocess.

Before we fix the TTY issue we should decide what beahaviour we want.
If you're running repo2docker from the command line with default options it'll output the logs to the console, in which case colour logs makes sense, and doing this through a pty(?) makes sense since in principle someone could provide input via stdin. If it's run as a background container or a Kubernetes pod you wouldn't expect a terminal to be connected so non-colour logs makes sense.

Thanks for taking the time to explain @tomyun! I had thought this was much simpler than it is :-/
Is it easier to always support colour output? I think that is what repo2docker used to do and that detecting in which scenario we are sounds like a tricky thing to do (and get right).
And it sounds like there are two issues: (1) find a way to not trigger the warning and keep buffering small (I think this is on purpose to solve the problem of very short output. getting lost) and (2) colour output. Should we make one issue for each?

I've created a new issue https://github.com/jupyterhub/repo2docker/issues/1033 for the log colourisation, let's keep this issue just for the line buffering problem.

I'm not familiar with history of the recent change, but is there any reason we can't use default buffer size (bufsize=-1 or leaving out = 8192 bytes) or maybe turning it off (bufsize=0)? bufsize=1 for binary mode is silently ignored and seems to fall back to the default buffer size anyways.

Is there any progress on this? In the meantime, how about reverting back to the previous code until we find a more solid solution on correctly handling log buffer and colored TTY output?

Is the line buffering problem just a warning, or is it breaking something?

I believe it's just a waring and if that's the case bufsize=1 should be removed as it's not used.


Is it possible to change the timezone?

What is the best approach to work with multiple timezones?
I have dim_calendar and dim_timer dimensions (UTC), but depending of the user, I need to change the timezone to calculate and display data.
I did using KeyExpression
EXTRACT(
 HOUR FROM (
   time at time zone 'america/sao_paulo'
 ))

But, I don't know if is the best way..
Is it possible to pass the timezone on schema, olap.execute or before run the mdx queries (SET TIMEZONE='america/sao_paulo');
Thanks

There is no good solution for supporting different time zones for each user in one Mondrian schema. Dimension members are cached at the schema level and they cannot be different in each query.
In our application, we offer to specify one specific time zone for reporting purposes and then we convert all timestamps to the corresponding date entry in the time dimension.
One approach would be to have a date dimension and a separate hour dimension and store there UTC date and hour. But then when presenting query results to users you could dynamically replace UTC hour with the user time zone hour by adding the corresponding offset.


Multiple Defintions error

Hi!
I'm trying to compile some code for Teensy 3.5 with PlatformIO  2.3.2, and I'm getting multiple definitions error.
I followed the procedure described in README, which sadly didn't work even though I cleaned the build environment and restarted VScode all the time.
Error message looks like this: link
I'm using Arch based linux distro:
Linux sz-81nh 5.11.14-arch1-1 #1 SMP PREEMPT Wed, 14 Apr 2021 12:06:34 +0000 x86_64 GNU/Linux
Also, the same code (the only difference being instead of TeensyTimer, I used STM32Timer) compiles and works perfectly on ST Nulceo F303RE
Steps to Reproduce
Minimal example: Teensy_minimal.zip
Expected behavior
Code compiles for Teensy
Actual behavior
Linker spews out a bunch of Multiple definitions error

I don't see you use this library anywhere in the code.
It seems that you're using TeensyTimers library and there is something wrong with it and PIO.
I'm closing the issue now as invalid, until you prove that the issue is caused by this library.

The library is included in include/I2CEEPROM.h file

You have to modify your code as follows

main.cpp

#include <Arduino.h>
#include "TimerInterrupt_Generic.h"
#include "I2CEEPROM.h"

I2CEEPROM mem;

void setup()
{
}

void loop()
{
}

I2CEEPROM.h

#include <Arduino.h>
#include <Wire.h>
#include <math.h>

#ifdef __STM32__
  STM32Timer DEVICE_TICK_TIMER(TIM2);
#elif defined(__TEENSY__)
  TeensyTimer DEVICE_TICK_TIMER(TEENSY_TIMER_3);
#endif

#ifndef I2CEEPROM_H
#define I2CEEPROM_H

//#define __STM32__
#define __TEENSY__

class I2CEEPROM {};

#endif

I2CEEPROM.cpp

#include "I2CEEPROM.h"

/*library code here*/

Compile will be OK as tested on Arduino IDE, unless your PIO settings /configuration are wrong

You have to modify like this to use new C++ style hpp

main.cpp or Teensy_minimal.ino

#include <Arduino.h>
#include "I2CEEPROM.h"

I2CEEPROM mem;

void setup()
{
}

void loop()
{
}

I2CEEPROM.h

#ifndef I2CEEPROM_H
#define I2CEEPROM_H

#include <Arduino.h>
#include <Wire.h>
#include <math.h>
#include "TimerInterrupt_Generic.h"

//#define __STM32__
#define __TEENSY__

class I2CEEPROM {};

#include "I2CEEPROM_Impl.hpp"

#endif

I2CEEPROM_Impl.hpp

#ifndef I2CEEPROM_IMPL_HPP
#define I2CEEPROM_IMPL_HPP

#ifdef __STM32__
  STM32Timer DEVICE_TICK_TIMER(TIM2);
#elif defined(__TEENSY__)
  TeensyTimer DEVICE_TICK_TIMER(TEENSY_TIMER_3);
#endif

/*library code here*/

#endif    // I2CEEPROM_IMPL_HPP

Compile is OK on Arduino IDE, and should be OK on PIO, unless you have wrong settings / configurations


Adds rtinfo template

closes #128

Couldn't test yet with rebooting node as it would boot up again using development branch of the templates, the robot would error out because it couldn't find the template.


build performance problems

We currently face some problems with meteor built time and the new build profile package revealed that this package is responsible for a huge part of our build time (30s with- and ~70s without package-cache).
I honestly have no idea what the reason could be. We don't use a lot of css (actually we use jss and all css we still use is legacy code), so I guess it's not really related to the css minification process at all.

Do you also see it with rebuild times (dev mode), are only production builds?
https://github.com/qualialabs/profile is a good start for tracing build time problems.

I think only on production builds (have to properly benchmark it though).
I suspect https://github.com/meteor/meteor/pull/10260 is somehow related related - at a first glance it seems to improve the build time with this package, but it might be a coincidence(only did a single run).


PT Run: ~ as user home directory

📝 Provide a description of the new feature
Really a small thing but I personally use cd ~ a lot in terminal.

If you'd like to see this feature implemented, add a 👍 reaction to this post.

@dedavis6797
we will have to update the PowerToys Run documentation to add the ~ symbol as prefix for the user's home folder.

Great news!  This was resolved in the newly released 0.33 version of PowerToys!  Head to https://aka.ms/installpowertoys to grab latest.


Provide Amazon Linux 2 or non CentOS Docker Images

See original description here:
https://github.com/opendistro-for-elasticsearch/opendistro-build/issues/696
Currently only CentOS 8 images are supported:
https://github.com/opensearch-project/OpenSearch/blob/0ba0e7cc26060f964fcbf6ee45bae53b3a9941d0/buildSrc/src/main/java/org/opensearch/gradle/DockerBase.java#L39
Would Alpine Images will be possible too?

The current docker images are on AL2.
https://github.com/opensearch-project/opensearch-build/tree/main/release/docker
https://hub.docker.com/r/opensearchstaging/opensearch
https://hub.docker.com/r/opensearchstaging/opensearch-dashboards
Will close this one for now.
Thanks.


MechanicalWorld and migration from v0.11

I notice that DefaultMechanicalWorld is not really a replacement for nphyics3d::world::World from v0.11 but rather a sub-set. Whatever reason the geometry/collider world, body, collider, joint and force sets are now separate, the examples show that these often do not need to be used separately.
Is it worth adding a type akin to the old World to wrap all the above (and likely also ease migration from v0.11)?

Hi! I'd prefer not to add a structure like the old World because of the added maintenance cost. The main differences between the new MechanicalWorld and the old one is that all features that belong to its other components (GeometricalWorld, body, collider sets, etc.) have to be accessed on those components directly now.
Unfortunately I did not have the time to write a migration guide, though feel free to open issues or ask on discord if you have any difficulty.

Whatever reason the geometry/collider world, body, collider, joint and force sets are now separate, the examples show that these often do not need to be used separately.

The reason why they are no separate is for two reasons:

Ease of interaction with other systems (like game engines) that may provide their own storage for all those components.
For removing limitation (due to borrowing rules) that prevented the user of nphysics from, e.g., get mutable access to bodies and colliders while iterating on the list of contacts.

Hi; in the mean-time I was able to work this out and update my code (a fork of the old testbed for 3D) to v0.22 with my own World definition and the sub-set of methods I needed:
/// A wrapper over collider and mechanical worlds and associated objects.
///
/// This is akin to `nphysics3d::world::World` from 0.11.
pub struct World<N: RealField> {
    mechanical_world: DefaultMechanicalWorld<N>,
    geometrical_world: DefaultGeometricalWorld<N>,
    /// The set of physics bodies
    pub bodies: DefaultBodySet<N>,
    /// The set of colliders, each attached to a body-part
    pub colliders: DefaultColliderSet<N>,
    /// Joint constraints on bodies
    pub joint_constraints: DefaultJointConstraintSet<N>,
    /// Forces acting on bodies
    pub force_generators: DefaultForceGeneratorSet<N>,
    /// We always have a ground body as a reference.
    /// Note that there need not be corresponding colliders or graphics.
    pub ground_handle: DefaultBodyHandle,
    /// Simulation time point
    pub time: N,
}

I further modified the code to avoid Kiss3D's Window struct (which ideally wouldn't embed the scene for similar reasons to the above) and use the underlying components directly (Canvas, SceneNode etc.).
If you like I could open a PR vs this repo, but it's not a drop-in replacement for the testbed due to removal of features I didn't need (e.g. fixed cameras) — primarily it shows that a lot cleaner code is possible when fewer generics and esp. run-time options are required.


caddy v2 support

I wonder if caddy v2 support is something someone's working on

@nodece @GopherJ @dovics @00LT00 can anyone take this issue?

We update the casbin from v2.0.1 to v2.8.6 and the caddy from v1.0.1 to v1.0.5. @SvenDowideit

@00LT00 that's for caddy version 1 - I'm curious about caddy version 2 support

@00LT00 please fix it.

@SvenDowideit update caddy from v1 to v2. #7

@SvenDowideit v2.0.0 has been released: https://github.com/casbin/caddy-authz/releases/tag/v2.0.0


Administrator

I net a free rdp
[Enter feedback here]

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 5a02d662-56db-4efc-52d4-61689541f632
Version Independent ID: f8692043-d888-09d3-1bf3-bff51ae20e34
Content: Install RDS client access licenses
Content Source: WindowsServerDocs/remote/remote-desktop-services/rds-install-cals.md
Product: windows-server
Technology: remote-desktop-services
GitHub Login: @lizap
Microsoft Alias: elizapo

User feedback is undeterminable. Closing thread.
#please-close

#assign:xelu86


⚠ Adacta-fintech.com has degraded performance

In 55a109b, Adacta-fintech.com (https://www.adacta-fintech.com) experienced degraded performance:

HTTP code: 200
Response time: 1363 ms

Resolved: Adacta-fintech.com performance has improved in fb2037c.


Standardise input behaviour and update mechanism across pages

Some input boxes update the page as you type, while others only update when pressing Enter. It would be clearer and more efficient to add an "Update" button across all input boxes to prevent unnecessary updates and to provide better control for the user.
Additionally, the arrows on many input boxes appear to be a remnant from copy-pasting and don't seem to serve a clear purpose.
It would be ideal to establish a consistent standard for these input boxes, possibly through a shared template, that can be imported whenever a specific type of input box is used. Also implementing a standard update mechanism with an "Apply Now" button or similar across all pages to ensure uniformity in how settings are saved and updated.
This issue relates to #25 and #6.

have one refresh button per page

We have looked through the page and decided that one single way of updating is less intuitive than picking from "automatic" updating and updating prompted by an update button. Thus we wont implement a single standard used on every page.


Initial German site localization

Once we have #62 working, let's support German!
Note the deeper spec / technical docs are out of scope for now as they're a heavy lift, and getting out of date might be worse than not having them.

@paddyson79 has basically done this :raised_hands:
The only thing we're missing to close this out IMO is the home page.


Fix JS object duplicate name issue

Description
It's expected that the actions inside a JS collection should always have a collectionId if it's not deleted. (Even if it's deleted thecollectionId should persist though).

There were some documents found in the newAction collection which don't have any collectionId and deletedAt attributes. But those documents were having the same pageId. Thus when the front-end sends some new actions be created by the server, it finds the name conflicting with the existing ones.
This could happen since several updates/inserts in the DB are taken place without sharing one single transaction. We haven't introduced transactional behavior in our application.
During the creation of an action under JS collection, the action is created first in the DB without having a collectionId. After the creation of the action, the collectionId gets updated with a separate update query. The latter update might fail which puts the action document in such an expected state.
This PR will ignore those JS actions that don't have any collectionId during the name-checking of new action creation.
RCA
Fixes #14755
Fixes #13407
Type of change

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

How Has This Been Tested?

Manual
JUnit

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

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

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test sha=b863fed

/ok-to-test sha=2f42769

Verified fix for JS object duplicate name issue
CC: @subrata71


TS typing and minor android fix

Added TS definiton
Improved the .npmignore (to future proof more work)
Fixed small items on README for readability.
Small Android bug when tts.Engine would be undefined thus throwing error that cannot read property KEY_PARAM_VOLUME of undefined.

Thanks Brad.
I no longer have the time to maintain this repo.
I'm wondering if you would be interested in either being added as a collaborator here or if you would be interested in it being transferred to your account?
I will update the README with some info about this if no one wants to take it over in the next few days

I'll take it. Myself and some others would greatly appreciate it and
maintain it. Thanks for letting me know.
On Thu, Sep 22, 2016, 6:07 AM anarchicknight<EMAIL_ADDRESS>wrote:

Thanks Brad.
I no longer have the time to maintain this repo.
I'm wondering if you would be interested in either being added as a
collaborator here or if you would be interested in it being transferred to
your account?
I will update the README with some info about this if no one wants to take
it over in the next few days
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/anarchicknight/nativescript-texttospeech/pull/5#issuecomment-248872005,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFulhGXWc-yxJXVtb0C2qtip1pIUx61cks5qsmFbgaJpZM4KDIgN
.

Will see if I can get the transfer sorted now.
Do you want to take ownership for the npm module too?

Yea need npm so I can publish so that would be awesome. Let me know if you
need anything from me. If you can't get a transfer sorted that's fine. Just
having permission to manage the repo and publish to npm is fine.
On Thu, Sep 22, 2016, 7:40 AM anarchicknight<EMAIL_ADDRESS>wrote:

Will see if I can get the transfer sorted now.
Do you want to take ownership for the npm module too?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/anarchicknight/nativescript-texttospeech/pull/5#issuecomment-248892318,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFulhIAdg8OQbx1_v9nFrrFeZCsSDfCLks5qsnc3gaJpZM4KDIgN
.

I edited my comment above about the transfer - once I am able to I will transfer to your github account.
I will look in to sorting the mp transfer/publish permissions when I get home tonight

you should now have rights to publish to npm - once you confirm that is the case I will remove myself as being an owner.
Can you sort out the removal or renaming of your fork of this module. Once that is done I can transfer ownership of the repo to you. I will then put a note on my page for it to say it is no longer maintained by me and to point to your page

Should be good to go. Let me know how the transfer works out.

cool - transfer initiated.
Will keep this open until we know transfer has gone through OK and that you have been able to publish to npm with no issues

@anarchicknight things went good, published just fine. Thanks for this 👍

Brilliant, thank you for taking it on to make sure it keeps being maintained :)
I will remove myself as owner on npm shortly and you will be sole owner then


ModuleNotFoundError: No module named 'gsnet'

Hi, i meet this error when i tried to run real_service.py by following the instruction.
did i miss something? hope you can help me fix the problem.
thank you very much!
encoder.load_state_dict(torch.load(f'{trained_path}/weights/encoder_latest.pth', map_location=torch.device('cuda')))
===> loaded roartnet 1.9277136325836182
===> loading graspnet
Traceback (most recent call last):
File "real_service.py", line 118, in
from gsnet import AnyGrasp
ModuleNotFoundError: No module named 'gsnet'

Hi,
Have you followed the instructions in AnyGrasp to download the license, weights and binary codes? There should exist gsnet.so in the root directory so that you can import it.
Hope this helps!


Hyper-V Virtual Ethernet Adapter for WSL disables and enables itself which drops X11 connection

Version
Microsoft Windows [Version 10.0.19042.1466]
WSL Version
WSL 2
Kernel Version
Linux version <IP_ADDRESS>-microsoft-standard-WSL2 (oe-user@oe-host) (x86_64-msft-linux-gcc (GCC) 9.3.0, GNU ld (GNU Binutils) <IP_ADDRESS>00220)
Distro Version
Distributor ID:	Ubuntu
Description:	Ubuntu Jammy Jellyfish (development branch)
Release:	22.04
Codename:	jammy
Other Software
vcxsrv x server version <IP_ADDRESS>
terminator 2.1.1
powershell 7.2.0
Repro Steps

Install and set up WSL2
Download and install VcXsrv from https://sourceforge.net/projects/vcxsrv/
Install Powershell as per instructions on https://docs.microsoft.com/en-us/powershell/scripting/install/installing-powershell-on-windows?view=powershell-7.2
Install terminator in WSL2 by running the following in Powershell

wsl
sudo apt install terminator

Start VcXsrv using the following command in Powershell

Start-Process -FilePath 'C:\Program Files\VcXsrv\vcxsrv.exe' -ArgumentList (':0'),('-ac'),('-terminate'),('-lesspointer'),('-multiwindow'),('-clipboard'),('-wgl'),('-dpi'),('auto') -NoNewWindow

Start terminator using the following command in Powershell (I have a shortcut for this)

Start-Process -FilePath (Get-Command bash.exe).Source -ArgumentList ('-c'),('-l'),('"DISPLAY=$(awk ''/nameserver / {print $2; exit}'' /etc/resolv.conf 2>/dev/null):0 LIBGL_ALWAYS_INDIRECT=1 exec $(wslpath ''\\wsl$\Ubuntu-22.04\usr\bin\terminator'')"') -NoNewWindow

Search for "Ethernet Settings" in windows search bar and click on "Change adapter options" on left
Disable network device that is connected to internet [either ethernet or wifi device, mine is named "Ethernet"]
Enable same network device
Watch "Hyper-V Virtual Ethernet Adapter" for WSL [mine is named "vEthernet (WSL)"] disable and enable itself which causes window containing terminator to automatically close [same behavior should be observed for any other X11 application]

Expected Behavior
The virtual ethernet adapter for WSL should not reset itself?
I don't know if the virtual ethernet adapter for WSL is supposed to reset itself but I do think it is an issue as network devices usually power down after some time when the machine is put in sleep mode.
This behavior is not seen if you physically unplug the ethernet cable and plug it back again [or disconnect and reconnect to wifi], only if the network device is reset [via disabling/enabling or power off/on].
Right now my workaround is to set the advanced properties of the network devices using Powershell via
Set-NetAdapterAdvancedProperty -Name Ethernet -DisplayName 'Energy Efficient Ethernet' -DisplayValue Off
Set-NetAdapterAdvancedProperty -Name Wi-Fi -DisplayName 'MIMO Power Saving Mode' -DisplayValue 'No SMPS'

so that they don't power down when the machine is in sleep mode but I am also not 100% sure that it works all the time
Actual Behavior
Watch "vEthernet (WSL)" closely and you should see it disable and enable itself when my ethernet device is successfully enabled again
https://user-images.githubusercontent.com/98415730/162987744-8edc6ed1-10ff-4532-976a-c750179edc83.mp4
Diagnostic Logs
No response

Thanks for reporting this @tan-wei-xin-alez.
Looks like you're using the host's IP address for $DISPLAY, which explains why the connection gets dropped when the adapter gets reset.
I'd say that the WSL network adapter being reset when the host's network adapter is reset is by design. Given that vcxsrv  isn't officially supported, I'd recommend looking at switching to wslg, or, giving that this is a custom solution, keep using the workaround that you found.

Thanks for the confirmation @OneBlue
Unfortunately, wslg requires Windows 11 and I'm stuck on Windows 10 due to other software requirements (p_-)
Are there any other alternatives to wlsg for Windows 10 that would not be affected by this issue?
I've seen some alternatives on https://derkoe.dev/blog/development-environment-in-wsl2/ but I do not know whether x2go would encounter the same issues.
Can anyone with experience in the matter chime in?


msvc: Incorrect CMake configuration for static lib when used to build Python lib

The static library project (cbsdk_static.vcxproj) is configured to use the Multi-threaded Dll runtime (/MD switch) however it seems the Python library wants the static version instead (/MT switch), relevant part of output of pip install .:
cbsdk_static.lib(cbsdk.obj) : error LNK2038: mismatch detected for 'RuntimeLibrary': value 'MD_DynamicRelease' doesn't match value 'MT_StaticRelease' in cbpy.obj

This is with VS2017, Qt5.11.2, Python 3.5 with pip 10.0.1 in a Miniconda environment. I don't have enough CMake nor pip experience to know how to fix this properly, but one of them needs to be changed so both runtimes are the same. Workaround for me was changing it in the vcxproj file and not run CMake again before installing with pip.
Sidenote: maybe the Python build instructions should say explicitly that right after running pip install one must leave the CereLink directory before trying import cerebus.cbpy in Python, otherwise it imports from the cerebus directory of this repo and error messages are confusing leading to frustration :P

Hm... I'm amazed I've never run into this problem. I'll take a look as soon as I can.

Sidenote: maybe the Python build instructions should say explicitly that right after running pip install one must leave the CereLink directory before trying import cerebus.cbpy in Python, otherwise it imports from the cerebus directory of this repo and error messages are confusing leading to frustration :P

This is true of any Python library and should be a good rule. If you ever want to debug a python library that you are working on and is under version control, then use pip install -e ., but yes, still change out of that directory. If we ever add instruction on how to use the Python package then I will try to be explicit about this.

Hm... I'm amazed I've never run into this problem.

I assume something is different on my setup. Forgot to mention cmake version: 3.12.1

As I tried to reproduce this on a fairly fresh Windows install, I ran into a different error because I had only Visual Studio 2017 installed and my particular configuration uses Python 3.5.4 which requires Visual Studio 2015. That made me realize that I've never actually tried this on VS 2017, I think I've only ever used 2015 for this project.
I'll try on a new conda env with Python 3.7 to see if I can get by with VS 2017. In case I don't have time to complete this fix today, and I don't answer back here in the next couple hours, you might want to stick to VS 2015 for now if you can.

I have things working with MSVC 2017 and I still can't reproduce your error.
I used the Windows command found in the "try me first" part of the BUILD instructions here to build with MSVC2017.
I've updated the Wiki a bit here but you can ignore the bits about Visual Studio 2015 because I used 2017 for the whole process and it still worked. I have VS 2015 installing in the background so there's a chance that it put some important files in place but I don't think that's the case.
pip install . failed to copy the Qt dll files (Qt5Core.dll and Qt5Xml.dll) in so I copied them manually from /dist to <path_to_python_env>/Lib/site-packages/cerebus.
After that everything worked.
I just finished getting CI working with another python library I work on (https://www.github.com/labstreaminglayer/liblsl-Python). Now that I'm familiar with how that works, I'll probably do the same for this project, and part of the output of that process will be building wheels that include all the dependencies that can then be installed with pip install cerebus.
Other details:

cmake version 3.12.18081601-MSVC_2
Be sure to delete your CereLink/build and CereLink/dist folders before any re-attempts. Qt-linked libraries have a nasty habit of keeping old configs and old objects around and using visual studio to clean the build dir is insufficient.
MSVC2015 and 2017 are binary compatible so the Qt binaries for either should work, as long as they match the platform architecture (i.e. x64).

Thanks for checking on this. I dived into it and finally found the problem. First I noticed for some reason my pip version was 10.something instead of a later one. After upgrading, which also upgraded setuptools and distutils etc I could build cbpy: it turns out distutils is the one responsible for adding /MT or '/MD' to the compile command used by pip. And the newer version of distutils used /MD on my machine so all is fine.
However, on another machine with a very very similar setup but a slight difference in VS2017 installation (and otherwise exactly same cmake, Qt, Python, distutils, ...) it does not work. In my opinion the logic in distutils is simply broken, for some reason it only wants to build with /MD if it finds the VC runtime redistributable dll in one specific directory (Microsoft.VC141.CRT) under the VS installation directory. That's bad, because those redistributable dlls aren't the ones used at runtime anyway and hence are far from related to whether using /MD is possible. It's also bad because this one machine doesn't have that particular directory since it hass Microsoft.VC150.CRT instead which is perfectly fine as far as VS goes. And it's also bad because that behaviour seems hardcoded and not really configurable so to get thinngs working you need to hack around: I ended up manually modifying _msvccompiler.py in the distutils module..

pip install . failed to copy the Qt dll files (Qt5Core.dll and Qt5Xml.dll)

Same here, but at runtime it picks up the dll from the miniconda/bin directory so I didn't even make a copy. Also my pyd file only depend on Qt5Core.dll but not Qt5Xml.dll.

And it's also bad because that behaviour seems hardcoded and not really configurable so to get thinngs working you need to hack around: I ended up manually modifying _msvccompiler.py in the distutils module.

It means filing a bug with distutils is the best course of action, to have a fix at one point. I think distutils has not yet picked up the subtleties of VS2017 vc runtime


Multiple script load bug fixed

Implemented completer based loadScript function so that multiple insertions by loadScript can be prevented.

Thanks.


Add Acknowledgements and Third-party software license.

Thank you for giving acknowledgements and software license, @yhirokawa san. I merge this PR.


Migrate SAML OneLogin configuration article

What changed, and why it matters
Depends on https://github.com/aiven/devportal/pull/1204 before merging. Added the OneLogin article and elaborated on text to substitute screenshots. Perhaps 2 screenshots of the aiven console and onelogin app would be beneficial but I am not sure.
Please check the formatting of values and field names in particular, I tried to stay consistent with existing entries but I am still not sure if it is right!
Addresses https://github.com/aiven/devportal/issues/825

Hi @encima! Good article, I changed the structure slightly to be more in line with devportal, could you please check? Also there's a missing image, do you still think it's needed?
The rest is ok from my side, once you validate from your side I'll ask a second pair of eyes on the wording and then merge

You are right, the image is useful but not necessary (especially when the design has changed quite a bit). Removed and added an explanation. The navigation reshuffle makes sense also, thank you!


Allow ContextMenuTrigger id prop to be of type number

I was surprised to see that ContextMenuTrigger wants a string for its unique id prop. I think it should also, if not only, allow numbers.

The TypeScript definitions was provided by a contributor. I do not have any idea why this decision was made.
You can open a pull-request with the required change?

It's not the typescript but the propTypes:
https://github.com/vkbansal/react-contextmenu/blob/master/src/ContextMenuTrigger.js#L11
They were added by you .
I opened the issue for discussion, before writing a change, since this is a design decision.

Oh. Sorry for the confusion. Yeah number can also be used for id.
You are welcome to open a PR.
On Tue 20 Nov, 2018, 8:34 PM Sam Bokai<EMAIL_ADDRESS>wrote:

It's not the typescript but the propTypes:
https://github.com/vkbansal/react-contextmenu/blob/master/src/ContextMenuTrigger.js#L11
They were added by you
https://github.com/vkbansal/react-contextmenu/commit/12752e2d6be868c67a83923f97416eef64675074#diff-8c52c6cf66a1c629f586678b831921a2
.
I opened the issue for discussion, before writing a change, since this is
a design decision.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/vkbansal/react-contextmenu/issues/246#issuecomment-440304591,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFp9bxxmKVBr9hUHe888XqHeHsEaIEHHks5uxBnigaJpZM4YmV4w
.

I'm busy right now, but will keep it in my mind for when i have a few hours. Let's keep the issue open, in case someone else wants to do this.
You could label this issue as beginner friendly to attract devs.

What's the motivation for allowing numeric IDs? DOM element IDs are always strings, so this seems like a weird request.
https://developer.mozilla.org/en-US/docs/Web/API/Element/id

I'm more used to numeric ids, but you're raising a good argument.


Rcint 4413

fix clear button in ie11

Coverage remained the same at 0.785% when pulling 5f15748ea55c5e8527082da042d10b578a3ec5bf on alvita:rcint-4413 into<PHONE_NUMBER>8d47172fe4ca7724b79a18e5c21c3a on ringcentral:dynamics-regression.


Blog Post: Access z/OS Performance and Management Data using CA SYSVIEW and Zowe CLI by Michael Heuzey.

@mpanch59 and @Joe-Winchester please review for content.

Added the {Ecosystem} tag as this article talks about the Zowe Conformant CA Sysview extensions.  Nice to see the Zowe Conformance badge at the end !  Good article and I learnt a lot reading it.


Histogram hover behavior on far right causes error

I believe this is due to getting an bin that is === bins.length.

I cannot seem to reproduce this.

Managed to reproduce this. It turns out to be a problem with xValues not lining up with the number of bins due to a error using d3.range.


merge package from arlac77/npm-package-template

package.json

chore(package<EMAIL_ADDRESS>

Codecov Report

Merging #301 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #301   +/-   ##
=====================================
  Coverage      75%    75%
=====================================
  Files           1      1
  Lines           8      8
=====================================
  Hits            6      6
  Misses          2      2

Continue to review full report at Codecov.

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

Coverage remained the same at 70.0% when pulling 60239e9eaab78dbe2b63f13d4b5863b50742f2a3 on template-sync-1 into b66e65e9fa7bb59041dff2ab89d303ae504cfdb1 on master.


How can I fix X-Frame-Options issue?

Hi,
Here is code. Please help me. Thanks.
import React from "react";
import ReactDOM from "react-dom";
import zoid from "zoid";

let ZoidComponent = zoid.create({
  tag: "test-component",
  // url: "https://delivery.wholefoodsmarket.com"
  url: "https://www.walmart.com/account/login"
});

let MyLoginReactZoidComponent = ZoidComponent.driver('react', {React, ReactDOM});
class Main extends React.Component {
  constructor(props) {
      super(props);
  }
  render() {
      return (
          <div>
              <MyLoginReactZoidComponent />
          </div>
      );
  }
}
ReactDOM.render(<Main />, document.getElementById('react-root'));

You need to set the x-frame-options header. You can only do this on sites you own; so it's unlikely you can make this work with walmart.com

Hi @bluepnume , Thanks for your quick response.
Please see it. I found it in description.
"Is this different to react-frame-component?
Yes. react-frame-component allows you to render html into a sandboxed iframe on the same domain. zoid is geared around sharing functionality from one domain to another, in a cross-domain iframe."
That's why I have decided to use Zoid.

Sure. But zoid can't get around browser restrictions. At the end of the day, an iframe needs that x-frame-options header otherwise it will be blocked by the browser.

Ah, so It's relation with Browser?

Yep. For security reasons, no way to get around this.


Dereference Null Return Value

Describe the bug
Dereferencing BucketPtr, which is known to be NULL.
Expected behavior
Check if BucketPtr is not NULL before dereferencing.
Code snips
https://github.com/nasa/cFE/blob/98f78e8604c19415fd1e199eae94196a781539b8/modules/es/fsw/src/cfe_es_generic_pool.c#L228
System observed on:
Coverity: https://scan.coverity.com/projects/arielsadamsnasa-cfs-jsf-rules?tab=overview
Reporter Info
Full name and company/organization if applicable

Isn't this checked here: https://github.com/nasa/cFE/blob/98f78e8604c19415fd1e199eae94196a781539b8/modules/es/fsw/src/cfe_es_generic_pool.c#L181?

I'm missing something here - where are the 3 retrievals of the BucketPtr?  The actual cited/linked line looks perfectly fine to me, I only see a single pointer get in this function.  In fact it looks perfectly fine to me - it is retrieved, immediately checked for NULL-ness, and the function only proceeds if it's non-null.  It does not retrieve BucketPtr again, at least not in this function.
However, inside the "Rebuild" function, I do see something that looks redundant here:
https://github.com/nasa/cFE/blob/98f78e8604c19415fd1e199eae94196a781539b8/modules/es/fsw/src/cfe_es_generic_pool.c#L587
The "BucketPtr" for that BucketId was already gotten and checked for NULL here, so yeah - this would just be getting the same pointer again.  I do concur with removing this line.
Other than that single extra get, I don't see any missing NULL checks in this file.


only output help for command errors

It's getting pretty bogged down in the help command when it outputs for every error. Having just CommandError logs when uses command incorrectly, but not when command fails. That way the failure output can direct them more clearly

Hey @LoganWright, you just merged a pull request, have a coin!
You now have 507 coins.


White screen on startup

Environment
Windows 10 (version 1903)
Windows build number: 18362.356
PowerToys version: 0.11.0
PowerToy module for which you are reporting the bug (if applicable):

After installing and starting the Powertoys, all I get to see is a white screen, where I can do basically nothing....
Screenshots

Hi, thanks for the report.
We are already tracking it in https://github.com/microsoft/PowerToys/issues/243


TypeError: require(...).options is not a function

Short Description:

Tried latest api version 0.8. Got error: TypeError: require(...).options is not a function

Platform:

macos

node version:

v9.11.1

Long descrption

Previous version of api is works fine. However when I tried latest version 0.8 got TypeError.
Updated npm, still same error. When I looked into node_modules folder there are many modules, previous version of api has only 2 modules. I am guessing newer version works this way.
How can I fix this error, please help.

code
const binance = require('node-binance-api').options({
APIKEY: '',
APISECRET: '',
useServerTime: true, // If you get timestamp errors, synchronize to server time at startup
test: true // If you want to use sandbox mode where orders are simulated
});
binance.prices((error, ticker) => {
console.log("prices()", ticker);
console.log("Price of BTC: ", ticker.BTCUSDT);
});
result
(function (exports, require, module, __filename, __dirname) { const binance = require('node-binance-api').options({
^
TypeError: require(...).options is not a function

Hmm strange, could you try it this way as well please?
const Binance = require('node-binance-api');
const instance1 = new Binance().options({
APIKEY: '',
APISECRET: '',
useServerTime: true
});

@jaggedsoft  It worked. thank you!!!


chore(readme): add branch information

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Improve user interface of STT section on Live mode page

Describe the bug
On the Lie mode page, when a lot of text is received from STT, the user has to scroll down. It would be great to have an auto scroll or have the latest text be displayed at the top of the box.
To Reproduce
Steps to reproduce the behavior:

Go to 'NSMQ AI Live Quiz'
Add in a live link
Notice you have to scroll a lot

Expected behavior
Auto scroll or display most current text
Screenshots

some interesting ideas here https://discuss.streamlit.io/t/how-do-i-move-the-default-scroll-of-st-text-area-to-the-bottom/39320/4


SCICD-580: Bootprep Dirs

Create a subdirectory within the media directory, so that when copying files, the iuf installer won't clobber any existing bootprep files.
Summary and Scope
Summarize what has changed. Explain why this PR is necessary. What is impacted? Is this a new feature, critical bug fix, etc?

Fixes an issue found with the https://jira-pro.its.hpecorp.net:8443/browse/SCICD-580 fix.

Tested on:

frigg

if would be good if we updated the guidance for the bootprep related files to give people a better idea of the allowable paths


Packaging all the dependencies

As this tool heavily relies on the exact version of the dependencies, it would be perfect if all the runtime dependencies were also delivered along with the release. For example, suitable versions of .NET core SDK, ASP .NET Core hosting tools, nuget.exe etc.
This process helps everyone to avoid issues like #243

Thanks for the suggestion! This is a great idea, I will considering making BaGet's release a self-contained application: https://docs.microsoft.com/en-us/dotnet/core/deploying/#self-contained-deployments-scd


Relative colors-wal.css symlink

It's weird (using six ../-es), but this should work on any system, regardless of username - as long as users are using the default installation locations for wal and Übersicht.

Thank you! Works great for me! I switched back to mono.css just because I was having some issues with default. Will need to get back to that in a bit.


Unable to export TRT model file : ERROR:root:Failed to save engine: [Errno 2] No such file or directory:

I have been trying to generate a trt file for this checkpoint merge. It seems to export the .onnx file and a build trt engine but apparently not a .trt save file and therefore it fails to save the engine as the file is never created in the first place...??? This has worked for a similiar sort of checkpoint merge of similiar size.
Is this error due to this tensor to python boolean conversion?
Any hints or pointers would be appreciated.
Exporting 0.5(abyssorangemix2_Hard) + 0.5(ambientgrapemixAnime2D_v20) to TensorRT
{'sample': [(1, 4, 64, 64), (2, 4, 64, 64), (8, 4, 96, 96)], 'timesteps': [(1,), (2,), (8,)], 'encoder_hidden_states': [(1, 77, 768), (2, 77, 768), (8, 154, 768)]}
No ONNX file found. Exporting ONNX...
Disabling attention optimization
C:\Users\Omen\Desktop\Engineering\Programming\AI\stable-diffusion-webui\venv\Lib\site-packages\einops\einops.py:314: TracerWarning: Converting a tensor to a Python boolean might cause the trace to be incorrect. We can't record the data flow of Python values, so this value will be treated as a constant in the future. This means that the trace might not generalize to other inputs!
known = {axis for axis in composite_axis if axis_name2known_length[axis] != _unknown_axis_length}
C:\Users\Omen\Desktop\Engineering\Programming\AI\stable-diffusion-webui\venv\Lib\site-packages\einops\einops.py:315: TracerWarning: Converting a tensor to a Python boolean might cause the trace to be incorrect. We can't record the data flow of Python values, so this value will be treated as a constant in the future. This means that the trace might not generalize to other inputs!
unknown = {axis for axis in composite_axis if axis_name2known_length[axis] == _unknown_axis_length}
C:\Users\Omen\Desktop\Engineering\Programming\AI\stable-diffusion-webui\repositories\stable-diffusion-stability-ai\ldm\modules\diffusionmodules\openaimodel.py:158: TracerWarning: Converting a tensor to a Python boolean might cause the trace to be incorrect. We can't record the data flow of Python values, so this value will be treated as a constant in the future. This means that the trace might not generalize to other inputs!
assert x.shape[1] == self.channels
C:\Users\Omen\Desktop\Engineering\Programming\AI\stable-diffusion-webui\modules\sd_hijack_unet.py:26: TracerWarning: Converting a tensor to a Python boolean might cause the trace to be incorrect. We can't record the data flow of Python values, so this value will be treated as a constant in the future. This means that the trace might not generalize to other inputs!
if a.shape[-2:] != b.shape[-2:]:
C:\Users\Omen\Desktop\Engineering\Programming\AI\stable-diffusion-webui\repositories\stable-diffusion-stability-ai\ldm\modules\diffusionmodules\openaimodel.py:109: TracerWarning: Converting a tensor to a Python boolean might cause the trace to be incorrect. We can't record the data flow of Python values, so this value will be treated as a constant in the future. This means that the trace might not generalize to other inputs!
assert x.shape[1] == self.channels
============= Diagnostic Run torch.onnx.export version 2.0.1+cu118 =============
verbose: False, log level: Level.ERROR
======================= 0 NONE 0 NOTE 0 WARNING 0 ERROR ========================
[I] Folding Constants | Pass 1
[I]     Total Nodes | Original:  8992, After Folding:  6216 |  2776 Nodes Folded
[I] Folding Constants | Pass 2
[I]     Total Nodes | Original:  6216, After Folding:  4968 |  1248 Nodes Folded
[I] Folding Constants | Pass 3
[I]     Total Nodes | Original:  4968, After Folding:  4968 |     0 Nodes Folded
Exported to ONNX.
Building TensorRT engine... This can take a while, please check the progress in the terminal.
Building TensorRT engine for C:\Users\Omen\Desktop\Engineering\Programming\AI\stable-diffusion-webui\models\Unet-onnx\0.5(abyssorangemix2_Hard) + 0.5(ambientgrapemixAnime2D_v20)_06ace631.onnx: C:\Users\Omen\Desktop\Engineering\Programming\AI\stable-diffusion-webui\models\Unet-trt\0.5(abyssorangemix2_Hard) + 0.5(ambientgrapemixAnime2D_v20)_06ace631_cc86_sample=1x4x64x64+2x4x64x64+8x4x96x96-timesteps=1+2+8-encoder_hidden_states=1x77x768+2x77x768+8x154x768.trt
[libprotobuf WARNING **************************************************************************\externals\protobuf\3.0.0\src\google\protobuf\io\coded_stream.cc:604] Reading dangerously large protocol message.  If the message turns out to be larger than<PHONE_NUMBER> bytes, parsing will be halted for security reasons.  To increase the limit (or to disable these warnings), see CodedInputStream::SetTotalBytesLimit() in google/protobuf/io/coded_stream.h.
[libprotobuf WARNING **************************************************************************\externals\protobuf\3.0.0\src\google\protobuf\io\coded_stream.cc:81] The total number of bytes read was<PHONE_NUMBER>
[libprotobuf WARNING **************************************************************************\externals\protobuf\3.0.0\src\google\protobuf\io\coded_stream.cc:604] Reading dangerously large protocol message.  If the message turns out to be larger than<PHONE_NUMBER> bytes, parsing will be halted for security reasons.  To increase the limit (or to disable these warnings), see CodedInputStream::SetTotalBytesLimit() in google/protobuf/io/coded_stream.h.
[libprotobuf WARNING **************************************************************************\externals\protobuf\3.0.0\src\google\protobuf\io\coded_stream.cc:81] The total number of bytes read was<PHONE_NUMBER>
[I] Loading tactic timing cache from C:\Users\Omen\Desktop\Engineering\Programming\AI\stable-diffusion-webui\extensions\Stable-Diffusion-WebUI-TensorRT\timing_caches\timing_cache_win_cc86.cache
[I] Building engine with configuration:
Flags                  | [FP16, REFIT, TF32]
Engine Capability      | EngineCapability.DEFAULT
Memory Pools           | [WORKSPACE: 8191.50 MiB, TACTIC_DRAM: 8191.50 MiB]
Tactic Sources         | [CUBLAS, CUDNN, EDGE_MASK_CONVOLUTIONS, JIT_CONVOLUTIONS]
Profiling Verbosity    | ProfilingVerbosity.LAYER_NAMES_ONLY
Preview Features       | [FASTER_DYNAMIC_SHAPES_0805, DISABLE_EXTERNAL_TACTIC_SOURCES_FOR_CORE_0805]
Building engine: 100%|███████████████████████████████████████████████████████████████████| 6/6 [01:52<00:00, 18.67s/it]
[I] Finished engine building in 113.046 seconds
[I] Saving tactic timing cache to C:\Users\Omen\Desktop\Engineering\Programming\AI\stable-diffusion-webui\extensions\Stable-Diffusion-WebUI-TensorRT\timing_caches\timing_cache_win_cc86.cache
[I] Saving engine to C:\Users\Omen\Desktop\Engineering\Programming\AI\stable-diffusion-webui\models\Unet-trt\0.5(abyssorangemix2_Hard) + 0.5(ambientgrapemixAnime2D_v20)_06ace631_cc86_sample=1x4x64x64+2x4x64x64+8x4x96x96-timesteps=1+2+8-encoder_hidden_states=1x77x768+2x77x768+8x154x768.trt
ERROR:root:Failed to save engine: [Errno 2] No such file or directory: 'C:\Users\Omen\Desktop\Engineering\Programming\AI\stable-diffusion-webui\models\Unet-trt\0.5(abyssorangemix2_Hard) + 0.5(ambientgrapemixAnime2D_v20)_06ace631_cc86_sample=1x4x64x64+2x4x64x64+8x4x96x96-timesteps=1+2+8-encoder_hidden_states=1x77x768+2x77x768+8x154x768.trt'

Windows has a limitation on the path length of 256 characters. And C:\Users\Omen\Desktop\Engineering\Programming\AI\stable-diffusion-webui\models\Unet-trt\0.5(abyssorangemix2_Hard) + 0.5(ambientgrapemixAnime2D_v20)_06ace631_cc86_sample=1x4x64x64+2x4x64x64+8x4x96x96-timesteps=1+2+8-encoder_hidden_states=1x77x768+2x77x768+8x154x768.trt
is 268 long. Either you need to move the project to a different path or enable long paths in Windows: https://learn.microsoft.com/en-us/windows/win32/fileio/maximum-file-path-limitation?tabs=registry#enable-long-paths-in-windows-10-version-1607-and-later


indexer: SQL query fix

Fixes an SQL query that was ambiguos according to MariaDB 5.5.

Note this is the master version of PR #2753
Signed-off-by: Samuele Kaplun<EMAIL_ADDRESS>

Reviewed-by: Tibor Simko<EMAIL_ADDRESS>Thanks, amended commit log message and merged as d446c9099e72cba3074feb3123914216e322ee89.


chef error during parallelcluster deployment

Hello,
I am trying to deploy parallelcluster but the cloudformation stack is failing.  Was hoping someone can help me figure out the issue with chef:
$ pcluster version
{
"version": "3.0.0"
}
Created a new custom AMI by running:
pcluster build-image --image-id ParallelCluster-3-0-ubuntu2004 --image-configuration image-config-pcluster.yaml --region us-west-2
$ pcluster list-images --region us-west-2 --image-status AVAILABLE
{
"images": [
{
"imageId": "ParallelCluster-3-0-ubuntu2004",
"imageBuildStatus": "BUILD_COMPLETE",
"ec2AmiInfo": {
"amiId": "ami-xxx"
},
"region": "us-west-2",
"version": "3.0.0"
}
]
}
Created a config for pcluster:
$ ls -l cluster-config.yaml
-rw-------  1 simran  staff  5589 Oct 19 16:48 cluster-config.yaml
Deploying the cluster using:
$ pcluster create-cluster --cluster-name simtest-cluster --cluster-configuration cluster-config.yaml
{
"cluster": {
"clusterName": "simtest-cluster",
"cloudformationStackStatus": "CREATE_IN_PROGRESS",
"cloudformationStackArn": "arn:aws:cloudformation:us-west-2:xxx:stack/simtest-cluster/29f076f0-3137-11ec-b247-02ca360f9713",
"region": "us-west-2",
"version": "3.0.0",
"clusterStatus": "CREATE_IN_PROGRESS"
},
"validationMessages": [
{
"level": "WARNING",
"type": "UrlValidator",
"message": "The url 'https://10.x.x.x/pcluster-setup.sh' causes URLError, the error reason is '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self signed certificate (_ssl.c:1125)'."
}
]
}
Cloudformation fails with this message:
WaitCondition received failed message: 'Failed to run cfn-init. If --norollback was specified, check /var/log/cfn-init.log and /var/log/cloud-init-output.log.' for uniqueId: i-02ffc3d49d1a82e8f
I logged into the head node during deployment and see the following in /var/log/cfn-init.log:
2021-10-19 23:56:18,716 [DEBUG] No test for command chef
2021-10-19 23:57:09,041 [ERROR] Command chef (chef-client --local-mode --config /etc/chef/client.rb --log_level info --logfile /var/log/chef-client.log --force-formatter --no-color --chef-zero-port 8889 --json-attributes /etc/chef/dna.json) failed
2021-10-19 23:57:09,041 [DEBUG] Command chef output: Redirecting to cinc-client
2021-10-19 23:57:09,041 [ERROR] Error encountered during build of chefConfig: Command chef failed
Traceback (most recent call last):
File "/usr/local/lib/python3.8/dist-packages/cfnbootstrap/construction.py", line 573, in run_config
CloudFormationCarpenter(config, self._auth_config).build(worklog)
File "/usr/local/lib/python3.8/dist-packages/cfnbootstrap/construction.py", line 272, in build
changes['commands'] = CommandTool().apply(
File "/usr/local/lib/python3.8/dist-packages/cfnbootstrap/command_tool.py", line 127, in apply
raise ToolError(u"Command %s failed" % name)
cfnbootstrap.construction_errors.ToolError: Command chef failed
2021-10-19 23:57:09,044 [ERROR] -----------------------BUILD FAILED!------------------------
2021-10-19 23:57:09,044 [ERROR] Unhandled exception during build: Command chef failed
Traceback (most recent call last):
File "/usr/local/bin/cfn-init", line 176, in
worklog.build(metadata, configSets)
File "/usr/local/lib/python3.8/dist-packages/cfnbootstrap/construction.py", line 135, in build
Contractor(metadata).build(configSets, self)
File "/usr/local/lib/python3.8/dist-packages/cfnbootstrap/construction.py", line 561, in build
self.run_config(config, worklog)
File "/usr/local/lib/python3.8/dist-packages/cfnbootstrap/construction.py", line 573, in run_config
CloudFormationCarpenter(config, self._auth_config).build(worklog)
File "/usr/local/lib/python3.8/dist-packages/cfnbootstrap/construction.py", line 272, in build
changes['commands'] = CommandTool().apply(
File "/usr/local/lib/python3.8/dist-packages/cfnbootstrap/command_tool.py", line 127, in apply
raise ToolError(u"Command %s failed" % name)
cfnbootstrap.construction_errors.ToolError: Command chef failed

This seems to have been related to the IAM role missing access.  I was able to resolve this issue by using the pcluster provided instance role/policies.


Swagger ui 3.26.2 oauth scopes

Resolves #1725 with an upgrade to swagger-ui 3.26.2.

Upgraded to swagger-ui via 5652e81 and OAuthScopes config option added via c6bf9a2

Upgraded to swagger-ui via 5652e81 and OAuthScopes config option added via c6bf9a2


Updated MCH version 2.10 => 2.11

Description
Updated the version in MCH to latest version ACM 2.11.
Related Issue
If applicable, please reference the issue(s) that this pull request addresses.
Changes Made
Updating MCH repo to reflect upcoming release for ACM 2.11.
Screenshots (if applicable)
Add screenshots or GIFs that demonstrate the changes visually, if relevant.
Checklist

[ ] I have tested the changes locally and they are functioning as expected.
[ ] I have updated the documentation (if necessary) to reflect the changes.
[ ] I have added/updated relevant unit tests (if applicable).
[ ] I have ensured that my code follows the project's coding standards.
[ ] I have checked for any potential security issues and addressed them.
[ ] I have added necessary comments to the code, especially in complex or unclear sections.
[ ] I have rebased my branch on top of the latest main/master branch.

Additional Notes
Add any additional notes, context, or information that might be helpful for reviewers.
Reviewers
Tag the appropriate reviewers who should review this pull request. To add reviewers, please add the following line: /cc @reviewer1 @reviewer2
/cc @cameronmwall @joeg-pro
Definition of Done

[ ] Code is reviewed.
[ ] Code is tested.
[ ] Documentation is updated.
[ ] All checks and tests pass.
[ ] Approved by at least one reviewer.
[ ] Merged into the main/master branch.

/lgtm


2.0.0-rc.8 getSession() return null after login

Bug report
auth.getSession() returns null
Describe the bug
I'm redirecting to '/' when I login but getSession() returns null which I'm not sure why is happening.
To Reproduce
This is my login function
const onSubmit: SubmitHandler<LoginValues> = async (loginValues) => {     const { data, error } = await supabase.auth.signUp(loginValues);     if (data.user) {       router.push("/");     } else {       alert(error ? error : "An unknown error ocurred");     }   }
and on the server  const { data, error } = await supabase.auth.getSession()
Expected behavior
session returns null
System information

OS: macOS
Browser  chrome
Version of supabase-js: 2.0.0-rc.8
Version of Node.js: 18.0.0

Hey @SatouKuzuma1,

and on the server const { data, error } = await supabase.auth.getSession()

To clarify, are you calling signUp in the browser and then calling getSession on the server side?
gotrue-js uses localStorage by default, meaning the session will not be synced to the server. You might like to check out the auth-helpers, which will help keep everything in sync for you. (Although I think you'll have to use the rc version there too at the moment!)

@alaister yes, just like you said ‘SignIn’ on the browser and ‘getSession’ on the server.

No worries. By the rc version, I mean the next version of the auth-helpers. For example npm install @supabase/auth-helpers-nextjs@next.
I'm going to close this as mentioned in my previous comment this isn't a (first party) supported use case of gotrue-js.
If I'm misunderstanding the issue you're having, please feel free to reopen this, and we can work it out.
Thanks!


move cloning of AbaqusReader.jl from .travis to build/deps.jl

Coverage remained the same at 90.635% when pulling b67eaa32c8ff2ec304ab4564cb1f8b0d5f3eab91 on deps/build.jl into 80b0db0ec888250dd642b5c88251689e3d196e9d on master.


getActivity() not working on 1.1.5

When calling the getActivity() on the Drawer object, it produces an error:
drawer.addEventListener('open', onNavDrawerWinOpen);
function onNavDrawerWinOpen(evt) {
this.removeEventListener('open', onNavDrawerWinOpen);
var actionBar = this.getActivity().getActionBar();
}
[ERROR] :  TiApplication: (main) [459,2124] Sending event: exception on thread: main msg:java.lang.RuntimeException: Unable to start activity ComponentInfo{com.webstreaming.drawer/org.appcelerator.titanium.TiActivity}: java.lang.NullPointerException; Titanium 4.0.0,2015/05/18 15:42,9239ff9
If I compile with Ti SDK 3.5.1GA it works perfectly, but with 4.0 its now working...
Any ideas?
Thanks

It's my same problem posted 28 days ago but seems not fixed ago


V9: Fix Markdown mobile editor

This fixes up the API Controller used in the mini AngularJS app for mobile posting with the url /a-new
Please thoroughly test the upload logic, and it works as expected still

Updated with your suggestions and I have done testing, but was asking for you to test it as well, but if you trust me then all good.
Let me know if there any other changes with this PR :)


Add a translation link for Polish

Added link for Polish lang. version

@hemanth changes here: https://github.com/git-tips/tips/pull/185/files


AlignTemplatedTestCases documentation error

AlignTemplatedTestCases:45 has a spelling error in only_with_haeders:True
At https://robotidy.readthedocs.io/en/stable/transformers/AlignTemplatedTestCases.html the documentation for enabling the transformer AlignTestCases instead of AlignTemplatedTestCases. The same spelling mistake for only_with_haeders is also in this documentation.
I also had difficulty trying to figure out what the second example was demonstrating. Was it that the tags would also be aligned or something else?

Thanks for the report, I will fix those issues. And I should give code example to the second example. In this case you can opt for not aligning the test case section if it does not contain headers like those:
*** Test Cases ***    argument1 argument1
Test case name        value            value2

With only headers set to True following code would not be aligned:
*** Test Cases ***
Test case name
    Step 1
    Step 2

Ah, and if you mean the second example under first subsection then it means that for templated test cases the settings will be aligned in one column like shown.


Fixed the image on the website

Description
This fixes the image which is not showing up on the website.
This PR fixes #1972
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

@vishalvivekm this logic should not be needed.


COMMUNITY [GLOBALIZATION, DIVERSITY]

I opened an issue for Node in Africa (NodeJS Africa) which was later approved and I have started pioneering this opportunity to make it a real success, likewise I think the foundation should look into diversity, encouraging developers or individual members around the globe to start up regional NodeJS community Like NodeJS Africa, having something like Node in Asia (NodeJS Asia), Node in Europe (NodeJS Europe) and others.
This will really assist the community to grow and get bigger. Giving individual responsibility to individuals is one of the fastest ways to grow and expand but sticking with the GitHub community that most people barely know about or heard of will just make the growth rate of the NodeJS Foundation community slow.
This is what I will love to speak on at the Collaboration Summit if I become a member of the foundation before time. I really believe bringing in diversity and sharing individual responsibility across the globe will speed up the Nodejs growth.
It may seem impossible at first and probably having the appropriate funding to make that possible but it’s more about giving the right support and getting the right backing from the foundation and less of funding. I believe they are individuals out there or even right here that will gladly take up the responsibility.

Hey!
Thanks for the issue. I’m glad you take on this subject, as it’s one of the aspects I’d like to push through with the issue I opened the other day ( #260 ).
I’d like to speak about this in the next CommComm meeting, and then if we can, definitely launch this initiative with volunteers.
Cheers!

@Tiriel Awesome! it will be a pleasure working together on this subject

Closing this issue. Feel free to re-open. 🙏 #308


Sparse diagonalization methods for getting the relevant spectrum of the projected Hamiltonian?

I’ve understood that a dense/exact diagonalization method (e.g., LinearAlgebra.jl's eigen()) is currently used for diagonalizing projected Hamiltonians during the sweeps (please correct me if I’m wrong). If this is the case, I'm curious if sparse diagonalization methods are already available (or easy to be implemented) as an alternative method for this step. Is this something that has been discussed?
At glance, it seems that DMRG implementation has an argument like solver_krylovdim, suggesting it might work with sparse diagonalization methods. I wonder if the same is true for DMRG-X as well, and if not whether we can implement this as an option.

That's correct, it just uses a dense diagonalization right now. I would say that my goal for dmrg_x was to present the simplest demonstration of a standard method, and not provide an extensive suite of DMRG-X-like algorithms (of which there are a surprising number of variants).
So my recommendation would be to look at how dmrg_x is implemented, which is very simple: https://github.com/ITensor/ITensorTDVP.jl/blob/46695737f57317d49e46a2e8c800cfb7228df0fc/src/dmrg_x.jl, and try out other variants yourself. Feel free to share an example/PR to discuss as an alternative implementation.
As for using a Krylov solver in particular, for this variant of DMRG-X where you try to find a state with the highest overlap with the previous state, I don't think there is a Krylov method for doing that. However, some kind of Krylov method could be used to target a certain energy, however I believe the energy targeting method is not as good (I think I read it causes the eigenstate to drift over the course of DMRG sweeps).