Update Slate project status - Low maintenance

Update README and git templates

Is this likely to affect https://github.com/shopify/theme-scripts as well?

Yikes! This is a pretty big setback, hopefully, other solutions and tools will emerge. I really liked where Slate was headed. 😞

@dan-gamble theme-script is not affected. It is an individual utilities library that can be used independently without Slate.
@jonathanmoore Thank you for your feedback. This is not an easy decision either. With the increasing Slate usage week by week, as of late too, it is really hard for us to make this decision as well. However, it is also why we put it in low maintenance. We want to be better equipped when we do come back to Slate, so that we could tackle on the real, hard problems.

Hire more talented developers as the ones you already have :) Anyway, hope you'll get back to it soon, this is a pretty ambitious & great project that I'll still use.


Add Beam 2.29.0 release blog post

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

[ ] 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.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

http://apache-beam-website-pull-requests.storage.googleapis.com/14562/blog/index.html
http://apache-beam-website-pull-requests.storage.googleapis.com/14562/blog/beam-2.29.0/index.html

Thanks Kenn! Does it make sense to include BEAM-11772 in the I/O section?

Thanks Kenn! Does it make sense to include BEAM-11772 in the I/O section?

Done


[cffLib] Fix encoding of Notice and Copyright values in XML on Python 2

In Notice and Copyright values, Latin-1 characters such as '©' or 'ç' are replaced with U+FFFD on Python 2.7.9:
<?xml version="1.0" encoding="UTF-8"?>
<ttFont sfntVersion="OTTO" ttLibVersion="2.5">

  <CFF>
    <CFFFont name="SabonNextLTPro-Regular">
      <version value="001.001"/>
      <Notice value="Copyright � 2002 - 2010 Linotype GmbH, www.linotype.com. All rights reserved. This font software may not be reproduced, modified, disclosed or transferred without the express written approval of Linotype GmbH. Sabon is a trademark of Linotype Corp. registered in the U.S. Patent and Trademark Office and may be registered in certain other jurisdictions in the name of Linotype Corp. or its licensee Linotype GmbH. This typeface is original artwork of Jean Fran�ois Porchez. The design may be protected in certain jurisdictions."/>
      <Copyright value="Copyright � 2002 - 2010 Linotype GmbH, www.linotype.com. All rights reserved. This font software may not be reproduced, modified, disclosed or transferred without the express written approval of Linotype GmbH. Sabon is a trademark of Linotype Corp. registered in the U.S. Patent and Trademark Office and may be registered in certain other jurisdictions in the name of Linotype Corp. or its licensee Linotype GmbH. This typeface is original artwork of Jean Fran�ois Porchez. The design may be protected in certain jurisdictions."/>

I think it should be as below, like on Python 3.4.0:
<?xml version="1.0" encoding="UTF-8"?>
<ttFont sfntVersion="OTTO" ttLibVersion="2.5">

  <CFF>
    <CFFFont name="SabonNextLTPro-Regular">
      <version value="001.001"/>
      <Notice value="Copyright © 2002 - 2010 Linotype GmbH, www.linotype.com. All rights reserved. This font software may not be reproduced, modified, disclosed or transferred without the express written approval of Linotype GmbH. Sabon is a trademark of Linotype Corp. registered in the U.S. Patent and Trademark Office and may be registered in certain other jurisdictions in the name of Linotype Corp. or its licensee Linotype GmbH. This typeface is original artwork of Jean François Porchez. The design may be protected in certain jurisdictions."/>
      <Copyright value="Copyright © 2002 - 2010 Linotype GmbH, www.linotype.com. All rights reserved. This font software may not be reproduced, modified, disclosed or transferred without the express written approval of Linotype GmbH. Sabon is a trademark of Linotype Corp. registered in the U.S. Patent and Trademark Office and may be registered in certain other jurisdictions in the name of Linotype Corp. or its licensee Linotype GmbH. This typeface is original artwork of Jean François Porchez. The design may be protected in certain jurisdictions."/>

it seems to be a Windows-only issue. At least, I wasn't able to replicate it neither on OS X nor on Linux.
When I tried to dump the CFF table of SabonNextLTPro-Regular like in your example, with fontTools checked at the latest HEAD, and using python 2.7.10 on OS X, or 2.7.6 on Ubuntu, the copyright characters were correctly encoded...

Sorry, forget what I just wrote...
I was misled by Sublime Text which automatically opened my ttx file using "Windows 1252" encoding (it was too clever!). When I force Sublime to reopen the file with UTF-8 encoding, the bug shows up in OS X and Linxu as well.
Because xmlWriter only supports writing in UTF-8, the input it receives must be either utf-8 encoded bytestrings, or unicode literals (which are then encoded as utf-8 inside the escape function).
the problem is the cffLib module is sending latin-1 encoded bytestrings. Because the latter are already instances of bytes, the tostr function (alias of py23.tobytes when running in python 2) does not encode them to utf-8, but leaves them encoded as latin-1.
@mashabow patch correctly fixes this, by sending the xmlWriter unicode literals, and by skipping str coercion if the value is already an instance of bytes or unicodes.

Thanks!
PS. I found that I also have a misunderstanding in my first comment:

Latin-1 characters such as '©' or 'ç' are replaced with U+FFFD on Python 2.7.9

It is not true. xmlWriter writes Latin-1 characters as raw bytes (e.g. 0xA9 for '©'). Because they are invalid as UTF-8, of cource, then my editor (Atom) replaces them with U+FFFD.

Well, it's a tricky issue. The strings inside the CFF Font Info structures are "PostScript strings", and, strictly speaking, these strings should only contain ASCII characters. Non-ASCII character codes should be escaped via an octal code (\ddd), and, strictly speaking, the interpretation of those is undefined (i.e. it’s not specified anywhere which “encoding” they represent). Strictly speaking, raw bytes with values >127 are not permitted at all there.
By convention, both the \ddd escaped codes and the (technically invalid) raw bytes with values >127 are best interpreted as ISO 8859-1 (ISO Latin 1).
Since the first 256 codes of ISO Latin 1 are identical to Unicode, the \ddd octal code convention could probably be extended to cover all of Unicode (using \dddd etc.), but I haven’t tested it.
Also, parentheses are dangerous. When the CFF is converted to a Type 1 (which happens in some printing environments), ) is actually interpreted as a terminator for the string, and only “by luck” a pair of parentheses as in (c) is interpreted as part of text. If one makes a CFF where the Copyright string contains only a closing paranthesis, such fonts may not print on some devices (it’s really funny).
However, parentheses can be escaped using \, i.e. \(, \) an \\ are permitted in PostScript strings.
The new FontLab codebase obeys the following guidelines within all Type 1 and CFF “Font Info” strings, such as Copyright or Notice:
Reading

\\ should be interpreted as “\”
\( should be interpreted as “\”
\) should be interpreted “)”
\ddd should be interpreted as octal numerical code in ISO 8859-1 encoding
all other ASCII characters should be interpreted as is
non-ASCII characters should be interpreted as ISO 8859-1
when inside a string an unescaped ( is found, it should be interpreted as “(”, and the matching closing ) should also be interpreted as “)” rather than as a terminator for the string

Writing

Only ASCII characters are permitted
The following characters must be escaped: “(” → \(, “)” → \), “\” → \\
The following characters should be unescaped:  !"#$%&'*+,-./0123456789:;<=>?@ABCDEFGHIJKLMNOPQRSTUVWXYZ[]^_abcdefghijklmnopqrstuvwxyz{|}~`
Three characters should undergo a special treatment: “©” → \(C\), “®” → \(R\) and “™” → \(TM\)
All other characters should be removed

Ps. It’s even more tedious. It seems that Adobe’s MakeOTF these days writes UTF-8 into the CFF Font Info fields. Which means, if I have 0xA9 in the CFF Notice field, a tool can interpret it as the ISO 8859-1 code for “©” but then another tool may write the UTF-8 representation of “©” into CFF, which is 0xC2 0xA9. Then, the tool that thinks it’s ISO 8859-1 will interpret that as “Â©”.

How about the following?

when reading such a string, check whether it is structurally valid UTF-8. If yes, decode as UTF-8; otherwise, decode as ISO-8859-1. Finally, unescape PostScript sequences such as \(.
when writing such a string, first escape for PostScript (replace ( by \(, etc.) Then, check whether it contains any characters above U+00FF. If yes, encode as UTF-8; otherwise, encode as ISO-8859-1.

I’ve seen that logic a few times in codecs for other ancient file formats, unrelated to fonts. This heuristic can obviously fail, but it works reasonably well in practice because Â© or Â£ are extremely rare.
It’s pretty easy to check whether a byte sequence is structurally valid UTF-8; see section 4 of RFC 3629 for a grammar.
One complication to keep in mind: A lot of libraries claim to write UTF-8 but actually they produce an esoteric encoding called http://www.unicode.org/reports/tr26/. There, codepoints above U+0FFFF are first encoded as UTF-16 surrogate pairs, and then the surrogates are sent separately to a naive UTF-8 encoder. The result is in explicit violation of the UTF-8 spec, but a lot of conversion code is doing it. When fonttools is going to decode UTF-8, it would be definitely worth testing how CESU-8 gets handled. (Probably Python’s UTF-8 codec will handle this transparently, which would be great; but I’d still test it).

@twardoch thanks for your investigation!

It seems that Adobe’s MakeOTF these days writes UTF-8 into the CFF Font Info fields.

Where did you get this information? I checked the afdko source code and could not find any reference of MakeOTF or any other FDK module re-encoding input strings to UTF-8 when writing CFF Font Info fields, neither in the python scripts nor in the C source files.
Are you sure it isn't something else which is passing UTF-8 bytes to MakeOTF, and the latter simply writes them like that in the CFF Font Info fields?
I like the idea of adding support for PostScript escape sequences when parsing/writing strings from/to CFF binary table (of course, the TTX output should always be written as UTF-8).
@brawer to test whether the input is structurally valid UTF-8, wouldn't it be enough to see if it raises or not a UnicodeDecodeError?

True, it looks it’s not FDK after all. I've noticed that Glyphs writes Unicode characters into the CFF copyright string as UTF-8 while RoboFont filters all non-ASCII out. So it seems like the tools do different things themselves when preparing the data that is then passed to MakeOTF.

If, like Adam says, the established convention is to interpret both non-ASCII characters in CFF copyright strings (either octal \ddd escaped codes or as raw bytes >127) as ISO 8859-1 (ISO Latin 1), and if Glyphs is the only editor which writes UTF-8 bytes in CFF, while others like Robofont or Fontlab filters out non-ASCII characters, wouldn't it be better to raise the issue with Glyphs app (cc @schriftgestalt), instead of allowing fontTools to also write such UTF-8 bytes in the binary CFF table?
by the way, ufo2fdk filters out not just  () parentheses, but also [](){}<>/% when normalising string for PostScript:
https://github.com/typesupply/ufo2fdk/blob/master/Lib/ufo2fdk/fontInfoData.py#L166

I fixed it. Thanks for the notice. It should follow Adams recommendations.

Thanks Georg! :+1:


Move wiki to documentation and clean up

Description
Moving the wiki  to the documentation  and clean up the content.
Related #750
Acceptance criteria

Tasks

[ ]

@christianbrb for this we have to evaluate what makes sense to be part of the documentation. I would suggest that the parts that are targeting a developer using the SDK would become part of the docs and the parts that for the developer developing the SDK should probably stay in the wiki


update headers for cl_intel_unified_shared_memory

The Khronos OpenCL-Headers repo was recently changed, see KhronosGroup/OpenCL-Headers#184.
This library should be updated to incorporate this change.

Incorporated in version 0.6.4.


Replaced dependency on immutable with Object.assign + lodash.isequal

Here is a new PR to address issue #41. I created a new branch instead of re-using the former PR branch in order not to pollute the project history.

@laaglu Awesome, thank you! I will update the docs and do a release today


Sema: Don't look at SubstitutedType in CSDiag

The code here is unprincipled, and mixes archetypes from
multiple sources:

The callee's generic environment

The caller's generic environment

Any random archetypes appearing in the original types of
typealiases, which could come from any generic environment

Initially, an UncurriedCandidate's type starts out as #1,
and then we sometimes set it to type #2 if the candidate is
a method on a base class.
We get #3 by virtue of walking the original types of
SubstitutedTypes created as part of dependent member type
substitution.
However, it turns out the real reason the SubstitutedType
walk existed was so that #2 archetypes that appear in the
UncurriedCandidate's type map to themselves. This doesn't
require looking at the original type of a SubstitutedType
at all; instead we just walk the UncurriedCandidate's type
normally, collecting archetypes.
If we do this, the code doesn't (erroneously) pick up random
archetypes from typealiases, and this changes the output in
the RangeDiagnostics test. While we can debate if the new
diagnostics are better or worse (I think it's either a wash,
or slightly better) the reason they changed is because more
in-depth diagnostic code is now executing, without breaking
things randomly as before. I suspect this is a good thing.

@swift-ci Please smoke test

@swift-ci Please smoke test Linux

@swift-ci Please smoke test

@swift-ci Please smoke test


Fixed: readme

Fixed: readme

Codecov Report

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

@@          Coverage Diff          @@
##           master    #51   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           3      3
  Lines         165    165
  Branches       24     24
=====================================
  Hits          165    165

Continue to review full report at Codecov.

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

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add ok-to-test trigger for functional tests

Description

Re-adding ok-to-test to trigger functional tests

Type of change

This pull request is a minor refactor, code cleanup, test improvement, or other maintenance task and doesn't change the functionality of Radius (issue link optional).

Fixes: #5632
Auto-generated summary

🤖 Generated by Copilot at fcbb71f
Summary
:no_entry_sign::clock1::point_right:

Remove pull_request trigger and add ok-to-test event for functional tests. This allows maintainers to manually run the tests on a pull request by sending a repository_dispatch event with the pull request number.

pull_request gone
functional tests run on demand
ok-to-test cuts

Walkthrough

Remove pull_request trigger and add ok-to-test event type for repository_dispatch trigger in functional test workflow (link). This allows maintainers to manually run the functional tests on a pull request by sending a repository_dispatch event with the ok-to-test type and the pull request number as the payload. The workflow file is .github/workflows/functional-test.yaml.

/ok-to-test

/ok-to-test

What is this change needed for? I added the ok-to-test comment to your PR which triggered https://github.com/radius-project/radius/actions/runs/6550667187.

/ok-to-test


fix: Adding referrerPolicy: "no-referrer" to logo images.

Some wallets feature a logo that won't resolve due to a 403 forbidden error. As an example, the fin wallet has a logo that points to:

https://lh3.googleusercontent.com/oKvetnosutN7Q7daICbew7k9kLLB1unNlggkA-mZZZZIdNWp7vJFpz5q9MZ73Wlw5S5KrHV5mEQ0UEKYaDWz5qmacHw=w128-h128-e365-rj-sc0x00ffffff.

Visiting the URL for this image directly works just fine. However, when this URL is used in  <img> tag, a 403 forbidden error results. Here are some screenshots for reference:

This issue seems to have been resolved for some Chakra UI components (https://github.com/chakra-ui/chakra-ui/issues/5909) but not for the one used to render the logo for each wallet.
This PR adds the referrerPolicy: "no-referrer" to logo images so that wallets using images served by googleusercontent.com will render as intended.

oh great! I didn't know about this issue at all, thanks for contributing!

seems like it's also good to add this by default to the Avatar component

I will batch this into release early next week!


Rewrite AssetStorage to the new ktx-async API

[x] Add ktx-assets-async module.
[x] AssetStorage rewrite.
[x] Usage documentation.
[x] Restore ktx-freetype-async, update to new API.
[x] Restore TextAssetLoader, move to ktx-assets.

AssetStorage was removed during the ktx-async overhaul in 1.9.9-b1 due to reliance on old coroutines API and threading issues. The goal of this task is to rewrite AssetStorage in a fully thread-safe way, without reliance on specific coroutines dispatchers for asynchronous tasks.

I'm making changes to a project I created two years ago in which I used the previous version of ktx-async which depended on the experimental coroutines, and specifically AssetStorage.
I see that AssetStorage is no longer available in KTX.
Would you say the wise move right now is to switch to using ktx-assets's AssetManager instead of AssetStorage?

@amitkot For now, yes. There were some issues with thread safety and unstable coroutines API usage, and I'm afraid AssetStorage needs a complete rewrite. Sorry.

@Quillraven @cypherdare @Jkly @maltaisn @jcornaz
I finally decided to rewrite the good old AssetStorage. While I still have to write usage examples and update ktx-freetype-async, I this point I'm happy with the AssetStorage implementation and its test suite.
To be honest, this is probably among top 5 most fun, challenging and complex pieces of code I've written. The initial implementation would definitely not pass the extensive tests I've written this time, including concurrent loading and unloading assets on multiple threads and coroutines.
I think the current implementation is fully thread-safe and non-blocking. Unless the user of the API blocks the rendering or loading threads manually with runBlocking and whatnot, it should handle anything you throw at it without deadlocks or unexpected exceptions. I've run hundreds of thousands of concurrent loading and unloading requests of assets with nested dependencies - I encourage you to play with should handle stress test test in storageTest.kt to try to break it.
Anyway, I would appreciate it very much if you took the time to try AssetStorage out as a replacement for AssetManager or even go through the sources and do some code review. I know it's a huge pull request, but even superficial review would help a lot. If you're interested and have some time:

ktx-assets-async - KTX module.
#258 - pull request.
ktx-assets-async version 1.9.10-SNAPSHOT should be available in the snapshots repository soon.

I'll leave the pull request open awaiting suggestions while I focus on the documentation and ktx-freetype-async.
Thanks.

ktx-freetype-async is now updated to the new AssetStorage API and available via snapshot releases.

I have finished the documentation and written all of the usage examples that I've planned. Let me know if anything is missing.

Ironically, after implementing this I found this obscure feature called LoadedCallback in AssetLoaderParameters, where you can actually add a callback that will be invoked after the asset is loaded by a AssetManager, which means coroutines support can be added to the original LibGDX manager relatively easy (well, easier than writing AssetStorage was).
AssetStorage still makes sense, of course, as it allows for true parallel asset loading, but I'm thinking we could also add coroutines support to AssetManager. loadAsync extension could return a Deferred reference to an asset that you can await for.

I believe the current AssetStorage available on develop is feature-complete and can be used as a replacement for AssetManager for the majority of use cases. @Quillraven also reported notable performance improvements, especially on mobile (see discussion in #258).
I want to thank @Quillraven again for testing and review of the new asset manager. I'd ship a lot less features and a clunky API if not for the input and feature requests.
Since there were no further comments, I'm closing this issue for now. If you'd like you make a feature request or comment regarding the AssetStorage API, use issue #259.

@czyzby I have been on a bit of LibGDX hiatus for the past three months. I looked this over briefly and it looks slick. I don't currently have a project in need of this feature to try it out in currently, though.


Module support fix

Allows for import "FLIP" in Webpack and other module systems.

I signed it!

@paullewis by any chance?


Add piyush1594 for access

Add user piyush1594 for access

Can one of the admins verify this patch?

[test]


Jenkins security setting is lost after service restart

I have configured my jenkins instance in ubuntu 12.04 using
  class { '::jenkins::security':
    security_model => 'full_control',
  }

which sets the settings to use user/password correctly, but  if I restart the service these settings are lost, giving the world full access without user/password...
Is this by design?

restarting sets the security back to what's in your config.xml. It seems to me that jenkins does not correctly persist the setting. Seems like a jenkins bug to me.

@iax7 / @Xylakant Could either of you try to reproduce this issue with the current master and reopen this issue if it is still present?


Fix test file for the trustyai image

I don't really understand how and why this file was broken by this commit aac06623b4942d9ceb6ea8ef5189639f4943fb1b . Our CI check notifies that something is broken in the file.
Please, review this thoroughly, I simply put back original values.
Looks like this issue isn't present in the upstream repository - https://github.com/opendatahub-io/notebooks/blob/main/jupyter/trustyai/ubi9-python-3.9/test/test_notebook.ipynb?short_path=7b4adf3.
Edit: CI passed the point of checking JSON files. The hadolint failure is gonna be fixed by #179.

/lgtm
/approve
thanks :100:


Eloquent relationship belongs to not working

I am trying to display data from a joined record, but I cannot get it to show the name. The code below doesn't work at all, but if I change the JS data attribute from 'company.name' to 'company' - I get the ID, but i need the name.
Controller:
namespace App\Http\Controllers;

use App\Licences;
use Yajra\DataTables\DataTables;

class DataTableController extends Controller
{
    private $dataTable;

    public function __construct(DataTables $dataTable)
    {
        $this->dataTable = $dataTable;
    }

    public function licences()
    {
        $licences = Licences::with('Company')->select('licences.*');
        return $this->dataTable->eloquent($licences)->make(true);
    }
}

JS:
jQuery(function($) {
        $('#licences-table').DataTable({
            processing: true,
            serverSide: true,
            ajax: '{!! route('datatable.licences') !!}',
            columns: [
                { data: 'identifier', name: 'identifier' },
                { data: 'company.name', name: 'company.name' },
            ]
        });
    });

This is based on the demo here https://github.com/yajra/laravel-datatables-demo/blob/master/app/Http/Controllers/RelationController.php

Better - its working with : DB and tables joining.


Sdc.com

Backstory
Went to website e found delete acount under acc settings
Data

name: Sdc.com
url: www.sdc.com/react/#/settings2
url_CODE:
difficulty: easy
notes:
email:
email_subject:
email_body:
domains:

Thanks for your contribution!


opens powershell

Windows Terminal version
No response
Windows build number
No response
Other Software
No response
Steps to reproduce
when i open the app it opens powershell instead
Expected Behavior
i was expecting it to open terminal
Actual Behavior
opened powershell

PowerShell runs inside Terminal.
Terminal is this part:

But there's no customisations on it?. It looks the same as.cmd
On Fri, 29 Sept 2023, 17:46 Dustin L. Howett, @.***>
wrote:

PowerShell runs inside Terminal.
Terminal is this part:
[image: image]
https://user-images.githubusercontent.com/189190/271667064-17858a9f-7bd9-4dee-be82-da2d84dc9fc6.png
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/terminal/issues/16064#issuecomment-1741199431,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASR5ZDWPS2QCYYXATWLHLKDX433ORANCNFSM6AAAAAA5MWLLSE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Can you clarify/? How did you launch the Terminal? Can you share a screenshot? (protip: Github lets you upload images just by pasting into the comment box)

I cant take a screen shot as I'm not near my computer. I launched it by
typing terminal in the Windows search bar, then it would just come up with
a default powershell window(powershell is my default thing, I can also
change it to cmd and stuff,but that made no difference)
I also then downloaded the preview terminal thing from Microsoft store but
that was no different.
On Fri, 29 Sept 2023, 19:34 Mike Griese, @.***> wrote:

Can you clarify/? How did you launch the Terminal? Can you share a
screenshot? (protip: Github lets you upload images just by pasting into the
comment box)
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/terminal/issues/16064#issuecomment-1741327672,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASR5ZDWUJHOI54DC2LIEPJDX44IELANCNFSM6AAAAAA5MWLLSE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Once you open the terminal, you have to click on this upside down caret sign.

Next, navigate to the Settings tab, and open the settings window.
Under "Default Terminal application" choose "Terminal".
And under "Default profile" choose which terminal profile you want to open when you run the terminal application.

I hope that has helped.

Thanks.
On Sat, 30 Sept 2023, 12:33 Igor Tadić, @.***> wrote:

Once you open the terminal, you have to click on this upside down caret
sign.
[image: image]
https://user-images.githubusercontent.com/97162709/271770353-e0acb2ff-9009-47a0-907d-265925166d47.png
Next, navigate to the Settings tab, and open the settings window.
Under "Default Terminal application" choose "Terminal".
And under "Default profile" choose which terminal profile you want to open
when you run the terminal application.
[image: image]
https://user-images.githubusercontent.com/97162709/271770461-66e0a36a-8882-4be3-bc26-bf15d92b8f86.png
I hope that has helped.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/terminal/issues/16064#issuecomment-1741745977,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASR5ZDU4UPAC5C5TRJTF2O3X477SLANCNFSM6AAAAAA5MWLLSE
.
You are receiving this because you authored the thread.Message ID:
@.***>

(That sounds like it answered your question - feel free to reply back here if it didn't)


[BUG] - Delete an image inside a DependencyContainer gives a 404

When add an Nova Image field inside the DependencyContainer and you try to delete an image in the edit entry screen then it does  perform a DELETE request but it gives a 404. Uploading a new image to overwrite the existing one does work .

Any news/workarounds on this?
I'm having the same issue with the nova File field.


Is this project maintained?

?

Ah, nvm, I see some recent commits. But just no replies about failed to import plain ESM code.

https://github.com/esm-dev/esm.sh/issues/729

This is a big effort to get all NPM stuff working. I hope it'll work out!


(maint) Move late-project? from queries to columns

This commit moves the late-project? field from the query-recs to
specific columns in the query-recs so that the specific fields that
queries can't do without are called out specifcally. The field is also
renamed to must-have? to align the name with the purpose.

ping @wkalt so here's a start at least

I imagine that we could could move from this to doing something like a late-join (keep the fact_value_id as a must-have field, do the group-by, and then a join of that whole thing with type/path.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1064/

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1062/

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1065/

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1063/

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1066/

@pljenkinsro retest this please

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1067/

I'm +1 on this -- makes things a bit more explicit. Ultimately we'll need to handle the underlying issue of late-project (which is relevant to group-by) by pushing all projections to the database and getting rid of the must-have flag. I've created https://tickets.puppetlabs.com/browse/PDB-1447 to cover this work.

Generally lgtm, just some small comments.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1076/

@pljenkinsro retest this please

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1077/

@mullr updated per your comments


Issue with directory separators

I use Windows as my dev environment. When I execute vendor/bin/vendor-patches generate it creates extra.patches entries like this in the composer.json:
"vendor/foobar": [
    "patches\\path-to-php.patch"
]

However, this will not work when deploying this to the live environment, which typically is not a Windows environment 😄
Directory separators in the composer.json should always be a forward slash, so that it works both on Windows and Unix based environments.

Thanks for feedback 🙂
Could you fix it here?
https://github.com/symplify/vendor-patches/blob/main/src/PatchFileFactory.php#L26

Yes, see #20


Added all jupyter notebooks to Travis

Last time I only added the BASIC and Cheryl notebook to the notebooks that are evaluated in Travis, now added all notebooks in the ipynb folder...
While testing on my local computer some notebooks did not fully run. If this is appreciated I might have some time to look into the possible issues next week. Let me know if this is something you want!

Related to this: some notebooks can only run in Python 2, others only in Python 3. Any preference for having them all in Python 3, and/or runnable in Python 2 by importing things from the future package?

Ah, that's the problem with having some notebooks from many years ago. My preference would be to do what is best for the user, and since we don't know what version a usert will be running, having them all run in both versions is best, if that's not too complicated. I think from __future__ import print_function should be used everywhere. The other major complication would be things like map being a list versus an iterator -- that might be more complicated to fix up.


Computation with where condition

Steps to reproduce
I have 2 measures as defined below:
m["test"] = tt.agg.mean(m["Value.SUM"] * 3, scope=tt.scope.origin(l["Hier1"]))

m["test2"] = tt.agg.mean(
    tt.where(
        l["Hier1"] == "001", # if condition
        m["Value.SUM"] * 10, # then
        tt.where(l["Hier2"] == "36yut", m["Value.SUM"] * 100, m["Value.SUM"] / 2), # Else. More "where" condition can be embedded under the else condition
    ),
    scope=tt.scope.origin(l["Hier1"]),
)

m["test"] computed as expected. l["hier1"] is computed using the formula m["Value.SUM"] * 3 as the scope is l["Hier1"].
However, m["test2"] value for l["hier1"] (in orange) is computed as the mean instead of using the conditional formula.

With another measure defined as below:
m["test3"] = tt.where(
    l["Hier1"] == "001",
    tt.agg.mean(m["Value.SUM"] * 50, scope=tt.scope.origin(l["Hier2"])),
    m["Value.SUM"] * 100,
)

How do we explain how the 2 values (circled in red) at the "Hier1" level is being derived?

Is this the expected behavior?
Environment

atoti: <IP_ADDRESS>11011091646
Python: 3.9.7
Operating System: win32

Hello,
I think this is fixed in atoti 0.7.0:
import atoti as tt
import pandas as pd
session = tt.Session()
table = session.read_pandas(
    pd.DataFrame({"h1": ["001", "001", "002", "002"], "h2": ["a", "b", "x", "y"], "value": [1,1,1,1]}),
    table_name="Test"
)
cube = session.create_cube(table)
h, l, m = cube.hierarchies, cube.levels, cube.measures
m["test"] = tt.agg.mean(m["value.SUM"] * 3, scope=tt.OriginScope(l["h1"]))
m["test2"] = tt.agg.mean(
    tt.where(
        l["h1"] == "001",
        m["value.SUM"] * 10,
        tt.where(l["h2"] == "x", m["value.SUM"] * 100, m["value.SUM"] / 2)
    ),
    scope=tt.OriginScope(l["h1"])
)
m["test3"] = tt.where(
    l["h1"] == "001",
    tt.agg.mean(m["value.SUM"] * 50, scope=tt.OriginScope(l["h2"])),
    m["value.SUM"] * 100,
)

@HuifangYeo if you cannot reproduce this too can you close this issue ?


Function return

Describe the bug
Functions don't properly resume from where called. Causes issues with recursive functions, like fractal trees
To Reproduce
Simple fractal tree:
move 375, 500
let colors =[RED, BLUE, GREEN, YELLOW]
let angle =30
rotate 90
fun y(level){
if(level >0){
color colors[level %4]
forward level10
rotate 360-angle
y(level-1)
rotate 2angle
y(level - 1)
rotate 360-angle
backward level *10
}
}
y(10)
Expected behavior
Should make a fractal tree, not a simple spiral.
Screenshots
If applicable, add screenshots to help explain your problem.
Smartphone (please complete the following information):

Device: moto pure
OS: Android 10
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Hello @kseiders,
Thank you for your report, I will test it and fix it in next release
Thanks
Amr Hesham

The bug is solved now :D, your code will work as expected in Release 1.3.1
Demo
https://user-images.githubusercontent.com/23631699/191572988-934b09a9-7025-4444-94ec-2ed849f67610.mp4


Server-Side Caching Support

This seems to be only a client-side caching mechanism - the only server-side approach doesn't seem to be supported on .NET Core yet. Can we get some clarification here that this does not cache on the server?
Reference: https://github.com/filipw/Strathweb.CacheOutput/issues/217

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

ID: b6728e59-93ec-891e-0416-94819296ad72
Version Independent ID: 5d581acc-c651-2112-844a-6b85dc5a99a6
Content: Response caching in ASP.NET Core
Content Source: aspnetcore/performance/caching/response.md
Product: aspnet-core
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Hello @alohaninja ... That's correct. It's a client-side cache that conforms to the HTTP spec, not a server-side cache. The request for server-side caching is at https://github.com/aspnet/Home/issues/2612. I'm going to close here, since this isn't actionable for the docs. You can ask the engineers further about it if you wish over on that issue.


Update to liberty <IP_ADDRESS>

Signed-off-by: Lee Surprenant<EMAIL_ADDRESS>

windows test failed, seems caused by windows encoding issue.
FFDC1015I: An FFDC Incident has been created: "com.fasterxml.jackson.core.JsonParseException: Invalid UTF-8 start byte 0x93
at [Source: (byte[])"
{
"openapi": "3.0.0",
"info": {
"title": "Simplified FHIR API",
"description": "A simplified version of the HL7 FHIR API",
"version": "4.0.1"
},
"servers": [
{
"url": "/fhir-server/api/v4"
}
],


Tuple Index Out of Range

Hello, when I try to run the script bellow I get an IndexError: tuple index out of range and I am not quite sure why.
`from pytorch_grad_cam import GradCAM, ScoreCAM, GradCAMPlusPlus, AblationCAM, XGradCAM, EigenCAM
from pytorch_grad_cam.utils.image import show_cam_on_image
trans=transforms.ToTensor()
model = model
target_layer = model.fc
input_tensor =trans(resize_image(image[14][0]))
input_tensor=input_tensor.unsqueeze(0)
input_tensor=input_tensor.to('cuda')
cam = GradCAM(model=model, target_layer=target_layer, use_cuda='args.use_cuda')
target_category = None
grayscale_cam = cam(input_tensor=input_tensor, target_category=target_category)
grayscale_cam = grayscale_cam[0,:]
visualization = show_cam_on_image(rgb_img, grayscale_cam)`
I then get the following error traceback:
`IndexError: tuple index out of range
IndexError                                Traceback (most recent call last)
 in
22
23 # You can also pass aug_smooth=True and eigen_smooth=True, to apply smoothing.
---> 24 grayscale_cam = cam(input_tensor=input_tensor, target_category=target_category)
25
26 # In this example grayscale_cam has only one image in the batch:
~/anaconda3/lib/python3.7/site-packages/pytorch_grad_cam/base_cam.py in call(self, input_tensor, target_category, aug_smooth, eigen_smooth)
127
128         return self.forward(input_tensor,
--> 129             target_category, eigen_smooth)
~/anaconda3/lib/python3.7/site-packages/pytorch_grad_cam/base_cam.py in forward(self, input_tensor, target_category, eigen_smooth)
75
76         cam = self.get_cam_image(input_tensor, target_category,
---> 77             activations, grads, eigen_smooth)
78
79         cam = np.maximum(cam, 0)
~/anaconda3/lib/python3.7/site-packages/pytorch_grad_cam/base_cam.py in get_cam_image(self, input_tensor, target_category, activations, grads, eigen_smooth)
44                       grads,
45                       eigen_smooth=False):
---> 46         weights = self.get_cam_weights(input_tensor, target_category, activations, grads)
47         weighted_activations = weights[:, :, None, None] * activations
48         if eigen_smooth:
~/anaconda3/lib/python3.7/site-packages/pytorch_grad_cam/grad_cam.py in get_cam_weights(self, input_tensor, target_category, activations, grads)
14                         activations,
15                         grads):
---> 16         return np.mean(grads, axis=(2, 3))
<array_function internals> in mean(*args, **kwargs)
~/anaconda3/lib/python3.7/site-packages/numpy/core/fromnumeric.py in mean(a, axis, dtype, out, keepdims)
3333
3334     return _methods._mean(a, axis=axis, dtype=dtype,
-> 3335                           out=out, **kwargs)
3336
3337
~/anaconda3/lib/python3.7/site-packages/numpy/core/_methods.py in _mean(a, axis, dtype, out, keepdims)
136
137     is_float16_result = False
--> 138     rcount = _count_reduce_items(arr, axis)
139     # Make this warning show up first
140     if rcount == 0:
~/anaconda3/lib/python3.7/site-packages/numpy/core/_methods.py in _count_reduce_items(arr, axis)
55     items = 1
56     for ax in axis:
---> 57         items *= arr.shape[ax]
58     return items
59
IndexError: tuple index out of range`
The image that I am feeding into it is 3,128,128 in dimension and I have added a 4th dimension with tensor.unsqueeze(0) as it would not be fed into the model properly without this pseudo "batch index". I do not understand which tuple it is finding to be out of range.

Hi,
I think the issue is target_layer = model.fc
The target layer output should have spatial dimensions.
The gradient then is 1d and np.mean(grads, axis=(2,3)) fails because axis 2 and 3 (X and Y) don't exist.

Ah I see yea seems I was trying to pass it a linear layer at the end of the model rather than a convolution layer. Thanks!


Cluster-autoscaler: unregistered nodes in cluster state registry

Ref: #2228 #2229
cc: @jszczepkowski @fgrzadkowski

This change is

Just one comment, LGTM after applying it.


Lowercase fr

This is just a style thing, right? The regex should be case-insensitive.

@squeek502 Yeah, this was actually my bad - I searched for some of the French phrases, and saw one that wasn't RT'd. I thought it was because of some case-matching, but I just misread the timestamp on it (it was a month old, I thought it was new). All good, either way!

:smile:


Make the set of request headers sent to Mixer configurable

We should make the set of request headers sent by the proxy to Mixer to be configurable such as to reduce the payload size of proxy RPCs to Mixer.
For now, the specific set will need to be configured by hand. But in the future, this can be handled automatically by observing what Mixer actually consumes and configuring the proxy accordingly.

Issue moved to istio/proxy #1194 via ZenHub


Add tests for MagicNumber:ignoreLocalVariableDeclaration

This commit adds test cases for the missing config option, which was
added in PR #2123.

Codecov Report

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

@@            Coverage Diff            @@
##             master    #2125   +/-   ##
=========================================
  Coverage     80.62%   80.62%
  Complexity     2015     2015
=========================================
  Files           336      336
  Lines          5802     5802
  Branches       1057     1057
=========================================
  Hits           4678     4678
  Misses          563      563
  Partials        561      561

Continue to review full report at Codecov.

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


Issue with 4.0.1 version

I am using version 4.0.1 version of inputmask.We copier from the dist folder of release version. It is restricting us to enter the ssn and date field in iPhone or Safari.Please assist

@wototech ,
There is an encoding problem with 4.0.1 caused by uglify.  Use 4.0.2.  If that doesn't work.  Please create a codepen with an example so I can test it myself.
Robin

@wototech ,
Did it work out?

We are still having this issue

See #2016


https://www.prisma.io/ 404's

Bug description
The website is unavailable.
How to reproduce
Go to the website.
Expected behavior
No response
Prisma information
N/A
Environment & setup
N/A
Prisma Version
N/A

The website should be back up. Thanks for reporting.


⚠ Bofrost.si has degraded performance

In 22d4db5, Bofrost.si (https://www.bofrost.si) experienced degraded performance:

HTTP code: 200
Response time: 2495 ms

Resolved: Bofrost.si performance has improved in 5f917ce after 6 minutes.


berks upload error

i am getting dis error while doing berks upload not able to solve it can any one help me out

please change metadata version
in metadata.rb

thank u
Thanks & Regards
Spandana Thirupari
On Tue, Aug 6, 2019 at 7:03 AM NareshAbbanapuri<EMAIL_ADDRESS>wrote:

please change metadata version
in metadata.rb
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/QT-DevOps/DevOpsIssues/issues/558?email_source=notifications&email_token=AMYHIALLJXBUCVXA7DALY4DQDDIFTA5CNFSM4IJNCDM2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3TRKAA#issuecomment-518460672,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMYHIAICUV66LH4KONP4DTTQDDIFTANCNFSM4IJNCDMQ
.

thank u for the help the error got rectified by changeing the version in metadata to 0.1.1 from 0.1.0


Update iis.txt

Description
Please provide a description of your proposed changes - providing obfuscated log/code examples is highly encouraged.
Related Issues
Are there any Issues to this PR?
Todos
Are there any additional items that must be completed before this PR gets merged in?

[ ]
[ ]

Created new request - discarding this one


why paper L is given by N = (n − 1) (n − 2) /2?it is not L is given by N = (n − 1) (n) /2

I suppose you are asking questions on pairwise distance calculation. Here we exclude the diagonal elements in the distance matrix as they are all zeros.

Thanks,I know what you mean.However,for a n*n matrix, the number of elements(i < j) is  (n − 1) (n) /2.But size of L is given by N = (n − 1) (n − 2) /2 in paper.Maybe I was stupid,thanks!


GraphQL pokémon sprites returns 404 status

I was taking a look in my pokedex project and it seems that something is broken on retrieving the Pokémon sprites via GraphQL.
Here's the projects link: https://pokedex-react-topaz-zeta.vercel.app/
I'm pretty sure its not a problem from my side, since I haven't done any changes in this project for quite some time.
The console returns a 404 error: "Failed to load resource: the server responded with a status of 404 ()"

Yes, there's a problem in our GraphQL API. Currently we strip away https://raw.githubusercontent.com/PokeAPI/sprites/master/, so you have to manipulate the link.
You need to change /media/ to https://raw.githubusercontent.com/PokeAPI/sprites/master/sprites/

I don't know if I'm doing it wrong, but it's still not working for me.
I have manipulated the link, but still get the same result, are any of these two images below the correct changes?


Added program for Swapping Nodes in Linked List- Python

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
Added program for swapping nodes in a Linked List in Python. All Pep8 rules have been followed and the code has been formatted. Doc strings for functions have been added, also inline and multi line comments have been added to explain the code.
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
#1572

Can any one of you please review this? @iamrajiv @atarax665 @shraddhavp @HarshCasper
Thanks :)

@Pranav016 Format your code according to PEP 8 rules. You can refer to this http://pep8online.com/.

I already did, before making this PR. It is giving no errors 😄

@Pranav016 Format your code according to PEP 8 rules. You can refer to this http://pep8online.com/.

Can you also suggest any VS code extension that helps to format the code like Pep8 ? That would be of great help! @iamrajiv

@Pranav016 Format your code according to PEP 8 rules. You can refer to this http://pep8online.com/.

Can you also suggest any VS code extension that helps to format the code like Pep8 ? That would be of great help! @iamrajiv


Choose encoding when export csv

config.py change CSV_EXPORT.encoding to gbk isn't effect download file encoding in SQL  Lab
CSV_EXPORT = {
'encoding': 'gbk',
}

Codecov Report

Merging #7141 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master    #7141   +/-   ##
=======================================
  Coverage   64.61%   64.61%
=======================================
  Files         422      422
  Lines       20605    20605
  Branches     2253     2253
=======================================
  Hits        13313    13313
  Misses       7170     7170
  Partials      122      122

Impacted Files
Coverage Δ

superset/views/core.py
74.92% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


test miq_request_workflow#make_request

We defined make_request wrong.
This puts a test around it so we don't run into this bug again.
/cc @gmcculloug

LGTM.  This fixes a bug introduced in PR #4332 where make_request was defined as a class method.


Nologin error

Motivation for this change
Fixes #16428
Things done

[x] Tested using sandboxing
(nix.useChroot on NixOS,
or option build-use-chroot in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] OS X
[ ] Linux

[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

The first commit seems like a no-brainer. The second one I cannot judge (yet). Are you waiting for feedback regarding the 2nd commit?

The second-one exposes the error of the first commit. For legacy reasons we also accept a path as an argument but if it's a package it should expose the shellPath passthru. I wrote all of that code initially so I'm quite confident of what code uses that util. I just like to send PRs just to make sure I didn't miss anything obvious.


🍒 [5.9][Constant Evaluator] Enable SILConstants::setIndexedElement function to work with aggregates containing unknown values

Such aggregates can be generated when an instruction is skipped during constant evaluation and its results are used to create a struct.
Original PR
Explanation: Compiler can crashes in passes using constant evaluator when evaluation encounters certain SIMD types.
Scope: Swift programs that use SIMD instructions along with the os_log APIs.
Risk: Low, this changes relaxes an assertion in an internal function of constant evaluator and instead returns an error in such cases, which is already gracefully handled by the constant evaluator.
Testing: Swift and SIL LIT tests. This PR landed in main for a while.
Reviewers: @matthewseaman and @kulpreetchilana
rdar://107344820

@swift-ci please smoke test

@swift-ci please test


Cleaning and adjusting layermask and PG tuts

Thank you...I really appreciate your work!

Thanks for the thanks.  It's my pleasure.  :)
The pictures for the PG tutorial... were moved to a different folder at urbanproductions.com.
The pictures won't work @ http://doc.babylonjs.com/generals/The_Playground_Tutorial until after a new build. SO, maybe build before July.  :)  thx

Should be good in less than 10minutes

@deltakosh  Doc is still down! Sorry, but I don't know how I could help more than warning you in every possible way I can!

Trying again

Oh, sorry, I'm slow to read your emails.  All looks fine to me.  Was their trouble?
I wasn't in a hurry.  Build in 2 weeks = fine.
Nobody ever reads the playground tutorial anyway.  The playground is very intuitive.  It teaches itself
Thanks for the notifications, and for trying those builds.  Was it refusing to build?  I hope I didn't break something.  :o

No worry everything is fine:)


Configs loaded from environmental variables aren't properly typed

When setting configs using environmental variables export SANIC_REQUEST_TIMEOUT=30
app = Sanic(__name__)
print(type(app.config.REQUEST_TIMEOUT))  #  <class 'str'>

The problem is in this function
# .../sanic/config.py
    def load_environment_vars(self):
        """
        Looks for any SANIC_ prefixed environment variables and applies
        them to the configuration if present.
        """
        for k, v in os.environ.items():
            if k.startswith(SANIC_PREFIX):
                _, config_key = k.split(SANIC_PREFIX, 1)
                self[config_key] = v  # os.environ values are always of type str

This could be solved by simply trying to cast v into int except ValueError then just leave it as string.

@mczp ya I was thinking that, but then I guess there could be other types as well.. :( 🤔  oh well, let's do that for now unless we come up with something better


Cascade saved windows with the same id

This causes saved windows with the same id to be cascaded. The cascade behavior was designed to mimic macOS:

Find top-most like window and add x/y offset to that position.
If the new window hits the right edge of the screen, change x position of new window to 0.
If it hits the bottom edge of the screen, change y position of the new window to 0 (under the top bar).

In OpenSphere, this results in windows being positioned on top of the top nav because the window container contains the navs. This feels a little odd, but the best way to resolve that would be changing the window container to the map container. That seemed like a potentially unwanted change, and perhaps not fit for this PR.

I updated this to cascade all windows based on id instead of just saved windows. This allows any like window to be opened multiple times and cascaded, if necessary.
I changed the Create/Edit place windows to all use the same id, so they'll all be cascaded against one another. Previously edit windows were given a unique id based on the feature, and would be centered and potentially overlap other similar windows.
Also found a bug where Modify Area wasn't updating the existing window if present, because the selector wasn't updated with Bootstrap 4.


[AMBARI-24107] Ranger & Atlas initial install require additional prop…

…erties to be explicitly configured in the UI.
What changes were proposed in this pull request?

Added a warning popups for Ranger if Solr is not selected.
Changed the warning popup for Atlas if Solr is not selected.
Added a warning popup for Atlas if HBase is not selected.
Have Ranger audit property is_solrCloud_enabled if both ranger and Solr are selected.

How was this patch tested?
Modified existing tests. Manually tested the fix on a remote cluster
21802 passing (26s)
48 pending

retest this please

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


[meta] issue and pull request templates

Hi,
Just found out there are templates for new issues and pull requests, do you think this can be useful for this project?
Create Pull request template.
Create issue template.

@C47D Yes, I intend to add these so that people will know to direct questions and enhancement requests to the forum rather than here.

@embeddedt Great, should we close this issue until the templates are added?

@C47D I'm working on them right now.

@kisvegabor Issue templates are added; let me know if you think something needs changing.

That was quick!, i don't know how often lvgl receive pull request, if so, do they need an special format?
It might be useful to add a template for them as well.

@C47D I don't think templates are necessary for pull requests right now, because usually a pull request is only opened after a discussion on the feature (unless it's a bugfix).
In the future it might be something to look into though.

I see, you are right, thanks for the quick reply!

Looks great! Thank you!
I've just renamed bug_report.md to bug-report.md for consistency.


Suggestion Pressing Missions

Hello,
Do you have a possibility to integrate the fact of recovering bales of straw, grass or other when there are pressing missions?

Done. Included as part of the latest release: <IP_ADDRESS>.
Please let me know if you have any issues.


TypeError: 'NoneType' object is not subscriptable

I am running conductor 0.3.1 and I get the following message:
Monitoring the witness, current misses: 1
Traceback (most recent call last):
  File "/home/witness/anaconda3/lib/python3.6/site-packages/conductor/watchdog.py", line 83, in watchdog
    diff = total_missed() - misses
  File "/home/witness/anaconda3/lib/python3.6/site-packages/conductor/watchdog.py", line 20, in total_missed
    return get_witness(witness('name'))['total_missed']
TypeError: 'NoneType' object is not subscriptable

It starts fine, and runs for a little while (5 to 30 minutes, maybe more), but then throws the error.  I am not sure if it is still running, I just have a blinking cursor.

This error is generally harmless (ignored).
I don't like the way its currently handled, so I will leave the issue open for now.

To be handled in https://github.com/Netherdrake/conductor/issues/14


[textanalytics] Update async samples

Related to #21209
I've updated  textanalytics samples to drop  loop.run_until_complete( . . )
(updated to asyncio.run( . . ))
This is my first time to send a Pull Reqeust, so it would be so Thankful if you tell me something went wrong or my pr is not good way.
Thank you!

@JoongWonChoi thanks for your contribution, it looks great! 😃
Just FYI, the test related failures in CI are unrelated to your PR changes so as soon as they are resolved I can help merge your PR. Thanks again!

@JoongWonChoi thanks for your contribution, it looks great! 😃
Just FYI, the test related failures in CI are unrelated to your PR changes so as soon as they are resolved I can help merge your PR. Thanks again!

Thanks a lot! I'll wait for your comment!

@JoongWonChoi the build is green, going to merge your PR. Thank you! 🥳


microsoft_gsl: 2.1.0 -> 3.1.0

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 130527 at fe195f61 run on x86_64-linux 1

1 package built successfully:

microsoft_gsl

1 suggestion:

warning: missing-phase-hooks
checkPhase should probably contain runHook preCheck and runHook postCheck.
Near pkgs/development/libraries/microsoft_gsl/default.nix:35:3:
   |
35 |   checkPhase = "make test";
   |   ^


Include query time in query results

Closed by https://github.com/matthewmturner/datafusion-tui/commit/897937cf1148b5434da5f6d06ca9f48f6caac4e4


No module named 'pkg_resources' when running pwndbg

Describe the bug
pwndbg fails to run because setuptools is no longer propagated implictly. If pwndbg were purely a python application built with buildPythonApplication then we would just need to add setuptools to propagatedBuildInputs and be done. However, pwndbg is not simply a Python application, but a wrapper around gdb, and isn't built usingbuildPythonApplication. The derivation patches the gdbinit.py script as follows:
 preFixup = ''
    sed -i "/import sys/a import sys; sys.path[0:0] = '$PYTHONPATH'.split(':')" \
      $out/share/pwndbg/gdbinit.py
  '';

This does result in the site-packages directories from all python packages ending up in sys.path, including setuptools (e.g. /nix/store/03rhr26hsrkz9hbcqqyb84zhiiprl5d9-python3.7-setuptools-41.2.0/lib/python3.7/site-packages). I'm not that familiar with Python packaging but from experimenting I found that the full path to the Python egg needs to be included (e.g. /nix/store/03rhr26hsrkz9hbcqqyb84zhiiprl5d9-python3.7-setuptools-41.2.0/lib/python3.7/site-packages/setuptools-41.2.0-py3.7.egg) in order for this method to work.
I made an attempt at resolving the issue but being new to Python packaging in Nix, neither are likely to be ideal:
We could use buildPythonPackage in place of mkDerivation, using format = "other" and use
preFixup = ''
  patchPythonScript $out/share/pwndbg/gdbinit.py
'';

Or continue to use mkDerivation and manually set program_PYTHONPATH for patchPythonScript:
preFixup = ''
  program_PYTHONPATH=$PYTHONPATH patchPythonScript $out/share/pwndbg/gdbinit.py
'';

I'm happy to submit a PR but I'm not sure what the recommended solution is here.
To Reproduce
Steps to reproduce the behavior:
❯ nix-shell -p pwndbg

[nix-shell:~]# pwndbg
Traceback (most recent call last):
  File "/nix/store/18c6hhprssnjpp6yf7gh5gbrw1ih5h22-pwndbg-2019.01.25/share/pwndbg/gdbinit.py", line 37, in <module>
    import pwndbg # isort:skip
  File "/nix/store/18c6hhprssnjpp6yf7gh5gbrw1ih5h22-pwndbg-2019.01.25/share/pwndbg/pwndbg/__init__.py", line 13, in <module>
    import pwndbg.arch
  File "/nix/store/18c6hhprssnjpp6yf7gh5gbrw1ih5h22-pwndbg-2019.01.25/share/pwndbg/pwndbg/arch.py", line 12, in <module>
    from capstone import *
  File "/nix/store/p477mn568d5byj67khyf833pz3qm9l03-python3.7-capstone-3.0.5.post1/lib/python3.7/site-packages/capstone/__init__.py", line 201, in <module>
    import pkg_resources
ModuleNotFoundError: No module named 'pkg_resources'
(gdb)

Expected behavior
gdb should launch with the pwndbg extension loaded:
❯ pwndbg
pwndbg: loaded 167 commands. Type pwndbg [filter] for a list.
pwndbg: created $rebase, $ida gdb functions (can be used with print/break)
pwndbg>

Metadata

system: "x86_64-linux"
host os: Linux 5.3.5, NixOS, 19.09.741.dbad7c7d59f (Loris)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.3
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

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

It seems pwndbg is normally used by adding a source $pwndbg/gdbinit.py to ~/.gdbinit, or calling gdb with -q -x path/to/gdbinit.py (what the wrapper registered in installPhase does).
This script then just does some sys.path addition calculations to register itself (we probably don't that at all) and some python 2 specific fixups, and then calls import pwndbg, which starts executing some code on import.
I'd suggest to package it as a regular python application first (via buildPythonApplication), so you can successfully import it inside a python3.withPackages (ps: with ps; [ pwndbg ]).
The python tooling should automatically take care of patching the python shebang properly if you make $out/share/pwndbg/gdbinit.py script executable.

proposed fix https://github.com/NixOS/nixpkgs/pull/71219


refactor!: simplify discv5 initialization

Motivation
Simplification of initialization of discv5 for the purpose of simplifying passing the enr across a thread boundary

@dapplion ready for re-review

:tada: This PR is included in v1.9.0 :tada:


Type inference failure with noncopyable types.

Description

The following code fails to compile with the error message: ref is borrowed and cannot be consumed.
struct Ref<T>: ~Copyable {
    var value: T
}

func consumingMap<A>(
    _ f: @escaping (inout A) -> Void
) -> (consuming Ref<A>) -> Ref<A> {
    { ref in
        var newRef = ref
        f(&newRef.value)
        return newRef
    }
}

The following, however, does compile:
func consumingMap<A>(
    _ f: @escaping (inout A) -> Void
) -> (consuming Ref<A>) -> Ref<A> {
    { (ref: consuming Ref<A>) -> Ref<A> in
        var newRef = ref
        f(&newRef.value)
        return newRef
    }
}

Steps to reproduce

Put the above code in a playground.  Observe that it does not compile.
Expected behavior

The first code should compile without having to put the explicit type information on the returned closure.

Environment

Swift compiler version info

swift-driver version: 1.87.3 Apple Swift version 5.9.2 (swiftlang-<IP_ADDRESS>.56 clang-15<IP_ADDRESS>.5) Target: arm64-apple-macosx14.0

Xcode version info

Xcode 15.1 Build version 15C5059c

Deployment target:

macOS 14

When run in a playground, the following, related, code fails to compile as well:
struct Ref<T>: ~Copyable {
    var value: T
}

func consumingMap<A>(
    _ f: @escaping (inout A) -> Void
) -> (consuming Ref<A>) -> Ref<A> {
    { (ref: consuming Ref<A>) -> Ref<A> in
        f(&ref.value)
        return ref
    }
}

with the message:
error: couldn't IRGen expression. Please enable the expression log by running "log enable lldb expr", then run the failing expression again, and file a bug report with the log output.


Update velero version

time="2024-07-02T15:40:48Z" level=info msg="updating velero addon"
time="2024-07-02T15:40:48Z" level=info msg="finding latest version of vmware-tanzu/velero"
time="2024-07-02T15:40:48Z" level=info msg="pulling velero chart version 7.1.0"
time="2024-07-02T15:40:49Z" level=info msg="downloaded velero chart: /tmp/velero-7.1.0.tgz"
time="2024-07-02T15:40:49Z" level=info msg="authenticating with "registry.replicated.com""
time="2024-07-02T15:40:50Z" level=info msg="Login Succeeded"
time="2024-07-02T15:40:50Z" level=info msg="verifying if destination tag already exists"
unable to mirror velero chart: unable to verify if tag already exists: failed to authorize: failed to fetch oauth token: unexpected status from GET request to https://registry.replicated.com/v2/token?scope=repository%3Aembedded-cluster-charts%2Fvelero%3Apull&service=registry.replicated.com: 401 Unauthorized

Closing, this will be reopened again in the future.


it crashes when connection with a relay is not established

Possible fix

874c8bf963a1101e2bb72a6d5155dfa8fb9d2792

@leesalminen not related to this issue but nice project btw.
I just forked the repo here and included YOUR_HEX_ENCODED_PUBLIC_KEY_HERE and TO_RELAY_URL as arguments.
Also made a Docker image,
docker run --rm -it jaonoctus/nostr-rebroadcast <YOUR_NOSTR_PUBLIC_KEY> <RELAY_TO_REBROADCAST_URL>


Send seems to close the websocket ?

Got the send functionality working per the example in the futures::websocket source, but it seems to close the websocket after send, and it seems wrong to do an open each time you send. Shouldn't the websocket be longlived, and is there a way to make it reconnect, if state is Closed or Closing ?. I might just not understand how this crate is supposed to be used.

I can try. Background is I'm using Sycamore, so have components where I want to send messages, but create the websocket in the main app component, and send it as a property to underlying components. So have a definition of open and send as this
`pub fn get_websocket() -> WebSocket {
WebSocket::open(WS_URL).unwrap()
}
pub fn send_msg(mut ws: WebSocket, msg: String) {
spawn_local({
async move {
ws.send(Message::Text(msg)).await.unwrap();
}
})
}`
These are used from other modules. first in the main app as
fn main() {     console_error_panic_hook::set_once();     console_log::init_with_level(log::Level::Debug).unwrap();     let ws = get_websocket();     sycamore::render(|| template! { App(AppProps{ws: ws}) }); }
Then in the router ws gets send further down with a clone, which might be the issue, not sure.
AppRoutes::Users => template! {            Users(pages::users::Props {ws: props.ws.clone()}) },
And in the User component is used inside a buttons onclick event as
button(class="col-span-4 bg-blue-500 hover:bg-blue-700 text-white font-bold py-2 px-4 rounded justify-self-center", on:click= move |_| {                     let u = HashMap::from([("user".to_string(), user.get().to_string()), ("name".to_string(),  name.get().to_string())]);                     let event = Event{service: "command".to_string(), source: "kedl.frontend".to_string(), detail_type: "Create User".to_string(), detail: serde_json::to_string(&u).unwrap()};                     let json = serde_json::to_string(&event).unwrap();                     log::info!("U WS STATE {:#?}",props.ws.state());                     send_msg(props.ws.clone(), json);                     log::info!("U WS STATE 2 {:#?}",props.ws.state());             }) { "Create" }
First time the button is clicked everything works fine and the state is Open, but the second time the state is Closed, in both log statements.
Don't know if this is possible to make sense off

WebSocket code was re-written in #4 but that isn't released yet (I thought I released it lol). Can you try depending upon reqwasm from git? Please let me know how the experience is. I shall release it based on your feedback.
You can look at this very simple test as an example:
https://github.com/hamza1311/reqwasm/blob/fb356775a579cd2cbad1111bcb77fa68c0b83366/src/websocket/futures.rs#L307-L319

I was already depending on git, but it still doesn't work for me. If I do 2 sends right after each other (in code, not by clicking a button), the second fails because the socket is closing. I'm not sure how to use the StreamExt::split, it seems to mandate a Context with some Waker stuff, that I have no idea how to do. Still using the  Sink directly, you seem to forego the purpose of having a wrapper around the whole websys::websocket thing, or maybe i'm misunderstanding it totally. What is the correct way to create a socket in one place of your wasm app, and then reuse that in all it's sub components ?. I don't think doing as in the example is viable for real apps with multiple pages and components, but again I might be totally wrong on the use case here.

I see. This seems to be a bug. The Clone and Drop implementations should not exist.

I'm not sure how to use the StreamExt::split, it seems to mandate a Context with some Waker stuff, that I have no idea how to do.

Sorry, I explained it badly. The split method can be used like this:
use futures::stream::StreamExt;
use wasm_bindgen_futures::spawn_local;

let ws = WebSocket::open(URL)?;

let (mut sender, mut receiver) = ws.split();

spawn_local(async move {
    sender.send(Message::Text(String::from("something"))).await;
});

spawn_local(async move {
    while let Some(msg) = receiver.next() {
        log!(format!("{:?}", msg))
    }
});

Still using the Sink directly, you seem to forego the purpose of having a wrapper around the whole websys::websocket thing, or maybe i'm misunderstanding it totally. What is the correct way to create a socket in one place of your wasm app, and then reuse that in all it's sub components ?. I don't think doing as in the example is viable for real apps with multiple pages and components, but again I might be totally wrong on the use case here.

This problem is exactly why I initially held off the release (then life got in the way and I forgot). This wrapper is based around Rust's Futures. It seems like splitting the future into Sink and Stream and passing those individually seems like the right approach.
You'd want to have some sort of communication mechanism in your application where a WebSocket instance is created at the top of the components tree and the messages are passed around using something like channels (see futures::channel::mpsc)

https://github.com/hamza1311/reqwasm/commit/445e9a5bf555a0e37d0ff7fb71e69d34cb8f2493 should resolve the issue related with the ws unexpectedly closing. See the updated test as an example on how to use it
https://github.com/hamza1311/reqwasm/blob/445e9a5bf555a0e37d0ff7fb71e69d34cb8f2493/src/websocket/futures.rs#L300-L323
I will resolve #11 before publishing the next release

Cool. Trying to make it work in my setup, but not being able to clone websocket gives all kind of lifetime issues when moving the socket between components, so have to figure out how to make that work

@surt666, was the issue resolved?

It seems that the close issue is resolved, but I'm somewhat unsure of how to move the socket around so it's only created in one place, now that it can't be cloned. It seems like i'm lacking some liftetime understanding, about how to move it as a reference, so if you have an example where the websocket is created outside of the function that uses it I think that would benefit a lot of people. I'm thinking taking the websocket as a parameter to the test function above

Sorry it took me a little while to respond. I recommend using channels or some other way to communicate between components inside your application instead of passing around the WebSocket instance. The Clone implementation was there to allow sending and receiving from multiple futures but that has been taken care of by ws.split()

Closing because this issue is resolved and 0.3.0 is now released


Exceptions defined in shaded dependencies break exception catching in clients

We added a shaded artifact in #630 to try to make this library "more portable".  It turns out this is not a drop-in replacement.
All classes defined in one of the shaded dependencies are relocated to a new hidden package to avoid conflicts. This includes exceptions. This means any consumer of this library that wants to specifically catch com.fasterxml.jackson.databind.JsonMappingException from the shaded library (as opposed to a general java.io.IOException) would would need to be rewritten to catch hidden.com.fasterxml.jackson.databind.JsonMappingException.
The only alternative to this is to define our own exceptions based on those in shaded exceptions and rethrow with them.  That would end up being a whack-a-mole finding all exceptions that might be thrown and in any case would still be a breaking change.
I'm inclined to give up on shading entirely.

I removed shading in this commit:  https://github.com/github-api/github-api/commit/c8115b1c10d62379459e9da64bd588fd2115c376
If someone cares about this enough to redo it at some point, they can.


Images embedded in documents are not displayed

In commit #c98e24c8f6f5224e1729284d6a3fc5acba6a14e3, the request handler regex was changed from "app.get()/" to "app.get(/^([^.])/", and the line res.sendfile(filePath); was removed from the handler. As a result, image files are not served anymore.

Good point, this is a problem that needs a long term solution.
In the short-term, you could utilize external hosting services for the images to get back up and running quickly.
Moving forward, we'll likely have a separate handler for content-related images.
If you have theme-related images, I'd recommend putting them in themes/default/images until we can extract the themes into separate packages


Vector3: Add operator+/- for scalars

Self-explanatory.  Adds ability to add/subtract a scalar value from all elements of a Vector3.

Add some quick testing to keep coverage at 100%?

Add some quick testing to keep coverage at 100%?

Will do :+1:


[kibana] add service.httpPortName config in chart

[X] Chart version not bumped (the versions are all bumped and released at the same time)
[ ] README.md updated with any new values or changes
[X] Updated template tests in ${CHART}/tests/*.py
[ ] Updated integration tests in ${CHART}/examples/*/test/goss.yaml

Fix https://github.com/elastic/helm-charts/issues/841: "Kibana: Missing service.httpPortName config in chart"

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

jenkins test this please

I updated the python test to be able to merge the PR. Thanks for your work @ebuildy 👍


Upgrade to flutter_map 5.0.0

Latest flutter_map package contains a lot of bug fixes and improvements

I updated the repository to flutter_map version 5.0.0 for my purposes and can do a pull request if wanted.
https://github.com/josxha/flutter_map_geojson


Incorrect Data Access X-Refs for PIE binaries

If you load a PIE ELF binary into Binary Ninja, the base address is set to zero.
This causes erroneous cross-references to be generated for any constant value.
Screenshots are attached.

Fixed in dev and stable.


Firmware Upgrade image is in Wrong Format

When I tried using the image "openwrt-ramips-rt305x-ht-tm02-squashfs-factory-r44945-ws.bin" to upgrade the firmware through the stock image's System's Firmware upgrade menu, it said that the image was in the wrong format. I had to use "openwrt-ramips-rt305x-ht-tm02-squashfs-factory-r42649.bin". Only that one was accepted.

I have this issue with every firmware you've provided. TripMate says it's the wrong format. openwrt-ramips-rt305x-ht-tm02-squashfs-factory-r42649.bin does not work as of September 2018. Attempted upgrade with OE firmware 2.000.046.


[Backport 2.x] Added Score Normalization and Combination feature

Backport PRs related to Hybrid Search feature:
Main PR with porting from feature branch:
https://github.com/opensearch-project/neural-search/commit/61e6e98bb6377d95d1043e1c4adcc0d4309b99ac from https://github.com/opensearch-project/neural-search/pull/241
Fix feature flag apporach:
https://github.com/opensearch-project/neural-search/commit/5819df61c6ce7d155463f8fcc0ffb4fd4ac3080d from https://github.com/opensearch-project/neural-search/pull/246
https://github.com/opensearch-project/neural-search/commit/2a9f70a86c1a4166fd9efa733eca57f815e1a1b8 from https://github.com/opensearch-project/neural-search/pull/247

Currently failing due to errors in ml-commons, they do have PR in progress to fix that https://github.com/opensearch-project/ml-commons/pull/1182

Closing this one as I messed up with 2.x branch in fork before this merge. Work in done under https://github.com/opensearch-project/neural-search/pull/263


[1476] copy course length and salary content

Context
Copying course length and salary content
Changes proposed in this pull request
Add "Copy from" select to enable users to copy content from another course within their org
Guidance to review
Example - /organisations/2AT/courses/3CXD/salary

 [S] Add sidebar with Copy functionality to "Course length and salary" edit page


ClassCastException: com.facebook.react.bridge.ReadableNativeMap cannot be cast to java.lang.String

After migrating the project to AndroidX the app shows me this error at startup.
I have no clues about where to look for this error, I also tried to load an initial blank page to eliminate all the possible code written by me, but the problem persists.
This error only appears on Android (tested on Android 5.1 and 6.0), everything works correctly on iOS
React Native version:
0.59.10
E/unknown:ReactNative: Exception in native call
    java.lang.ClassCastException: com.facebook.react.bridge.ReadableNativeMap cannot be cast to java.lang.String
        at com.facebook.react.bridge.ReadableNativeArray.getString(ReadableNativeArray.java:135)
        at com.facebook.react.bridge.JavaMethodWrapper$5.extractArgument(JavaMethodWrapper.java:73)
        at com.facebook.react.bridge.JavaMethodWrapper$5.extractArgument(JavaMethodWrapper.java:69)
        at com.facebook.react.bridge.JavaMethodWrapper.invoke(JavaMethodWrapper.java:359)
        at com.facebook.react.bridge.JavaModuleWrapper.invoke(JavaModuleWrapper.java:158)
        at com.facebook.react.bridge.queue.NativeRunnable.run(Native Method)
        at android.os.Handler.handleCallback(Handler.java:815)
        at android.os.Handler.dispatchMessage(Handler.java:104)
        at com.facebook.react.bridge.queue.MessageQueueThreadHandler.dispatchMessage(MessageQueueThreadHandler.java:29)
        at android.os.Looper.loop(Looper.java:194)
        at com.facebook.react.bridge.queue.MessageQueueThreadImpl$4.run(MessageQueueThreadImpl.java:232)
        at java.lang.Thread.run(Thread.java:818)

RN 0.60 migrated to AndroidX, could you please try to upgrade to it, and check if it works?

@dulmandakh there are no other solutions to try first? I'm afraid of making the situation worse, given the breaking changes introduced

AndroidX is not supported by React Native < 0.60 so you will probably run into all kinds of hard to debug issues if you try to run older versions of RN on AndroidX. I recommend upgrading.

Hey everyone @Ilario17  @bartolkaruza @dulmandakh ! I just upgraded to RN 0.60 and migrated to AndroidX. Then, I experience the same error. For the record, I haven't been able to get my project to work since the migration. Thanks if you've any clue !

Hi @clementinoux after hours of testing I discovered that the error comes from an incompatible library, in my case the library is react-native-default-preference.
If I comment on the snippet in which I use this lib the error disappears


setup react up

Set up react app called Math magicians
Initialized it to this git repository
Installed eslint and stylelint

I let the linter errors be because I will be deleting the boilerplate template in the next part of the project

Hi @TracyMuso, Please fix the linter errors, the linters have to pass before you can get a code review, it is part of the requirements 👍🏿

Hi @TracyMuso,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a project, that  has linter errors, please fix it 👍🏿
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification @prolajumokeoni.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


Issue636: fix tense of verbs

Tell me if this fix is too jank. I thought of other ways but that would involve more lines of code.

Codecov Report

Merging #664 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #664      +/-   ##
==========================================
+ Coverage   88.82%   88.83%   +<.01%
==========================================
  Files          47       47
  Lines        3419     3421       +2
==========================================
+ Hits         3037     3039       +2
  Misses        382      382

Impacted Files
Coverage Δ

vent/menus/main.py
78.66% <100%> (+0.14%)
:arrow_up:

Continue to review full report at Codecov.

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

there's more than just remove that needs to be fixed...

Roger.


Sidebar improvements

Fixes #163

Use built in Gatsby active link styling feature instead of homegrown one.
Refactor all of the sidebars so they share a common set of components.
Add table of contents sidebar to the legal notices.
Make docs and legal notices full width.

https://app.clubhouse.io/larder/story/1783/full-width-docs-and-legal-notices


AttributeError: type object 'MyModel' has no attribute 'split'

Hey, I am not sure if I misunderstood the docs but when I add widgets = {'myfield': SearchableSelect(model='MyModel', search_field='myfield', many=True, limit=10)} to my form it throws this error. Any pointers would be highly appreciated 😄

Found the error...it wasn't a string...sorry

what was your solution? i have the same issue

Sorry for getting back to you so late. I looked at my code - honestly it was a while ago I worked with this so I am not completely sure. But I think the 'myfield': SearchableSelect(...) needs to be a in double quotes like so "myfield". Never got this component to work the way I wanted though. 😓


Add information about "profile" param in state backend

Oh, I've been wanting to do this for almost a year!
Depends on the unreleased version of the cli.
After that's released with the updated go-cloud library, it'll become a breeze to work on multiple projects on AWS without thinking about env vars.
Even though we have a workaround of using direnv to set the AWS_PROFILE, this new approach is much cleaner and consistent across multiple people.
This PR is a sibling of #1499, which covers the encryption part.

@ekini thank you for opening a pr!! would you be up for accepting @jkodroff's changes before we merge this?

Thank you! Should be all fixed, along with the freshly released 3.33.1 version.


chore(firestore-bigquery-export): increment change tracker version to…

fixes: https://github.com/firebase/extensions/issues/621

Codecov Report

Merging #645 (3a5815e) into next (9a60a06) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             next     #645   +/-   ##
=======================================
  Coverage   50.38%   50.38%
=======================================
  Files          30       30
  Lines        1324     1324
  Branches      194      189    -5
=======================================
  Hits          667      667
  Misses        656      656
  Partials        1        1


Create Bobby gosselin huntsville recommendation file

This is a bullet point.

[ ] This is a task list item.

Good job

Hello!  I am leaving a general comment!  Who is this General Comment and why is he looking at your project?


Add session_traffic_source_last_click fields

Description & motivation
Google recently released a new set of fields, session_traffic_source_last_click, documented as:
The session_traffic_source_last_click RECORD contains the last-click attributed session traffic source data across Google ads and manual contexts, where available.

These fields have been added to the base package model.
Checklist

[X] I have verified that these changes work locally
[ ] I have updated the README.md (if applicable)
[ ] I have added tests & descriptions to my models (and macros if applicable)
[ ] I have run dbt test and python -m pytest . to validate existing tests

@dgitis I ran this and noticed differences between our stg_ga4__sessions_traffic_sources_last_non_direct_daily model and Google's last_click fields. When I looked at the raw events, it looked like our model was more accurate which was weird.
I think it will only confuse people to have 2 definitions of 'session last click attribution' in the package. I suppose we should just include these fields and remove stg_ga4__sessions_traffic_sources_last_non_direct_daily just thought I'd check with you first.

@adamribaudo-velir can I help to push this pull request?

Sorry @FloDevelops , I'll review this now. I thought that I had reviewed it.

I'll go ahead and merge. There is some overlap with these new fields and our existing 'last touch' calculated model, but offering both options does no harm. If we want to deprecate or further clarify in the future we can.


NuGet package is out of date and unlisted

I am trying to use Raylib-cs with fsharp on dotnet 3.1.
I am able to install the lib on a console app using dotnet add Raylib-cs
But now in my code doing
open Raylib
gives me the error The namespace or module 'Raylib' is not defined.

I have not used fsharp so I cannot help there I am afraid. There are bindings for it here https://github.com/dallinbeutler/RaylibFS although I think it would need to be updated.

I quickly testing this in c# and I am getting the same issue.
I started by creating a new C# console app and then added Raylib-cs to it. However in the code, it is not recognising raylib-cs
PS C:\Users\risharan\DevSpace\fsharp\test> dotnet new console                                                                                The template "Console Application" was created successfully.

Processing post-creation actions...
Running 'dotnet restore' on C:\Users\risharan\DevSpace\fsharp\test\test.csproj...
  Restore completed in 119.24 ms for C:\Users\risharan\DevSpace\fsharp\test\test.csproj.

Restore succeeded.

PS C:\Users\risharan\DevSpace\fsharp\test> dotnet add package Raylib-cs                                                                                Writing C:\Users\risharan\AppData\Local\Temp\tmpD0D6.tmp
info : Adding PackageReference for package 'Raylib-cs' into project 'C:\Users\risharan\DevSpace\fsharp\test\test.csproj'.
info : Restoring packages for C:\Users\risharan\DevSpace\fsharp\test\test.csproj...
info :   GET https://api.nuget.org/v3-flatcontainer/raylib-cs/index.json
info :   OK https://api.nuget.org/v3-flatcontainer/raylib-cs/index.json 908ms
info : Package 'Raylib-cs' is compatible with all the specified frameworks in project 'C:\Users\risharan\DevSpace\fsharp\test\test.csproj'.
info : PackageReference for package 'Raylib-cs' version '1.4.0' added to file 'C:\Users\risharan\DevSpace\fsharp\test\test.csproj'.
info : Committing restore...
info : Generating MSBuild file C:\Users\risharan\DevSpace\fsharp\test\obj\test.csproj.nuget.g.targets.
info : Writing assets file to disk. Path: C:\Users\risharan\DevSpace\fsharp\test\obj\project.assets.json
log  : Restore completed in 2.55 sec for C:\Users\risharan\DevSpace\fsharp\test\test.csproj.

You should copy the bindings into your project directly instead of adding via the package. The package is unlisted and out of date. I had issues getting it to work. I may revisit it although I have been making breaking changes on the dev branch and it is not a priority at this time.

Thanks. Thta makes sense.
I have edited the title of the issue so that it refers to updating the lib on dotnet package repo.

The point of having it be a package is to make it easy for people to install and use so I would want it to include the native libraries too although I am not sure how it should be done. The dev branch is nearly ready to merge into master so I will work on that and take another look at this after.

Hey Chris, I know you will have 3.0 as a priority but have you a timeline planned for this fix?

I am not sure. There are different ways I could package it and I have not found a clear guide for nuget packages that use native libraries. I have tried dotnet pack and that produces a package for Raylib-cs.dll but I am stuck on figuring out how to add the native libraries so they are included and also are copied to the output directory based on the platform being used.

Update. Taking another look to see if there was something I missed. Looks like there is a open issue abou usingt native libraries in nuget packages here.

Good news! We have just released a NuGet package targeting netstd2.0 (which should support Framework 4.7.2) & Core 3.1
So far we have tested it with the examples and with Github Actions, but please feel free to open new issues if there are problems with the package.
https://www.nuget.org/packages/Raylib-cs
dotnet add package Raylib-cs

Seems like the current solution does not work for netfx under certain conditions. Refer to #3 of this NuGet issue  and this comment
Current options seem to be:

Add a target file (which means writing a target for each arch & platform we want to support)
rename the runtime folder to libs - this is documented to cause issues/warnings with MSBuild and I'd rather not do something hacky just for netfx

@9ParsonsB I say we should look into the target files. I agree that renaming the folder to libs is hacky and sounds like it will cause more problems. The nuget package should be able to be built using dotnet pack with no other steps needed. If netfx can not work with that then we won't be able to support it.

To make it clear on the runtimes we are currently supporting. The libraries we include for these runtimes are taken from the official raylib release:

linux-x64
linux-x86
osx-x64
win-x64
win-x86

It is possible that linux-x86 and win-x86 support is dropped in the future as I do not use them myself and there is a extra cost in having to support/test them. Currently they are the least tested part because of this.

After some more investigations, I have another approach for packaging native libraries after reviewing this: https://github.com/NuGet/Home/issues/8623 (now using Content rather than Reference in the .targets file)
I will push a new package version now and hope someone can test it for me join us on discord!

does this use 64 bit binaries? I keep getting this..

I downloaded the latest raylib native library and both 32 and 64 bit doesn't seem to work

@Shadowblitz16 The latest nuget package currently includes the x64 and x86 libraries. Check that you are using the latest nuget release. You should not need to download the latest raylib library yourself to use the nuget package.

I get the incorrect format when trying to run win-x86, x64 is fine though.
I was running from fork before but have just started to use the nuget.

@ChrisDill the nuget package doesn't even install the C# bindings for me
I am using a .net framework 4.6

@ChrisDill it doesn't work

this is a blank console app targeted at .net framework 4.7.2

Set to x64.
Please set x86 and report that doesn't work.

nope that doesn't work either

@Shadowblitz16 Are you targeting .Net Framework or .Net Core?
.Net Framework doesn't handle cross-platform native libraries that well - in this case you will need to copy the native raylib dll into your output directory.
In any case, you will probably have better luck by only building x64. I know AnyCPU is broken. not sure about x86.
I am currently testing a fix for this (see pre-release NuGet packages).


feat(PageHeader): add SidebarToggle to header

This PR adds the sidebar toggle based on the work in https://github.com/dcos/dcos-ui/pull/3090 but be aware the design will be added by @tnguyen0 in another PR, this is only an implementation of the button.
Testing

npm start
Click on the header hamburger button and note the sidebar open/close

@weblancaster I changed base to master as #3090 got merged, now there are conflicts. Hope this was okay to you 😇

Wouldn't @tnguyen0 styling PR be a dependency then?

@Poltergeist correct, that's expected behavior.

@weblancaster Would be cool if you could add the lines I need to change to see the headerbar in the PR next time ;)

@weblancaster conflicts need to be resolved


add sort_by_weight unittest, extra yaml tag ctors

Motivation for changes:
Add missing unittest for sort_by_weight plugin and add yaml tag constructors for Quality, Interval types to conftest.py. This allows to directly construct those types via yaml and mock plugin.
Config usage if relevant (new plugin or updated schema):
config = """
  tasks:
    test1:
      sort_by_weight:
      ....

  mock:
    - title: 'title'
      age: !Interval '7 days'
      quality: !Quality '720p webrip'
      date: !!timestamp '2016-2-14 21:59:43.10 -5'  # can use builtin tag type (yaml 1.1)

Whats the hold up on this pull request atm?

@Andy2244 I know we have neglected some of the larger PRs for a long time. I think this one could be merged if a few changes are made. If you're still interested...


The station has no Firefox plug-in connection option

firefox: v106
Terra Station Wallet: v2.8.1

The station extension does not currently work on Firefox. Unable to connect to my wallet.


How can use multiple image input on same page ?

Hello I want to use multiple file input area on same page but they all are picking same data when I add files to one of them.

It's Done.


Disable a few flaky tests

Tests have failed a few times on PR builds.

Closing in favor of https://github.com/aspnet/AspNetCore/pull/9088


Implement Amazon Appstore support

Checklist

[ ] I've run bundle exec rspec from the root directory to see all new and existing tests pass

Tests currently fail (see below)

[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context
Fixes #15148
This changeset introduces a new plugin to help Fastlane interface with the Amazon Appstore App Submission API, much like supply does with the Google Play Store API.
This pull request is currently a work-in-progress with some changes soon to come from @joshdholtz to help tie things into Fastlane plugin ecosystem.
Testing Steps
Testing steps are coming soon, and unit tests are currently unhappy due to the use of a new ActionSets namespace (this is part of what @joshdholtz will be helping with).

          1 Warning

:warning:
Big PR

  Generated by :no_entry_sign: Danger

Wouldn't it make more sense to have those plugins outside of fastlane?

Hi, is this still in progress?
How does it compare to https://github.com/ugroupmedia/fastlane-plugin-amazon-app-store-submission

ping

I'm going to move this out to a separate gem and plugin repo that I'll link here momentarily. Closing this PR since it's been dead for a while!