Did the renaming of Owners to HeadAdmins break anything

I am not sure if I am correct however if I are then backwards compatibility MUST BE ADDED!

No it didn't. I still use a 2017 loader and everything works fine.

Ok going to close this soon if no issues are found

Ok it did not. I will close this.


Adds the OpenSSL license

I believe since we copied the BIO function from OpenSSL, we must include the OpenSSL license as well. I removed the NOTICE thing, since the copyright attribution can be put where the source is.

@jablko Did any of this OpenSSL code go into 7.1.x? If so, we need this change there as well.

Intel CC build successful! See https://ci.trafficserver.apache.org/job/icc-github/18/ for details.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/freebsd-github/1585/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/linux-github/1479/ for details.

clang-analyzer build successful! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/150/ for details.


[Feature request] add colors to the toolbar buttons

For example, when VM is on, this button should ideally be green showing that VM is on, when VM is off, it can be in red colors

the rest of the buttons can have colors too for better visibility and recognition.
Thanks

Could you make a recommendation for an icon set that is free and open source?

Color is possible on the current font icons, I believe. But that's what I recommended in the first option. I think maybe we spoke past one another. Or I did a poor job at conveying my thoughts.
I'll try some color adjustments when I have time and post a screenshot here for your input.

Here is the first pass, I don't really like how this is at the moment, but to give you an idea:


Add unit tests for Translation and implement override of get_message

This PR switches Translation::get_message and Translation::get_plural_message virtual call to GDScript compatible call. Also it introduces unit test for translations.
Closes https://github.com/godotengine/godot-proposals/issues/1926

Are there any news regarding this PR?

Ok, crashes now: https://github.com/godotengine/godot/pull/44670/checks?check_run_id=1952034051#step:9:59.
I suspect this is because TranslationServer is not actually instantiated in the test environment. So, we have two options:

Instantiate and load translations in test_main().
Make a local instance of TranslationServer.

So, the original:
TranslationServer *translation_server = nullptr;
translation_server = memnew(TranslationServer);
translation_server->setup(); //register translations, load them, etc.
translation_server->load_translations();

may be acceptable way to do this, but it would only work fine for the test case itself, but may cause problems down the road if someone would want to test TranslationServer anywhere else.

@Xrayez Should I only initialize TranslationServer and it will be finalized itself. Or I should finalize it after test_context.run() too?

Sorry, I've been working on resolving this issue, see #46313.

I suppose it better to add files into .gitignore because it possible on Windows I cannot delete them while tests are executing.

Rebased. Could it be also backported to 3.2 branch?

I think this can be done, but without tests because 3.2 doesn't have tests implemented as in 4.0 (master).
Also, by rebasing I also meant that commits must be squashed into one before merging, see PR workflow in Godot. 🙂
Akien is responsible for merging and cherry-picking PRs to 3.2.

Squashed it.

Does something else is needed to accept this PR?

It needs a review by core engine maintainers (not sure why the previous review request was discarded). I don't know if the changes done to core/string/translation.cpp are the proper way to support the use case in godotengine/godot-proposals#1926.

Rebased and moved test to recently introduced translation tests.

Thanks!


Couldn't Mark Message as Read [matrix => xmpp]

Describe the bug
Frequently, when sending messages from matrix to xmpp, an error shows that a message couldn't be marked as read with a Content Not JSON error.
To Reproduce
Set up a bridge between Matrix and XMPP with pantalaimon and attempt to send a message from Matrix to XMPP.
Expected behavior
The message sends without error.
Screenshots/debug logs
[0016] ERROR matrix:       couldn't mark message as read contents=[REDACTED] msg=Failed to POST JSON to [REDACTED LINK] code=400 wrapped=M_NOT_JSON: Content not JSON.

Environment (please complete the following information):

Debian 11
version: 1.25.0 26596acf

Additional context
Please add your configuration file  (be sure to exclude or anonymize private data (tokens/passwords))
[xmpp.myxmpp]
Server="slipfox.xyz"
#Jid your userid
<EMAIL_ADDRESS>Password="[REDACTED]"
Muc="xmpp.slipfox.xyz"
Nick="xmppbot"
RemoteNickFormat="[{PROTOCOL}] <{NICK}> "

[matrix.mymatrix]
# Server="https://matrix.org"
# Matterbridge does not support encrypted group chats.
# Therefore, use Pantalaimon to MiTM myself:
Server="http://localhost:20662"
# Dedicated user
# Messages sent from this user will not be relayed to avoid loops.
Login="sovalt"
Password="[REDACTED]"
RemoteNickFormat="[{PROTOCOL}] <{NICK}>: "
#Whether to send the homeserver suffix. eg ":matrix.org" in @username:matrix.org
#to other bridges, or only send "username".(true only sends username)
NoHomeServerSuffix=true
HTMLDisable=true

[[gateway]]
name="foobar"
enable=true
[[gateway.inout]]
account="xmpp.myxmpp"
channel="XMPP_NAME_HERE"

[[gateway.inout]]
account="matrix.mymatrix"
channel="MATRIX_NAME_HERE"

This has been fixed in master for a while.
Soon in 1.25.1


Unable to access account with unverified email

Describe the bug
The account https://www.nuget.org/profiles/wallymathieu is linked to my email, but I'm unable to access it due to the email not being verified, the password being lost.
To Reproduce
Create an account. Have the delivery of the email fail (for instance due to some system filtering emails in between or a failure to send out the email)...
Try to restore the password using your email you get "Unverified email"
Expected behavior
Either:

An account with no activity and unverified email (and no nuget packages) should perhaps be deleted?
An account with an unverified email should be able to use password reset.

Additional context
The above mentioned nuget account is quite old. I got reminded of it https://github.com/fsprojects/FSharp.Formatting/pull/510#issuecomment-613568562 recently.

Hey @wallymathieu, for account management issues, I'd recommend reaching out to<EMAIL_ADDRESS>

I have sent them emails, but since I can't follow the documentation due to the above issue, we haven't gotten anywhere.

The solution is to delete the account so that it can be recreated.


macOS Sierra + Alfred 3 issues

For macOS Sierra, make sure to install 'appscript', as mentioned in another thread. I could not get this workflow to work on my machine without it.
sudo gem install rb-appscript

life saver! this should really add to the Readme!

$ sudo gem install rb-appscript
Fetching: rb-appscript-0.6.1.gem (100%)
Building native extensions. This could take a while...
ERROR:  Error installing rb-appscript:
	ERROR: Failed to build gem native extension.

    current directory: /usr/local/lib/ruby/gems/2.5.0/gems/rb-appscript-0.6.1
/usr/local/opt/ruby/bin/ruby -r ./siteconf20180305-74820-1sopcm8.rb extconf.rb
*** extconf.rb failed ***
Could not create Makefile due to some reason, probably lack of necessary
libraries and/or headers.  Check the mkmf.log file for more details.  You may
need configuration options.

Provided configuration options:
	--with-opt-dir
	--with-opt-include
	--without-opt-include=${opt-dir}/include
	--with-opt-lib
	--without-opt-lib=${opt-dir}/lib
	--with-make-prog
	--without-make-prog
	--srcdir=.
	--curdir
	--ruby=/usr/local/Cellar/ruby/2.5.0_2/bin/$(RUBY_BASE_NAME)
extconf.rb:44:in `<main>': uninitialized constant Config (NameError)
Did you mean?  RbConfig
               CONFIG

extconf failed, exit code 1

Gem files will remain installed in /usr/local/lib/ruby/gems/2.5.0/gems/rb-appscript-0.6.1 for inspection.
Results logged to /usr/local/lib/ruby/gems/2.5.0/extensions/x86_64-darwin-17/2.5.0/rb-appscript-0.6.1/gem_make.out

$ sudo gem install rb-appscript
Fetching: rb-appscript-0.6.1.gem (100%)
Building native extensions. This could take a while...
ERROR:  Error installing rb-appscript:
	ERROR: Failed to build gem native extension.

    current directory: /usr/local/lib/ruby/gems/2.5.0/gems/rb-appscript-0.6.1
/usr/local/opt/ruby/bin/ruby -r ./siteconf20180305-74820-1sopcm8.rb extconf.rb
*** extconf.rb failed ***
Could not create Makefile due to some reason, probably lack of necessary
libraries and/or headers.  Check the mkmf.log file for more details.  You may
need configuration options.

Provided configuration options:
	--with-opt-dir
	--with-opt-include
	--without-opt-include=${opt-dir}/include
	--with-opt-lib
	--without-opt-lib=${opt-dir}/lib
	--with-make-prog
	--without-make-prog
	--srcdir=.
	--curdir
	--ruby=/usr/local/Cellar/ruby/2.5.0_2/bin/$(RUBY_BASE_NAME)
extconf.rb:44:in `<main>': uninitialized constant Config (NameError)
Did you mean?  RbConfig
               CONFIG

extconf failed, exit code 1

Gem files will remain installed in /usr/local/lib/ruby/gems/2.5.0/gems/rb-appscript-0.6.1 for inspection.
Results logged to /usr/local/lib/ruby/gems/2.5.0/extensions/x86_64-darwin-17/2.5.0/rb-appscript-0.6.1/gem_make.out

I encountered the same problem during installation。


Unmaintained underlying package(s)

@databases/mysql is still running v2.x.x of mysql2. The latter is not receiving any security backports which directly affect the sql-mapper package in here. See https://github.com/ForbesLindesay/atdatabases/issues/323 as an example.
Not sure what the best course of action is, but this could prove to be an issue in the future.

Working on it at https://github.com/ForbesLindesay/atdatabases/pull/335

Fixed in https://github.com/platformatic/platformatic/pull/3459.


[Feature] Counter limits

Add an option to set a limit on counters ("Value limit"), and get a popup message when this limit is reached. This could be set during the creation of counters or editing.

We would need to redesign the counter create/edit dialog first (perhaps using a ModalBottomSheet?) as the current AlertDialog-based one would get too cluttered.
Can you give examples of use cases for this feature? Global thresholds are already in place to prevent users from making values too high or too low.

We would need to redesign the counter create/edit dialog first (perhaps using a ModalBottomSheet?) as the current AlertDialog-based one would get too cluttered.
Can you give examples of use cases for this feature? Global thresholds are already in place to prevent users from making values too high or too low.

It's not a top priority feature, but might be useful e.g. when you count something for a long period of time and want to be alerted when reaching a certain value. For example, how many packs of cigarettes you've smoked throughout the year, and you want to be alerted when you reach 1000!😋 Of course that has no meaning if you already have that limit in your mind. But in case you don't remember it after a long time, a reminder might be useful.
It's up to you my friend. I don't bother if you don't implement it. You can then close this issue as not planned.


Decouple in-memory storage updates from on-disk storage updates

PR Description
Adds the ability to apply updates to the in-memory store without first waiting for the on-disk store to be updated.  The on-disk storage will now continually retry updates if they fail to prevent it from winding up in an inconsistent state.  If the error isn't recoverable the event channel will eventually back up and block any further progress - we will likely need to treat too many repeated failures as a fatal error and shutdown to avoid these stalls.
When using archive mode, there is now potentially a period where finalized blocks have been removed from the in-memory store but are not yet available from the on-disk store.  We'll need to resolve that before this can be toggled on by default.
Change is feature toggled off by default.
Fixed Issue(s)
#1934
Documentation

[x] I thought about documentation and added the documentation label to this PR if updates are required.

Going to close this for now to keep things tidy.  It does have some performance benefit but I'm not sure its worth it and there's still a few key things to work through.  Will keep the branch around to refer to in the future.


Do not stop RabbitMQ before creating the Erlang Cookie

When boostraping a cluster, Puppet will:

create the cookie and manage permissions
if there is a change on this resource (create or update), thanks to
the notify in rabbitmq::config, RabbitMQ service will be restarted by
Puppet.

Given that, we should not manage the service in the provider itself,
since it's already done in the manifest.
This patch aims to delete the 'puppet resource' command in the
rabbitmq_erlang_cookie provider that ensured RabbitMQ service was
stopped before the cookie file management.
This patch has been tested on a new environment and an existing one, to be sure RabbitMQ is updated with the right cookie, on both use-cases.

This patch is related to PDB-1223

If you run the acceptance tests, you can see this causes serious problems with the rabbitmq-server. If the cookie changes while the server is still running, it becomes unable to connect to itself and therefore unable to restart itself.
An alternative to using the puppet resource command would be stopping the service via the init system. This means dealing with all supported init systems and all possible service names, so using the puppet resource command is much simpler.

The result of the test was: FAIL
Details at http://planck.nibalizer.com:8080/buildlogs//puppetlabs+puppetlabs-rabbitmq+306+1424889518+FAIL

I am a beta ci bot. I am probably lying to you.
You can contact nibalizer for more details

Closing due to inactivity and the concerns raised by @cmurphy.
Feel free to open a new PR
Thanks!


Don't set background to black when long-pressing buttons.

Before Gtk 3.18 the buttons were fully grey when they
were clicked (long press).
With gtk 3.18 it shows a black square, rounded with a
grey border.
This is a regression, so this restore the previous
behaviour
Also tested on XO-4 (with old Gtk version) and this patch doesn't broke the normal behaviour

I will test

reviewed.

Is this change compatible with Gtk < 3.18 ?

"Also tested on XO-4 (with old Gtk version) and this patch doesn't broke
the normal behaviour"
On Sun, 11 Oct 2015 5:02 pm Gonzalo Odiard<EMAIL_ADDRESS>wrote:

Is this change compatible with Gtk < 3.18 ?
—
Reply to this email directly or view it on GitHub
https://github.com/sugarlabs/sugar-artwork/pull/76#issuecomment-147161351
.

Not yet mentioned above is that XO-4 with 13.2.5 has Gtk+ versions 3.6 and 2.24, and that with 14.1.0 has Gtk+ versions 3.10 and 2.24.  Not tested this patch myself.

14 is unreleased isnt it?

Complexity involved that question is difficult to simplify.  We have one customer using it at least, as custom build they make from our collection of git repositories and packages.  But my builds don't meet my requirements for stability and usability so I've not made a release yet.  Several of the problems have been impossible to solve with the available resources.  There has been very little interest or contributed work.

OK, I'll give it a test on 14.  Is there a 106 build, or is there only the 104 version?

Only 0.104.  Link by mail.


Grader data performance improvements

Motivation and Context

I noticed a few slow-loading actions related to TA management: loading an assignment grade summary (and specifically the TA grade distribution graphs); and updating the graders table.
Your Changes

Description: Improved the performance of the relevant methods. Notably, I consolidated the grader mark calculations into Result to align with other places where the calculate marks.
Type of change (select all that apply):

[x] Refactoring (internal change to codebase, without changing functionality)

Testing

Tested in the UI; ran test suite.
Questions and Comments (if applicable)

Checklist

[x] I have performed a self-review of my own code.
[ ] I have verified that the pre-commit.ci checks have passed.
[ ] I have verified that the CI tests have passed.
[ ] I have reviewed the test coverage changes reported on Coveralls.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

16 of 16 (100.0%)  changed or added relevant lines in 4 files are covered.
6 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.02%) to 91.673%

Files with Coverage Reduction
New Missed Lines
%

spec/models/course_spec.rb
1
99.21%

app/models/ta.rb
5
87.8%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
37113

Relevant Lines:
39949

💛  - Coveralls

Note: I guess the test coverage decreased because of some old code removal, which I think is okay.


Handle Non-Zero Error Codes in Allocation update_all() Method

Expected Behavior
When one or more nodes in the Allocation data model tree fails to update properly, the user should be informed (via dialog). Additionally, details regarding which node(s) failed to update should be written to the debug log.
Actual Behavior
RTK simple prints 'FIXME: Handle non-zero error codes in rtk.allocation.Model.update_all()' to the terminal. This print statement is simply a placeholder and useless to the end user.
Steps to Reproduce the Problem

Launch RTK.
Open a Program database.
Select the Hardware tab in the Module Book.
Select the Allocation analysis in the Work Book.
Press the 'Save All' button in the Module Book.

The top node in all data model trees does not contain a data object, so the update() method will return a non-zero error code.
Proposed Solution (optional)
1.Create a requirement/specification for the update_all() method to handle non-zero error codes from the update() method. This requirement will cover the update_all() method in all RTK Modules.
2.Create a test to verify the requirement is implemented properly.
3.Ignore the top node in the update_all() method.
4.Create a descriptive error message to, ultimately, pass to the data controller. This error message will be written to the debug log for later inspection. Since the update() method that is called for each node in the update_all() method returns an error code and message, the update_all() method will need unique error code and message variables. The error code variable must retain a non-zero value after one update error. The error message variable must be appended to for each node that raises a non-zero error code in the update() method.
5.Have the Allocation Module View raise a Warning dialog informing the user one or more Allocations failed to update properly and to check the debug log for more details.
Operating Environment (not required for REQUIREMENTS errors)

The AttributeError also needs handling in the update_all() method.  Refer to the HazOps module for reference on the AttributeError.

Fixed in commit 9442979f257c107974e1881b3cbc6fe7ad22aae5


fix: build on Mac

Fix https://github.com/Taraxa-project/taraxa-node/issues/1594

prnet cleaned-up successfully


Handling big numbers in elastic search driver

Thanks for your great work and tool. Here's a minor issue i've encountered today and tried to fix appropriately.
I had problems when the data in ElasticSearch was Int64.MaxValue or -Int64.MaxValue. In that case when the result data from ES is parsed through JSON.parse the result is<PHONE_NUMBER>854776000 instead of<PHONE_NUMBER>854775807. Seems that javascript has problems working with numbers > 2^53.
So I've found this json-bigint module which seems to work good for such cases.

Thanks for the patch. I'd rather overwrite JSON in this context to make sure all instances have the same behavior. Expect an update from my side to the master branch.


Add Laravel 7 Build to Travis

Add PHP 7.2 and 7.3 for Laravel 7.x in Travis.

It's being replaced by Github Action in #20


Thread Safety

Once done you can use it to verify any number of tokens. Since verifing does not modify the verifier you can reuse it as often as you like.

Does it mean that verify method is thread-safe?

Yes

Thanks for the clarification.


Documentation stuck in loading page in case of unsupported test references in schema.yml

Describe the bug
dbt Documentation will stuck in loading page in case model tests are written in bundled format in schema.yml
Steps To Reproduce
In schema.yml referencing the not null tests the following way will cause the issue to surface:
models:
  - name: my_sql_model
    description: something my sql model does
    tests:
      - not_null:
          column_name:
            - date
            - customer_id
            - value
            - amount
            - etc...

Expected behavior
After dbt docs generate and dbt docs serve and opening the documentation in the browser, it will be stuck in loading page, never rendering the catalog (neither will this give a notification that schema.yml tests are the cause of the issue) .
Additional context
A workaround exists for the issue in case not null tests are referenced in a different (although more redundant) manner:
models:
  - name: my_sql_model
    description: something my sql model does
      - name: date
        tests:
          - not_null
      - name: customer_id
        tests:
          - not_null
      - name: value
        tests:
          - not_null
      - name: amount
        tests:
          - not_null
      - etc...

@zhegedus13 thanks for writing this up and providing a clear example.  The example you provide is actually not a valid way to define tests.  columns_name cannot take a list of columns.  You could define it as:
models:
  - name: my_sql_model
    description: something my sql model does
    tests:
      - not_null:
          column_name: 'date'

I'm guessing that your docs are not loading because they have not generated.  When I try to generate docs with a list of columns as you describe above, I get a Parsing Error and they do not generate.
I'm going to close this issue since it is not a bug in the docs.  Feel free to comment if you disagree.


Bump version to 3.2.2

edx-platform will temporarily use this edX fork of django-celery until Celery is upgraded to v4.
edx-platform currently installs django-celery v3.2.1. That version was not on master in the upstream repo - it was on an unmerged branch:
https://github.com/celery/django-celery/compare/master...v3.2.1
This PR bumps the version on the edX fork to avoid confusion around the installation of django-celery. Bumping the version will cause installation of the edX fork version on all edx-platform installations.

@jmbowman Ready for re-review.


Update Pages “PestControlPage”

Automatically generated by Netlify CMS

👷 Deploy Preview for chic-shortbread-66a66b processing.

Name
Link

🔨 Latest commit
ca9e9a57b348cdb6c7def52cf9f7c5b275169df8

🔍 Latest deploy log
https://app.netlify.com/sites/chic-shortbread-66a66b/deploys/6552c5837268cb000869af23


Fixing performance issues

@etrepum This PR moves the prop mapping to the internal <Control> state, which prevents it from recreating the mapped props unless the field/control is supposed to rerender, anyway.

I've tried adding this version as a dependency and it doesn't appear to fix the performance problem, I'm still seeing the same behavior inside my components (control, controlProps, onFocus, onBlur, onChange, onLoad, onSubmit changing for every component on every redux store update). I'll dig deeper later today.

I can't seem to reproduce the problem in isolation with this PR, it definitely seems like an improvement :+1: Not sure why we're still having problems, but it could be on our end or perhaps there was some pollution in node_modules when I tried to pull it commit from git into our app. I don't see a reason not to merge this as-is.

Thanks!

Yep! There's more wins to be had, but this is a good start.


Confd spawning application hijacks stdout logging in Docker

CMD bin/confd -confdir etc/confd/ -log-level info -backend etcd -node http://etcd-client:2379
Hi, so I am using this Dockerfile command to make confd spawn my own application so it can monitor key changes in my etcd instance and automatically reload and check everything for me. However, because confd is monitoring the spawned app, it is logging to Docker's log file (stdout) only confd messages not my app messages any more, as it used to be before confd.
Is confd hijacking my app logging because I don't use -onetime with it? Is there a workaround for this?

I would use something like s6 or another lightweight init system to spawn the processes (including confd).
Confd isn't really meant to spawn and monitor subprocesses.
You may also get some problems with zombie processes if you have more than 1 process per container without an init system (https://blog.phusion.nl/2015/01/20/docker-and-the-pid-1-zombie-reaping-problem/) -- i ran into this with a simple haproxy setup.

It's a common misconception that docker picks up on anything shipped to STDOUT. This is incorrect. Docker picks up on anything shipped to PID 1's STDOUT.
The reason this doesn't work is because /dev/stdout is a link to STDOUT of the process accessing it. So by letting confd run as PID 1 and booting your application inside the container as part of confd, anything it logs will not be shipped to docker logs.
What you want to do is to redirect output to STDOUT of PID 1 (in this case, confd). PID 1 is the process launched by docker, and its STDOUT will be what docker picks up.
A much better solution would be to use an init system like what @HeavyHorst suggests. No init system = zombies.


Note Doug as a maintainer

Doug does a lot of good work and per conversation with
other maintainers is already recognized as core.
This is not reflected in the MAINTAINERS file, fixing.
Signed-off-by: Eric Windisch<EMAIL_ADDRESS>

/cc @duglin @crosbymichael @LK4D4

I never knew that @duglin wasn't core maintainer :)
LGTM

LGTM

LGTM

Janky objects :P

Lol race condition or something

Isn't this missing the section with his name and email address?

I think he already there from builder maintainers
On Thursday, May 14, 2015, Arnaud Porterie<EMAIL_ADDRESS>wrote:

Isn't this missing the section with his name and email address?
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/13216#issuecomment-102098238.

Hell yeah! LGTM

Discussing with @crosbymichael, it's inappropriate for me to propose this PR (although @duglin  is awesome).


Correct the Chewy configuration in Readme

Correct:
Chewy.settings
to
Chewy.configuration

https://github.com/toptal/chewy/blob/master/CHANGELOG.md - it was renamed in master

oh... I was just doing a pull request...

Thanks :)


Add optional precipitation sinks, make precipitation sources optional

This PR:

adds precipitation sinks (evaporation, deposition, sublimation and melting) into 1-moment microphysics tendency. They can be switched on/off with the command line argument --precip_sinks,
adds a command line argument to switch on/off the autoconversion and accretion in the 1-moment microphysics --precip_sources.

Both process groups are switched on by default. The parameter parsing function did not accept Bool as parameter type. Thats why I'm passing them as Int.
Also, the github marking of changes got very confused in the microphysics tendency part. In most of the file I just increased the indent because of the additional if/else.

bors r+


Update the self host deployer message so published application comman…

…d is accurate
otherwise we get messages like

cc @BrennanConroy @kichalla

:shipit:


Except trigger condition

Would be really handy to be able to have a « ! » condition. Kind of inverted conditions
For example « !3/4 »

This requires one more bit for the condition. There is exactly one bit left per step, so it would be an option to implement this. Not sure how important that feature is though.

Will be released with 0.1.37.


Fix sample app source link

The current link links to master branch and gets a 404.

Closes #2716


Fixed Issue #423

I certify that I own, and have sufficient rights to contribute, all source code and related material intended to be compiled or integrated with the source code for the SharpZipLib open source product (the "Contribution"). My Contribution is licensed under the MIT License.

As mentioned by @Numpsy, we have to make a decision about how to correctly handle this. It might very well be that this is the end result, but if so, other places need to be update as well.


[BEAM-9890] Try fixing BIT_AND(NULL) issue

R: @amaliujia
Created a new Accum class to try fixing BIT_AND(NULL) issue

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

Run SQL_Java11 PreCommit

@Imfuyuwei
Can you add a unit test to test NULL value?

@Imfuyuwei
Can you add a unit test to test NULL value?

I will try. Right now not very clear how to do a 'NULL' test using PCollection, do you have any example?

NULL value can be tested: https://github.com/apache/beam/blob/master/sdks/java/extensions/sql/zetasql/src/test/java/org/apache/beam/sdk/extensions/sql/zetasql/ZetaSQLDialectSpecTest.java#L506

Sometimes you might need a CAST though: https://github.com/apache/beam/blob/master/sdks/java/extensions/sql/zetasql/src/test/java/org/apache/beam/sdk/extensions/sql/zetasql/ZetaSQLDialectSpecTest.java#L276

Sometimes you might need a class casting though: https://github.com/apache/beam/blob/master/sdks/java/extensions/sql/zetasql/src/test/java/org/apache/beam/sdk/extensions/sql/zetasql/ZetaSQLDialectSpecTest.java#L276

Yes, I tried several ways but the results are different from what I expected (neither -1L nor null). Do you know if there are some handy methods to print the elements of a PCollection to the console?

The easiest way I know is to write a ParDo by your own, which

pass through elements.
but also save elements in a List.

So you can append this ParDo to the PCollection that you want to inspect and then add breakpoint accordingly to suspend execution and check elements.

@Imfuyuwei
can you revert changes in BeamBuiltinAggregations.java  and BeamSqlDslAggregationTest.java


[TEST] Fix a broken MLIR test (reorder-instr)

Fix a broken unit test. I didn't check when the test broke, but it looks like the pass does that it was intended.  The test sink_convert_dealloc checks that we are able to sink convert_layout below the dealloc instructions. I fixed the test to check the intended behavior.
Tested with 'lit test'.
...
PASS: TRITON :: TritonGPU/reorder-instructions.mlir (12 of 49)
...

I suspect there's an issue with your setup, since this PR fails CI.
I kicked off tests at main, but my guess is that this test is actually passing.  https://github.com/openai/triton/actions/runs/8889569308

For my info, are you contributing on behalf of an employer, or is this a side project for you?  Thanks for your interest.

Please reopen the PR if a change is actually needed.

Must be an issue on my side. I'll take a look. Yes, I work at Meta.
Thank you.

OK, thanks.  For Meta engineers I think we were asking that you all get a review on your side before sending to OAI.


issue with using node-usb native module

Found an issue or bug? Tell me all about!
Describe the issue / bug.
When you put usb inside app/package.json and import it inside a vue file. Then it tried to parse tons of files. By doing some reading from the internet, seems like putting externals: ['usb'] inside webpack config will solve the problem but then I met usb module's not found error. Any idea why?
On the other hand, when I tried to do the same in Electron React Redux boilerplate, it went smooth without any problem. Make me wondering how did they solve the problem and I started to compare the two projects. But given that I just started trying out Electron for not long, I couldn't have much clue about what's going on inside.
How can I reproduce this problem?
do npm install --save usb in app folder then put import usb from 'usb'; console.log(usb). Then you will find it.
Tell me about your development environment.

Node version: 6.9.2
NPM version: 3.10.9
vue-cli version: 2.6.0
Operating System: Mac OS X Sierra

@lyonlai
Sorry for the late reply, I've been away for the holidays.
Sorry for your trouble with usb. Its hasn't ever been the easiest thing to use native node modules within webpack. I'm glad that you mentioned Electron React boilerplate, there's some setup there that I haven't seen before and might be the answer to some other improvements I've been trying to make. It looks like their setup uses externals for all node_modules instead of bundling everything together like electron-vue does. For the moment, you might be able to add usb as an external and then update the ignore filter in config.js to include node_modules in the production environment.

On a related note, I'm having a similar issue with nodegit.
More specifically, nodegit depends on promisify-node which in turn uses dynamic requires (as in '(module) => require(module)') which breaks in webpack.
Currently I can only use nodegit in the main process, which is less than ideal given how electron-vue is built, and will be even more so once the main process is part of the webpack build.
@SimulatedGREG
I've now added nodegit as an external as 'nodegit': 'commonjs nodegit', but I'm getting a module not found error in the dev build.
Nodegit is included as a dependency in 'app/package.json', so I'd assume it should be found?
The ignore filter you're referring to concerns only the production packaging if I'm understanding things correctly, yes?
Any pointers where I should look at?

@Flaw
Been working on migrating to webpack 2, and decided to treat dependencies as externals. I found that when in development, since files are served over http electron doesn't provide the proper context for require statements. You must manually add the path to app/node_modules for it to import properly. You can either wait out a few more days (webpack should be released very soon) and wait for a full release to master or take a look at the dev branch. Going to go ahead a close this, feel free to comment anything.
Specifically at...
https://github.com/SimulatedGREG/electron-vue/commit/033ab27231d5f94482ada7a9bad8aec9bba34cf3#diff-e4b797c6da6781c6f502d68ce67ddc2b
https://github.com/SimulatedGREG/electron-vue/commit/033ab27231d5f94482ada7a9bad8aec9bba34cf3#diff-c2daed2a6716bfe5579bc5ea723bd0c6
https://github.com/SimulatedGREG/electron-vue/commit/033ab27231d5f94482ada7a9bad8aec9bba34cf3#diff-f8f5519ac7b9fc97b564a2f1982f7dbdR16
https://github.com/SimulatedGREG/electron-vue/commit/033ab27231d5f94482ada7a9bad8aec9bba34cf3#diff-f8f5519ac7b9fc97b564a2f1982f7dbdR78

I'll give the dev branch a spin when I have time.
In the interim I took electron-react, nuked everything relating to React and fabricobbled Vue on top of it. Managed to get it working at least in dev mode, externals and all, but have no clue if packaging and such will work.
Frankly I'd much rather use electron-vue, you've done a great job with this. As someone who writes embedded C for a living and drives himself crazy with the web side of things in his free time, the documentation has been a life saver.

did this problem solve? i get the same problem and make me cazzy!~

[https://github.com/bgyhnddr/vue-electron.git](here is my code) take a look at it if you have some time


feat(focus indicators): Add config map to base focus indicators mixin and tweak some default styles.

This is the first PR in a series of PRs to refresh the focus indicators API in preparation for formal documentation being added. This PR includes the following changes:

The mixin mat-strong-focus-indicators (and the MDC equivalent) now both accept a $config map that developers can use to configure focus indicator styles (e.g. border-style, border-width, border-radius).
I changed some of the default focus indicator styles (e.g. increased width from 2px -> 3px, updated some of the default offsets).

Future PRs will:

Add per-component customization mixins.

Looks like you just need to rebase this branch on master


feat: use json encoder when no terminal is detected

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

5 of 5 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 98.851%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
172

Relevant Lines:
174

💛  - Coveralls


examples/shared should have a reference to LBaaS certificates creation

The provided example creates a LB with a certificate that has a reference to pre-existing local certificates folder:
                        "certificates": {
                            "LB-SHARED-CERT-1-KEY": {
                                "ca_certificate": "~/certs/ca.crt",
                                "certificate_name": "lb1-cert1",
                                "private_key": "~/certs/my_cert.key",
                                "public_certificate": "~/certs/my_cert.crt"
                            }
                        },

We should indicate the need to have valid PEM certificates for the Load Balancer. For educational purposes, and if the user has not trusted CA certificates we can point to a public reference with instructions on how to setup self-signed certificates in the shared MD documentation or a link to a secondary MD with some instructions like:
	a. Create a Self-Signed Root CA:
	openssl req -x509 -sha256 -days 1825 -newkey rsa:2048 -keyout ca.key -out ca.crt

	b. Create a cert key and certificate signing request (CSR):
	openssl req -nekey rsa:2048 -nodes -keyout my_cert.key -out my_cert.csr

	c. Sign the certificate CSR with Root CA:
	cat my_cert.txt
	authorityKeyIdentifier=keyid,issuer
	basicConstraints=CA:FALSE
	subjectAltName = @alt_names [alt_names]
	DNS.1 = oe01.com

	openssl x509 -req -CA ca.crt -CAkey ca.key -in my_cert.csr -out my_cert.crt -days 365 -CAcreateserial -extfile my_cert.txt

	d. Check the cert:
	openssl x509 -text -noout -in my_cert.crt

I agree, I think this is a prerequisite for the template provided, and should be detailed.

Instructions created in the common_lbaas_self-signed_certificates_howto.md and references in the shared & OE examples.


unit tests for summarize_provenance

My naive attempt to fix this (per this old commit) failed because most of the test cases for provenance files do not include a file for each patient. IE there are either 3 or 4 patients in the test data & only 1 of them has a provenance file.
Perhaps this is a realistic scenario? Currently summarize_data_sources() fails when there isn't a provenance file for each patient.  I don't know the correct behavior in this case, so for now postponing the problem.

Added unit tests - per 0220965


change color of the popups

since the image is white i can no see the select button how can i fix this

The buttons are using Android's buttonBarButtonStyle style.
I think as text color this style uses colorAccent.
Did you by incident set colorAccent to white?
If so you should be fine by defining colors like this in your src/main/res/values/colors.xml (if you derive your theme from RichTextEditorAppThemeLight or RichTextEditorAppThemeDark):
<?xml version="1.0" encoding="utf-8"?>
<resources>

    <!-- light theme -->

    <color name="colorPrimary_Light">#0D47A1</color>
    <color name="colorPrimaryWithTransparency_Light">#570D47A1</color>
    <color name="colorPrimaryDark_Light">#1A237E</color>
    <color name="colorAccent_Light">#FF9100</color>

    <color name="primaryTextColor_Light">#000000</color>

    <color name="backgroundColor_Light">#969696</color>

    <!-- dark theme -->

    <color name="colorPrimary_Dark">#212121</color>
    <color name="colorPrimaryWithTransparency_Dark">#57212121</color>
    <color name="colorPrimaryDark_Dark">#000000</color>
    <color name="colorAccent_Dark">#FF9100</color>

    <color name="primaryTextColor_Dark">#bababa</color>

    <color name="backgroundColor_Dark">#303030</color>

</resources>

Does this fix the issue?

yes it does thanks
i overlooked it


304 Not modified and Cache-Control

https://github.com/kfiroo/react-native-cached-image/blob/master/ImageCacheProvider.js#L126
https://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html#sec14.9.4

@iegik I believe you are correct!
Cache responses will be handled as errors, though I didn't encounter this before.
Care to spin a quick PR for this? :)

Will be soon :)


METRON-1696: Create the HDFS directory for pcap sequence files and add required privileges to metron user

Contributor Comments
PCAP parser fails to write pacap sequence file to hdfs on kerberized cluster due to insufficient privileges to hdfs folder for 'metron' user. Create the HDFS directory for PCAP sequence files and add the required privileges to metron user.
Pull Request Checklist
Thank you for submitting a contribution to Apache Metron.
Please refer to our Development Guidelines for the complete guide to follow for contributions.
Please refer also to our Build Verification Guidelines for complete smoke testing guides.
In order to streamline the review of the contribution we ask you follow these guidelines and ask you to double check the following:
For all changes:

[x] Is there a JIRA ticket associated with this PR? If not one needs to be created at Metron Jira.
[x] Does your PR title start with METRON-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.
[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

For code changes:

[ ] Have you included steps to reproduce the behavior or problem that is being changed or addressed?

[ ] Have you included steps or a guide to how the change may be verified and tested manually?

[ ] Have you ensured that the full suite of tests and checks have been executed in the root metron folder via:
mvn -q clean integration-test install && dev-utilities/build-utils/verify_licenses.sh

[ ] Have you written or updated unit tests and or integration tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

[ ] Have you verified the basic functionality of the build by building and running locally with Vagrant full-dev environment or the equivalent?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered by building and verifying the site-book? If not then run the following commands and the verify changes via site-book/target/site/index.html:
cd site-book
mvn site

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.
It is also recommended that travis-ci is set up for your personal repository such that your branches are built there before submitting a pull request.

@MohanDV thanks for the contribution! I actually have the same concern/thoughts on this PR as I made on @anandsubbu's here - https://github.com/apache/metron/pull/1132#issuecomment-409447590. I think PCAP is really trying to be a first class citizen in Metron. We had fairly manual support for PCAP before. I think if we're going to make these changes, we should make PCAP a proper component for Metron similar to [parsers, enrichment, indexing, profiler].

See comment here - https://github.com/apache/metron/pull/1132#issuecomment-409656807
I'm +1 via inspection pending a follow-on Jira being created, having it linked to this Jira, and also linked for reference in the comments on this PR.

Just FYI, this work was already done in https://github.com/apache/metron/pull/1019.  I do not see much value in merging this into master since it will likely be overwritten when the pcap feature branch is complete.

Thanks for pointing this out @merrimanr, Yes this will overwrite the the #1019 .  If the feature branch is not getting to master sooner then we can keep this change and rebase the feature branch else we can discard this. @mmiklavc metron:metron should allow pcap topology to write to hdfs directory. I started the topology as metron user.

metron:metron did not allow pcap topology to write as the Storm user.
Yup @mmiklavc .. on a non-kerberized cluster, I faced the same issue.

@anandsubbu @MohanDV - if we set this to metron:hadoop will that cover both cases? If it does, I would request you make this change in the feature branch. It should work for both kerberized/non-kerberized setups.

@mmiklavc I see the 'storm' user as the owner of pcap topology in non-kerberized setup, where as the 'metron'  as the owner in kerberized setup. IMHO The pcap should have 'metron' as owner of the topology, then having the hdfs permissions set to metron:hadoop will work on both the environment.

Non Kerberized pcap topology runs under 'storm' which requires write under the 'hadoop' group so we should provide mode 0775 when the cluster is un secure and mode 0755 when the cluster is secure as the topology runs under 'metron' user.

The REST component in Ambari is currently responsible for setting up Pcap HDFS directories.  It looks like this PR duplicates some of that.  I think we will have a dedicated Pcap Ambari component at some point so I don't think it matters that much where this setup logic lives in the meantime.
However, this PR doesn't line up with the feature branch because it only creates one directory where the Pcap query feature requires 3.  Also the property names in Ambari need to match REST.  I would recommend reviewing how this is currently implemented in REST.  Most of the work was done in https://github.com/apache/metron/pull/1124.  Whether this ends up in the REST or parsers component doesn't matter to me as long as it functions.
Also, unless we can adjust this PR to only include a couple minor changes I would strongly suggest this be tested end to end.  I doubt it works in it's current state.

I am fine wuth closing this PR as this is currently implemented in REST. When pcap gets its own start/stop script in Ambari we can move the code from REST to respective file.

Should this be closed out?

closing as it addressed in METRON-1738

Closed as it is addressed in  METRON-1738


Allow distribute scala 2.12 and update to spark 2.4.3

What do you think about distributing Elasticsearch spark" in Scala 2.12 as well?

Hi @neoramon, we have found your signature in our records, but it seems like you have signed with a different e-mail than the one used in your Git commit. Can you please add both of these e-mails into your Github profile (they can be hidden), so we can match your e-mails to your Github profile?

Please_DocuSign_Individual_Contributor_Licen (1).pdf

Sorry for the wait on this issue folks. I'll look into getting this tested and reviewed soon. The way we cross compile Scala in the build has been up in the air for a while due to some external forces on the project (mostly needing to upgrade our version of Gradle, but that upgrade broke our Scala cross compile stuff). Now that the concerns around our compile logic are clearing up, we can start looking at this in earnest.

@jbaiera Any update here?

@neoramon @jbaiera any update on the issue? do you need any help?
current MR has a conflict that is easy to merge into master.
I will build with 2.12 locally because I depend on it for my project, feel free to ask for some fixes to end it in master.

Done

Any updates? Looks like it's almost there. :)

At least Java 11 is required to use elasticsearch gradle tools

It seems that Travis doesn't have Java 11 config.

At least Java 11 is required to use elasticsearch gradle tools

It seems that Travis doesn't have Java 11 config.

I would ask the legitimacy of that move (java 8 -> java 11) for build, up to my knowledge spark(2.4, i haven't dig into 3.0 topic yet) doesn't explicitly support java 11.

At least Java 11 is required to use elasticsearch gradle tools

It seems that Travis doesn't have Java 11 config.

I would ask the legitimacy of that move (java 8 -> java 11) for build, up to my knowledge spark(2.4, i haven't dig into 3.0 topic yet) doesn't explicitly support java 11.

I'm not sure why this build needs java 11, there is a validation on build for Java >= 11.
buildSrc/build.gradle
if (JavaVersion.current() < JavaVersion.VERSION_11) {
    throw new GradleException('At least Java 11 is required to use elasticsearch gradle tools')
}

At the end, it also needs Java 8 for hadoop.
And I don't know how to set two jdk in travis.
:(

@neoramon I share your confusion on the Travis CI situation. I've tried a number of configurations and none of them have panned out. We are working on changing the CI for the project to use the public Jenkins builds for the CI checks instead.
As for the reasoning behind the move to Java 11+ for the build: We use the Elasticsearch project Gradle build tools to stand up clusters for our testing. In order to stay in sync with the versions of the build tools (which only support Java 12), we need the build to run on that same version of Java. That said, we still need Java 8 in an environment variable in order to compile Scala with the correct versions.

👍  @neoramon @jbaiera - thank you for working on this!
I've compiled on jdk8 and scala 2.12.10 with basic app tests passing. Just the src code changes (I was half way though before I found this PR ;)
Dataproc on GCP image preview is 2.12 now, so this is becoming more critical - adding my +1 here
@neoramon - thank you for the PR!

Hi there! Any updates on this? Is there any way to help? Thanks for your work on this issue so far! 😄

Hi there! Any updates on this? Is there any way to help? Thanks for your work on this issue so far! 😄

@rlmark  - I'm watching this as well, here is the latest relevant comment: https://github.com/elastic/elasticsearch-hadoop/issues/1412#issuecomment-579464498

Hi all, a quick update here: I've logged an issue on the project pertaining to our growing pains with building Scala. This PR will most likely need to wait for that issue to be resolved before it can be moved any further.

Do you have a rough timeline for when you think these things will happen @jbaiera?
And, is it possible to hack this together by building some of the parts myself?

Any updates on this?

@neoramon i'm trying to build your fork locally https://github.com/neoramon/elasticsearch-hadoop and it seems to fail with
> Task :buildSrc:compileGroovy FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':buildSrc:compileGroovy'.
> Compilation failed; see the compiler error output for details.

please help?

Any updates on this? /2

This is PR is old now and it will need an update from master.
Someone know how to fix the problem of running Travis with two jdk?

I just did merge with master and saw that travis file was removed, and it looks that all checks have passed! 👍

Ccan someone review and merge?

Is there any news on when it'll be merged?

I'm closing this PR.
@jbaiera already updated spark and scala version in this PR: https://github.com/elastic/elasticsearch-hadoop/pull/1551/files

Even im facing this issue with Spark 3.0 and no option to use scala 2.11 as spark supports min 2.12 onwards from 3.0.0


Removing VM Disk, Why run Update-AzVM

Hi,
After removing a VM Disk with Remove-AzVMDataDisk, the doc example indicates that one needs to run Update-AzVM to " updates the state of the virtual machine". I wanted to ask, why is it required to update this state, and what would happen if Update-AzVM is not run?
It would be nice to add this explanation in the following pages

https://learn.microsoft.com/en-us/azure/virtual-machines/windows/detach-disk#detach-a-data-disk-using-powershell
https://learn.microsoft.com/en-us/powershell/module/az.compute/remove-azvmdatadisk?view=azps-10.0.0#example-1-remove-a-data-disk-from-a-virtual-machine

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

ID: bc8c8392-0eb4-cda5-c282-0656869b7a6b
Version Independent ID: 5d4d2a61-9d83-a09e-2c33-36465a04e5c5
Content: Detach a data disk from a Windows VM - Azure - Azure Virtual Machines
Content Source: articles/virtual-machines/windows/detach-disk.md
Service: storage
Sub-service: disks
GitHub Login: @roygara
Microsoft Alias: rogarana

@saryeHaddadi
Thanks for your feedback! We will investigate and update as appropriate.

@saryeHaddadi
Thank you for your feedback!
I'd recommend working closer with our support team via an [Azure support request] (https://docs.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request). Or you can leverage our Q&A forum by posting your issue there so our community, and MVPs can further assist you in troubleshooting this issue or finding potential workarounds.
[Teams Q&A forum] (https://docs.microsoft.com/en-us/answers/topics/46488/office-teams-windows-itpro.html) for technical questions about the configuration and administration of Microsoft Teams on Windows.
[Microsoft Teams Community forum] (https://answers.microsoft.com/en-us/msteams/forum?sort=LastReplyDate&dir=Desc&tab=All&status=all&mod=&modAge=&advFil=&postedAfter=&postedBefore=&threadType=All&isFilterExpanded=false&page=1)
Thank you for your time and patience throughout this issue.


Exception raised when running under Python 3

The sky subtraction step in astrodrizzle will raise an exception, at least when run under Python 3; namely,
Exception in Tkinter callback
Traceback (most recent call last):
File "/user/hack/dev/release/anaconda3/envs/rtdev35/lib/python3.5/site-packages/drizzlepac/util.py", line 219, in wrapper
func(*args, **kwargs)
File "/user/hack/dev/release/anaconda3/envs/rtdev35/lib/python3.5/site-packages/drizzlepac/astrodrizzle.py", line 204, in run
sky.subtractSky(imgObjList, configobj, procSteps=procSteps)
File "/user/hack/dev/release/anaconda3/envs/rtdev35/lib/python3.5/site-packages/drizzlepac/sky.py", line 203, in subtractSky
_skymatch(imageObjList, paramDict, inmemory, clean, log)
File "/user/hack/dev/release/anaconda3/envs/rtdev35/lib/python3.5/site-packages/drizzlepac/sky.py", line 312, in _skymatch
fi.mask_images[k], fi.maskext[k], in_memory)
File "/user/hack/dev/release/anaconda3/envs/rtdev35/lib/python3.5/site-packages/drizzlepac/sky.py", line 450, in _buildStaticDQUserMask
randomize_prefix=False)
File "/user/hack/dev/release/anaconda3/envs/rtdev35/lib/python3.5/site-packages/stsci.skypac-0.9.5-py3.5.egg/stsci/skypac/utils.py", line 280, in temp_mask_file
# write data to the "temporary" file
File "/user/hack/dev/release/anaconda3/envs/rtdev35/lib/python3.5/site-packages/astropy/utils/decorators.py", line 492, in wrapper
return function(*args, **kwargs)
File "/user/hack/dev/release/anaconda3/envs/rtdev35/lib/python3.5/site-packages/astropy/io/fits/hdu/hdulist.py", line 864, in writeto
fileobj = _File(fileobj, mode='ostream', overwrite=overwrite)
File "/user/hack/dev/release/anaconda3/envs/rtdev35/lib/python3.5/site-packages/astropy/utils/decorators.py", line 492, in wrapper
return function(*args, **kwargs)
File "/user/hack/dev/release/anaconda3/envs/rtdev35/lib/python3.5/site-packages/astropy/io/fits/file.py", line 128, in init
objmode = _normalize_fits_mode(fileobj_mode(fileobj))
File "/user/hack/dev/release/anaconda3/envs/rtdev35/lib/python3.5/site-packages/astropy/io/fits/file.py", line 86, in _normalize_fits_mode
"files must be opened in binary mode".format(mode))
ValueError: Text mode 'w' not supported: files must be opened in binary mode
This looks to be caused by a change in the behavior of io in Python 3.

@stsci-hack This error fundamentally appears to be related to https://github.com/spacetelescope/drizzlepac/issues/84 - see this issue for more details. Specifically, I have identified the reason for this error to be in the https://github.com/astropy/astropy/pull/6373 PR by @drdavella

@stsci-hack Did you set in_memory to "yes"?

Fixed by @stsci-hack in https://github.com/spacetelescope/stsci.skypac/pull/16


KeyRepeat tab is gone

What's the reason behind removing the Key Repeat tab?

Karabiner-Elements use virtual keyboard to post key events since v0.90.69.
(We have to use the virtual keyboard to support sticky modifiers in Accessibility features.)
Apple does not provide a way to change the key repeat rate of keyboard devices.
Thus, the key repeat configuration was vanished.

Is there any way to re-add this feature? Something I really enjoyed!

Hi @jackzampolin,
You can build the v0.90.68 from the source code as I did :)

You can do it by defaults command without Karabiner-Elements.
(You have to logout and login to apply the setting.)
Example:
defaults write NSGlobalDomain KeyRepeat -int 10

I do not know how Karabiner-Elements does it but I am able to set the KeyRepeat and InitialKeyRepeat from the Karabiner-Elements UI with a much more responsive feeling compared to editing the defaults. Is it just changing the defaults under the hood?

NSGlobalDomain KeyRepeat and NSGlobalDomain InitialKeyRepeat don't seem to affect my system (karabiner elements currently installed)
macOS 10.11.6


fire from PolymerBase doesn't convert detail

While porting iron-signals I have this code
fire('iron-signal', detail: convertToJs({'name': "foo", 'data': "Foo!"}));

Without convertToJs(...) iron-signals didn't recognize the event.
Does it make sense to automatically convert?

Definitely just a bug, thanks for the report!


Requesting special history search feature: If matching zero, upon entry don't nuke my command, emit it.

[x] I have read through the manual page (man fzf)
[x] I have the latest version of fzf
[x] I have searched through the existing issues

Info

OS

[x] Linux
[x] Mac OS X
[x] Windows
[x] Etc.

Shell

[x] bash
[x] zsh
[x] fish

Problem / Steps to reproduce
Typical workflow is I first hit Ctrl+R to do a history search. I type a command, and it turns out after all that i've never run that particular command yet on the system (or it hasn't made its way into that particular shell instance's history buffer). At this point I have lost the work I've put into typing this buffer if I'm not careful. Hitting enter on a fzf search with 0 results yields nothing, which nukes my command I typed. Same thing happens if we cancel instead of accept by hitting ctrl+c, query is nuked.
So i hope to implement a special behavior in the case of acceptance on zero results to just emit the query instead of nothing. Can this be done?
I am aware that semantically speaking this is sloppy because we're in a history search buffer and why are we typing a whole command intended for execution, well I've accepted that humans aren't naturally driven by logic truly, which is why we all have jobs as software makers anyhow. So this has become a bit of a rough edge for me.

Would it help to have an `accept-or-print-query' action (tentative naming) so you could bind it to the Enter key?
export FZF_CTRL_R_OPTS='--bind enter:accept-or-print-query`

for sure that should work! Thank you so much for looking into this :D

Hmm, on second thought, I realized that this is already possible with become.
export FZF_CTRL_R_OPTS='--bind "enter:become:if [[ -n {} ]]; then echo {}; else echo {q}; fi"'

However, become is not supported on Windows, so I'm going to keep the new action. The implementation is quite simple so it doesn't hurt to have it.

The suggested snippet works when there are no matches in the history.
I often find that there are wrong matches only and I want to cancel search but keep the input.
Is it possible to do that?
I think CTRL+C should keep the input by default if possible - user may hit CTRL+C once again to clean if they want to.

@AlexanderYastrebov Agreed, this would be the next sharpest edge to polish. In my experience though it's been fluid enough for me to hit enter to take one of those choices and edit it. Reason being at least 85% of the time the way I got those results surfaced involved a fuzzy entry into the query box. So there is usually incentive to grab one full command and edit it.
It does look like enhancing the ctrl+c behavior would be ideal though. We can keep Esc to do as it does now. Or flip them around, which would make ctrl+c more consistent at wiping out what you typed.

Why don't use just bind CTRL-C to become?
export FZF_CTRL_R_OPTS='--bind "ctrl-c:become:echo {q}"'

However, I keep thinking that I don't want to suggest this approach to the users because as @unphased mentioned above, we're typing in a special fuzzy matching syntax instead of a full command in the prompt, something likegitch-b !comm.
Also, https://junegunn.github.io/fzf/search-syntax/#how-to-get-better-results

@unphased @junegunn Thanks for replies. bind ctrl+c snippet does what I wanted but I also get the point of fuzzy syntax.

nice. I take it we would be able to do what I suggested above by binding become to Esc.


Left vs right eigenvectors

We need to explain the difference (#136) and why we are using right eigenvectors in figure (3.c).

This is now figure 4.c right?


Installation works but no functionality on my machine

I just installed the dbml-viewer from VS Marketplace but it seems to not provide any functionality on my machine: The eye icon does not show (and there is no .DBML syntax highlighting in case this also comes with dbml-viewer).
I am on Windows 10. There were no errors during installation. I tried to restart VS Code and Windows but it did not help.
Any idea what could be the problem?

I deinstalled and reinstalled the extension. Now it works.


Using Wiki lookup (use on object NPC) on Estate Agent brings to Wizard Wiki page

Please check if your issue is not a duplicate by searching existing issues
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

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

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

OS: [e.g. Windows, Ubuntu, macOS]
RuneLite version: [e.g 1.4.6]
Launcher version: [e.g 1.5.2]

Additional context
Add any other context about the problem here, e.g. logs. Your client logs can usually be found in
C:\Users\<your_user_name>\.runelite\logs on Windows and ~/.runelite/logs on Linux and macOS.

this should be fixed now


Add timeouts for reqwest clients

Motivation
Asynchronous reqwest::Client do not have timeouts be default resulting in tests which can hang indefinitely.
Proposal
Add a timeout of 30 seconds to all reqwest::Client.
Test Plan
CI should catch regressions.

Thanks for doing this, now I can push again 😁


[BUG] Change of Api version not being reflected when using SetApiVersion for configurationAssignment in Microsoft.Maintenance

Library name and version
Azure.ResourceManager.Maintenance 1.2.0-beta.4
Describe the bug
Using SetApiVersion to override version using SDK libraries like so
ArmClient Client { get; set; }
ArmClientOptions clientOptions = new ArmClientOptions(){Environment = environment };
clientOptions.SetApiVersion(new ResourceType("Microsoft.Maintenance/configurationAssignments"), "2021-05-01");
Client = new ArmClient(cred, subscriptionId, clientOptions);

Using the following code for creating the configuration assignment
await resourceGroupResource.CreateOrUpdateConfigurationAssignmentAsync(ProviderName, ResourceType, VMSSResourceName, VMSSConfigurationAssignmentName, data);

Getting an exception while making the call which shows that the version was not overridden. The error is expected as the endpoint I'm calling does not support this version.

Further Information
Upon debugging I saw that the ApiVersionOverrides property of the ArmClient is being populated correctly
The issue is that the ResourceGroupResourceExtensionClient is not initializing the ConfigurationAssignmentsRestClient object with the correct ApiVersion, instead the default constructor is being used default _apiVersion
Code:
"private ConfigurationAssignmentsRestOperations ConfigurationAssignmentsRestClient => _configurationAssignmentsRestClient ??= new ConfigurationAssignmentsRestOperations(Pipeline, Diagnostics.ApplicationId, Endpoint);"

Reference of the above code ResourceGroupResourceExtensionClient.cs#L49
ConfigurationAssignmentsRestClient is being used to call the CreateOrUpdateAsync of configurationAssignments

Reference of the above code ResourceGroupResourceExtensionClient.cs#L554
The autorest.md file indicates that the code is auto generated so this might not be the right place to look for such errors.

Expected behavior
After calling SetApiVersion, The specified version should be sent in the request while making the call to configurationAssignments.
Actual behavior
The default Api version ('2023-04-01' in this case) is being sent in the request.
Reproduction Steps
ArmClient Client { get; set; }
ArmClientOptions clientOptions = new ArmClientOptions(){Environment = environment };
clientOptions.SetApiVersion(new ResourceType("Microsoft.Maintenance/configurationAssignments"), "2021-05-01");
Client = new ArmClient(cred, subscriptionId, clientOptions);

Using the following code for creating the configuration assignment
await resourceGroupResource.CreateOrUpdateConfigurationAssignmentAsync(ProviderName, ResourceType, VMSSResourceName, VMSSConfigurationAssignmentName, data);

Environment
No response

For the Reference
MaintenanceConfiguration in Microsoft.Maintenance working fine, it is using the specified Api version to request which I set using setApiVersion
clientOptions.SetApiVersion(new ResourceType("Microsoft.Maintenance/maintenanceConfigurations"), "2021-05-01");
It is initializing the _maintenanceConfigurationRestClient object with the correct ApiVersion using that TryGetApiVersion

Reference of the above code MaintenanceConfigurationCollection.cs#L47
and then this client object is being used to create maintenanceConfiguration

Reference of the above code MaintenanceConfigurationCollection.cs#L90

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

New version released contains the fix: https://www.nuget.org/packages/Azure.ResourceManager.Maintenance/1.2.0-beta.7


Palindrom extension

Description
added palindrome check to string
Basic Checklist

[x] 🛠 Added proper documentation
[x] ❌ No warnings or errors in the code
[x] ✅ If added a new extension, added in Readme and example too

Type of Change

[ ] ✨ New feature (non-breaking change which adds functionality)
[ ] 🛠 Bug fix (non-breaking change which fixes an issue)
[ ] ❌ Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 Code refactor
[ ] ✅ Build configuration change
[ ] 📝 Documentation
[ ] 🗑 Chore

@jasimameen thanks for the PR. Please fix the pipeline and write test cases for the same.

LGTM! Thanks for the functionality :)


rename coreos to containerLinux

We use operatingSystem:coreos.
As it got renamed to ContainerLinux we should adapt

Doesn't Digital Ocean image thingie still identify it as coreos?
The system itself definitely contains ID=coreos in its /etc/os-release.

Yes, this issue is just about the renaming the the constant used for specifying the Operating system via the machine manifest. For backwards compatibility we should though accept both.

I think this is a won't fix now, as [it is now named coreos again] (https://www.redhat.com/de/about/press-releases/red-hat-unveils-roadmap-coreos-integration-red-hat-openshift)


java.lang.NoClassDefFoundError: org/spongepowered/plugin/meta/PluginMetadata

There is an error from version 4.0.0 on SpongeVanilla :
SpongeVanilla 1.8.9-4.1.0-BETA-253 : Error (http://pastebin.com/Kk5XdNsh)
SpongeForge 4.0.1-BETA-1184 : Ok

Fixed in https://github.com/SpongePowered/SpongeVanilla/commit/eac1a58f3c27d58fd8ebc4bb21a030a3d6b6a2f8: Pending merge


Uncaught TypeError: undefined is not a function

I'm getting this exception when I toggle any of the search options (weird!). Sometimes, when I'm viewing a file I haven't opened in a while, I can toggle options one or a few times without seeing this issue, but it eventually shows up.
TypeError: undefined is not a function
    at ~/.atom/packages/color-picker/lib/ColorPicker-view.coffee:242:21
    at Object.Emitter.emit (~/.atom/packages/color-picker/lib/ColorPicker-view.coffee:6:9)
    at ~/.atom/packages/color-picker/lib/ColorPicker-view.coffee:206:14

This is occurring with all files, although regex search successfully toggles (i.e., successfully enables/disables).
Looks like this may (or may not) be related to bug https://github.com/thomaslindstrom/color-picker/issues/59. Edit: Upon further reading of bug 59, it looks like it very well may be related/the same..... (see the comment from ekonstantinidis)
OS X 10.10.3
Atom 0.201.0
User packages:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

Affects me too!

Same here.

Try switching search options

Atom Version: 0.201.0
System: Mac OS X 10.9.5
Thrown From: color-picker package, v2.0.1
Stack Trace
Uncaught TypeError: undefined is not a function
At /Users/chris/.atom/packages/color-picker/lib/ColorPicker-view.coffee:209

TypeError: undefined is not a function
  at /Users/chris/.atom/packages/color-picker/lib/ColorPicker-view.coffee:209:56
  at Object.Emitter.emit (/Users/chris/.atom/packages/color-picker/lib/ColorPicker-view.coffee:172:31)
  at /Users/chris/.atom/packages/color-picker/lib/ColorPicker-view.coffee:185:65

Commands
     -3:40.3.0 editor:select-to-end-of-line (atom-text-editor.editor.is-focused)
  2x -3:40 core:backspace (atom-text-editor.editor.is-focused)
     -3:39.3.0 editor:move-to-first-character-of-line (atom-text-editor.editor.is-focused)
     -3:36.9.0 core:save (atom-text-editor.editor.is-focused)
  6x -3:26 core:undo (atom-text-editor.editor.is-focused)
     -3:21.2.0 core:save (atom-text-editor.editor.is-focused)
     -2:16.8.0 editor:newline (atom-text-editor.editor.is-focused)
     -2:16.5.0 core:paste (atom-text-editor.editor.is-focused)
     -2:11 core:move-down (atom-text-editor.editor.is-focused)
     -2:09.8.0 core:undo (atom-text-editor.editor.is-focused)
     -2:08.9.0 find-and-replace:show (atom-text-editor.editor.is-focused)
     -1:19 editor:consolidate-selections (atom-text-editor.editor.is-focused)
     -1:19 core:cancel (atom-text-editor.editor.is-focused)
     -1:12.9.0 deprecation-cop:view (atom-workspace.workspace.scrollbars-visible-when-scrolling.theme-seti-syntax.theme-seti-ui)
  3x -0:43.8.0 core:close (atom-text-editor.editor.is-focused)
     -0:03.1.0 find-and-replace:show (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "themes": [
      "seti-ui",
      "seti-syntax"
    ]
  }
}

Installed Packages
# User
color-picker, v2.0.1
seti-syntax, v0.4.0
seti-ui, v0.7.1

# Dev
No dev packages

This is really weird. I can't seem to be able to reproduce it. If anyone has time, would it be too much to ask for a simple video showing the bug? Anything to help me track this down will be greatly appreciated.

Oddly enough, after restarting atom (not as a part of trying to debug this issue, it just happened at some point today), the issue went away for me.

Huh. I tried to reproduce it as well. Nothing. O-o

Quick screengrab:
https://www.dropbox.com/s/cs44mhfjk6it0ug/colorpicker-error.mov?dl=0
But after reading @bmhkim’s comment, I restarted and similarly can no longer reproduce. Some kind of update error seems plausible.

I can't speak to whether or not I saw this issue directly after updating, I didn't really pay attention to which plugins were updated today. However, I can say I agree it seems plausible, especially since it went away after restarting atom.

Thanks, @delucis. It's hard to say what this might be.  Parallel to the new one.
If it disappears after restarting Atom, for all of you guys, I'm going to blatantly assume that is the case.
Line 209, in v2.0.0, doesn't seem to be doing anything interesting, anyway. It's just a function definition:
 emitClose: ->

https://github.com/thomaslindstrom/color-picker/blob/16b46256275a579f2f7fb2e01bf6c3cff860106d/lib/ColorPicker-view.coffee#L209

I'm going to close this for now. Please re-open this issue if the issue persists!


Terminating app due to uncaught exception 'NSRangeException'

macos: 14.2.1 (23C71)
wechat: 3.8.1
Error report: Terminating app due to uncaught exception 'NSRangeException' reason: '*** -[__NSArrayM objectAtIndexedSubscript:]: index 0 beyond bounds for empty array'

Solved! By delete data cache directory.


repo no longer updating and just failures since January

I've been trying to update my site since January but I just get repo failure in the repository history. Now it doesn't even update the repository. Everything seems to go through from vs code side. What is happening?!

Error "   Error: fatal: No url found for submodule path 'polygonpalm' in .gitmodules
Error: The process '/usr/bin/git' failed with exit code 128@
I can't find a .gitmodules folder


[BUG] "undefined" is not valid JSON

What happened?
Click on "Offer Accepted" in Transaction history in GUI.
Outcome:  "Something went wrong" error displayed.

GUI
Relevant log output or stacktrace
"undefined" is not valid JSON

URL
#/dashboard/offers/view

Stacktrace
at webpack://@chia-network/gui/src/components/offers2/OfferBuilderViewer.tsx:68:36 useEffect(
at webpack://@chia-network/gui/node_modules/react-dom/cjs/react-dom.production.min.js:172:114 useLayoutEffect
at webpack://@chia-network/gui/node_modules/react/cjs/react.production.min.js:23:112 useLayoutEffect
at webpack://@chia-network/gui/src/components/offers2/OfferBuilderViewer.tsx:67:27 useEffect(
at webpack://@chia-network/gui/node_modules/react-dom/cjs/react-dom.production.min.js:157:136 Ch
at webpack://@chia-network/gui/node_modules/react-dom/cjs/react-dom.production.min.js:267:459 b?b.sibling:null}I
at webpack://@chia-network/gui/node_modules/react-dom/cjs/react-dom.production.min.js:250:346 Pj
at webpack://@chia-network/gui/node_modules/react-dom/cjs/react-dom.production.min.js:250:277 Pj
at webpack://@chia-network/gui/node_modules/react-dom/cjs/react-dom.production.min.js:250:137 Pj

Dupe of #1418


Multiple Fonts

For a project we have different fonts. The regular fonts, for "normal" characters. And "fallback"-fonts, for ❤ (emojis).
Our example for this fallback-method looks like this:
    private void printWithFallback(final String line, final int fontSize, final float startX, final float startY) throws IOException {
        this.contentStream.beginText();
        this.contentStream.newLineAtOffset(startX, startY);

        for (int offset = 0; offset < line.length(); ) {

            PDFont fontToPrint = this.document.getFonts().get(0);
            final int codePoint = line.codePointAt(offset);
            offset += Character.charCount(codePoint);
            String codePointAsString = new String(new int[]{codePoint}, 0, 1);

            //try all available fonts
            for (final PDFont fontToTry : this.document.getFonts()) {
                try {
                    fontToTry.encode(codePointAsString);
                    fontToPrint = fontToTry;
                    break;
                } catch (final IllegalArgumentException e) {
                    //try next font
                }
            }

            try {
                fontToPrint.encode(codePointAsString);
            } catch (final IllegalArgumentException e) {
                log.error("Codepoint {} could not be printed to PDF and has been replaced by the fallback character {}. " +
                                "This probably means that a font which can encode this codepoint should be included in the next release.",
                        codePoint, FALLBACK_CHARACTER);
                codePointAsString = FALLBACK_CHARACTER;
            }

            this.contentStream.setFont(fontToPrint, fontSize);
            this.contentStream.showText(codePointAsString);
        }

        this.contentStream.endText();
    }

I think this is pretty closely related to https://github.com/vandeseer/easytable/issues/29. Because if the TableDrawer class would allow for easier reuse from outside one could just override the respective methods for this case.

That would be nice.

Please check out the develop branch, I updated the code accordingly and also added an example class (together with a hint in the README). For your specific use case it should exactly work as in the example class: EasytableCustomTableDrawer
For installing the current snapshot version locally just use: mvn install -DskipTests -Dgpg.skip
I didn't yet release this though – but may do so soon.
If it works and you have some nice code and tables to share, please let me know. ;-)

Did it work @AndreKoepke? Just wanted to confirm it also from your side before I do the next release.

I didn't tested it yet. I will try to do it in the next few days.

https://github.com/vandeseer/easytable/blob/799929001c5f3184b2ba16fe6078d96972cc41c9/src/main/java/org/vandeseer/easytable/TableDrawer.java#L49
The constructor must be public. I cant use in the current version.

See https://github.com/vandeseer/easytable/pull/35

It is protected in the develop branch, as I pointed out above (see here).
You would need to install that branch manually via mvn install -DskipTests -Dgpg.skip. Then you can reuse the TableDrawer like so:
@SuperBuilder
private static class CustomTableDrawer extends TableDrawer {

    CustomTableDrawer(float startX, float startY, PDPageContentStream contentStream, Table table, float endY) {
        super(startX, startY, contentStream, table, endY);
    }

    @Override
    protected void drawText(String text, PDFont font, int fontSize, Color color, float x, float y) throws IOException {
        System.out.println("My custom table drawer is called :-)");
        super.drawText(text.toUpperCase(), font, fontSize, color, x, y);
    }

}

In case you are using lombok at least. I created two examples hence:

CustomTableDrawer.java
NoLombokCustomTableDrawer.java

Whuups, was on the wrong branch. My bad. :D

Iam done. All our features working now, but easyTable needs some changes (look at pull-request).
import lombok.Getter;
import lombok.experimental.SuperBuilder;
import org.apache.pdfbox.pdmodel.PDPageContentStream;
import org.apache.pdfbox.pdmodel.font.PDFont;
import org.vandeseer.easytable.TableDrawer;
import org.vandeseer.easytable.structure.Table;

import java.awt.*;
import java.io.IOException;
import java.util.ArrayList;
import java.util.List;
import java.util.stream.IntStream;

@SuperBuilder
public class EasyTableDrawer extends TableDrawer {

    private static final String FALLBACK_CHARACTER = "?";
    private final List<PDFont> fonts;

    @Getter
    private float drawSizeCurrentPage;

    EasyTableDrawer(final float startX,
                    final float startY,
                    final PDPageContentStream contentStream,
                    final Table table,
                    final float endY,
                    final List<PDFont> fonts) {

        super(startX, startY, contentStream, table, endY);
        this.fonts = fonts;
    }

    @Override
    protected void drawText(final String text, final PDFont font, final int fontSize, final Color color, final float x, final float y) throws IOException {
        super.contentStream.beginText();
        super.contentStream.newLineAtOffset(x, y);

        for (int offset = 0; offset < text.length(); ) {

            PDFont fontToPrint = font;
            final int codePoint = text.codePointAt(offset);
            offset += Character.charCount(codePoint);
            String codePointAsString = new String(new int[]{codePoint}, 0, 1);

            final List<PDFont> fonts = new ArrayList<>(this.fonts);
            fonts.add(0, font);

            //try all available fonts
            for (final PDFont fontToTry : fonts) {
                try {
                    fontToTry.encode(codePointAsString);
                    fontToPrint = fontToTry;
                    break;
                } catch (final IllegalArgumentException e) {
                    //try next font
                }
            }

            try {
                fontToPrint.encode(codePointAsString);
            } catch (final IllegalArgumentException e) {
                codePointAsString = FALLBACK_CHARACTER;
            }

            super.contentStream.setFont(fontToPrint, fontSize);
            super.contentStream.showText(codePointAsString);
        }

        super.contentStream.endText();
    }

    @Override
    public void draw() throws IOException {
        final int startIndex = this.rowToDraw;

        super.draw();

        final int lastIndex;

        if (this.rowToDraw == startIndex) {
            // all draw
            lastIndex = this.table.getRows().size();
        } else {
            lastIndex = this.rowToDraw;
        }

        this.drawSizeCurrentPage = IntStream.range(startIndex, lastIndex)
                .mapToObj(index -> this.table.getRows().get(index).getHeight())
                .reduce(0f, (left, right) -> left + right);
    }

}

Glad you made it now and thanks for your pull request. I will look into it shortly and will then release a new version.

Included in version 0.4.2.


Multi tag search

Fixes Issue
Minor tweaks from the great work by @Ananya2001-an on https://github.com/EddieHubCommunity/LinkFree/pull/4030

Changes proposed

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

@eddiejaoude Hey Eddie. I will review the code in 1 hour or so. I have got some urgent work right now. Sorry for the delay but I will definitely check everything and tell you if it's working fine or not.

No problem @Ananya2001-an , please do not stress, I look forward to hearing from you when you have some time


st cannot display CJK chars or emojis if invoked by dwm

Hi Luke,
So what happened is, when I open a st instance using
$ ~/.config/st/st
$ /usr/local/bin/st
or simply
$ st (which i believe points to the binary in /usr/local/bin)
they all work just fine: I can copy & paste CJK characters/emojis into the terminal:

and if I open a newsboat instance from there, the rss feeds with CJK characters are perfectly displayed.
Now, if I open a st from dwm keybindings, it looks like this:

I was wondering if you have any idea why this could happen? Thanks in advance!
Best regards,
Greg

I've had that happen when I was editing Luke's bash dwmblocks scripts via the st terminal to update the icons with FontAwesome icons. I'm not entirely sure of this but I guess it's a bug with how st renders emojis

If you're not using my exact build of dwm, it's possible that your build is enforcing its own font configuration for st instances spawned by dwm. The default builds of dwm do that for some reason.


Installation issue

Hey folks, trying to install on Ubuntu 14.04, getting this error:
ubuntu@ip-10-1-0-33:~$ sudo sift install
><EMAIL_ADDRESS>> sift-version: notinstalled

Installing and configuring SaltStack properly ...

Command failed: apt-get install -y --allow-change-held-packages salt-minion
E: Command line option --allow-change-held-packages is not understood

Error: Command failed: apt-get install -y --allow-change-held-packages salt-minion
E: Command line option --allow-change-held-packages is not understood

    at ChildProcess.exithandler (child_process.js:205:12)
    at emitTwo (events.js:106:13)
    at ChildProcess.emit (events.js:194:7)
    at maybeClose (internal/child_process.js:899:16)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:226:5)

apt seems to work fine as far as I can tell, installing other things like curl works. That flag doesn't seem to though, not sure why.
What's the best Ubuntu version to use here, maybe that's the issue? This is for AWS ec2, so currently trying the 14.04 AMI.

14.04 is not supported
Sent from my iPhone

On Dec 11, 2018, at 16:50, Jason Rosenbaum<EMAIL_ADDRESS>wrote:
Hey folks, trying to install on Ubuntu 14.04, getting this error:
ubuntu@ip-10-1-0-33:~$ sudo sift install

<EMAIL_ADDRESS>sift-version: notinstalled

Installing and configuring SaltStack properly ...
Command failed: apt-get install -y --allow-change-held-packages salt-minion
E: Command line option --allow-change-held-packages is not understood
Error: Command failed: apt-get install -y --allow-change-held-packages salt-minion
E: Command line option --allow-change-held-packages is not understood
at ChildProcess.exithandler (child_process.js:205:12)
at emitTwo (events.js:106:13)
at ChildProcess.emit (events.js:194:7)
at maybeClose (internal/child_process.js:899:16)
at Process.ChildProcess._handle.onexit (internal/child_process.js:226:5)

apt seems to work fine as far as I can tell, installing other things like curl works. That flag doesn't seem to though, not sure why.
What's the best Ubuntu version to use here, maybe that's the issue? This is for AWS ec2, so currently trying the 14.04 AMI.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Gotcha -- what should I use then? 18.04 didn't work either (read here that wasn't supported). What's the recommended distro?

16.04 is the only supported, 18.04 is being worked on.
Sent from my iPhone

On Dec 11, 2018, at 17:00, Jason Rosenbaum<EMAIL_ADDRESS>wrote:
Gotcha -- what should I use then? 18.04 didn't work either (read here that wasn't supported). What's the recommended distro?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Ok great, will try 16.04 then!

Hm, got a lot farther on the install this time, but still failed out. Not quite sure what I'm looking at in terms of how to debug:
>> Incomplete due to Failures -- Success: 529, Failure: 11

>>>> List of Failures (first 10 only)

     NOTE: First failure is generally the root cause.

     IMPORTANT: If opening a ticket, please include this information.

      - ID: sift-config-terminal-profiles-install
        SLS: sift.config.user.terminal
        Run#: 474
        Comment: Command "dconf load /org/gnome/terminal/ < /usr/share/sift/terminal-profiles.txt" run
      - ID: sift-config-theme-gtk
        SLS: sift.config.user.theme
        Run#: 477
        Comment: Command "gsettings set org.gnome.desktop.interface gtk-theme Arc" run
      - ID: sift-config-theme-icon
        SLS: sift.config.user.theme
        Run#: 478
        Comment: Command "gsettings set org.gnome.desktop.interface icon-theme Arc-Icons" run
      - ID: sift-config-unity-launcher-user
        SLS: sift.config.user.unity
        Run#: 485
        Comment: Command "gsettings set com.canonical.Unity.Launcher launcher-position Bottom" run
      - ID: sift-config-unity-disable-scopes
        SLS: sift.config.user.unity
        Run#: 486
        Comment: Command "gsettings set com.canonical.Unity.Lenses disabled-scopes "['more_suggestions-amazon.scope', 'more_suggestions-u1ms.scope', 'more_suggestions-populartracks.scope', 'music-musicstore.scope', 'more_suggestions-ebay.scope', 'more_suggestions-ubuntushop.scope', 'more_suggestions-skimlinks.scope']"" run
      - ID: sift-config-unity-favorites
        SLS: sift.config.user.unity
        Run#: 487
        Comment: Command "gsettings set com.canonical.Unity.Launcher favorites "['application://gnome-terminal.desktop', 'application://firefox.desktop', 'application://org.gnome.Nautilus.desktop', 'application://unity-control-center.desktop', 'unity://running-apps', 'unity://expo-icon', 'unity://devices']"" run
      - ID: sift-config-unity-icon-size
        SLS: sift.config.user.unity
        Run#: 488
        Comment: Command "dconf write /org/compiz/profiles/unity/plugins/unityshell/icon-size 32" run
      - ID: sift-config-unity-minimize-window
        SLS: sift.config.user.unity
        Run#: 489
        Comment: Command "dconf write /org/compiz/profiles/unity/plugins/unityshell/launcher-minimize-window true" run
      - ID: undefined
        SLS: sift.config.user
        Run#: 490
        Comment: One or more requisite failed: sift.config.user.theme.sift-config-theme-icon, sift.config.user.unity.sift-config-unity-icon-size, sift.config.user.unity.sift-config-unity-favorites, sift.config.user.terminal.sift-config-terminal-profiles-install, sift.config.user.unity.sift-config-unity-launcher-user, sift.config.user.unity.sift-config-unity-disable-scopes, sift.config.user.theme.sift-config-theme-gtk, sift.config.user.unity.sift-config-unity-minimize-window
      - ID: undefined
        SLS: sift.config
        Run#: 547
        Comment: One or more requisite failed: sift.config.user.sift-config-user

nevermind, I got it with packages only mode and a python update, thank you!


Avoid resource fragmentation at the node level

This is copied from a thread on wg-batch slack channel
We schedule jobs on a set of nodes that vary in capacity (i.e. RAM, CPUs).  Ideally, a queued job should run on the smallest node with sufficient unused capacity, but it should not be admitted by Kueue if there is not such a node.
For example, suppose I have nodes 1=1CPU and 2=2CPU.  If I submit the following jobs:
Job A: 0.5 CPU
Job B: 1 CPU
Job C: 1.5 CPU
Job D : 0.5 CPU
I would like job A to be started on node 1, job B to be started on node 2, and job C to wait patiently until there's room on node 2.  Finally, when Job D is submitted, it should be admitted to schedule (ideally on node 1).
In testing the above scenario, if I define a single ClusterQueue with a default ResourceFlavor, containing in total 3 CPUs, Kueue will admit job C immediately, but once admitted, it will wait in a pending status.  Job D will not be admitted, because job C has allocated the remaining CPUs -- even though it is not using them, because it cannot cobble together capacity across multiple nodes.
One workaround is to instead create a ResourceFlavor for each individual node in my cluster, so that individual jobs are only admitted if there is capacity to run them on one of the individual nodes alongside whatever other jobs are already present?  E.g. above, I would create ResourceFlavors "small" and "large", with capacity of 1/2 CPUs, and label the nodes similarly.
Another solution is to group nodes of the same size as one flavor. This will work if the nodes sizes are multiple of the job sizes.
Both solutions above are fairly hacks, the first will not scale and the scale is too restrictive.
How can we enhance Kueue to solve this problem in a better way? For example, for each quota, I wonder if we can bubble up information related to available fragments

/cc @eskaug

Node resources awareness seems the concern of scheduler, but yes back and forth the allocation is a great waste. At the least, we should support something like https://github.com/kubernetes-sigs/kueue/issues/349 for basically.

Maybe the key point locates at the integration with scheduler(not blocking the optimizations in kueue). Currently we support semantic all-or-nothing allocation in kueue, if we can get the scheduling result from scheduler(maybe through the ongoing pod group api), we can suspend the job again for insufficient resources.

/lifecycle frozen


Make columns that must have values not nullable

I don't remember what specific columns should be made non nullable. So I would recommend revisit database structure and make changes if required.


Need tflite inference example for tiny llama in python

I have converted tiny llama to tflite by running ai_edge_torch/generative/examples/tiny_llama/convert_to_tflite.py . Is there an example of how to run inference in python using the generated tflite model? Thanks in advance.

Hi @rajendra-ml, Here's the documentation on how to run arbitrary .tflite files: https://ai.google.dev/edge/litert/inference#run-python let me know if you have any additional questions.

Thanks a lot, that helps. This issue can be closed now.

closed


Bug - Minor - PDP - Hero not fully in view

The hero should be always 100% of the viewport height minus the header. On the PDP we use the inpage navigation.
The height of the inpage navigation block needs to be removed from the height of the hero. This way the pause button #156 and scroll indicator remain in view.
Expected:

Result:

@TomaszDziezykNetcentric please approve this

confirmed as fixed
as tested on https://175-hero-not-fully-in-view--vg-volvotrucks-us-rd--netcentric.hlx.page/pdp

confirmed as fixed as tested on Safari and on iPhone 13 and Ipad 10 (browserstack)


fix(material-experimental/mdc-chips) adjust margins

mat-chip-grid goes inside a mat-form-field, and the 4px margin on the
individual chips is causing the mat-chip-grid element to be too tall,
and thus the form field is also too tall. Give the mat-chip-grid a
negative margin so that it fits in the form field. The same thing was
done for non-mdc chips:
https://github.com/angular/components/blob/master/src/material/chips/chips.scss#L195

Hey jelbourn@ and mmalerba! Would it be possible to merge this one?

@mmalerba PTAL


Find all missing number in array

Enter your question -
Find all missing number in array
you have given a array [1, n], you have to find the all missing number in the array
time complexity: O(n)
Sample input: [1 3 2 5 8 7 2 5]
output: [4 6]
Enter link to the question(if question belongs to any online platform) -
https://leetcode.com/problems/find-all-numbers-disappeared-in-an-array/description/
Tags for the question(eg - Array, Basic, Stack, etc.) -
Array, Searching

@SarthakKeshari
kindly assign this issue to me

@SarthakKeshari
Kindly assign this issue to me

@MohammedHassan07,
Kindly add your solution in "Arrays/1D Arrays" folder
Deadline - 04/10/2021
Gentle advice for futher Issues -
Medium and hard level questions will be given preference for Hacktoberfest tag.