Finalize switch from Travis CI to GH actions

Travis CI is pratically dead for OSS: https://blog.travis-ci.com/2020-11-02-travis-ci-new-billing
In fact, it already stopped working for this repo. The following note is printed on travis-ci.com:

Builds have been temporarily disabled for public repositories due to a negative credit balance. Please go to the Plan page to replenish your credit balance or alter your Consume paid credits for OSS setting.

I've already pushed a small first iteration of a GitHub Actions CI: https://github.com/vackosar/gitflow-incremental-builder/commit/77cb9eda57c4c85a9cd717c0ca0e721d29eda2b2
This needs to be finalized:

add ssh-agent handling
archive test logs
use a matrix to build all viable Maven + JDK combinations
add Codacy
add jacoco rendering via lynx?
replace badge
remove .travis.yml
... ?

/cc @vackosar I think I will re-setup the Codacy part after we have moved this repo to an organization.
I will try to come up with a good name for it. 😉 Recommendations welcome!

/cc @vackosar I think I will re-setup the Codacy part after we have moved this repo to an organization.
I will try to come up with a good name for it. 😉 Recommendations welcome!

I'll add Codacy coverage reporting once we moved to a GH org.

I'll add Codacy coverage reporting once we moved to a GH org.


Make it possible to pass options to driver creation helper

 Make it possible to pass options to driver creation helper


Keyboard interrupt

FAQ

[X] I have checked the FAQ and it didn't resolve my problem.

Announcement

[X] I have checked Breaking change announcement.

Minimal reproducible full config

Description
I got this error but I don't know how to reproduce it. This error just pops up sometimes and it's not the first time.
Error executing vim.schedule lua callback: ...m/site/pack/packer/start/nvim-cmp/lua/cmp/utils/misc.lua:243: Keyboard interrupt
stack traceback:
        [C]: in function 'cmd'
        ...m/site/pack/packer/start/nvim-cmp/lua/cmp/utils/misc.lua:243: in function <...m/site/pack/packer/start/nvim-cmp/lua/cmp/utils/misc.lua:241>

Steps to reproduce
Unknown
Expected behavior

Actual behavior
It just shows the error.
Additional context
Here is my config if it helps.

You have pressed C-c to leave insert mode.
You should not do it.
It breaks plugins.

@Shougo didn't know about it. Well, looks like I'll have to bear with it

						*i_CTRL-C*
CTRL-C		Quit insert mode, go back to Normal mode.  Do not check for
		abbreviations.  Does not trigger the |InsertLeave| autocommand
		event.

C-c  is emergency feature.  It is not just back to Normal mode.
It is similar with C-c interrupt in shell.

@Shougo thanks for the clarification. Btw, since this is not a cmp problem, do I need to close this issue?

Yes.  It should be closed.

Oh, I came here looking for the same thing. That's good to know! I use it because it's easier then reaching for escape. Are there any alternatives?

You can user other map to escape like this.
inoremap <C-l> <Esc>
I don't recommend for you to map <C-c> to <Esc>.
Because <C-c>map is only used in Insert mode.


<Box> not documented (but used in other elements' examples)

Maybe other elements miss there too?  <Root> is not in the index for example, even if it is mentioned in "Usage" section.

card, box, root, calendar, a whole lot are missing from the docs page.


Camera Movement Bug

@tufanaydin6340
Hey question, i can't change "MinVerticalAngle" and "MaxVerticalAngle" when scene runs it turns default "-90 & +90" how can we change? i can't change.

I wasn't able to repro this bug but logging it for note. I was using the nickmaltbie.OpenKCC.CameraControls.CameraController class

Problem is i can't change max and min vertical angle because in runtime angels are going default. Also in runtime can't change because not updating.
1- In editor change default camera angles to another different angle for example "-75"
2- Hit play button and look is it changed "probably changed and get back default"
3- Also change in runtime again another different angle "-75" test it in game window it affects from first default floats "-90"
4- If you want to affect it change in "slider" because code uses "OnValidate()"

Problem is i can't change max and min vertical angle because in runtime angels are going default. Also in runtime can't change because not updating.
1 - In editor change default camera angles to another different angle for example "-75"
2 - Hit play button and look is it changed "probably changed and get back default"
3 - Also change in runtime again another different angle "-75" test it in game window it affects from first default floats "-90"
4 - If you want to affect it change in "slider" because code uses "OnValidate()"

Hey @Tufis16, I wasn't able to repro the bug locally on my machine. Could you share a test project where you can re-create the issue and the version of unity and the OpenKCC plugin you are using? I'm currently using the latest master branch of the project and unity version 2023.1.6f1
Also, as per point 4, the current version of the CameraController does not have an OnValidate() method defined.
The min and max pitch are defined as minPitch and maxPitch in the CameraConfig

I am really sorry, i realized something, entire time i am looking for this assets bug "Kinematic Character Controller"  not "Open Kinematic Character Controller"
https://assetstore.unity.com/packages/tools/physics/kinematic-character-controller-99131

I am really sorry, i realized something, entire time i am looking for this assets bug "Kinematic Character Controller" not "Open Kinematic Character Controller"
https://assetstore.unity.com/packages/tools/physics/kinematic-character-controller-99131

No problem, sorry for confusing names. Hope the project and adjustments work out well!


HTCondorJobTask (submission to an HTCondor batch system)

Description
Implementing the HTCondorJobTask to allow for submission to the HTCondor batch system.
Motivation and Context
In the High-Energy Particle physics (HEP) community, HTCondor is a very popular batch system.
This PR is the first implementation of HTCondor submission for local batch systems.
Furthermore, it opens the possibility to submit workflows across data centres (e.g. Open Science Grid (OSG) infrastructure).
Given the correct HTCondor configuration (e.g. JobRouter), this PR could be also used to submit jobs to the Worldwide LHC Computing Grid (WLCG).
For both the handling of X509 certificates needs to be added, which will be done in a follow-up PR.
Have you tested this? If so, how?
I have included unit tests (based on SGE tests) and ran my jobs on a fresh install on my DEV machine (all-in-one install & configuration).

@Tarrasch : thanks for the comments, fixed the things you pointed out.
As for the condor details, maybe @bbockelm might want to have a look :) (or recommend someone).

By the way, do you have any clue for the failing test? Also, anyone from your organization can review it too.

Regarding the failed tests: fixed some of the flake8  warnings and the other test seems to have timed out. Seems to be all clear now.

@kreczko - Why did you utilize the CLI interface instead of the python bindings?  One major motivator of the python bindings was to avoid having to screen-scrape the outputs of things like condor_q as you did in this PR.

@bbockelm That was my original plan, but I could not get the bindings installed on Ubuntu 16.04 (see email on htcondor-users mailing list).
Also, most of our users use python >=2.7 on ScientificLinux6 which is not the default python distribution (hence no condor python bindings out of the box).
Ideas are welcome.

If you don't mind helping a bit with what will be the world's weirdest setup.py, I think we should be able to get the native bindings going.

OK, that sound like a plan.
This PR should be hold off until then.

Apologies for the delays, this is a 0 hour project :).
After sorting out the htcondor binding installation (now available via conda*), I've changed the implementation to use the htcondor python modules.
As a result some of the code has been removed.
@bbockelm could you please have a look again?
*
# default boost does not have boost_python, conda-forge does
wget https://repo.continuum.io/miniconda/Miniconda2-latest-Linux-x86_64.sh -O miniconda.sh
bash miniconda.sh -b -p <path to conda installation>
export PATH=<path to conda installation>/bin:$PATH
conda create -n condor python=2.7
source activate condor
conda install boost -c conda-forge
# match htcondor version to what is installed on the system
conda install htcondor-python=8.4.3 -c kreczko

I think it would be better to let this one incubate inside one of our current projects first.
I will make another PR once this is more fleshed out and better tested.


Fix some L7 policy cases in ENI mode

ENI mode was disabling L7 proxy entirely previously due to an overeager check in the datapath to avoid redirecting to the proxy in cases where the endpoint devices are IPVLAN devices.
Furthermore even in endpoint routes mode where the endpoint devices know how to handle traffic from the proxy, the stack was still forwarding traffic towards cilium_host. Disable these routes in this case.
These fixes enable the following cases in ENI mode:

Egress L7 policy plus no ingress policy
Ingress L7 policy plus no egress policy
Egress L7 + Ingress L7 policy within a single node

Future work:

Egress L7 policy + Ingress L7 policy across multiple nodes (it's not so much that this completely doesn't work, rather in this combination the destination sees source id world rather than of the pod)

This change is

test-me-please

Coverage increased (+0.006%) to 44.189% when pulling d9dd383db1594303906ca5ed1a49497c408fc1dc on joestringer:submit/eni-proxy into 23ec9760431a89c3b132adf7194d67cad26a4c64 on cilium:master.

@joestringer I remember that for the lyft-cni chaining, it was required to specifically route via cilium_host in order to get the redirect to work. It's interesting that this is no longer the case apparently. Could also be specific to kernel versions. I remember @jrfastab saw similar issues related to the encryption work.

Long and short is that right now if I just enable host redirect it causes ingress proxy at destination to receive SYN, reply with SYN/ACK, then fail to receive the ACK (and retransmit SYN/ACK ad infinitum). SYN/ACK reaches source node which responds with ACK, which arrives to eth1 of destination but I haven't been able to trace it further than that yet.

I've updated the PR to simplify it, this is easier to justify and it fixes the specific case that was reported.
Getting ingress policy to work in ENI mode across multiple nodes will require additional work.

test-me-please


Drop help button

Closes #1882

Coverage remained the same at 80.933% when pulling 666ff804d155961f003a6d20942006f9b8a2676a on eatonphil:drop-helper-button into 307dd97aaee02db14674d4a8e46e99afb9b60890 on linode:master.


Passing variables to patterfly-notification-drawer triggers infinite loop

If I pass the template includes via $scope variables like this:
          angular.extend($scope, {
            notificationGroups: [{
              heading: 'Tab 1',
              subHeading: 'woohoo!',
              notifications: [{
                unread: true,
                message: 'Hello world'
              }]
            }],
            headingInclude: 'views/directives/notifications/heading.html',
            subheadingInclude: 'views/directives/notifications/subheading.html',
            notificationBodyInclude: 'views/directives/notifications/notification-body.html',
            notificationFooterInclude: 'views/directives/notifications/notification-footer.html',
            customScope: {}
          });

  <div
    pf-notification-drawer
    heading-include="headingInclude"
    subheading-include="subheadingInclude"
    notification-body-include="notificationBodyInclude"
    notification-footer-include="notificationFooterInclude"
    custom-scope="customScope">
  </div>

My console logs some pretty gnarly things:

And the chrome-helper process associated with this page very quickly consumes over 1GB of memory (this is when I force quite, it seems to climb indefinitely).

The includes are expected to be attributes, I believe you need to use heading-include=“{{headingInclude}}” .
Not saying it is OK to infinite loop on error but does this fix your particular issue?

As far as that loop error goes, I'm guessing that it attempted to make an $http request for the template on the string headingInclude which fails and probably returns index.html.  The directive is likely trying to put index.html into the page, which reloads angular & triggers the whole cycle again.

Great. Should we close this then? I'm not sure we could fix this in the drawer code unless we were to expect variables instead and that would be a breaking change.

I'm fishing around a bit to see if this is a common problem with using includes, or if there is a good way to check against it.


Switch to securityFileStatus (fix failing ci)

This PR fixes the failing CI:

~some formatting issues in packages/tasks~ already done in https://github.com/huggingface/huggingface.js/pull/990
in listFiles, entry.security has been replaced by entry.securityFileStatus server-side. This PR updates the data structure + test to address this change.

Expectation: green CI :smile:

Super comforting to see those green badges :) Thanks a lot @Wauplin!

Thanks @Wauplin !


Adding new HHEM supported languages and some content suggestions

Adding HHEM's new supported languages and proposed some content changes. I think these changes may improve clarity of this page and how HHEM and FCS relate to query result scoring.

Looks good! Feel free to merge!

Thanks for the additional review 🙇♂ I'll wait until we have a final 👍 that the feature is ready for release before merging to avoid any potential confusion 🙂


Resolver: tighter requires-python bound in consumer fails to satisfy

I have two projects, both of which I control -- so this is not a blocker for me, but I think it's doing the wrong thing.
I have a consuming project, seattle-2025, that has requires-python = "~=3.12.3" and a dependency project, nomnom-hugoawards that has requires-python = "~=3.12.4"
On the surface, these seem to be satisfiable when resolving dependencies for seattle-2025; there are plenty of Python versions in the range, including 3.12.4.
However, when I attempt to sync, here's what I get:
$ uv sync
  × No solution found when resolving dependencies:
  ╰─▶ Because the requested Python version (>=3.12.3, <3.13) does not satisfy Python>=3.12.4,<3.13
      and nomnom-hugoawards==2025.0.0b5.dev1 depends on Python>=3.12.4,<3.13, we can conclude that
      nomnom-hugoawards==2025.0.0b5.dev1 cannot be used.
      And because only nomnom-hugoawards==2025.0.0b5.dev1 is available and your project depends on
      nomnom-hugoawards, we can conclude that your project's requirements are unsatisfiable.

      hint: The `requires-python` value (>=3.12.3, <3.13) includes Python versions that are not
      supported by your dependencies (e.g., nomnom-hugoawards==2025.0.0b5.dev1 only supports >=3.12.4,
      <3.13). Consider using a more restrictive `requires-python` value (like >=3.12.4, <3.13).
$ uv --version
uv 0.4.27 (36b729e92 2024-10-25)

Maybe I'm missing something, but I would expect dependency resolving to find the intersection of the required python versions, rather than require the consumer to be a superset. I wasn't able to infer this behavior from the resolver docs.
In this case, I'd expect 3.12.4 to satisfy the resolver.

Your project needs to be a subset, since right now, your project says it supports Python 3.12.3, but you have a dependency that doesn't support that version. We're not looking for a single version that works; we're looking to create a resolution that works for all the stated Python versions.

Oh... okay, that is very much not how I interpreted those bounds.
Fair enough, and I'll make the changes on my end -- the nice part about controlling both sides of this!. I suppose on some level this is a documentation bug report, then. Thanks!


entproto: support enum values with special characters

This fixes: https://github.com/ent/ent/issues/3498

I like this . Please handle the case of two enums that normalize to a identical string with an informative error?

Thanks for your feedback, that's a good point. I assume you mean two values within a single enum right? I updated the PR accordingly and also added a test.
I also factored out a entproto.NormalizeEnumIdentifier function so that we don't rely on the same normalization logic being implemented in multiple different places. It is still being defined twice though, in entproto.NormalizeEnumIdentifier as well as in the enum template.
I also have a question: We are not being consistent in error messages with strings vs. quoted strings and I assume we are stuck with this for backwards compatibility?

@jamslinger CI is red, can you check ?

I did a go mod tidy and it should be fixed now.

Thanks for the contribution @jamslinger !

Thanks for the support @rotemtam :)


updata cqu.txt

@Tianranse Thank you for your request. Regrettably, this domain is blacklisted because one of the following happened:

the college allows to register an email address there even if you are not a student of the college.
someone from the college shared their account details or activation code publicly, or sold it, and it lead to broad account misuse and license terms violation.
Therefore, we are not able to identify if the people with @cqu.edu.cn addresses are actual students or not.
If you are the student of this college, please apply with official document ("OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students).


unit test changes and adb sync command addition

I have removed kill-server and start-server testing from unit test code. Generally kill-server followed by start-server will put our code into infinite wait and its behavior is unpredictable. Please merge this code.

Thanks!


Leaking 'exports' into global

I believe the offending code is related to the snippet below, but what is happening is that, when including gss as a plain tag in a project that also has amd components, exports is getting added to window which prevents modules that check for both commonJS and AMD fail if they look for the existence of exports as does PIXI.js, for example. (I'm guessing scope == window when this runs).
// For Node...not that I'm bitter. No no, not at all. Not me. Never...
if (typeof require == "function" &&
    typeof module != "undefined" &&
    typeof load == "undefined") {
  scope.exports = c;
}
// ...well, hardly ever.

I put together a very basic pen http://codepen.io/adampilks/pen/vExNZe that show the errors with requirejs.  Requirejs introduces a globals of require and define which are conflicting with gss.
Is there a way to have these work side by side currently?

This was addressed in acfae7d2b3ba710f926b0434adea05707592c857 in case that's an option for you until it gets merged to master.

Actually, the relevant commits are 8334f868da3c524e1f53e9f37fb1cb3a132fccac and 77e1a7e3c888613d4736139bb169cbf4f7a1f1e9 but it's all part of the move to browserify that exists on the queries branch.

Great, we are in early development so will point our bower config to that branch for now.

We tagged v2.0.0 which includes those commits.


As a user of the marketplace I want to have the app linked to my owned domain

Right now, the grid infra controls the domains on demo.grid.tf and demo2.grid.tf, I'd like to be able to have these apps on $app.$mydomain or on $mydomain directly

what this will mean:

changes to all chatflows
requires more experienced end-users (chat flow to create a domain)
use name.com API to create new domains
user can choose to use a domain they already own


Vendor DuckDB sources, bumping to duckdb's version v0.10.1

Update: something is off with the vendor.py script, but I was too lazy to properly debug this.
Documenting the workaround (that should probably directly added to CI):
cd ..
git clone https://github.com/duckdb/duckdb.git --branch vX.Y.Z fresh_duckdb
cd duckdb-node
./vendor --duckdb ../fresh_duckdb
git add *
git commit -m "Vendor DuckDB vX.Y.Z"

Also, I need to generate the results for test_all_types's new types.
For now EXCLUDED them from testing, that should make this already mergeable.


Invalid hook call

This lib is just what I'm looking for but I'm running into this issue:
Error: Invalid hook call. Hooks can only be called inside of the body of a function component. This could happen for one of the following reasons:
1. You might have mismatching versions of React and the renderer (such as React DOM)
2. You might be breaking the Rules of Hooks
3. You might have more than one copy of React in the same app
See https://reactjs.org/link/invalid-hook-call for tips about how to debug and fix this problem.

at resolveDispatcher (/Users/jonathanlibov/node_modules/react/cjs/react.development.js:1476:13)
    at Object.useState (/Users/jonathanlibov/node_modules/react/cjs/react.development.js:1507:20)
    at InstagramEmbed (/Users/jonathanlibov/node_modules/react-social-media-embed/dist/components/embeds/InstagramEmbed.js:57:37)
    at renderWithHooks (/Users/jonathanlibov/apps/bathonea/node_modules/react-dom/cjs/react-dom-server.browser.development.js:5471:16)
    at renderIndeterminateComponent (/Users/jonathanlibov/apps/bathonea/node_modules/react-dom/cjs/react-dom-server.browser.development.js:5544:15)
    at renderElement (/Users/jonathanlibov/apps/bathonea/node_modules/react-dom/cjs/react-dom-server.browser.development.js:5759:7)
    at renderNodeDestructive (/Users/jonathanlibov/apps/bathonea/node_modules/react-dom/cjs/react-dom-server.browser.development.js:5898:11)
    at renderNode (/Users/jonathanlibov/apps/bathonea/node_modules/react-dom/cjs/react-dom-server.browser.development.js:6030:12)
    at renderHostElement (/Users/jonathanlibov/apps/bathonea/node_modules/react-dom/cjs/react-dom-server.browser.development.js:5456:3)
    at renderElement (/Users/jonathanlibov/apps/bathonea/node_modules/react-dom/cjs/react-dom-server.browser.development.js:5765:5)

This is a Next app running React 18 on Node v16.14.2

Did you figure this out? Are you calling the hook from inside a function component?

Let me rephrase - Are you using this component inside a function component or in a class component? I built this using React hooks and function components so not sure how well it performs using the alternative.

Please reopen if the issue persists.

@libovness Glad you got it working. I opened a feature request issue for that, great suggestion. I'll take a look into that next time I do maintenance on the project


index error when using interpolate with get_data

I was playing around with the smhi client and tried to use interpolate. I got a dict cannot have index error and took a closer look at the code. It seems like there is a small bug with the order in which the get_data values are assigned. It can be found in line 169 in smhi.py and looks like this:
_, tmpdata = self.get_data(parameter, nearby_station[0])
I think you only need to switch places on the underscore and tmpdata and it will do as intended.
Thank you for providing this client!
Br,
Karl

Thank you for noting. I will try to get a bugfix in tonight.


Path order in Tree search and node compare

Split off the first part of https://github.com/gg-scm/gg-git/pull/32.
Git doesn't use alphabetic order when it sorts entries in a directory - it uses the so called "path order". The main difference is that if an entry is a subdir, a "/" character is appended during the sorting. This PR aims to fix that. It does change a public API - tree.Search() now also takes a Mode param so that the search function can properly adding the trailing slash to it when necessary.
As discuss in the other PR, i also could not find any real doc on the path order. This is the comment in the Git source code where it mentions the subdir behavior:
https://github.com/git/git/blob/15030f9556f545b167b1879b877a5d780252dc16/fsck.c#L529-L536
Sorry I didn't add new tests for this. I wasn't sure what's the correct way of generating the testdata files. One type of Git repos that I think can easily reproduce the problem is anything that's created from Xcode, because it seems such projects always have the following directory structure:
myProject/
myProject.xcodeproj/
(... other stuff)

For example, https://github.com/orta/OROpenInAppCode

I ended up wanting more out of this PR than what I thought would be reasonable to ask as a review:

Avoiding the string allocation while comparing entries
Keeping the API the same by implementing a two-pass search
Creating a small test repository with the desired tree and running git cat-file tree $(git write-tree) to generate the test data
Adding tests for Tree.Search and Tree.Sort, which were totally missing before
Checking for duplicate entries between files and directories
Updating documentation to explain the sort order in an end-user friendly manner

In the interest of time, I treated this PR as a bug report and wrote 04343f2f4c173de5eaf2c2f01e64c569bcd37aeb from scratch. Thanks for the assistance, and especially for the code comment explaining this behavior. The tree format is weirdly undocumented (even more than other Git internals) and that has always bothered me.


Indexing XMPP Extension Protocols (XEPs)?

Description
I wonder if indexing XMPP Extension Protocols (XEPs) would be an option for bib.ietf.org. We, i.e., the XMPP Standards Foundation, currently provides bibxml files for all published XEPs at https://xmpp.org/extensions/refs/. Currently as one file per XEP, but it would be easy to also provide an archive with all bibxml files (e.g. https://xmpp.org/extensions/refs/all.tgz).
If you are interested, then please reach out to the XSF (editor), or just respond to this issue. I'll make sure that it reaches the right people.
Code of Conduct

[X] I agree to follow the IETF's Code of Conduct

@Flowdalic we (the Relaton project) would be very interested in distributing the bibliographic information of XEPs. All we need is confirmation that the bibliographic data is openly licensed that allows re-distribution.
Once Relaton is allowed to distribute the data, the BibXML Service can consume it easily.

Thanks for the fast reply. We are always happy to have consumers of the XEP bibliography and I believe the XSF should be able to provide such a confirmation.
Could you elaborate how the Relaton project is related to bib.ietf.org? Will bibliopgrahy consumed by Relaton automatically appear on bib.ietf.org?

Thanks for the clarification. Would it be enough if we put a text file at e.g. https://xmpp.org/extensions/refs/LICENSE that states something like

The bibliopgrahic data available under https://xmpp.org/extensions/refs/ is available openly and free of charge

(suggestions for the license text are welcome).

Thank you to @Flowdalic the XEPs bibliographic data are now under CC0, which allows redistribution.

Got it @Flowdalic -- now that https://github.com/xsf/xeps/pull/1220 is merged, we shall await your confirmation that the XSF board approves the change.

Got it @Flowdalic -- now that xsf/xeps#1220 is merged, we shall await your confirmation that the XSF board approves the change.

The Board approved the change, hence the PR was merged. Unfortunately, it was reverted due to technical reasons. There is now https://github.com/xsf/xeps/pull/1221 that fixes those.

Thank you @Flowdalic !


Image not showing on Hover but on click popup working, what I need to show both actions

What I want to make is to show an image on hover on draw layer (it can be marker, poligon, anything) and on click show a popup with GEOjson Property. I can manage to show popup but not able to show image on hover. Those are visible on console.log but on screen not appearing.
Please help me and very sorry for my bad English.
`var drawControl = new L.Control.Draw(options);
map.addControl(drawControl);
var myGeometry = '{"type":"Feature","properties":{"title":"My school","message":"This is description","school":"DPS School","name":"Rehul Pal","class":"iV"},"geometry":{"type":"Point","coordinates":[88.43603074550629,22.570362143570495]}}';
var myGeoJSON = JSON.parse(myGeometry);
//switcher to add all functions to each feature (department)
function onEachFeature(feature, layer)
{
layer.on({
click: htmlData,
mouseover: highlight,
mouseout: resetHighlight,
        });

    }

function htmlData(e){
var layers = e.layers;
html = '';
var html = "" +"Tis is POPUP"  +
"" +
""
        var options = {minWidth: 500}
        this.bindPopup(html, options);
    console.log(html, options);

}
function highlight(e)
{
var layers = e.layers;
images_url ='';
var images_url = "";
this.bindPopup(images_url);
console.log(images_url);
}
    function resetHighlight(e)
    {
       this.bindPopup();
    }

var geoJsonGroup = L.geoJson(myGeoJSON, {
onEachFeature  }).addTo(map);
addNonGroupLayers(geoJsonGroup, drawnItems);
// Would benefit from https://github.com/Leaflet/Leaflet/issues/4461
function addNonGroupLayers(sourceLayer, targetGroup) {
if (sourceLayer instanceof L.LayerGroup) {
sourceLayer.eachLayer(function (layer) {
addNonGroupLayers(layer, targetGroup);
})

} else {
targetGroup.addLayer(sourceLayer);
}
}
`

Hi, great to hear that you use Leaflet!
However, this issue tracker is used for reporting bugs and discussing new features. For questions on using Leaflet, please use gis.stackexchange.com or stackoverflow.


remove abstractscimllinearoperator in favor or trait islinear(L)

close https://github.com/SciML/SciMLOperators.jl/issues/106

@ChrisRackauckas


Arangodb 2.5 - model schema not updated in test interface

Hello,
I'm starting a foxx app with the new arangodb 2.5. It' really great! I think I found a little bug, when I change my model schema, it's not updated into the test interface. I attach an screenshot.
Thanks!
Diego

Hi Thanks for reporting this i will have a look at it now.

Thanks a lot for reporting this. In my tests i was lucky enough not to hit this bug


Can you provide a docker deployment method?

Can you provide a docker deployment method?

This is the source code of the website mclo.gs, while self-hosting is possible, it is not supported, and there are no plans to provide further configuration and deployment options for that.


Adding a parameter to plot_confusion_matrix() to hide overlaid counts

Hi @reiinakano,
Thank you for this great repo! I am using plot_confusion_matrix() but my counts are quite large so the overlaid counts end up overlapping each other and result in a cluttered plot. I was wondering if I could submit a pull request to update this function to add a hide_counts parameter to give the option to not plot the counts? I've already forked and created a branch with the changes. Thank you!

That sounds good to me. Could you also provide an example image before and after?

Great, and sure, an example image of before vs after would be:

pull request is now here: https://github.com/reiinakano/scikit-plot/pull/90

I tried pip install scikit-plot (after pip uninstall scikit-plot) but it seems like it's not installing the updated version with the new parameter (tried the function but it's not recognizing hide_counts). It however did work if I installed via pip install git+https://github.com/reiinakano/scikit-plot. What needs to be done to make it work the first way? I realized I forgot to increase the version in setup.py and I'm not sure if that's related. Thanks!

Hi @echan5, I haven't released a new version of scikit plot yet so you won't get it on pypi. I'll probably do a release over the weekend. For now, please stick with the solution you found, which is installing directly from master

Got it, thank you!

New version with your patch is now out. Thanks!


Remove leading + sign for sorting

Fixes #74

Thanks! One thing: return "\(key)" could be better written as return key though. If you could change that I'll be happy to merge this.

Done, thanks!


No brightness attribute for entity - No Clear Way To Fix

System Health details
System Information

version
core-2023.10.3

installation_type
Home Assistant OS

dev
false

hassio
true

docker
true

user
root

virtualenv
false

python_version
3.11.5

os_name
Linux

os_version
6.1.56

arch
x86_64

timezone
America/New_York

config_dir
/config

Home Assistant Community Store

GitHub API
ok

GitHub Content
ok

GitHub Web
ok

GitHub API Calls Remaining
5000

Installed Version
1.33.0

Stage
running

Available Repositories
1318

Downloaded Repositories
38

HACS Data
ok

Home Assistant Cloud

logged_in
true

subscription_expiration
July 16, 2024 at 20:00

relayer_connected
true

relayer_region
us-east-1

remote_enabled
true

remote_connected
true

alexa_enabled
false

google_enabled
true

remote_server
us-east-1-1.ui.nabu.casa

certificate_status
ready

can_reach_cert_server
ok

can_reach_cloud_auth
ok

can_reach_cloud
ok

Home Assistant Supervisor

host_os
Home Assistant OS 11.0

update_channel
stable

supervisor_version
supervisor-2023.10.0

agent_version
1.6.0

docker_version
24.0.6

disk_total
234.0 GB

disk_used
70.2 GB

healthy
true

supported
true

board
generic-x86-64

supervisor_api
ok

version_api
ok

installed_addons
Z-Wave JS (0.1.98), File editor (5.6.0), Samba share (10.0.2), Home Assistant Google Drive Backup (0.111.1), Z-Wave JS UI (2.1.2), Mosquitto broker (6.3.1), chrony (2.6.2), MariaDB (2.6.1), Grocy (0.20.1), Advanced SSH & Web Terminal (15.1.0), NGINX Home Assistant SSL proxy (3.5.0), AdGuard Home (4.8.16), RTSPtoWeb - WebRTC (1.4.0), phpMyAdmin (0.8.9), Glances (0.19.4), AWNET to HASS (1.0.1), Grafana (9.1.0), InfluxDB (4.8.0), RTSPtoWebRTC (1.0.1), Auto Guest Login (1.3.1), ESPHome (2023.9.3), OneDrive Backup (2.2.2), Tautulli (3.2.10), Emulated HUE (0.3.0), SQLite Web (3.10.0)

Dashboards

dashboards
5

resources
22

views
51

mode
storage

Recorder

oldest_recorder_run
October 2, 2023 at 14:19

current_recorder_run
October 17, 2023 at 14:20

estimated_db_size
4215.87 MiB

database_engine
sqlite

database_version
3.41.2

Checklist

[X] I have read the FAQ to see if there is a known solution to my problem.
[X] I have enabled debug logging for my installation.
[X] I have filled out the issue template to the best of my ability.
[X] This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).
[X] This issue is not a duplicate issue of currently previous issues..

Describe the issue
`This error originated from a custom integration.
Logger: custom_components.powercalc.strategy.linear
Source: custom_components/powercalc/strategy/linear.py:148
Integration: Powercalc (documentation, issues)
First occurred: 14:24:04 (1 occurrences)
Last logged: 14:24:04
No brightness attribute for entity: light.node_35`
I see this error regularly and have been unable to find a way to resolve this.   This light is set up with an automation that syncs the state with another light switch in the same room (so one of them turns on/off the other).  This switch is rarely actually hit and instead is activated via zwave commands from the other switch.  And the manual brightness values have been set for this since it is a dimmer switch, but usually used at 100% brightness.
Reproduction steps

Use automation to sync the state of two z-wave light switches
Manually set the brightness values to correspond to the power usage of the bulb(s).
turn on the switch from the other synced light switch.
...

Debug logs
2023-10-17 14:44:58.832 WARNING (MainThread) [custom_components.powercalc.strategy.linear] No brightness attribute for entity: light.kitchen_overhead
2023-10-17 14:44:58.832 DEBUG (MainThread) [custom_components.powercalc.sensors.power] light.kitchen_overhead: State changed to "on". Power:None
2023-10-17 14:44:58.897 WARNING (MainThread) [custom_components.powercalc.strategy.linear] No brightness attribute for entity: light.node_35
2023-10-17 14:44:58.897 DEBUG (MainThread) [custom_components.powercalc.sensors.power] light.node_35: State changed to "on". Power:None
2023-10-17 14:44:59.901 DEBUG (MainThread) [custom_components.powercalc.strategy.linear] light.kitchen_overhead: Linear mode state value: 255 range(255-0)
2023-10-17 14:44:59.902 DEBUG (MainThread) [custom_components.powercalc.sensors.power] light.kitchen_overhead: State changed to "on". Power:105.00
2023-10-17 14:44:59.954 DEBUG (MainThread) [custom_components.powercalc.strategy.linear] light.node_35: Linear mode state value: 255 range(255-0)
2023-10-17 14:44:59.954 DEBUG (MainThread) [custom_components.powercalc.sensors.power] light.node_35: State changed to "on". Power:15.00

Diagnostics dump or YAML config
{
"home_assistant": {
"installation_type": "Home Assistant OS",
"version": "2023.10.3",
"dev": false,
"hassio": true,
"virtualenv": false,
"python_version": "3.11.5",
"docker": true,
"arch": "x86_64",
"timezone": "America/New_York",
"os_name": "Linux",
"os_version": "6.1.56",
"supervisor": "2023.10.0",
"host_os": "Home Assistant OS 11.0",
"docker_version": "24.0.6",
"chassis": "embedded",
"run_as_root": true
},
"custom_components": {
"hacs": {
"version": "1.33.0",
"requirements": [
"aiogithubapi>=22.10.1"
]
},
"ge_home": {
"version": "0.6.8",
"requirements": [
"gehomesdk==0.5.23",
"magicattr==0.1.6",
"slixmpp==1.8.3"
]
},
"zha_toolkit": {
"version": "v1.1.2",
"requirements": [
"pytz"
]
},
"redfin": {
"version": "1.1.4",
"requirements": [
"redfin==0.1.1"
]
},
"adaptive_lighting": {
"version": "1.19.0",
"requirements": [
"ulid-transform"
]
},
"xiaomi_cloud_map_extractor": {
"version": "v2.2.0",
"requirements": [
"pillow",
"pybase64",
"python-miio",
"requests",
"pycryptodome"
]
},
"awnet_local": {
"version": "1.2.0",
"requirements": []
},
"winix": {
"version": "1.1.4",
"requirements": [
"winix==0.2.1"
]
},
"soma_connect": {
"version": "2023.8.1",
"requirements": [
"aiosoma==1.1.3"
]
},
"mail_and_packages": {
"version": "0.3.19",
"requirements": [
"beautifulsoup4",
"Pillow>=9.0"
]
},
"webrtc": {
"version": "v3.4.0",
"requirements": []
},
"google_home": {
"version": "1.10.0",
"requirements": [
"glocaltokens==0.7.0"
]
},
"powercalc": {
"version": "v1.9.5",
"requirements": [
"numpy>=1.21.1"
]
},
"watchman": {
"version": "0.5.1",
"requirements": [
"prettytable==3.0.0"
]
},
"average": {
"version": "2.3.1",
"requirements": [
"colorlog==6.7.0"
]
},
"tesla_custom": {
"version": "3.18.0",
"requirements": [
"teslajsonpy==3.9.5"
]
},
"jvcprojector": {
"version": "2.0.0a3",
"requirements": [
"jvc-projector-remote == 0.2.1"
]
},
"fordpass": {
"version": "0.1.57",
"requirements": []
}
},
"integration_manifest": {
"after_dependencies": [
"integration",
"utility_meter"
],
"codeowners": [
"@bramstroker"
],
"config_flow": true,
"dependencies": [
"light",
"group",
"template",
"select",
"utility_meter"
],
"documentation": "https://github.com/bramstroker/homeassistant-powercalc",
"domain": "powercalc",
"iot_class": "local_polling",
"issue_tracker": "https://github.com/bramstroker/homeassistant-powercalc/issues",
"name": "Powercalc",
"requirements": [
"numpy>=1.21.1"
],
"version": "v1.9.5",
"is_built_in": false
},
"data": {
"entry": {
"entry_id": "a3bda9b9847c06b69a3d267c2ae68b0f",
"version": 2,
"domain": "powercalc",
"title": "Sink",
"data": {
"entity_id": "light.node_35",
"mode": "linear",
"create_energy_sensor": true,
"create_utility_meters": false,
"linear": {
"min_power": 1.5,
"max_power": 15.0
},
"energy_integration_method": "left",
"unique_id": "pc_3600815020.35-38-0-currentValue",
"sensor_type": "virtual_power",
"name": "Sink",
"_power_entity": "sensor.sink_power",
"_energy_entity": "sensor.sink_energy"
},
"options": {},
"pref_disable_new_entities": false,
"pref_disable_polling": false,
"source": "user",
"unique_id": "pc_3600815020.35-38-0-currentValue",
"disabled_by": null
}
}
}

This is a problem somehow in your HA installation specific.
Whenever a light is on and is in color_mode brightness, it must have a brightness attribute.
This is also according to HA specs how a light entity should behave.
It's not something I can fix in powercalc, as it needs the brightness attribute to correctly lookup power value.
So you'll have to investigate why the brightness attribute is not set. Probably an issue in your automation.
You can have look into developer tools -> states to further investigate.
You should see:
color_mode: brightness
brightness: 144

But in your case brightness would be missing.

Interesting.  I'm quite sure why because after the light is on, the state shows the following in developer tools.  The other light has the same configuration, so the automation just copies the state between them.   Appreciate the response and please let me know if there's anything else I should check.
`supported_color_modes:

brightness
color_mode: brightness
brightness: 255
icon: mdi:light-recessed
friendly_name: Sink
supported_features: 32`


Added description field to the Community struct

Fixes the issue #50.
This pull request was created for https://app.gib.work/bounties/0b022a92-303f-4a02-9c99-9e0c3973a4cf in an attempt to solve a bounty #50 . Payment for the bounty is immediately sent to the contributor after merge.

Thanks for your contributions! I made some more tweaks, added the last missing pieces and merged this in!

Thanks for your contributions! I made some more tweaks, added the last missing pieces and merged this in!

Hi @beeman! I just saw your commit and noticed you handled most of the work—thank you for stepping in! Apologies for the lower-quality PR; I'm actually a bit new to Rust and Solana programming.


can't reach IdentityServer4 with Ocelot route

Perhaps i'm doing something wrong, but i'm completely stuck and don't know how to resolve this problem.
Expected Behavior / New Feature
i expect to reach to identityserver instance at desired url route defined in ocelot configuration file.
Actual Behavior / Motivation for New Feature
We're building a bigger apps deployed to Kubernetes with Ingress in-front. I struggle a lot to make to make IdenityServer4 reachable. The first stop is Ocelot. In our app, i can only reach IdenityServer4 if its route is defined at the root url of ocelot (so, http://ocelot). It will not work at any other sub-url (like http://ocelot/identity). I've created a demo app (docker-based) that describes this problem - it's available here .
Here's my ocelot.config:
{   "ReRoutes": [     {       "DownstreamPathTemplate": "/{catchAll}",       "UpstreamPathTemplate": "/identity/{catchAll}",       "UpstreamHttpMethod": [ "Get", "Options", "Post" ],       "DownstreamScheme": "http",       "ServiceName": "identity"     }   ],   "GlobalConfiguration": {     "RequestIdKey": "OcRequestId",     "AdministrationPath": "/administration"   } }
Steps to Reproduce the Problem
Please refer the demo app. It's the simplest possible app with these two components onboard.

Run demo app
IdServ4 should be visible under localhost:7060/identity

Specifications

Version: Ocelot 12.0.1
Platform: .NET Core 2.2 + Docker (<IP_ADDRESS>-win81 (29211))
Subsystem: IdentityServer4 (ver. 2.3.2)

@przemekwojcik
Hi Przemek!
Seems this issue is rather question than real issue ready for development.
Next time please open a discussion with your questions. And community will help you.

Platform: .NET Core 2.2 + Docker (<IP_ADDRESS>-win81 (29211))

Sorry, we don't support outdated versions of .NET framework. Since November'23 we support only .NET 6+ as Microsoft does it.
Stay with Ocelot!


[Quilt] java.lang.RuntimeException

Minecraft .1.19.3
Quilt 0.17.10
Game crashes on load.
Crash report
crash-2023-02-01_18.28.01-client.txt

Any updates on this? It seems to simply not work at all, I don't understand how it works fine for others and just crashes the game for me. I seemingly have everything needed as well...


add a way to run custom scripts for environment setup

Introduce: VCPKG_ENVIRONMENT_SETUP_SCRIPTS to run custom scripts instead or supplementing the vcvars set up environment.
If the script is a cmake file -> call cmake with the paths vcpkg would use -> allows downloading of stuff before vcpkg runs any other stuff like compiler detection etc.

A bit of story what I did hear: I want to install&setup Intel (Fortran) compilers before running vcpkg and have successfully compiled lapack-reference with it.
Triplet x64-win:
set(VCPKG_TARGET_ARCHITECTURE x64)
set(VCPKG_CRT_LINKAGE dynamic)
set(VCPKG_LIBRARY_LINKAGE dynamic)

set(VCPKG_ENVIRONMENT_SETUP_SCRIPTS
  "${CMAKE_CURRENT_LIST_DIR}/x64-win/download-tools.cmake"
  "${CMAKE_CURRENT_LIST_DIR}/x64-win/setup.bat" # Just calls out to generated scripts
)

# This below could be in a toolchain instead: (basically copied and adapted from one of my Fortran PRs)
set(PATH_SUFFIX "bin")
find_program(Fortran_COMPILER NAMES ifort ifort.exe
  PATHS "${DOWNLOADS}/tools/intel.2024.1/compiler/latest"
  PATH_SUFFIXES "${PATH_SUFFIX}"
  NO_DEFAULT_PATH
)
if(NOT Fortran_COMPILER AND DEFINED CURRENT_PACKAGES_DIR)
  message(FATAL_ERROR "unable to find Fortran compiler!" )
endif()

set(mach_flag "/Qm64 /QxSSE4.2")

list(APPEND VCPKG_CMAKE_CONFIGURE_OPTIONS
  "-DCMAKE_TRY_COMPILE_PLATFORM_VARIABLES=CMAKE_Fortran_FLAGS\;CMAKE_Fortran_FLAGS_RELEASE\;CMAKE_Fortran_FLAGS_DEBUG\;CMAKE_Fortran_STANDARD_LIBRARIES\;CMAKE_EXE_LINKER_FLAGS\;CMAKE_EXE_LINKER_FLAGS_RELEASE\;CMAKE_EXE_LINKER_FLAGS_DEBUG\;CMAKE_STATIC_LINKER_FLAGS\;CMAKE_STATIC_LINKER_FLAGS_DEBUG\;CMAKE_STATIC_LINKER_FLAGS_RELEASE\;CMAKE_SHARED_LINKER_FLAGS\;CMAKE_SHARED_LINKER_FLAGS_RELEASE\;CMAKE_SHARED_LINKER_FLAGS_DEBUG\;CMAKE_REQUIRED_LINK_OPTIONS"
  "-DCMAKE_Fortran_COMPILER=${Fortran_COMPILER}"
  "-DCMAKE_Fortran_FLAGS_INIT:STRING=/Z7 /names:lowercase /assume:underscore /assume:protect_parens /fp:strict ${mach_flag} /Qopenmp-"
  "-DCMAKE_Fortran_FLAGS_DEBUG_INIT:STRING=/Od /Ob0"
  "-DCMAKE_Fortran_FLAGS_RELEASE_INIT:STRING=/O2 /Ot"
)

x64-win/download-tools.cmake:
# Available variables: VCPKG_ROOT_DIR PACKAGES_DIR BUILDTREES_DIR _VCPKG_INSTALLED_DIR DOWNLOADS

# vcpkg is not passing the scripts folder down to cmake. It just invokes ports.cmake directly. However, it internally knows where the scripts are.

set(SCRIPTS "${VCPKG_ROOT_DIR}/scripts" CACHE PATH "Location to stored scripts")
list(APPEND CMAKE_MODULE_PATH "${SCRIPTS}/cmake") # This is necessary since some script is using `inlude(vcpkg_*)` instead of full paths

include("${SCRIPTS}/cmake/execute_process.cmake")
include("${SCRIPTS}/cmake/vcpkg_acquire_msys.cmake")
include("${SCRIPTS}/cmake/vcpkg_add_to_path.cmake")
include("${SCRIPTS}/cmake/vcpkg_apply_patches.cmake")
include("${SCRIPTS}/cmake/vcpkg_download_distfile.cmake")
include("${SCRIPTS}/cmake/vcpkg_download_sourceforge.cmake")
include("${SCRIPTS}/cmake/vcpkg_execute_required_process.cmake")
include("${SCRIPTS}/cmake/vcpkg_execute_required_process_repeat.cmake")
include("${SCRIPTS}/cmake/vcpkg_extract_archive.cmake")
include("${SCRIPTS}/cmake/vcpkg_extract_source_archive.cmake")
include("${SCRIPTS}/cmake/vcpkg_extract_source_archive_ex.cmake")
include("${SCRIPTS}/cmake/vcpkg_find_acquire_program.cmake")
include("${SCRIPTS}/cmake/vcpkg_from_bitbucket.cmake")
include("${SCRIPTS}/cmake/vcpkg_from_git.cmake")
include("${SCRIPTS}/cmake/vcpkg_from_github.cmake")
include("${SCRIPTS}/cmake/vcpkg_from_gitlab.cmake")
include("${SCRIPTS}/cmake/vcpkg_from_sourceforge.cmake")
include("${SCRIPTS}/cmake/vcpkg_get_program_files_platform_bitness.cmake")
include("${SCRIPTS}/cmake/vcpkg_host_path_list.cmake")
include("${SCRIPTS}/cmake/vcpkg_list.cmake")

include("${SCRIPTS}/cmake/z_vcpkg_apply_patches.cmake")
include("${SCRIPTS}/cmake/z_vcpkg_forward_output_variable.cmake")
include("${SCRIPTS}/cmake/z_vcpkg_function_arguments.cmake")
include("${SCRIPTS}/cmake/z_vcpkg_prettify_command_line.cmake")

message(STATUS "***** Running env setup cmake script *****")

#vcpkg_find_acquire_program(PYTHON3)
vcpkg_find_acquire_program(7Z)

# Setup Intel Compiler for Fortran

# Download link from https://github.com/oneapi-src/oneapi-ci/blob/master/.azure-pipelines.yml
block()
  find_program(Fortran_COMPILER NAMES ifort ifort.exe
    PATHS "${DOWNLOADS}/tools/intel.2024.1/compiler/latest"
    PATH_SUFFIXES "${PATH_SUFFIX}"
    NO_DEFAULT_PATH
  )
  if(NOT Fortran_COMPILER)
    vcpkg_download_distfile(archive_path
        URLS "https://registrationcenter-download.intel.com/akdlm/IRC_NAS/c95a3b26-fc45-496c-833b-df08b10297b9/w_HPCKit_p_2<IP_ADDRESS>1_offline.exe"
        FILENAME "w_HPCKit_p_2<IP_ADDRESS>1_offline.exe"
        SHA512 84d94b3b6e240325e8d8b5018ed57df888653b912e3132ec91b7d89164f6d36b411975d65f981e366012fa05a582c0451d55dd81409a312baa3f1c6b80cae87b
    )

    set(extract_path "${DOWNLOADS}/tools/intel-extract")
    file(MAKE_DIRECTORY "${DOWNLOADS}/tools/intel-extract")
    vcpkg_execute_in_download_mode(
                            COMMAND "${7Z}" x "${archive_path}" "-o${extract_path}" "-y" "-bso0" "-bsp0"
                            WORKING_DIRECTORY "${extract_path}"
                        )

    set(packages
            "intel.oneapi.win.openmp,v=2024.1.0+964"
            "intel.oneapi.win.ifort-compiler,v=2024.1.0+964"
            "intel.oneapi.win.compilers-common,v=2024.1.0+964"
            "intel.oneapi.win.compilers-common-runtime,v=2024.1.0+964"
            "intel.oneapi.win.oneapi-common.vars,v=2024.1.0+582"
        )

    foreach(package IN LISTS packages)
      vcpkg_execute_required_process(
          COMMAND "${CMAKE_COMMAND}" "-E" "tar" "-xf" "${extract_path}/packages/${package}/cupPayload.cup"
          WORKING_DIRECTORY "${extract_path}"
          LOGNAME "extract-${package}"
      )
    endforeach()

    file(REMOVE_RECURSE "${DOWNLOADS}/tools/intel.2024.1")
    file(RENAME "${extract_path}/_installdir/" "${DOWNLOADS}/tools/intel.2024.1")
    # The env setup script assumes that the "latest" folder exists, so rename the version folder (alternnative would be to create a link)
    file(RENAME "${DOWNLOADS}/tools/intel.2024.1/compiler/2024.1"  "${DOWNLOADS}/tools/intel.2024.1/compiler/latest")
  endif()

  # Generate setup script
  file(MAKE_DIRECTORY "${CMAKE_CURRENT_LIST_DIR}/gen") # Here goes the generated scripts
  file(WRITE "${CMAKE_CURRENT_LIST_DIR}/gen/setup-intel-env.bat"
  "
  set \"ONEAPI_ROOT=${DOWNLOADS}/tools/intel.2024.1\"
  \"${DOWNLOADS}/tools/intel.2024.1/setvars.bat\" \"compiler\"
  ")
endblock()
# Setup LLVM Compiler

message(STATUS "***** ENV setup cmake script finished *****")

What do I get here:

every download performed in download-tools.cmake runs through vcpkg and is thus asset cached.
The scripts run before the compiler detection, so the environment is correctly setup
The script is only run once instead for every port (don't know if per port customization is possible?)

Unwanted behavior:

If CMake errors vcpkg just continues execution as normal? (Need to test if that also applies to other scripts?)

Extensions:

Use this to obtain MSBuild itself?
Use this to obtain EMSCRIPTEN toolchain?
Use this to obtain a MSYS environment?
Use this to install system deps automatically?

Note: I now also have a env setup script which can setup emscripten

Could this be done inline in the triplet file by defining an "IN_PASS_0" variable? Then the user could execute_process() whatever they'd like.


Update now - add formula conflict

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} reports no offenses.
[x] The commit message includes the cask’s name and version.

https://github.com/Homebrew/homebrew-core/pull/14827

Why does this conflict with the formula? It doesn't seem to install a binary.

The Now.app installs and updates the same binary as now-cli.

$ ls -lh /usr/local/bin
total 123488
lrwxr-xr-x  1 commitay  admin    28B 29 Jun 16:58 brew -> /usr/local/Homebrew/bin/brew
-rwxr-xr-x  1 commitay  staff    60M 30 Jun 11:24 now


google_sign_in: Not possible to re-authenticate with existing credentials

The google_sign_in plugin at 5.0.7 deliberately gives back the same user from signIn and signInSilently once a sign in has succeeded. This is great for working with access tokens because a sign in is not required to refresh them.
The Android docs  for silentSignIn recommend its use to get a new ID token when required.
It would be great if the application using the plugin could make the decision about whether the platform silent sign in should be called.

I have the same problem.  Steps to reproduce:

Sign in with Google on App
Go to Google security settings and revoke access to App
Sign in with Google on App again

At this point, it will give me an access token, but if I try to use it, it will throw 401.  The access token is supplied.


Respect mutes and notification type settings for push notifications

We currently only respect mutes for push notifications wrt replies.
We don't respect any of the notification type settings (as set in /settings) for push notifications afaict.

We don't respect any of the notification type settings (as set in /settings) for push notifications afaict.

We have this function in api/webPush.js:
const createUserFilter = (tag) => {
  // filter users by notification settings
  const tagMap = {
    REPLY: 'noteAllDescendants',
    MENTION: 'noteMentions',
    TIP: 'noteItemSats',
    FORWARDEDTIP: 'noteForwardedSats',
    REFERRAL: 'noteInvites',
    INVITE: 'noteInvites',
    EARN: 'noteEarning',
    DEPOSIT: 'noteDeposits',
    STREAK: 'noteCowboyHat'
  }
  const key = tagMap[tag.split('-')[0]]
  return key ? { user: { [key]: true } } : undefined
}

which is called inside of sendUserNotification to filter push subscriptions by user settings:
const userFilter = createUserFilter(notification.tag)
const payload = createPayload(notification)
const subscriptions = await models.pushSubscription.findMany({
  where: { userId, ...userFilter }
})

Is this not what you mean?

Oh my bad, the abstraction is so lightweight I missed it! I'll update the issue

I think it would be helpful to enumerate the push notifications supported today, and then build a truth table as to whether they should take mutes into account, based on the code in lib/webPush.js:

Push Notification Type
Mutes should apply?
Does mute currently apply?
Required Fix?

REPLY
Yes
Yes
❌

MENTION
Yes
No
✅

FOLLOW
Yes
No
✅

TERRITORY_POST
Yes
No
✅

TERRITORY_TRANSFER
No
N/A
❌

TIP
No
N/A
❌

FORWARDEDTIP
No
N/A
❌

REFERRAL
No
N/A
❌

INVITE
No
N/A
❌

EARN
No
N/A
❌

DEPOSIT
No
N/A
❌

WITHDRAWAL
No
N/A
❌

STREAK
No
N/A
❌

STREAK-LOST
No
N/A
❌

So to summarize, this issue will require updating the push notification code to honor mutes in the following scenarios:

Don't send a push if a stacker that you've muted, mentions you in an item.
Don't send a push if a stacker that you've muted, and also subscribed to, creates an item type to which you're subscribed. This one is funny, are people really muting subscribed users?
Don't send a push if a stacker that you've muted submits a post to a territory to which you've subscribed.

Did I get all of them? Any others? Anything wrong?

Mhh, yes, I think this makes sense. The alternative would be to allow it and overwrite the previous setting but I like explicit things and this should be an edge case so it's probably not worth thinking too much about it. A simple check in the backend and throwing an error seems best.

Will do! Also, please review the above table to see if I missed anything in my assessment, if you get a chance :) I have changes locally that I'll probably push for review this weekend.


catkin_make fails to find proper python env

The setup script c:\opt\ros\melodic\x64\setup.bat properly sets the PYTHONHOME env var, but it does not set PYTHON_EXECUTABLE, which catkin_make references independently.  On a system with many python interpreters, this means catkin_make uses the wrong environment entirely (it uses the environment of the first-found interpreter, and not the one in PYTHONHOME).
I'd call this a bug in catkin_make, unless they have a good reason (unlikely) for using a random interpreter.  But nevertheless this appears to happen only on Windows.
To reproduce, install two python 2 environments and then ROS using the second environment.  Then install any ROS repo and run catkin_make.  Unless both python environments have empy It will fail with "python can't find library em..." (meaning empy).
Workaround:  manually add that PYTHON_EXECUTABLE env var to the ROS setup.  It must have the full path to the python executable.  (But you can't add it to setup.bat because it's autogenerated.)
OS: Windows 10
Chocolatey:  v0.10.15

Thank you for the writeup and investigation. We've been trying to map the ROS environment to a locally installed python path in c:\opt, so that it is isolated from potentially tainted environments outside of the ROS environment.

we will consider checking PYTHONHOME in the setup and offer a warning during setup that a previous python environment was detected. the user can still click to continue setup.


Add calibration set for RNN-T benchmark

Add calibration set for RNN-T benchmark
@galv @christ1ne @guschmue @psyhtest  Could you guys take a look? Thanks
All the credit to my team, not me :)

@christ1ne Could you merge this if there is no other issue? Thanks


v.3.1.0 alpha automatically opening gw2 news page after login

I did not like this new feature implemented in 3.1.0 and I'd like to disable it if I can.
However after checking all settings I did not find and option to disable it. Am I missing something?
Not fun to have multiple tab wasting resources and having to manually close them when you login 5 accs at a time, 40 a day
Ty!!
ps: at least for me it's this page it's opening

It's doing the same for me.

It's doing the same for me.

I was trying to troubleshoot this problem so I could give more info about it and apparently it has something to do with "Press gameclient login button on gamestart". If this option is not ticked then it won't open the news page.
If "Press gameclient login button on gamestart" is ticked, I have the feeling that launchbuddy is auto clicking twice. Once at login button and a second time at the banner that appears at the same place when you login successfully. And this is what is redirecting to gw2 news page.
So I do think we're talking about a possible bug and not a feature that was implemented on purpose.

Same issue for me on german client.
It is good to see, that my wish to skip 2FA reminder is implemented, but the mouse clicking feature maybe really double clicks at 2FA/Login area or is not yet ready for other language UI sizings or something like that.

Setting can be disabled here

Currently working on language detection so I can press buttons accordingly. Workarround would be to use anything other than the "none" authentication type in the account settings.

Fixed in the next release ;)

Fixed in the next release ;)

Sounds better than disabling "Press gameclient 'Login Button' on gamestart" completely, because that's the reason we use GW2LB ;D
This news page opening drives me crazy ... but it is not always happening ... strange behaviour.
Hopefully the new version is coming soon to github ;)
--
For the 2FA topic:
I realized, that not only SMS is a 2FA possibility but also an time based authenticator.
Using such an authenticator app does not need personal phone numbers for 2FA.
Additionally ArenaNet also gives the possibility to save your current network where you come from.
Now I don't have that nasty 2FA reminder anymore for my accounts :D

Ya. I've had the odd one opening up from time to time but just had the news page open up in 8 browser tabs while logging in 1 account


When working with weapons with skill melee or brawl, lock the damage field to prevent confusion.

Base damage for those types is based on brawl, and actual damage is listed as a damage modifier.  When the melee or brawl skill is selected, disable the damage field input.

Added to dev branch.


Validate radio button group not individual inputs.

If I do the following there is a blip on the validation.

From the Zip Code tab to the Yes & No radio group
The individual input Yes has focus, but not checked.
Click on No. This will cause a 'blur` event on the Yes input which will cause validation.
You will see the validation error blip in the split second going from Yes to No.
Then once No is checked the validation passes and the error is removed.

I can understand why my question is can this be fixed and if so how?

I just noticed that if I have the focus on the Yes but not checked then move to the right using the cursor key I don't get this behavior.

would work better using <yes-no value.bind="multipleLocations & validate"></yes-no>

How? What if I did not have a yes-no component? How do I blur on components?

if you need to validate on blur, and the component that represents your property is made up of two radio inputs, the best thing to do is go ahead and create that yes-no component.

Can you show in code rather than words? I'm having a hard time wrapping my brain around how to take a group of HTML inputs and know when I leave one those inputs that I'm leaving the component instead of leaving the one of the inputs to another within the component. How would I write the code or what would the code look like that says you have left the component now when to fire a 'blur' event?

for the record, https://github.com/aurelia/validation/issues/271#issuecomment-235113128 worked for me as follows:
<template bindable="value">

<input type="radio" name="b_${field}" model.bind="opt[1]" checked.bind="value"/>${opt[0]}
</template>

<template>
<require from="components/yes-no.html"></require>
Do you agree?

<yes-no value.two-way="result & validate"></yes-no>

@jdanyow could you please clarify how to fire the 'blur' event?

ah, looks like https://github.com/aurelia/validation/issues/297 is supposed to answer this.  Currently, it only links to a solution when the custom element has one focusable element.

I was able to get it to work by using validateOnChange. yay!

http://stackoverflow.com/questions/43894214/aurelia-validation-validating-multiple-checkboxes/43896083


Feature request: Route tables showing subnet associations

aws ec2 describe-route-tables shows an array of associations. It would be great if awless list routetables would do the same. awless show rtb-xxxxxxxx also does not show the associations.
Would love to have this information, or perhaps I am looking in the wrong place? Thanks so much.

Thanks for the suggestion, this is a good idea. This will also make more easy the usage awless detach routetable association=....


Run on master

When creating a new WC, the order in which Apps are installed is random and unpredictable.
It might happen that VPA app is installed before cilium, thus trying to run in a cluster which is fundamentally not working (no CNI).
This Chart has 2 jobs that run as hooks on helm release creation: certgen and crd-patch.
Those jobs can't run if the cluster is not ready and the helm install operation hangs for several minutes (sometimes even half an hour) blocking installation of other applications and causing a lot of trouble.
Now, the 2 jobs only require to talk to the API server to do what they're meant to do, so they not need any CNI or working cluster, but really just a working API server.
This PR makes the following changes to the 2 jobs:

run on host network
run on master nodes only
tolerate notReady taint
add 2 env variables to reach API server on localhost

with this trick, the jobs can safely run even on very broken clusters and chart installation time is dramatically improved.
Now, I understand this is hacky, but considering those are one-off jobs we could accept them.
WDYT?

Is there a way to tweak the helm timeout so that these jobs fail earlier instead of blocking helm while trying to reach the api when the CNI is not there? That way it would fail earlier and retry later, when the CNI may be already running.

Unfortunately I have no idea

For example kyverno sets a 10m timeout for installing / upgrading. Maybe we can do the same with a small timeout, so that it fails quick and retry later.

Will give it a try thanks

For example kyverno sets a 10m timeout for installing / upgrading. Maybe we can do the same with a small timeout, so that it fails quick and retry later.

doing this is a no-go IMHO. In legacy, default apps' CRs are created by cluster operator and I want to avoid yet another
if app.Name == "BLAH" {
  yet := "another hack"
}


fix: extension settings

:tada: This PR is included in version 1.15.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.14.0-dev.9 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Finnish Pack

Published to Play-Store.
Waiting for FDroid: https://gitlab.com/fdroid/fdroiddata/merge_requests/2014


As a CD automation service account I need to completely create/delete the landing zone from the target organization

use case: full CD automation of the boostrap/common/non-prod/prod cloud build deployments - with full tear down of everything

A side case would be to pull from upstream and merge into the CSR
this case is for a complete reuse of the org - back to a clean state (minus required roles)

indirect collaboration: https://github.com/GoogleCloudPlatform/pubsec-declarative-toolkit/issues/91

ol side - bootstrap dev env on remote gcp alternate.gcp.zone
welcome to Cloud Shell! Type "help" to get started.
To set your Cloud Platform project in this session use “gcloud config set project [PROJECT_ID]”
root_@cloudshell:~$ git config --global user.email "mi...rg"
root_@cloudshell:~$ git config --global user.name "Mich...en"
root_@cloudshell:~$ gcloud projects create pbmm-on-gcp-onboarding-agz --name="pbmm-on-gcp-onboarding-agz" --labels=type=dev
Create in progress for [https://cloudresourcemanager.googleapis.com/v1/projects/pbmm-on-gcp-onboarding-agz].
Waiting for [operations/cp.6587421635639881883] to finish...done.
Enabling service [cloudapis.googleapis.com] on project [pbmm-on-gcp-onboarding-agz]...
Operation "operations/acat.p2-494519740861-8267d647-dbcf-4721-93ed-8287af4aec46" finished successfully.
root_@cloudshell:~$ gcloud config set project pbmm-on-gcp-onboarding-agz
Updated property [core/project].
root_@cloudshell:~ (pbmm-on-gcp-onboarding-agz)$ git clone https://github.com/obriensystems/pbmm-on-gcp-onboarding.git
Cloning into 'pbmm-on-gcp-onboarding'...
remote: Total 2563 (delta 588), reused 315 (delta 264), pack-reused 1827
Receiving objects: 100% (2563/2563), 3.11 MiB | 16.60 MiB/s, done.
Resolving deltas: 100% (1616/1616), done.
root_@cloudshell:~ (pbmm-on-gcp-onboarding-agz)$ cd pbmm-on-gcp-onboarding/
root_@cloudshell:~/pbmm-on-gcp-onboarding (pbmm-on-gcp-onboarding-agz)$ vi README.md
root_@cloudshell:~/pbmm-on-gcp-onboarding (pbmm-on-gcp-onboarding-agz)$ git status
On branch main
Your branch is up to date with 'origin/main'.
Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
        modified:   README.md
root_@cloudshell:~/pbmm-on-gcp-onboarding (pbmm-on-gcp-onboarding-agz)$ git add README.md
root_@cloudshell:~/pbmm-on-gcp-onboarding (pbmm-on-gcp-onboarding-agz)$ git commit -m "#166 - verify upstream from sandbox gcp"
[main c605281] #166 - verify upstream from sandbox gcp
 1 file changed, 1 insertion(+), 1 deletion(-)
root_@cloudshell:~/pbmm-on-gcp-onboarding (pbmm-on-gcp-onboarding-agz)$ git push origin main
Username for 'https://github.com': obriensyste..ms
Password for<EMAIL_ADDRESS>Enumerating objects: 5, done.
Counting objects: 100% (5/5), done.
Delta compression using up to 2 threads
Compressing objects: 100% (3/3), done.
Writing objects: 100% (3/3), 321 bytes | 321.00 KiB/s, done.
Total 3 (delta 2), reused 0 (delta 0), pack-reused 0
remote: Resolving deltas: 100% (2/2), completed with 2 local objects.
To https://github.com/obriensystems/pbmm-on-gcp-onboarding.git
   06c833f..c605281  main -> main
root_@cloudshell:~/pbmm-on-gcp-onboarding (pbmm-on-gcp-onboarding-agz)$ git status
On branch main
Your branch is up to date with 'origin/main'.
nothing to commit, working tree clean

20240406: Closing issue during retrofit/rebase of this TEF V1 based/modified repo to TEF V4 standards
This issue may participate in the LZ refactor after rebase
Query on all issues related to the older V1 version via the tag
https://github.com/GoogleCloudPlatform/pbmm-on-gcp-onboarding/labels/2024-pre-tef-v4


libsql: add durable_frame_num caching and metadata file

This PR adds both a metadata file that is used to cache the durable_frame_num to allow us to only send the frames the server needs. This also adds some basic logging to allow us to track what frames and generations are being pushed to the server.
Closes #1818
❯ RUST_LOG=libsql::sync=debug cargo run --example offline_writes
    Blocking waiting for file lock on build directory
   Compiling libsql v0.6.0 (/home/lucio/code/libsql/libsql)
    Finished `dev` profile [unoptimized + debuginfo] target(s) in 10.14s
     Running `target/debug/examples/offline_writes`
2024-11-19T19:57:20.054954Z DEBUG libsql::sync: no metadata info file found
Please write your entry to the guestbook:
hello
You entered: hello

Guest book entries:
  cargo run --example offline_writeshello

  hello

  hello

  hello

  hello

Syncing database to remote...
2024-11-19T19:57:23.677365Z DEBUG push_one_frame{generation=1 frame_no=1}: libsql::sync: pushing frame
2024-11-19T19:57:23.781639Z DEBUG push_one_frame{generation=1 frame_no=1}: libsql::sync: frame successfully pushed durable_frame_num=6
2024-11-19T19:57:23.781975Z DEBUG push_one_frame{generation=1 frame_no=7}: libsql::sync: pushing frame
2024-11-19T19:57:23.837472Z DEBUG push_one_frame{generation=1 frame_no=7}: libsql::sync: frame successfully pushed durable_frame_num=6
Done!
libsql on  lucio/cache-server-frame [$!?] via 🦀 v1.81.0
❯ RUST_LOG=libsql::sync=debug cargo run --example offline_writes
    Finished `dev` profile [unoptimized + debuginfo] target(s) in 0.23s
     Running `target/debug/examples/offline_writes`
Please write your entry to the guestbook:
bye
You entered: bye

Guest book entries:
  cargo run --example offline_writeshello

  hello

  hello

  hello

  hello

  bye

Syncing database to remote...
2024-11-19T19:57:42.074752Z DEBUG push_one_frame{generation=1 frame_no=7}: libsql::sync: pushing frame
2024-11-19T19:57:42.118522Z DEBUG push_one_frame{generation=1 frame_no=7}: libsql::sync: frame successfully pushed durable_frame_num=7
2024-11-19T19:57:42.118936Z DEBUG push_one_frame{generation=1 frame_no=8}: libsql::sync: pushing frame
2024-11-19T19:57:42.162425Z DEBUG push_one_frame{generation=1 frame_no=8}: libsql::sync: frame successfully pushed durable_frame_num=7
Done!

@penberg in the logs above I see it is re-sending frame 7 because the server on the previous call returned max_frame_no 6 when we sent frame_no 7. Is there an off by one on the server side or is that to be expected with the protocol?

@LucioFranco What happens if the metadata file is corrupted? The code should recover from that by trying to push the whole log.

@LucioFranco I corrupted the info file as follows:
{"hash":3168545583"version":0,"durable_frame_num":120,"generation":1}

and now I see this error:
Running `/Users/penberg/src/tursodatabase/libsql/target/debug/examples/offline_writes`
thread 'main' panicked at /Users/penberg/src/tursodatabase/libsql/libsql/src/sync.rs:161:78:
called `Result::unwrap()` on an `Err` value: Error("expected `,` or `}`", line: 1, column: 19)
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

As I said before, the info file is for caching purposes. If we fail to read it, we should just discard it and regenerate it after chatting with the server.

@penberg that has been fixed/implemented and there are a bunch of tests including a corrupted metadata file.


Leading comments on export statements are not attached properly

Leading comments on export statements are not attached properly.
Steps To Reproduce
Considering this file:
/* LEADING */
export {/* INSIDE */ FOO} from './bar';

The comments are attached as expected:
Program {
  directives: Array []
  comments: Array [
    CommentBlock {
      id: '0'
      value: ' LEADING '
    }
    CommentBlock {
      id: '2'
      value: ' INSIDE '
    }
  ]
  body: Array [
    ExportExternalDeclaration {
      leadingComments: Array ['0']
      source: StringLiteral {value: './bar'}
      namedSpecifiers: Array [
        ExportExternalSpecifier {
          exported: Identifier {name: 'FOO'}
          leadingComments: Array ['2']
          local: Identifier {name: 'FOO'}
        }
      ]
    }
  ]
}

LEADING is attached to ExportExternalDeclaration
INSIDE is attached to ExportExternalSpecifier

However, if INSIDE is removed:
/* LEADING */
export {FOO} from './bar';

Then LEADING is attached to both nodes, which makes formatting way more complicated since the specifier is tokenized before the declaration.
Program {
  directives: Array []
  comments: Array [
    CommentBlock {
      id: '0'
      value: ' LEADING '
    }
  ]
  body: Array [
    ExportExternalDeclaration {
      leadingComments: Array ['0']
      source: StringLiteral {value: './bar'}
      namedSpecifiers: Array [
        ExportExternalSpecifier {
          exported: Identifier {name: 'FOO'}
          leadingComments: Array ['0']
          local: Identifier {name: 'FOO'}
        }
      ]
    }
  ]
}

Expected Results
In the last example, LEADING must be attached only to ExportExternalDeclaration.
Note: It works as expected for import statements!

I think this has something to do with us converting ExportLocalSpecifier into a ExportExternalSpecifier, and when we do the comment attachment, we correctly try to remove the leading comment from the specifier, but since we've cloned the node it has no effect.


[admin] Allow administrators to delete other users #128

Fixed the issue by adding a has_delete_permission to
UserAdmin which returns true when the necessary
conditions are met. I Also updated the Administrators'
default permissions by adding email permissions to it
Fixes #128

Coverage decreased (-0.2%) to 97.244% when pulling 3c1af2bbc5189c7eb5b82adec0e8d6430af5afd8 on NoumbissiValere:allow_admins into e731a45d384fb1145bcec4fedab579d1e2988b14 on openwisp:master.

@nemesisdesign it is done :smile:

Thank you @nemesisdesign

@NoumbissiValere After this was merged, openwisp-radius's extended openwisp-user is giving error:
======================================================================
FAIL: test_staff_bulk_delete (openwisp2.sample_users.tests.TestUsersAdmin)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/python3.7/unittest/case.py", line 59, in testPartExecutor
    yield
  File "/usr/lib/python3.7/unittest/case.py", line 615, in run
    testMethod()
  File "/home/atb00ker/.local/share/virtualenvs/openwisp-radius-6lLKGd6R/lib/python3.7/site-packages/openwisp_users/tests/test_admin.py", line 769, in test_staff_bulk_delete
    self.assertEqual(r.status_code, 200)
  File "/usr/lib/python3.7/unittest/case.py", line 839, in assertEqual
    assertion_func(first, second, msg=msg)
  File "/usr/lib/python3.7/unittest/case.py", line 832, in _baseAssertEqual
    raise self.failureException(msg)
AssertionError: 403 != 200

======================================================================
FAIL: test_staff_delete_staff (openwisp2.sample_users.tests.TestUsersAdmin)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/python3.7/unittest/case.py", line 59, in testPartExecutor
    yield
  File "/usr/lib/python3.7/unittest/case.py", line 615, in run
    testMethod()
  File "/home/atb00ker/.local/share/virtualenvs/openwisp-radius-6lLKGd6R/lib/python3.7/site-packages/openwisp_users/tests/test_admin.py", line 674, in test_staff_delete_staff
    self.assertEqual(r.status_code, 200)
  File "/usr/lib/python3.7/unittest/case.py", line 839, in assertEqual
    assertion_func(first, second, msg=msg)
  File "/usr/lib/python3.7/unittest/case.py", line 832, in _baseAssertEqual
    raise self.failureException(msg)
AssertionError: 403 != 200

Is this a backward incompatible change? Do I need to perform any steps to make it work in openwisp-radius's sample_users app?
Thanks! :smile:

@atb00ker it's a permission error. Have you ran the migrations ?

@NoumbissiValere
 SAMPLE_APP=1 python manage.py makemigrations sample_users
No changes detected in app 'sample_users'

Obviously @atb00ker since you've not perform changes to the models.   Please try SAMPLE_APP =1 ./manage.py migrate

Obviously @atb00ker since you've not perform changes to the models. Please try SAMPLE_APP =1 ./manage.py migrate

You don't need to migrate when running tests. Just to avoid confusion, the error was noticed when running tests.

No manual migrations don't get automatically created. (P.S when extending apps, we have to manually create such migration changes as done for 0002  migration as well, it's manually created in extended apps! :smile: )

Ok :smile:


Hot Module Reloading not working

Maybe it is a conflict with React Router.
Error example:
GET http://<IP_ADDRESS>:8080/chat/d1207e136d3a4f3ee911.hot-update.json 404 (Not Found)
Should /chat be part of the path?
HMR works at root path.

Solved when we started using create-react-app


[Feature] Add ability to set the caller display name

By default we set the display name to the phone number. I propose we modify the TwilioVoice.connect method to accept a contactIdentifier string similar to https://github.com/react-native-webrtc/react-native-callkeep#startcall and add a new method to update active call invites similar to https://github.com/react-native-webrtc/react-native-callkeep#updatedisplay.

We're now looking into CallKeep and if we should implement it.
Read our update here #158

This issue is partly resolved. This PR works for incoming calls: https://github.com/hoxfon/react-native-twilio-programmable-voice/pull/170


this.scale.setGameSize(window.width, window.height) throwing error

Version

Phaser Version: 3.52.0
Operating system: Linux 5.4.0-58-generic #64-Ubuntu SMP
Browser: 1.0.4044.129 (Official Build) (64-bit)

Description
The scene.scale.setGameSize() method results in an error with the following stack trace:
Uncaught Error: Framebuffer status: Incomplete Attachment
    at WebGLRenderer.createFramebuffer (phaser.js:81326)
    at RenderTarget.resize (phaser.js:35261)
    at WebGLRenderer.emit (phaser.js:1928)
    at WebGLRenderer.resize (phaser.js:80268)
    at WebGLRenderer.onResize (phaser.js:80191)
    at ScaleManager.emit (phaser.js:1935)
    at ScaleManager.refresh (phaser.js:96561)
    at ScaleManager.setGameSize (phaser.js:96397)
    at Game.create (main.js:12)
    at SceneManager.create (phaser.js:98721)

Example Test Code
import Phaser from './lib/phaser.js'

class Game extends Phaser.Scene {
  constructor () {
    super('game')
  }

  preload () { }

  create () {
    this.scale.setGameSize(window.width, window.height)
  }

  update (time, delta) { }
}

export default new Phaser.Game({
  type: Phaser.AUTO,
  scale: {
    parent: 'mygame',
    mode: Phaser.Scale.FIT,
    autoCenter: Phaser.Scale.CENTER_BOTH,
    width: 800,
    height: 600
  },
  audio: {
    disableWebAudio: true
  },
  dom: {
    createContainer: true
  },
  scene: [Game]
})

Additional Information
I have no reason to believe this is specific just to one browser. I also tried to move the setGameSize invocation to update(), but the same result happens.

This issue has been mentioned on Phaser. There might be relevant details there:
https://phaser.discourse.group/t/full-screen-scaling-and-resizing-question/8710/4

This issue has been mentioned on Phaser. There might be relevant details there:
https://phaser.discourse.group/t/full-screen-scaling-and-resizing-question/8710/4

My bad. I should have pased window.innerWidth, window.innerHeight, not width and height, which are not defined.

My bad. I should have pased window.innerWidth, window.innerHeight, not width and height, which are not defined.


Replaced CRA with Vite and Started work on Automatic Frontend Build

Changes

Replaced Create-React-App with Vite

speeds up dev load time drastically
dependancies are cleaner and easier to audit in package.json
ability to speed things up further in the future with plugins
may want to change eslint config in the future

Added environment variable api to build

setting VITE_API_ENDPOINT on build controls what api the frontend will use

Testing building the frontend within the code pipeline

this does not do anything yet, its just to see if the build is successful and we can see its artifacts in the bucket
this shouldn't cause any problem on prod, at most it will fail the pipeline and we can fix it

Deployment
https://d1kquov5x5h6vg.cloudfront.net/

Consider splitting the CRA -> Vite & automatic CodePipeline build changes into separate PRs. I think we could merge the Vite change as is, but we should wait to merge the CodePipeline changes until they're more complete.
As a general rule, smaller PRs are often easier to review. That will allow you to merge & get your changes to the rest of the team faster.

Consider splitting the CRA -> Vite & automatic CodePipeline build changes into separate PRs. I think we could merge the Vite change as is, but we should wait to merge the CodePipeline changes until they're more complete.
As a general rule, smaller PRs are often easier to review. That will allow you to merge & get your changes to the rest of the team faster.

Just removed the pipeline testing change commit, i'll open a PR specifically for that; so we can easily revert it if pipeline addition doesn't work. @ophilli we should discuss in more detail what kind of changes are required for this,  as I think the complexity of our cdk deployment makes building the site for each stage really difficult to accomplish.


think through the model of sanity schemas

Before diving head first into sanity integration, it would be wise to think hard about each schema we're going to be using in this project and store them somewhere canonically (should be fine to store in this issue thread)

Data needed for each page that should come from sanity (as per @melanie-y-zhang's figma mockup):
Home Page

Main Announcement Image
Latest Youtube Videos

About Page

About text

Schedule Page
nothing currently, all data is served from API
Podcasts
Latest Podcasts (from spotify?)
Sports
Show Archives
Articles


BUG - Plant renamed to "alias"

After HA restart now plant name is renamed to "alias"

@Saentist I've just logged into https://server.growatt.com using the credentials you gave me a while back.
The plugin pulls the names of the plant and inverters from the server directly, the inverter is called "alias" in the Growatt UI:

Did you change it recently??
This is definitely not a bug in the integration.

Not changed by me, I use user account and no one use admin account with is strange.
and all HA graphs are gone for last days.
Guess someone have greatest idea to exclude non Latin letters i names.

Have you tried naming it back to what it was before? Either way, this isn't a problem with the Integration it's just pulling the values from the server.

Yes I rename it and all entities light as normal on embeded.
but on 1.1 need removing...

It should work the same with this integration as well, the code is identical

different naming
v1.1
sensor.home_total_total_money_today_2
sensor.alias_ac_frequency

embeded
sensor.home_total_total_money_today
sensor.pokriv_ac_frequency

Ahhh that would have happened if you were trying to use both integrations at the same time, they provide the same values so you can't have them both running at the same time.


licensee directory fails, but licensee file succeeds

This behaviour occurred when working with the repository https://github.com/evhub/coconut.
When running licensee ./coconut, the correct file, LICENSE.txt, is identified, but licensee fails to detect the license. If I instead directly run licensee ./coconut/LICENSE.txt, however, licensee correctly detects that the file is a 96% match for Apache 2.0.
I don't understand how this could be anything other than a bug, since licensee is detecting the correct license file, and knows what type of license that license file is, but still can't detect the license. Some help understanding what's going on and potential workarounds would be appreciated.
Here's the full log from my terminal session:
$ licensee ./coconut
License file: LICENSE.txt
Attribution: Copyright 2015-2016 Evan Hubinger
License: Not detected

Here's the closest licenses:
$ licensee ./coconut/LICENSE.txt
License file: LICENSE.txt
Attribution: Copyright 2015-2016 Evan Hubinger
License: Apache License 2.0
Confidence: 96.58%
Method: Licensee::Matchers::Dice

I can't speak to the inconsistent behavior (and can't recreate it), but the the license not being recognized is the expected behavior, as it has the apache boilerplate prepended. See https://github.com/benbalter/licensee/issues/103.

@benbalter Modified the license file not to have the boilerplate prepended (https://github.com/evhub/coconut/blob/develop/LICENSE.txt), and I still get the same behaviour.

@evhub compare that license text with http://choosealicense.com/licenses/apache-2.0/. The licenses has a copyright header added to the top and the license boilerplate removed from the end and is properly being detected as not matching the Apache license.

@benbalter For other repositories with similar Apache 2.0 licenses (with no boilerplate at the bottom), however, licensee seems to work correctly. For example, this repository also has a version of the Apache 2.0 license without the boilerplate at the bottom and with a copyright header at the top, and yet is detected as Apache 2.0. Thus, it seems to me like something more is going on than just that the license file doesn't include the boilerplate (which isn't part of the legal license, and so theoretically shouldn't be necessary).


core/shared/platform/nuttx: mock several APIs for libc-wasi

Is it better to add these dummy implementations to NuttX ?

Is it better to add these dummy implementations to NuttX ?

i don't see the point to make an OS provide non-working impl.

But once these API implemented, we must remove these code.

But once these API implemented, we must remove these code.

yes. what's wrong with it?

I just think it's better to put the os implementation related code on the os side, especially these API are defined by POSIX.

I just think it's better to put the os implementation related code on the os side, especially these API are defined by POSIX.

ideally, yes.
however, implementing openat family is not trivial.
i think this makes sense as a stop-gap solution.

@no1wudi , @Donny9 has a patch which is almost finished. Maybe, we can boost the progress.

i dropped preadv/pwritev mock after https://github.com/apache/incubator-nuttx/pull/6168

@no1wudi , @Donny9 has a patch which is almost finished. Maybe, we can boost the progress.

i guess we can revert (part of?) this PR when/if his patch gets ready and merged to nuttx.

@xiaoxiang781216 do you want to keep this blocked until @Donny9's work gets ready?

Let's merge it first? @Donny9 is busy on other task.

@no1wudi do you have any concern?

It's LGTM now, and once these APIs implemented on NuttX side, we must sync the changes to WAMR.

It's LGTM now, and once these APIs implemented on NuttX side, we must sync the changes to WAMR.

@yamt @no1wudi Shall we merge the PR?

It's LGTM now, and once these APIs implemented on NuttX side, we must sync the changes to WAMR.

@yamt @no1wudi Shall we merge the PR?

i prefer to merge it because it would make my work simpler.

Yes we can merge it, it works with current tagged NuttX release at least.


Add a formal LICENSE

This code is very interesting, and looking at your package.json looks like you’re releasing it under the MIT license.
Could you please state it in the README and add a proper LICENSE file?

Hello @yuchi. I will do so as soon as I have some time to wrap up a first releasable version of the lib.

I added the LICENSE. Are you the person working on react-titanium @yuchi?

Yes I am! I stole most of the infrastructure from this module, and I wanted to be in line with your interests.
Do you mind if I place you and this project on the acknowledgements?

I don't mind at all @yuchi. Please come back here if you hit issues with custom rendering so we can fix it together.


Sorting: Desc Folder or file

VSCode Version:1.14.1
OS Version:Win 10

1.The Folder or file is :
a.x
b.x
c.x
2.What i want is:
c.x
b.x
a.x
Sorry, my English is poor
<EMAIL_ADDRESS>

This feature request will not be considered in the next 6-12 months roadmap and as such will be closed to keep the number of issues we have to maintain actionable. Thanks for understanding and happy coding!


Standardize progress spinners

All the indeterminate ProgressBar definitions have been changed to use the Font Awesome spinner for their drawable, by using the new IconProgressBar implementation with appropriate default styles. Progress indicators have also been implemented for the various add buttons in the discussions module.
Fixes MA-1365.

Great work with the app-wide standardisation :star2:
Following problems found:

Height of button changes slightly before and after tapping the login button:

Before tapping
After tapping

Spinner alignment issue on Announcements screen:

"Add response" button is not disabled upon tapping, so, it allows adding a response multiple times upon multiple taps:

The first two issues were introduced here, and have been fixed. The third one was an existing issue that was not directly affected by the changes here, so I have opened a separate pull request for it at #517.

:+1:


Animation callback occurring before tween is finished

Here I have a site, the intention is for the image to expand and then become the banner of the next page. This is the transition I'm using.
var ExpandTransition = Barba.BaseTransition.extend({
  start: function() {
    this.originalThumb = lastElementClicked;

    Promise
      .all([this.newContainerLoading, this.enlargeThumb()])
      .then(this.showNewPage.bind(this));
  },

  enlargeThumb: function() {
    var deferred = Barba.Utils.deferred();
    var thumbPosition = this.originalThumb.getBoundingClientRect();

    this.cloneThumb = this.originalThumb.cloneNode(true);
    this.cloneThumb.style.position = 'fixed';
    this.cloneThumb.style.top = thumbPosition.top+'px';
    this.oldContainer.appendChild(this.cloneThumb);

    TweenLite.to(this.cloneThumb, 0.3, {
      top: 0,
      height: window.innerHeight,
      onComplete: function() {
        deferred.resolve();
      }
    });

    return deferred.promise;
  },

  showNewPage: function() {
    this.newContainer.style.visibility = 'visible';
    this.newContainer.appendChild(this.cloneThumb);
    this.done();
    // this.cloneThumb.style.position = 'relative';
      }
});

The problem I'm having is that as soon as the image begins to expand, the callback occurs and the image immediately jumps to the end of the animation. I'm not sure but I think by clearing the cache/offline website data, it fixes it one time. I've tried using deferred.resolve (no parenthesis) and putting a promise() at the end of the animation but I can't seem to get it working.

It was a css transition applied to the anchor tag causing the slow down in the animation.