Cogljj/python3 changes no default

Pull Request Description
What issue does this change request address? (Use "#" before the issue to link it, i.e., #42.)
This makes all the changes in #954 except defaulting to Python 3
What are the significant changes in functionality due to this change request?
See #954
This is to see if anything other than python3 is causing the qsubs to fail.

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[ ] 1. Review all computer code.
[ ] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[ ] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[ ] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass.
[ ] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True.
[ ] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[ ] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[ ] 8. If an analytic test is changed/added is the the analytic documentation updated/added?
[ ] 9. If any test used as a basis for documentation examples (currently found in raven/tests/framework/user_guide and raven/docs/workshop) have been changed, the associated documentation must be reviewed and assured the text matches the example.

Job Test qsubs on a752528 : invalidated by @joshua-cogliati-inl
trying to figure out why qsubs fail.

CCB chair approves!

This was a temporary pull request used to test the build system.  Closing.


Autogenstudio Add CSV file support

Why are these changes needed?
Being able to render and  download generated CSV in autogenstudio will allow more fluid interactions with additional workflows
Related issue number
Closes #1328
Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (edce174) 34.57% compared to head (cb30f36) 32.87%.

Additional details and impacted files
@@                Coverage Diff                @@
##           autogenstudio    #1355      +/-   ##
=================================================
- Coverage          34.57%   32.87%   -1.71%
=================================================
  Files                 38       38
  Lines               4587     4587
  Branches            1060     1060
=================================================
- Hits                1586     1508      -78
- Misses              2875     2957      +82
+ Partials             126      122       -4

Flag
Coverage Δ

unittests
32.87% <ø> (-1.66%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

@microsoft-github-policy-service agree


Change the "Cross-context transaction (relational databases only)"

Moved from aspnet/Docs#4220. Filed by @Ciantic, pertaining to the following doc:
https://docs.microsoft.com/ef/core/saving/transactions
I propose to change the section "Cross-context transaction (relational databases only)".
Current approach as it now stands suggest to change the super class constructor signature of DbContext, this can be cumbersome since everyone who develops DbContext super classes need to change their constructors.
With my approach below it is not required to refactor your precious DbContexts which may be separated in different projects etc. The super classes of DbContext can be "regular" and have a DbContextOptions as a argument as in every other example. The trick is then to provide the connection to shared contexts using the regular AddDbContext configuration.

Create a simple DI class that wraps the shared connection e.g.

/// <summary>
/// This allows to share a DbConnection within a scope, e.g. between DbContexts so they can perform transactions
/// </summary>
public class DbContextConnectionConfiguration : IDisposable
{
    private readonly DbConnection _dbConnection;
    private readonly Action<DbConnection, DbContextOptionsBuilder> _configuration;
    private readonly bool _dispose;

    public DbContextConnectionConfiguration(DbConnection dbConnection, Action<DbConnection, DbContextOptionsBuilder> configuration, bool dispose = true)
    {
        _dbConnection = dbConnection;
        _configuration = configuration;
        _dispose = dispose;
    }

    public void Configure(DbContextOptionsBuilder optionsBuilder)
    {
        _configuration(_dbConnection, optionsBuilder);
    }

    public void Dispose()
    {
        if (_dispose) {
            _dbConnection.Dispose();
        }
    }
}

Now only thing required is to inject this to AddDbContext calls in Startup.cs:
public void ConfigureServices(IServiceCollection services)
{
    services.AddScoped(t => new DbContextConnectionConfiguration(
            new SqliteConnection(Configuration.GetConnectionString("Default")),
            (c, o) => o.UseSqlite(c)
    ));
    services.AddDbContext<YourFirstDbContext>((s, o) =>
    {
        s.GetService<DbContextConnectionConfiguration>().Configure(o);
    });
    services.AddDbContext<YourSecondDbContext>((s, o) =>
    {
        s.GetService<DbContextConnectionConfiguration>().Configure(o);
    });
}

That's it, there is no need to refactor your DbContext's just to share a connection.

@ajcvickers to investigate


Assets from newer Ace Attorney for better image quality?

... although perhaps some people would prefer the older one. Also I'm not sure what it would legally mean, as newer ones are surely more relevant for companies...
Anyway, doing so may ease the font choice for #53.

It could be added as an option. In fact there is an unofficial fork of the bot for the Russian Social Network VK which uses the new renders. However the owner failed to disclose the source (which is fine, the bot is MIT licensed anyway)
This would also help with #12 .
As I've said this should be an option rather than the only option. About the copyright issue, I feel like Capcom is not too protective of this game. Luckily

This issue should NOT be worked on until #97 is solved

This issue should NOT be worked on until #97 is solved

That's solved now


Cast Numbers to Strings

dashtable.data2rst.TableContentError: Table must be a list of lists of strings
There may be a technical reason why, but it seems like forcing the input to be strings only could be avoided if numbers were cast using str().
That is what I do in my code to "fix" this error.
Thanks for considering the improvement!

Ok, I made it so that data is converted to strings automatically. I can't see a good reason to keep throwing an error.

Much appreciated, @doakey3!


Domain Verification by DNS txt entry over CNAME

Hi,
Can someone explain why a verification of a TXT record (postal-verification xxx...) does not work if the domain has only one CNAME record and the TXT record exists in the target domain?
Many thanks in advance.
Bye
Ingo

see #868

Yes hole zone, but the record is shown as invalid in postal.
https://www.whatsmydns.net/ shows the correct value.

I am not sure whether Postal would be able to follow the CNAME of the whole domain to be honest, sounds like it is not.
Do you know enough ruby to investigate what Postal sees or can you share the domain you're using?


Consider removing lodash-es

See:
https://www.reddit.com/r/webdev/comments/osy47i/is_lodash_still_a_thing_or_are_there_already/
https://youmightnotneed.com/lodash/
We might be able to remove another dependency that way

closing the issue now, let's just not use loadash anymore on new parts of the project instead.


DRA download failing

Scheduled loads are failing.
+ ogr2ogr -f PostgreSQL 'PG:***' --config PG_USE_COPY YES -lco GEOMETRY_NAME=geom -lco FID=transport_line_id -nln bcdata.transport_line /vsizip/tmp/dgtl_road_atlas.gdb.zip TRANSPORT_LINE
ERROR 1: Couldn't fetch requested layer 'TRANSPORT_LINE'!
Error: Process completed with exit code 1.

New data structure:
$ ogrinfo /vsizip/tmp/dgtl_road_atlas.gdb.zip
INFO: Open of `/vsizip/tmp/dgtl_road_atlas.gdb.zip'
      using driver `OpenFileGDB' successful.
Layer: VERSION (None)
Layer: DGTL_ROAD_ATLAS_MPAR_SP (Multi Line String)
Layer: DGTL_ROAD_ATLAS_DPAR_SP (Multi Line String)

New layers have shortened, shapefile compatible names :(

scheduled load has failed: https://github.com/smnorris/bcfishpass/actions/runs/8945871747/job/24575590978

GeoBC confirms that what is currently posted is what we get:

the public DRA data model will not be changing any time soon. We will be updating the public data dictionary

This is unfortunate, given the shortened names in the posted data (that do not match the short names provided by the catalogue api) it is easier to just download from WFS using bcdata.
Download is straightforward but I'm not certain if it is simpler to update all DRA queries to use the new data model or add a script that maps WFS/BCGW column names to the existing column names based on the TRANSPORT_LINE layer.

Current workaround is to use a cached DRA download from April, allowing modelled crossing rebuild with latest FTEN.


Sprint 4:  Team Portfolio

PDF of Team Portfolio based on manual guidelines, with an emphasis on Think, Innovate and/or Control Award criteria.  Be sure review the portfolio and award criteria  in the manual
Define and assign the distribution of duties here and create the necessary sub issues. Be sure to set the necessary assignees and label (Portfolio) on related issues.

I will begin the portfolio and lay the groundwork for it while I wait for the others to complete their necessary tasks before joining me. So in short I will take lead on the completion of the team portfolio.


[Request] Add adb-sync

adb-sync is a tool to synchronize files between a PC and an Android device using the ADB (Android Debug Bridge).
https://github.com/google/adb-sync

Deprecated and no longer being maintained.


Attributes for Encounter

@mickohanlon23  - We added "encounter" as a data element on the DER for eCQMs and CDS Evaluation Metrics.  Would you mind taking a look at the FHIR attributes I listed to see if they make sense to you. Is there anything else I should add? (I realize that 'encounter' is not used in the CDS, so this is outside the immediate context of our CDS).

@ssebast2  Are you referring to the last two rows within CCSM Data Element Requirements 6.21.22?
The CDS looks for Encounters in one place - to check for an Encounter that had a reasonCode of an abnormal bleeding code. These Encounters are used as inclusion criteria for LP1 and are also displayed in the Dashboard. The following Encounter elements are used:

reasonCode (must be in "Abnormal Uterine or Vaginal Bleeding Disorders" or "Abnormal Uterine or Vaginal Bleeding Findings" value set)
status (triaged, in-progress, or finished)
period (to get the start and end time)

As for the use of Encounters within the eCQMs I would defer to @matthewtiller.

@ssebast2

Added row to the DER looks good!
We don't restrict these encounters to any value set. What element are you talking about restricting specifically? Does this matter? Shouldn't any encounter with a reasonCode referring to abnormal uterine or vaginal bleeding be considered relevant?
We currently restrict the status of these encounters to triaged, in-progress, or finished).

I asked about the value set constraint from an eCQM and pilot evaluation perspective. It makes sense for you not to restrict it from a CDS/L3 perspective - if a clinician is in the midst of an encounter and they record info about abnormal bleeding, that should 'count'.
From an eCQM and pilot eval perspective - we would only 'ding' a clinician if they didn't perform evidence-based care during specific types of encounters (i.e., the types defined in our value sets).
I think it will be clearest to create a 3rd row for encounters in the VS and DER spreadsheets. The CDS row will not constrain the type of encounter, whereas eCQMs and pilot metrics will.

That makes sense to me! I'll close out this issue here, since it requires no further updates to the L3 CDS.


Fix notification of service on config changes

The IPv6 support broke the notification for cases where the default is
not to manage the service but service management is explicitly enabled
as a paramter to the openvpn class.

Thank you for the contribution.


i18n_sync.rb:99:in read_file': private method load' called for YAML:Module

Just giving i18n_sync a spin and on first run get the following error.  I'll admit to not looking into it, but wanted to get this posted in order to remind me or see if you've got any insights.
$ i18s config/locales/en.yml
/Users/philip/.rvm/gems/ruby-1.8.7-p352@ni/gems/i18n_sync-0.3.0/bin/../lib/i18n_sync.rb:99:in `read_file':
     private method `load' called for YAML:Module (NoMethodError)
from /Users/philip/.rvm/gems/ruby-1.8.7-p352@ni/gems/i18n_sync-0.3.0/bin/../lib/i18n_sync.rb:54:in `initialize'
from /Users/philip/.rvm/gems/ruby-1.8.7-p352@ni/gems/i18n_sync-0.3.0/bin/../lib/i18n_sync.rb:29:in `new'
from /Users/philip/.rvm/gems/ruby-1.8.7-p352@ni/gems/i18n_sync-0.3.0/bin/../lib/i18n_sync.rb:29:in `work_on'
from /Users/philip/.rvm/gems/ruby-1.8.7-p352@ni/gems/i18n_sync-0.3.0/bin/i18s:55
from /Users/philip/.rvm/gems/ruby-1.8.7-p352@ni/bin/i18s:19:in `load'
from /Users/philip/.rvm/gems/ruby-1.8.7-p352@ni/bin/i18s:19

Relevant RVM info...
system:
  uname:       "Darwin Philip.local 10.8.0 Darwin Kernel Version 10.8.0: Tue Jun  7 16:33:36 PDT 2011; root:xnu-1504.15.3~1/RELEASE_I386 i386 i386"
  bash:        "/bin/bash => GNU bash, version 3.2.48(1)-release (x86_64-apple-darwin10.0)"
  zsh:         "/bin/zsh => zsh 4.3.9 (i386-apple-darwin10.0)"

rvm:
  version:      "rvm 1.7.0 by Wayne E. Seguin<EMAIL_ADDRESS>[https://rvm.beginrescueend.com/]"

ruby:
  interpreter:  "ruby"
  version:      "1.8.7"
  date:         "2011-06-30"
  platform:     "x86_64-darwin10.8.0"
  patchlevel:   "2011-06-30 patchlevel 352"
  full_version: "ruby 1.8.7 (2011-06-30 patchlevel 352) [x86_64-darwin10.8.0]"

Relevant gems...
 i18n (0.5.0)
 i18n_sync (0.3.0)
 ya2yaml (0.30)

Running bundle exec i18s config/locales/en.yml yields...
/Users/philip/.rvm/gems/ruby-1.8.7-p352@ni/gems/ya2yaml-0.30/lib/ya2yaml.rb:25:in `_ya2yaml': set $KCODE to "UTF8". (RuntimeError)
from /Users/philip/.rvm/gems/ruby-1.8.7-p352@ni/gems/ya2yaml-0.30/lib/ya2yaml.rb:369:in `ya2yaml'
from /Users/philip/.rvm/gems/ruby-1.8.7-p352@ni/gems/i18n_sync-0.3.0/bin/../lib/i18n_sync.rb:107:in `write_file'
from /Users/philip/.rvm/gems/ruby-1.8.7-p352@ni/gems/i18n_sync-0.3.0/bin/../lib/i18n_sync.rb:105:in `open'
from /Users/philip/.rvm/gems/ruby-1.8.7-p352@ni/gems/i18n_sync-0.3.0/bin/../lib/i18n_sync.rb:105:in `write_file'
from /Users/philip/.rvm/gems/ruby-1.8.7-p352@ni/gems/i18n_sync-0.3.0/bin/../lib/i18n_sync.rb:84:in `sync'
from /Users/philip/.rvm/gems/ruby-1.8.7-p352@ni/gems/i18n_sync-0.3.0/bin/../lib/i18n_sync.rb:72:in `each'
from /Users/philip/.rvm/gems/ruby-1.8.7-p352@ni/gems/i18n_sync-0.3.0/bin/../lib/i18n_sync.rb:72:in `sync'
from /Users/philip/.rvm/gems/ruby-1.8.7-p352@ni/gems/i18n_sync-0.3.0/bin/../lib/i18n_sync.rb:61:in `work'
from /Users/philip/.rvm/gems/ruby-1.8.7-p352@ni/gems/i18n_sync-0.3.0/bin/../lib/i18n_sync.rb:56:in `initialize'
from /Users/philip/.rvm/gems/ruby-1.8.7-p352@ni/gems/i18n_sync-0.3.0/bin/../lib/i18n_sync.rb:29:in `new'
from /Users/philip/.rvm/gems/ruby-1.8.7-p352@ni/gems/i18n_sync-0.3.0/bin/../lib/i18n_sync.rb:29:in `work_on'
from /Users/philip/.rvm/gems/ruby-1.8.7-p352@ni/gems/i18n_sync-0.3.0/bin/i18s:55
from /Users/philip/.rvm/gems/ruby-1.8.7-p352@ni/bin/i18s:19:in `load'
from /Users/philip/.rvm/gems/ruby-1.8.7-p352@ni/bin/i18s:19

Looks to be a Ruby 1.8 vs 1.9 issue...

I didn't.  Adding in a "require 'yaml'" gets past this error, but then it complains about calling [] on FalseClass.  Which baffled me as to why it would do that...

Hmm, I got the same thing.
Installed (note warning from ya2yaml):
Fetching: ya2yaml-0.30.gem (100%)
WARNING: ya2yaml-0.30 has an invalid nil value for @cert_chain
Fetching: i18n_sync-0.3.0.gem (100%)
Successfully installed ya2yaml-0.30
Successfully installed i18n_sync-0.3.0
2 gems installed
...
```
Then ran:
```
i18s locale/en.yml
/Users/mikemee/.rvm/gems/ruby-1.9.2-p290/gems/i18n_sync-0.3.0/lib/i18n_sync.rb:99:in `read_file': private method `load' called for YAML:Module (NoMethodError)
	from /Users/mikemee/.rvm/gems/ruby-1.9.2-p290/gems/i18n_sync-0.3.0/lib/i18n_sync.rb:54:in `initialize'
	from /Users/mikemee/.rvm/gems/ruby-1.9.2-p290/gems/i18n_sync-0.3.0/lib/i18n_sync.rb:29:in `new'
	from /Users/mikemee/.rvm/gems/ruby-1.9.2-p290/gems/i18n_sync-0.3.0/lib/i18n_sync.rb:29:in `work_on'
	from /Users/mikemee/.rvm/gems/ruby-1.9.2-p290/gems/i18n_sync-0.3.0/bin/i18s:55:in `<top (required)>'
	from /Users/mikemee/.rvm/gems/ruby-1.9.2-p290/bin/i18s:19:in `load'
	from /Users/mikemee/.rvm/gems/ruby-1.9.2-p290/bin/i18s:19:in `<main>'
```
If I get a chance I'll dig into a little and post updates here and/or a pull request.


how to use 'chunk_time_interval' parameter

Could you please explain me how to use 'chunk_time_interval'?
I've tried below but it didn't work.
timescaledb_hypertable={
'time_column_name': 'time',
'chunk_time_interval': '7 days',
}
If create a table and after that I convert it to a hypertable in a normal SQLAlchemy ways, it works though.
Thank you!

Hello @vssssssssss , thanks for the question. Here is a working example using the library:
import datetime

from sqlalchemy.orm import declarative_base
from sqlalchemy import Column, Float, String, DateTime

Base = declarative_base()

class Metric(Base):
    __table_args__ = ({
        'timescaledb_hypertable': {
            'time_column_name': 'timestamp',
            'chunk_time_interval': '7 days'
        }
    })

    name = Column(String)
    value = Column(Float)
    timestamp = Column(
        DateTime(), default=datetime.datetime.now, primary_key=True
    )

A slightly non-obvious point is that the hypertable is created after the creation of the original table: https://github.com/dorosch/sqlalchemy-timescaledb/blob/master/sqlalchemy_timescaledb/dialect.py#L18
This means that all changes to the hypertable are applied at the time the original table is created.
I hope this answers your question. If you have any more questions please let me know.


Notebook Timeline does not reflect true state of notebook & does not work as expected

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:  OSS
OS Version:

Steps to Reproduce:

Create a new notebook on disc, save it => First entry Create xx.ipynb
Add a new cell & save => Next entry edit
Add another cell & save => Next entry Insert Cell
Repeat this, a few times, and pause a few seconds before saving
The items in the time line are

Either edit or Insert Cell, seems to alternate
Sometimes we end up with Update Cell Metadata as well
Some times we do not have an entry in Timeline for the cell inserted, hence timeline is now completely wrong.

Problems

Create a new notebook and hit Ctrl+Save multiple times in quick succession and you'll end up with at least two Create xxx.ipynb in the timeline.

If we create 10 cells, I would expect to see 10 different entries in Timeline, this is not the case
By design, also see https://github.com/microsoft/vscode/issues/207523

We seem to alternate between edit and Insert Cell
Fixed https://github.com/microsoft/vscode/pull/207525

Clicking on one of the items such as edit entry in Timeline displays a broken editor (The editor could not be opened due to an unexpected error....)
Filed here https://github.com/microsoft/vscode/issues/207418
If one of the edit works, then clicking another fails.
This is very flaky, sometimes the edit entry that worked, no longer works at all.

Hitting save without making any changes results in either edit or Update Cell Metadata appearing in the timeline (ensure notebook format is 4.5)
Whats weird is the fact that SCM view shows now difference.
Create a notebok with a markdown cell
Possible the notebook metadata diffing works based on Object refs
Fixed https://github.com/microsoft/vscode/pull/207377

Unwanted Update Cell Metadata entry
This seems to be related to attachment changes.
Also, for some reason this entry shows that the metadata is removed.

When you add a md cell
We end up with one item edit and the diff view shows that the metadata of the md cell is removed.
When in fact we should have an entry for Insert MD as mentioned earlier.
A few issues here (metadata getting removed in diff view and missing entries)
Fixed https://github.com/microsoft/vscode/pull/207525

Work on a notebook and add 10 cells and edit tyhem, etc
Eventually use the key z to remove all the added cells (i.e. undo them all)
Next Save and you'll end up with an Undo / Redo entry
Clicking this item results in a completely broken view.

Use this notebook to replicate issue 4

IPYNB
{
 "cells": [
  {
   "cell_type": "code",
   "execution_count": null,
   "id": "42e5cfd1",
   "metadata": {},
   "outputs": [],
   "source": [
    "11111111"
   ]
  },
  {
   "cell_type": "code",
   "execution_count": null,
   "id": "32680134-ac16-4256-bb58-a80f9db9bea0",
   "metadata": {},
   "outputs": [],
   "source": [
    "one"
   ]
  },
  {
   "cell_type": "markdown",
   "id": "bea1797c-58b7-4409-a7a9-70d11366c2f2",
   "metadata": {},
   "source": [
    "# Hello\n",
    "## One"
   ]
  },
  {
   "cell_type": "code",
   "execution_count": 1,
   "id": "7d82c306-1a04-4458-9c61-6e2bebd08210",
   "metadata": {},
   "outputs": [
    {
     "data": {
      "text/plain": [
       "1234"
      ]
     },
     "execution_count": 1,
     "metadata": {},
     "output_type": "execute_result"
    }
   ],
   "source": [
    "1234"
   ]
  },
  {
   "cell_type": "code",
   "execution_count": null,
   "id": "253bad58-aff2-4cc3-9d48-b6b2d3898bf0",
   "metadata": {},
   "outputs": [],
   "source": []
  },
  {
   "cell_type": "code",
   "execution_count": 1,
   "id": "fce962ad",
   "metadata": {},
   "outputs": [
    {
     "data": {
      "text/plain": [
       "234"
      ]
     },
     "execution_count": 1,
     "metadata": {},
     "output_type": "execute_result"
    }
   ],
   "source": [
    "234"
   ]
  },
  {
   "cell_type": "code",
   "execution_count": null,
   "id": "a380a147",
   "metadata": {},
   "outputs": [
    {
     "data": {
      "text/plain": [
       "234"
      ]
     },
     "metadata": {},
     "output_type": "display_data"
    }
   ],
   "source": [
    "234"
   ]
  },
  {
   "cell_type": "code",
   "execution_count": null,
   "id": "00e8c3b8",
   "metadata": {},
   "outputs": [
    {
     "data": {
      "text/plain": [
       "234"
      ]
     },
     "metadata": {},
     "output_type": "display_data"
    }
   ],
   "source": [
    "234"
   ]
  },
  {
   "cell_type": "code",
   "execution_count": null,
   "id": "16c55c56",
   "metadata": {},
   "outputs": [
    {
     "data": {
      "text/plain": [
       "234"
      ]
     },
     "metadata": {},
     "output_type": "display_data"
    }
   ],
   "source": [
    "234"
   ]
  },
  {
   "cell_type": "code",
   "execution_count": 2,
   "id": "d1b35634",
   "metadata": {},
   "outputs": [
    {
     "data": {
      "text/plain": [
       "1234"
      ]
     },
     "execution_count": 2,
     "metadata": {},
     "output_type": "execute_result"
    }
   ],
   "source": [
    "1234"
   ]
  },
  {
   "cell_type": "code",
   "execution_count": null,
   "id": "4093e13d",
   "metadata": {},
   "outputs": [
    {
     "data": {
      "text/plain": [
       "1234"
      ]
     },
     "metadata": {},
     "output_type": "display_data"
    }
   ],
   "source": [
    "1234"
   ]
  },
  {
   "cell_type": "code",
   "execution_count": null,
   "id": "b665b601",
   "metadata": {},
   "outputs": [
    {
     "data": {
      "text/plain": [
       "1234"
      ]
     },
     "metadata": {},
     "output_type": "display_data"
    }
   ],
   "source": [
    "1234"
   ]
  },
  {
   "cell_type": "code",
   "execution_count": null,
   "id": "eb3fe4a5",
   "metadata": {},
   "outputs": [],
   "source": []
  }
 ],
 "metadata": {
  "kernelspec": {
   "display_name": "Python 3 (ipykernel)",
   "language": "python",
   "name": "python3"
  },
  "language_info": {
   "codemirror_mode": {
    "name": "ipython",
    "version": 3
   },
   "file_extension": ".py",
   "mimetype": "text/x-python",
   "name": "python",
   "nbconvert_exporter": "python",
   "pygments_lexer": "ipython3",
   "version": "3.12.1"
  }
 },
 "nbformat": 4,
 "nbformat_minor": 5
}

Steps

Open a workspace folder
Create a Jupyter notebook
Save this file
Verify there's an entry in the timeline view for this notebook
Save this file a few more times without making any changes
Verify there are no new entries in the timeline view (previously we'd have one entry for each save operation)


Magic Leap Audio + Mic

This binds <audio> and WebAudio to the native Magic Leap audio APIs.
Most of the work is coming in from https://github.com/modulesio/LabSound/pull/15.
Fixes https://github.com/webmixedreality/exokit/issues/484.
Fixes https://github.com/webmixedreality/exokit/issues/477.

Should be good to go in.
This adds both regular WebAudio bindings and getUserMedia({audio: true}) bindings for Magic Leap. Almost all of the code remains the same; only the render endpoint for ML is affected.
Related LabSound PRs: https://github.com/modulesio/LabSound/pull/15, https://github.com/modulesio/LabSound/pull/16, https://github.com/modulesio/LabSound/pull/17, https://github.com/modulesio/LabSound/pull/18


Feat  add gcp support

What kind of changes does this PR include?

[X] Platform support

Checklist:

[X] All the contributions made are solely the work of me and my co-authors
[X] I tested the changes in this PR (if applicable)
[X] I added tests to ensure my change works (if applicable)
[X] The PR passes pre-commit hooks and automated tests

Fixes #157


Fargate vs Node for running Karpenter

Description
How can the docs be improved?
The suggestion to use Fargate for running Karpenter was raised in #1763 and the two options are listed in https://aws.github.io/aws-eks-best-practices/karpenter/#run-the-karpenter-controller-on-eks-fargate-or-on-a-worker-node-that-belongs-to-a-node-group
This documentation doesn't give any particular guidance on how to choose whether Fargate or a small node group is more appropriate. In particular, which option costs more?
From the linked earlier issue I gather there's security and reliability concerns with ensuring a managed node set up for Karpenter won't run workload pods; but no clear guidance on the cost / speed / reliability trade-offs of Fargate vs manager node?
I'm setting up a Karpenter based system for managing Gitlab CI build nodes and it's working really well. This is the first time I've used Kubernetes so there's probably a bit of background knowledge I'm missing.

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

but no clear guidance on the cost / speed / reliability trade-offs of Fargate vs manager node

I think the answer here is: it depends. Do you want lower overhead at a higher cost? Then, you should run on Fargate. Do you want to manage more of your infrastructure, get the new pod identity feature (https://docs.aws.amazon.com/eks/latest/userguide/pod-identities.html) and potentially run other workloads on the same capacity, run on MNG.
Traditionally, we (as the maintainers) have been running our capacity on Fargate because it's lower overhead for us to deal with, but it really comes down to your personal requirements and there's definitely not a clear-cut one-size-fits-all approach. TLDR: YMMV

Thanks for the reply.... what exactly do you mean when you refer to overhead?
Just the fact there'll be a node running all the time, as opposed to the 4 fargate instances?
I'm setting up a new clean cluster as a testbed and from a configuration perspective it looks like it's just selecting one of
managedNodeGroups:
- instanceType: m5.large
  amiFamily: AmazonLinux2
  name: ${CLUSTER_NAME}-ng
  desiredCapacity: 1
  minSize: 1
  maxSize: 10

or
fargateProfiles:
- name: karpenter
 selectors:
 - namespace: "${KARPENTER_NAMESPACE}"

Once either of these is configured won't it just run unattended from that point on?
Otherwise... I guess I don't really know what my requirements are, other than running this cluster for the sole purpose of hosting karpenter based ephemeral build machines :-D

what exactly do you mean when you refer to overhead

Sorry, I realize looking back that that statement wasn't clear. I meant operational overhead. You have to maintain things like image updates on MNG. Do you want the burden of having to update those to the latest all of the time or manage when the node goes bad for whatever reason.
If you're just running Karpenter, I'd go with Fargate. Worth not having to manage the infrastructure for Karpenter and let Karpenter handle the rest.

Ah, yes I hadn't thought of the ongoing overhead of upgrading a managed node (which runs karpenter) to keep up with security updates, as well as any rebuilding needed for cluster version updates, etc. That does make sense, thanks!

Closing as it seems the question has been answered.


Toggle search up or down

Added checkbox to toggle search up/down
Moved search to the top row of the step panel, replacing empty space

I haven't been able to open the GUI to check, but I am pretty sure that I am not gonna like the "replacing empty space" change, given that it most likely will move the buttons, which I have gotten quite used to. :)
I get this when I try to compile and open the application:

It is not very different

You need to delete generator-settings.json as the new format of it is not backwards compatible. (or delete shortcuts parts)

This might be a good time to add some kind of "use the old GUI layout" toggle thing - I don't really want this change, but understand that it does make sense to implement it.
Isn't it possible to make it backwards compatible, so it checks if it is the old format and change it. I don't want to make a breaking change for something which seems relatively minor. I know it isn't a breaking change in terms of code, but if I need to write something like "ohh btw, you need to delete some stange file in the folder of your run to make this work" I consider it breaking for the user. :)

You might have misunderstood me, it does upgrade the file correctly. At least in my test. The problem is that the old version of the generator can't use the new file format, and since you loaded up a version with the new format your settings file was upgraded. Then when you returned to a branch that doesn't use the new file format, the generator can no longer read the settings file.
I am not sure what you mean by old GUI layout?

I see - that makes sense. 👍
So it is possible to revert back to the GUI which is the current GUI ("old" if we implement this change).

It sounds laborious to have a separate old GUI. Given how integrated the code is with the interface.

Yeah. It would be pretty difficult to actually have two different GUIs, but in this case it is just the layout of the GUI components and it seems to be possible to change the layout either on startup or "on click".  It is done for some things in cMain:

It might be possible to also do it for the placement of components.

I don't question that it is possible.
Just how much extra work it would add in the future, and by extension who would do that work.

Fair point. It looks better anyway - I will just have to get used to the new placement of the buttons. :)


WIP - add 1.0.0-beta2 section to CHANGELOG.md

Clean up various markdown linter warnings.
Let's use this PR for on-going tweaks to the CHANGELOG.md. Although I'm not anticipating making any more changes myself.  Although I need to wrap up the README changes and tweak the Wiki on customizing your prompt function.
Should we shoot for a Feb 14th release of beta 2? I think we could easily pull that up if we wanted to.

@rkeithhill I've rebased this changes onto master, with a bit of cleanup and the newest changes. Please have a look at the latest commit (and the full changelog).

Weird, any idea why the reference-style links don't seem to be working?


Drop support for v1 of NDEx

From version 4.0.0 on the python client will no longer support the pre 2.4.x endpoints (those without the v2)

84d6d4e..6c72dbc  v4.0.0 -> v4.0.0


Notebook to deploy a trained SKLearn model on satellite data

Hi @cisaacstern , I added the notebook deploy_trained_model.ipynb to the repo. It's in a 'working' mode but does have some comments included. In it I attempt to deploy my model trained using scikit-learn on the satellite data, but it doesn't seem to be working, I think mainly when I am flattening the satellite data, removing rows with NaNs, and then re-shaping.

UPDATE: I re-worked everything and fixed it so that the flattening and re-shaping seems to be working now, at least to the extent that the land shows up as white (NaN values). However, there are some strange stripe effects in the plot that I'm not sure about, and it seems like there should be more valid data that what we see (in the updated deploy_trained_model.ipynb notebook)


docs: initial documentation added

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

v1.0.0
npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Citeproc/CSL support

started from #515. watch the mentioned dspace issues...
not mentioned issues (other than accents escaping) with citeproc-java:

a wrapper around citeproc-js which apparently has license incompatible with dspace
the released version (0.6) is broken with java8 (due to changed default js interpreter in java8). Seems this was fixed but not released later on.

https://www.atmire.com/articles/detail/university-of-bordeaux-releases-citation-styles-for-dspace?utm_source=generallist&utm_medium=email&utm_campaign=cslpatch

Is the situatiuon still unchanged and Citeproc-Java is still broken?
In case this issue is better postponed to new UI, depending on what UI it is, maybe the implementation in Invenio / Zenodo would be relevant. I think this is the most common approach and I like it: include all the hundreds of CSL styles, pick one default and add a search box to change the citation style. Also allow export of the CSL itself. This is how they implemented it https://github.com/inveniosoftware/invenio/issues/1416

look at https://opencitations.net how it relates

maybe related to #1115


Scripts need  =

I believe I should be able to create a script with a different name than filename. I mean I should do this:
Recipe:
            <key>SCRIPT_NAME</key>
            <string>Add Printer</string>
            <key>SCRIPT_FILENAME</key>
            <string>add_printer.sh</string>
[snip]
                    <key>scripts</key>
                    <array>
                        <dict>
                            <key>name</key>
                            <string>%SCRIPT_FILENAME%</string>
                            <key>template_path</key>
                            <string>S-Standard.xml</string>
                        </dict>
                    </array>

Script Template:
<script>
    <name>%SCRIPT_NAME%</name>
    <category>%CATEGORY%</category>
    <filename>%SCRIPT_FILENAME%</filename>
    <info />
    <notes />
    <priority>After</priority>
    <parameters />
    <os_requirements />
</script>

Unfortunately, I get this error:
JSSImporter: Found file: /Users/admin/Library/AutoPkg/RecipeRepos/com.github.ftiff.mac-autopkg/S-Standard.xml
Response Code: 409  Response: Conflict. Error: Duplicate name
Failed.
Receipt written to /Users/admin/Library/AutoPkg/Cache/com.github.ftiff.mac-autopkg.jss.followme/receipts/com.github.ftiff.mac-autopkg.jss-receipt-20160118-130317.plist

The following recipes failed:
    com.github.ftiff.mac-autopkg.jss.followme
        Error in com.github.ftiff.mac-autopkg.jss.followme: Processor: JSSImporter: Error: Response Code: 409   Response: Conflict. Error: Duplicate name

First of all, am I right?
If yes, I'm ok to work on this and create a PR, as I believe @sheagcraig is quite busy.

Yes-scripts and packages can have different names and filenames. Specifically, at least with non-cloud/JDS distribution points, the filename had to point to the filename of the actual file, whereas the name could be something more human-readable.
The 409 error that you are getting is due to the fact that there is already a name or filename on your JSS that the attempt to update/post is conflicting with. If you remove all similarly named scripts, does this fail the first time, or only on subsequent runs?
I'm a little bit rusty on why this happens, as it has been awhile, but remember having similar issues in the past that revolved around which name the JSS uses for the actual identity of the object. You should be able to sort out the issue by playing around with some combination of removing conflicting named scripts and running multiple times. I can say with authority that name must be unique. I don't know whether filename needs to be unique, although I think it probably has to be too. Imagine what would happen if you had two script objects referencing the same filename, perhaps with the intention of being analogous to a link in the filesystem-save disk space by only having one copy of an otherwise duplicated file. However, using the web-interface to delete this script would result in the second script object have a reference to a now-non-existent file. So I would think the filename also has to be unique. But I don't think they need to be the same thing. Indeed, I'm sure you can prove that using Casper Admin pretty quickly.
I do believe this should work correctly without changing the JSSImporter code; I never did it because I wanted my name and filename to match, but I don't see any reason why you shouldn't be able to do it that way if you want to.
Let me know what you find out and hopefully we can make some progress.

Any updates on this?

Guessing this has been abandoned, based on the dates. Feel free to reopen if you feel like working on it again @ftiff!


JWS-Test Suite update for JWT Representations

Add JWT representation tests to the DIF test suite https://github.com/decentralized-identity/JWS-Test-Suite
After the merge of https://github.com/TBD54566975/did-sdk/pull/47

https://github.com/decentralized-identity/JWS-Test-Suite/pull/56


Fullscreen window focusing is incomplete.

When in fullscreen mode it's not possible to go backwards through the stack of windows, you can only go forwards.
movefocus l and movefocus r both go forwards in the window stack when they should go opposite directions.
Same thing for movefocus u and movefocus d.
cyclenext prev starts cycling in the opposite direction of cyclenext but after the 2nd call it will go back to the original window, so if we have windows 1-2-3 and we are on window 1 it will do this:
-> window 1
cyclenext prev -> window 3
cyclenext prev -> window 1 (should be 2)
cyclenext prev -> window 3 (should be 1)
cyclenext prev -> window 1 ...

movefocus improved in 6fec5bfbeb80a5b2231af2ac5710a8b9c8bbc08a
cant seem to repro the cyclenext thing tho

can you check if the issue is resolved for you

still bugged altough I'm using the nixos stable version right now and cannot verify on latest commit

@leg7 this should be resolved


Added Multi Step Animating Button

Fixes Issue
Closes #1427

Changes proposed
I want to add a MULTI-STEP GROWING LIKE BUTTON
https://github.com/Spyware007/Animating-Buttons/assets/119161467/4646bacc-217c-436f-99ae-cee374b5a80b

Screenshots

Checklist

[x] You've linked this PR to the correct issue.
[x] You have checked that the code is working correctly.
[x] You ⭐ the repository!

Note to reviewers
@Souravvmishra  @TANYA-CHAN  I am writing to kindly request your attention and consideration in reviewing and accepting my pull request. I would greatly appreciate it if you could allocate some time to review the pull request and provide your feedback. If there are any areas that require further attention or modifications, please let me know, and I will be more than willing to address them promptly.

@Souravvmishra  @TANYA-CHAN I am writing to kindly request your attention and consideration in reviewing and accepting my pull request. I would greatly appreciate it if you could allocate some time to review the pull request and provide your feedback. If there are any areas that require further attention or modifications, please let me know, and I will be more than willing to address them promptly.

Please check #1346

@TANYA-CHAN   PLease merge my pull request


common-grpc: correctly define default service options

Fixes #2190
According to this comment, I was improperly defining gRPC default options.
@murgatroid99 does this look better? Thanks!

Coverage remained the same at 100.0% when pulling 5dc78ca4a8294d634c30dc372d9e164f01ed62e5 on stephenplusplus:spp--2190 into aa9525dda48cd83d9067f42f8e47dccd7ea34ef4 on GoogleCloudPlatform:master.

I think this should work


About video predict

I see you have a demo of video segment, but only find the image finetune code.  Did you achieve this effect by finetune only on images, or did you finetune on videos as well?

Hi @staple-pi ,
Video fine-tuning is much more challenging. we are still working on it.
the SAM2 team has released part of the training code: https://github.com/facebookresearch/sam2/tree/main/training


Feature logo card component

Implemented LogoCard component.

Codecov Report

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

@@            Coverage Diff             @@
##           master     #530      +/-   ##
==========================================
+ Coverage   99.73%   99.73%   +<.01%
==========================================
  Files         216      217       +1
  Lines        5241     5275      +34
  Branches     1226     1235       +9
==========================================
+ Hits         5227     5261      +34
  Misses         14       14

Impacted Files
Coverage Δ

src/components/logoCard/LogoCard.tsx
100% <100%> (ø)

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 0216239...a48e6e1. Read the comment docs.

Demo: https://coveo.github.io/react-vapor/feature-logo-card-component/


Database creation problem

When I was creating the database, I got a problem, the error is that :
root@labserver:~/my-postgresql/data/eicu-code/build-db/postgres# make eicu datadir=/root/my-postgresql/data/

-- Checking for data --
All data present!

-- Building eICU --

-- Creating tables --
psql "dbname=eicu user=eicu options=--search_path=eicu_crd" -v ON_ERROR_STOP=1 -f postgres_create_tables.sql
psql: FATAL:  Peer authentication failed for user "eicu"
Makefile:313: recipe for target 'eicu-build' failed
make: *** [eicu-build] Error 2
how can I solve it ?

It is failing to authenticate as database user "eICU". Rerun the scripts
setting the database username to be the same as your operating system user
name.

Thinks a lot，It helps！Think you fot replying me!
------------------ 原始邮件 ------------------
发件人: "Alistair<EMAIL_ADDRESS>发送时间: 2019年7月14日(星期天) 晚上10:20
收件人<EMAIL_ADDRESS>抄送: "幻世忧伤"1557679224<EMAIL_ADDRESS>主题: Re: [MIT-LCP/eicu-code] Database creation problem (#77)
It is failing to authenticate as database user "eICU". Rerun the scripts
setting the database username to be the same as your operating system user
name.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Fix bar series border typos

Changes all instances of bodrder to border inside the bar series type

Thanks for your contribution!


Insert Virtual Media not working

Hey there, I'm trying to attach an iSO file into my iLO:
- name: Insert Virtual Media
   community.general.redfish_command:
     category: Manager
     command: VirtualMediaInsert
     baseuri: "{{ ansible_host }}"
     username: "root"
     password: "{{ password }}"
     virtual_media:
     image_url: 'http://....iso'
     media_types:
       - CD
       - DVD
     resource_id: BMC
   tags: test

fatal: [..]: FAILED! => {"ansible_facts": {"discovered_interpreter_python": "../bin/python3.12"}, "changed": false, "msg": "System resource BMC not found"

Did I miss something?

Instead of using BMC for your {ManagerId}, have you tried tried to use 1 ?
iLO's manager URI is /redfish/v1/Managers/1 , not /redfish/v1/Managers/BMC.

Hi @donzef,
you mean the task should look this?
- name: Insert Virtual Media
   community.general.redfish_command:
   category: Manager
   command: VirtualMediaInsert
   baseuri: "{{ ansible_host }}"
   username: "root"
   password: "{{ password }}"
   virtual_media:
   mage_url: 'http://.....iso'
   media_types:
     - CD
     - DVD
   resource_id: 1

Cheers - Willi

Yes. give it a try.

fatal: [...]: FAILED! => {"ansible_facts": {"discovered_interpreter_python": ".../bin/python3.12"}, "changed": false, "msg": "Unable to find an available VirtualMedia resource supporting ['CD', 'DVD']"}

So there is some progress.
Then, from the DMTF doc (the community.general.redfish_command is from DMTF, not from HPE) I can see and indentation after virtual_media:

FYI, I could make it work by removing the media_types entry:

Awesome, that works fine.


Auth annotations on service have no effect when using 'use-annotation'

---
apiVersion: v1
kind: Service
metadata:
  name: api
  labels:
    app.kubernetes.io/name: api
  annotations:
    alb.ingress.kubernetes.io/auth-type: none
spec:
  type: NodePort
  ports:
    - port: 8080
      targetPort: http
      protocol: TCP
      name: http
  selector:
    app.kubernetes.io/name: api

---
apiVersion: networking.k8s.io/v1beta1
kind: Ingress
metadata:
  annotations:
    alb.ingress.kubernetes.io/auth-type: oidc
    alb.ingress.kubernetes.io/actions.api-ann: |
      {
        "Type":"forward",
        "ForwardConfig":{
          "TargetGroups":[
            {
              "ServiceName":"api",
              "ServicePort":"8080"
            }
          ]
        }
      }
    alb.ingress.kubernetes.io/conditions.api-ann: |
      [
        {
          "Field":"http-header",
          "HttpHeaderConfig":{
            "HttpHeaderName": "x-api-key",
            "Values":["*"]
          }
        }
      ]
...
spec:
  rules:
      - path: /api/*
        backend:
          serviceName: api-ann
          servicePort: use-annotation

I want to disable oidc authentication on API endpoint when x-api-key is defined. I need oidc enabled on ingress because it's not possible to enable it separately on redirects and external targetgroup forwards.
Annotation on alb.ingress.kubernetes.io/auth-type: none on service has no effect, because of this check here https://github.com/kubernetes-sigs/aws-alb-ingress-controller/blob/206154d817ae0c49cd1be475992768a57771469c/internal/ingress/auth/auth.go#L103

Hi,
By ALB's design, the authentication is based on path instead of backends.
We originally supported the feature to "configure annotations on service"  to workaround k8s IngressSpec's limitation that there is no way to configure path specific configuration, since there is always only one service per path.
However, with ALB's weighted targetGroups support, a single path can have multiple serviceBackend, but auth is still based on path. That's why i decided to not support auth on these services configured via annotation.
In summary, the annotation schema is like this:

configurations that applies to ALB Rules TargetGroup: we support configure them via annotation on either Service or Ingress.
configurations that applies to ALB Rules:

route conditions like alb.ingress.kubernetes.io/conditions.api-ann, where 'api-ann' is the referenced serviceName.
auth like alb.ingress.kubernetes.io/auth-type, it's supported on Ingress and Service when the service is not a "use-annotation" port.
We could add a new annotation like alb.ingress.kubernetes.io/auths.pi-ann.auth-type to support path based auth, and deprecate the support for configure it on services.

BTW, for your case, since you are only use one service backend, you don't need to use the 'use-annotation', the conditions annotation can be applied to serviceNames that are not annotations based as well(i'll fix the docs about this). something like below:
annotations:
    alb.ingress.kubernetes.io/conditions.api: |
      [
        {
          "Field":"http-header",
          "HttpHeaderConfig":{
            "HttpHeaderName": "x-api-key",
            "Values":["*"]
          }
        }
      ]
...
spec:
  rules:
      - path: /api/*
        backend:
          serviceName: api
          servicePort: 8080

@M00nF1sh Thanks! This resolves my issue.


feature: Support icon = ".." in key spec

Did you check the docs?

[X] I have read all the lazy.nvim docs

Is your feature request related to a problem? Please describe.
You just added icon support in which-key.nvim - it'd be great to be able to specify an icon in a lazy key spec, and when which-key is loaded, use it.
Describe the solution you'd like
See above.
Describe alternatives you've considered
See above.
Additional context
No response

I don't think this makes much sense
Because the place where the icons are used is which-key. So icons should be added there.
For all users which don't use which-key this field isn't of much use and will just make the plugin spec even more complicated than it already is.

I hear what you're saying - and I was a little unsure if I should file this.
I have some key specs that include icons in them - but I also want to use which-keys icons otherwise.

Notice the Terminal map. Ideally I'd like to replace the which-key icon with the one I selected.

Check the opts.icons.rules setting.
If you use lazy.nvim keys, then you can add icon rules for the plugin like { plugin = "toggle-term.nvim", icon= ...}
That way, all lazy.nvim keys for the toggle-term plugin will get that icon.
(I dont use toggleterm, so the plugin name might be incorrect)

{
  icons = {
    rules = {
      { plugin = "toggleterm.nvim", icon = "<my icon>", hl = "MyHighlightGroup" }
    }
  }
}

works for me

That did the trick! Thanks!

You're welcome, glad I could help


The Infinite Scroll of Articles in "Stories" are looping Twice... Then Halts.

Hi,
The first 30 articles in the "Stories" page are being repeated the second time by the infinite-scroll feature.
Then, the infinite-scroll feature stops completely even when scrolled to the last article.
I can confirmed this issue with Google Chrome and Microsoft Edge.
Here's my Ghost Instace Details:
Version 2.2.3
Environment production
Database mysql
Mail SMTP

Hey! Thanks for raising the issue :relaxed: This has literally just been fixed in version 2.2.4! https://github.com/TryGhost/Ghost/commit/e865d2218c1b6e392831db67552fa397ccd31eac


WXKG17LM (lumi.remote.acn004) incorrect battery level

Fixed in latest master version. Will be in next release


UploadFile::fake() not working when fake file uploads in phpunit

my test case looks like this
public function test_submit_edit_profile_picture()
    {
        $user = $this->userAccount();

        Storage::fake('local');
        $width = 200;
        $height = 200;
        $file =  UploadedFile::fake()->image('avatar.jpg', $width, $height)->size(100);
        $response =   $this->actingAs($user)->call('PATCH',
            '/panel/profile/edit-profile-picture',
            array(
            '_token' => csrf_token(),
                'file' => $file
        ));

//        Storage::disk('local')->assertExists($file->hashName());

        $response->assertJson([
            'success'=>true
        ]);

    }

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.


update form 0.6.3 to 0.7.1 sidenav stops working

I updated the version and the sidenav i had working fine with version 0.6.3 doesn't work. Also i had a nested accordion menu which worked fine but doesn't in the new version...

Steps to reproduce
Which browser?

What is expected?
What is actually happening?

Reproduction Link

Can you provide some code examples?

@eackland I had the same problem at first. But in my case it turned out that the menu trigger button needs the .native modifier since 0.7.1. The same solution applied to a few more components too actually, starting from 0.7.1. Also, as @marcosmoura pointed out, please fill out the bug template in your comment.

@eackland Your problem is similar to @praneetloke?

Having the same problem with my sidebar. Have a nested accordion menu in the sidebar that no longer works.
@praneetloke where did you add the .native modifier? Where are the 'menu trigger buttons' for list items?

@web-surfer I had to add it to the @click(or v-on:click) handlers. For list items, the event handlers are in the list items themselves. If you post a codepen link, I could help you better.


Update Javadoc to be compliant with Java-17

Java-17 is a bit stricter on Javadoc. Unmatched tags are sometimes throwing an error.
Furthermore, the Version with a $Revision$ from SVN is still part of some class descriptions. These lines can be removed.

No warnings anymore on maven clean javadoc:javadoc in the entire project.


Make it work in Internet Explorer

This is the problem:
The development branch does not work in Internet Explorer. There are visual problems like non-hidden selects and functional problems like non-working multi-selects.
Steps to reproduce:
Launch the development branch in IE.
This is my solution:

IE does not support multiple-parameter classList.add() / classList.remove(); Replace it with single-parameter calls

IE needs polyfills for Array.prototype.find / Array.prototype.includes; Include them from core-js

Eventually it makes sense to investigate how to use useBuiltIns: 'usage' regarding the Polyfills.

Which version of IE? Choices only supports >IE11

The problem appears in IE 11 which happens to be the latest version of Internet Explorer.

Yeap, on IE11 this lib is pretty much useless.


🐛 Bug Report: Steps output is always empty while passing with other steps inputs

📜 Description
I am writing some custom actions; where outputs of one action is always empty(undefined). Even though we have values assigned in ctx.output

Here is steps in template file.
- id: createTFWorkspace
      name: Create Terraform workspace
      action: terraform:cloud:workspace:create
      input:
        workspaceName: ${{ parameters.component_id }}
        organizationName: "sample_org"
        projectID: "sample_project_ID"
        allowDestroyPlan: true
        autoDestroyActivityDuration: "2d"
        repoIdentifier: ${{ (parameters.repoUrl | parseRepoUrl).owner }}/${{ (parameters.repoUrl | parseRepoUrl).repo }}

    - id: attachVariablesetToWorkspace
      name: Attach variables to workspace
      action: terraform:cloud:variableset:attach
      input:
        workspaceID: ${{ steps.createTFWorkspace.output.workspaceID }}
        organizationName: "sample_org"
        variablesetID: "sample_variableSet"

Inside the custom action terraform:cloud:workspace:create I am adding the following
ctx.logger.info(`Successfully created workspace: ${workspaceNameInResponse}`);
ctx.logger.info(`workspaceURL: ${workspaceURL}`);
ctx.logger.info(`workspaceID: ${workspaceID}`);

ctx.output('workspaceURL',`${workspaceURL}`);
ctx.output('workspaceID', workspaceID);

Logs of the above:
{"level":"info","message":"\u001b[32minfo\u001b[39m: workspaceURL: https://app.terraform.io/app/sample_org/workspaces/parameter-group","plugin":"scaffolder","service":"backstage"}
{"level":"info","message":"\u001b[32minfo\u001b[39m: workspaceID: ws-sample_org","plugin":"scaffolder","service":"backstage"}

logs of the action terraform:cloud:variableset:attach
ctx.logger.info(`workspaceID: ${workspaceID}, organizationName: ${organizationName}, variablesetID: ${variablesetID}`)

{"level":"info","message":"\u001b[32minfo\u001b[39m: workspaceID: , organizationName:sample_org, variablesetID: varset-sample","plugin":"scaffolder","service":"backstage"}

Logs in Web UI:
2024-09-18T23:58:48.671Z Beginning step Attach variables to workspace
2024-09-18T23:58:48.684Z info: workspaceID: , organizationName: sample_org, variablesetID: varset-<sample>
2024-09-18T23:58:48.685Z info: API call data being sent to Terraform API: {"data":[{"type":"workspaces","id":""}]}

Here it is getting empty.
👍 Expected behavior
Need to get the output values of one step to another
👎 Actual Behavior with Screenshots
Pasted logs above
👟 Reproduction steps
This is the custom actions.
📃 Provide the context for the Bug.
No response
🖥 Your Environment
No response
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
None

@acierto can you take a look at this? :pray:

Hi @sarathchandra24 , we have a lot of custom actions at bol with output values, and for us it works well.
I can't spot any problems in the snippets what you placed in the ticket.
I can only offer you to spend 15-30 min together to debug your case.
It might be that the problem somewhere else.

Thanks @acierto for the reply!
I was the one who created these actions and I fixed the problem it's because of the wait. Code is returning before completing the action, fixed this with an await statement.
Thanks again for responding.


What is the function of the parameter max_feats?

Thank you for the great work and open source code!
I have been exploring the codebase and documentation, and I came across the max_feats parameter. However, I couldn't find a clear explanation for its purpose. Could you please provide some insights into what role max_feats plays in the functionality of the project?
Thank you in advance for taking the time to address this inquiry.

Thanks for your interest in our work.
max_feats denotes the maximum number of frames.
The larger max_feats generally yields the higher performance.


button click is clicking on an incorrect element for version 21,20,19 works for v18

Expected behavior
Button Click with Text displayed should click on the right element.
UI Element -

DOM -

Actual behavior

Using
await highlight(button("Submit"));
await click(button("Submit"));

clicks on the following element
UI -

DOM-

I have tried with some of the taiko versions

v1.0.21 - Fails
v1.0.20 - Fails
v1.0.19 - Fails
v1.0.18 - Works

But this works in REPL for all versions, but when using it with Gauge scripts in a step definitions, it fails and clicks on the incorrect element.
Versions
node - v12.16.1
Gauge version: v1.1.1

What happens when u try click('Submit')

click('Submit') also doesnt work - tried with v1.0.21
only thing that works currently -  await click($(#assignPhysicalCardSubmitBtn));

We have this issue, too. We could only solve it ATM like this:
  const scheduleButtonElement = $('.mileage-form-schedule-btn');
  await evaluate(scheduleButtonElement, (el) => el.click());

This worked in v1.0.18, but not in later versions of taiko anymore.

Will try reproducing this from our end, meanwhile if anyone of you can share a repro will help us fixing the issue sooner.

@mhp-borisbojic can you please confirm if your buttons label is also 'Submit' ? I could see it broken for type submit

@NivedhaSenthil No, the button name is irrelevant.
This worked with 1.0.18:
  await click('Further details');

With 1.0.19 and above, it's broken and it could't find and click the button anymore. I had to change it having a class and evaluate it like this:
  const scheduleButtonElement = $('.schedule-btn');
  await evaluate(scheduleButtonElement, (el) => el.click());

@mhp-borisbojic That shouldn't be happening.. atleast the click api should have worked with css/xpath selectors.
Can you please provide a sample html where this is happening ?

I forgot to mention that the button itself is a Web Component that makes use of the shadow DOM, too.
Here is the generated markup:

test.html
<html>
  <script>
    class ShadowButton extends HTMLElement {
      constructor() {
        super();

        var button = document.createElement('button');
        // button.setAttribute('type', 'button');
        // button.setAttribute('value', 'Shadow Click');
        button.addEventListener("click", event => {
          alert("Hello from the shadows");
        });
        button.textContent = "Hello";

        var shadow = this.attachShadow({mode: 'open'});
        shadow.appendChild(button);
      }
    }

    customElements.define('shadow-button', ShadowButton);
  </script>
  <body>
     <shadow-button/>
  </body>
</html>

code.js
(async () => {
    try {
        await openBrowser();
        await goto("file:///.../test.html");
        await alert(()=>accept());
        await click("Hello");
    } catch (error) {
        console.error(error);
    } finally {
        await closeBrowser();
    }
})();

I am able to successfully click the text of button within shadow DOM like in the above. These kind of screenshots does not really help catch the issue, please provide a repro like above. The issue with button having label as submit is fixed as part of  #1485.

Please try out this form: https://login.porsche.com/login/gb/en_GB#
We're only able to click the buttons with this piece of code - this works without issues:
const protractor = (protractorId) => {
  return $(`//*[@data-protractor-id="${protractorId}"]`)
}

const clickLoginBtn = async() => {
  warn('Clicking on login button');
  const loginBtn = protractor('login-btn');
  await evaluate(loginBtn, (el) => el.click());
  success('Clicked on login button');
}

Whereas this doesn't work at all - it just doesn't go an further and waits endlessly for the click to happen:
const clickLoginBtn = async() => {
  warn('Clicking on login button');
  const loginBtn = protractor('login-btn');
  await click(loginBtn');
  success('Clicked on login button');
}

Can I provide you with anything more?

Thanks, that was very helpful for understanding the issue. Fix done via https://github.com/getgauge/taiko/pull/1500, should be available i next release once reviewed and merged.
Meanwhile you could try installing from the branch.


Client library cannot reconnect after a failure.

Since #1007 added grpc.FailOnNonTempDialError(true) to the client's dial options I am finding with swarmd's containerd executor that if I glitch containerd (by restarting) it is never reconnecting and is always seeing the same error:
rpc error: code = Internal desc = connection error: desc = "transport: dial unix /run/containerd/containerd.sock: connect: no such file or directory"

This persists even after containerd is fine again (i.e. a new ctr invocation works)
I patched ctr event to make it easier to demonstrate see https://github.com/ijc/containerd/commit/869ec23e1fc0b8960a64f8acb7cddf5d522c44c2 which ports it to the client library and adds some retry loops with some logging. With that I run containerd and ctr events, if I then bounce the containerd process I get:
Recv failed: rpc error: code = Internal desc = transport is closing
Retrying
Retry failed: rpc error: code = Internal desc = connection error: desc = "transport: dial unix /run/containerd/containerd.sock: connect: no such file or directory"
Retry failed: rpc error: code = Internal desc = connection error: desc = "transport: dial unix /run/containerd/containerd.sock: connect: no such file or directory"
Retry failed: rpc error: code = Internal desc = connection error: desc = "transport: dial unix /run/containerd/containerd.sock: connect: no such file or directory"
Retry failed: rpc error: code = Internal desc = connection error: desc = "transport: dial unix /run/containerd/containerd.sock: connect: no such file or directory"
Retry failed: rpc error: code = Internal desc = connection error: desc = "transport: dial unix /run/containerd/containerd.sock: connect: no such file or directory"
Retry failed: rpc error: code = Internal desc = connection error: desc = "transport: dial unix /run/containerd/containerd.sock: connect: no such file or directory"
[...forever at 1s intervals...]

Note that "no such file or directory" is considered a non-temporary error (by google.golang.org/grpc/transport.isTemporary, which relies ultimately on syscall.Error.Temporary(), which considers ENOENT to be non-temp).
AFAICT google.golang.org/grpc/clientconn.go.addrConn.wait ends up latching the failure in ac.tearDownErr and is never reset or retried. I patched WithTimeout down to 10s, it does not reconnect after more than 1m even with that setting.
I'm not entirely sure what #1007 (which came about from #989) was attempting to do, in my usage it seemed like ctr event (patched to use the client library) fails immediately if containerd is not running irrespective of the use of WithBlock/WithFailOnNonTempDialError or WithTimeout. The use of grpc.FailFast(false) on individual calls seems to make much more difference.
For my purposes with the swarmd executor I'm was generally happy with the default behaviour with no-WithBlock or WithFailOnNonTempDialError and with FailFast enabled (the default) for most calls (there are small number where I want to manually retry, but mostly I can rely on the orchestrator to retry things in most cases).
/cc folks involved with #1007/#989: @dmcgowan @estesp @crosbymichael @stevvooe

Do we have a backoff maximum configured?

We don't seem to. I'll play with that today.

I added grpc.WithBackoffMaxDelay(20 * time.Second),  to the DialOption slice with no change in behaviour after 300+ seconds. Not too surprising since the default is 120 * time.Second and I left it running for longer than that previously.

I think we need to extend the new client opts to allow for people specifying their own timeout and blocking grpc opts.  We still need a good default that works but a way to setup the connection for their use case.  I'll try to work something up

Going to keep this open because I don't think my  PR solves the entire issues.  It just makes it more flexible for consumers of the client to specify different timeouts and options.


uncomment osm.de and added mapilion vector tiles

I investigated a bit these CORS issues and for OSM.de it was a problem that their redirection did not contain the CORS headers and failed (combined with the fact that we did not used https). But when using the correct URLs that are not redirected this works now.
The other layers do not have a proper CORS support it seems and probably do not want to be included elsewhere.
The performance of the sorbian layer is very bad (in Firefox/Linux) and so I left it commented out. My suspicion is that it uses some kind of redirect mechanism to fetch normal tiles for the area outside of Germany/Lusatia and fetches Sorbian tiles only for Germany/Lusatia and maybe the mapbox client is not able to handle this redirection trick properly.
I included the mapilion vector tiles as alternative to MapTiler. I also tried the vector tiles from Thunderforest but I got JS errors.
Two questions - maybe we should clarify them in a different issue:

Should we add the 2x to the URLs for bigger devices instead of hardcode them?
Unfortunately the formatting kicked in - should I revert this and we do some formatting for the whole project once or automatically somehow for every commit?

Unfortunately the formatting kicked in - should I revert this and we do some formatting for the whole project once or automatically somehow for every commit?

Yes, sorry I auto-formatted everything using the wrong formatting rules (see here: https://github.com/graphhopper/graphhopper-maps/pull/88#discussion_r643434996). So let's do the formatting again in all branches?

This is related to #57

Ok, created #90 for formatting the master branch.

Yes, sorry I auto-formatted everything using the wrong formatting rules (see here: #88 (comment)).

Ok, yeah, this is all a bit confusing. Also in VSCode.

Should we add the 2x to the URLs for bigger devices instead of hardcode them?

I always thought this is for high dpi devices regardless of the screen size? I think we could also test for this as well of course.

Ah, ok. Yes, sounds more logical :D


Contract call error

When I am trying to upload the wasm file in substrate node I am getting error
Code is -
const keyring = new Keyring({ type: 'sr25519' });
const alice = keyring.addFromUri('//Alice');
const code = fs.readFileSync('./flipper.wasm').toString('hex');
const result = await api.tx.contracts.putCode(code).signAndSend(alice);
Error is -
submitExtrinsic(extrinsic: Extrinsic): Hash:: 1002: Verification Error: Execution: Could not convert parameter tx between node and runtime: No such variant in enum MultiSignature: RuntimeApi("Execution: Could not convert parameter tx between node and runtime: No such variant in
Error: 1002: Verification Error: Execution: Could not convert parameter tx between node and runtime: No such variant in enum MultiSignature: RuntimeApi("Execution: Could not convert parameter tx between node and runtime: No such variant in

@kumari23anjali I think you've hit the wrong repo. SubstraTEE is not substrate. Our SubstraTEE-node does not feature the contract pallet. I guess you're looking for either ink! or paritytech/substrate


Website

[x] Website Design coordinate with Vendor
[ ] Deploy Website and setup dns
[ ] Update all places
[ ] Announce

Let's close this as its completed.


[Automated] Upgrade to latest dependencies

Produced via:
./hack/update-deps.sh --upgrade
./hack/update-codegen.sh

/cc we don't use Prow
/assign we don't use Prow

Codecov Report

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

@@           Coverage Diff           @@
##           master      #96   +/-   ##
=======================================
  Coverage   96.18%   96.18%
=======================================
  Files          16       16
  Lines         446      446
=======================================
  Hits          429      429
  Misses         15       15
  Partials        2        2

Continue to review full report at Codecov.

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

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@8506b21). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #96   +/-   ##
=========================================
  Coverage          ?   96.18%
=========================================
  Files             ?       16
  Lines             ?      446
  Branches          ?        0
=========================================
  Hits              ?      429
  Misses            ?       15
  Partials          ?        2

Continue to review full report at Codecov.

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


[Bug Fix] Auth screens images issues

What does this PR?

Fixed image clipping issue in register screen
Fixed image clipping issues in signin screen

Issue number
fixes #2632
Screenshots/Video

@aliseyalvi also please make sure both mascots are same size/visibility wise.

@noumantahir @feruzm I had changed mascots dimensions to percentages and its parent container to flex so it would be least affected by device dimensions.

@aliseyalvi regarding sign in mascot, cannot we reduce the percentage assigned to make the both look somewhat the same size... from UI perspective, the sign in mascot looks too big for the text content on left....
Do also share screenshots of both on different simulators like iPhone 6, nexus 5, pixel 3a as well


Export ssdMobilenetv1 as alias of locateFaces -  no build

@justadudewhohacks Same as #84 but without the build/dist.

Thanks!


Broken version 0.0.4

Got this error on start after updating to 0.0.4
undefined method supervise_as' for Sidekiq::Batching::Actor:Class /Users/antoshalee/.rbenv/versions/2.1.1/lib/ruby/gems/2.1.0/gems/sidekiq-batching-0.0.4/lib/sidekiq/batching/supervisor.rb:9:in run!'

Try 0.0.5, please


Refactor Hero

What kind of change does this PR introduce?
Make a generic Hero component and update all existing Heros

Issue Number:

Related to #558

Screenshots/videos:

Before
After

If relevant, did you update the documentation?

Summary

Does this PR introduce a breaking change?

Moreover, it seems like we are missing the new hero for the query plans comparison, is that planned for later or was that an oversight?

Good point, I just forgot to do it!


Added recursive image search to image gallery

Resolves Issue #20 in adding recursive search for image gallery.
To test, you can do the following:

git clone https://github.com/johnsbuck/obsidian-image-gallery.git
git checkout issue-20
npm run build
Copy main.js from dev folder to <vault_folder>/.obsidian/plugins/obsidian-image-gallery
Test with example query below:

```image-gallery
path: path/to/img/super/folder
type: vertical
recursive: true
```

This works great, thanks! Really useful addition.


Update concept-primary-refresh-token.md

If the user signed-in without internet, and the PRT is expired, after the device is re-connected to the internet while the user is already signed-in, the CloudAP will not renew the PRT automatically unless a new Windows Sign-in or Lock\Unlock is done.
The current phrase in the article: "The CloudAP plugin renews the PRT every 4 hours during Windows sign in. If the user doesn't have internet connection during that time, CloudAP plugin will renew the PRT after the device is connected to the internet." may give the impression that after the device is connected to the internet the PRT will be renewed automatically, when in reality it will not.

The PR's build is failing on an unrelated article. Later, we can close and reopen the PR to see if the issue is resolved upstream.
@MicrosoftGuyJFlo
Can you review the proposed changes?
IMPORTANT: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


TuneTwins 2nd card's equalizer animation is not playing

Describe the bug
Every TURN in TuneTwins has a first card and a second card. Now equalizer animation stops when the 2nd card in a TURN is clicked, in other words, the animation is not playing for the 2nd card.
Expected behavior
For every card in TuneTwins, the equalizer animation of the card should play when the card's audio is on. Other behaviors should not be affected, for example, the cards will shake if two cards are not a match while the animation of the second card should still be playing when the audio is playing.

Don't know if it helps, but it seems like in a video recording from August I made, the second card's animation was still on. So the feature seems to be there and it is a bug?

Don't know if it helps, but it seems like in a video recording from August I made, the second card's animation was still on. So the feature seems to be there and it is a bug?

Good to have that confirmed. I think I fixed the issue, I just need a code review and I can merge & deploy it to ACC. :-)


Adding Epoxy, Fakeit, Kluent, Mockito, Mosby, Robolectric

.

@mikepenz Any thoughts on this?

@JosephRoskopfOPI  @j-roskopf thanks a lot for the PR. I will review the changes and get it merged as soon as I can. currently a little bit busy.

Thanky ou very much!


pynitrokey >=0.4.26 not installable on Arm

On Arm systems, pip only installs pynitrokey up to version 0.4.25.  This is probably due to the indirect dependency pc_ble_driver_py (pulled in by nrfutil), see https://github.com/NordicSemiconductor/pc-ble-driver-py/issues/202.
Related issues:

https://github.com/NordicSemiconductor/pc-nrfutil/issues/337
https://github.com/NordicSemiconductor/pc-nrfutil/issues/361
https://github.com/NordicSemiconductor/pc-ble-driver/pull/271

@robin-nitrokey is there any known workaround?

Yes.  You should be able to install Python and pynitrokey in a Rosetta terminal (see e. g. this discussion).  Or you could build pc_ble_driver_py and pc_ble_driver_py from source and install the wheel with pip before installing pynitrokey (as described in this comment – though you should be able to use the upstream master as the PR has been merged).

This also block FreeBSD support to pynitrokey 0.4.25 (with local patches for libusbio 2.1.11, and a minimal patches for nitropy to launch on FreeBSD)

@daringer installation worked fine on a Purism Librem5 (aarch64). If this is really resolved https://docs.nitrokey.com/software/nitropy/all-platforms/installation#arm-not-supported should be updated.


What should I fill in if my proxy does not require an account password?

I only need to fill in the proxy type, server and port in tools like SwitchyOmega to use the proxy.
Also, how do I use the SOCKS 5 type of proxy? Does this software support it?

It turns out that I need to manually add the proxy address in the settings file, and the one filled in the software can't be saved to take effect (or I didn't figure out how to save it).


My Device Doesn't Suppert HID and i dont Have hidg0 & hidg1 drivers

Hi Gays
The Title is shownig my problem and i will explain it more.
First of all
My Device J2 Prime G532F
I want to add hidg0 and hidg1 to /dev
and i will try to run the payloads with juice ssh i hope it will work :(
i want to show me how to add hidg0 & hidg1 to /dev & will it work or not
Tanks :)

You will have to compile the kernel for your device, as described in README.

Ok now where i can found hidg0 and hidg1 files -?

If the kernel on your android device has USB-HID keyboard support, you can find /dev/hidg1 and /dev/hidg0 on your android device.
You will also need root access and either a terminal app or adb.

Hi Everyone
I'm a new in Githib and in Linux Kernel.
Would you please tell me how to unpack/repack zIamge in my boot.img file of Android TV, so as to have access to HID files and make some modifications ??
Please I need help :D :D


Is it possible to remove FMRadio funciton but make some code change that user can input Radio freq directly in VFO mode to listen to broadcast, to reduce the size of firmware.

You like long titles ;)

I suppose it could auto switch to FM Radio mode if frequency is within the broadcast band

In my personal opinion, the search, save, etc. functions of the FM radio function are not so necessary, so I came up with this idea, wondering if it could reduce the firmware size.

I'm sure it's late at your place, good night!

One thing I found out recently was that LTO builds w/o the FM radio option made binaries too small to flash. It took me by surprise as I usually compile with it included. Maybe I can look at it when I have time so 1o11 can focus on bug fixes.

Too small ?  .. do they still work I wonder

One thing I found out recently was that LTO builds w/o the FM radio option made binaries too small to flash. It took me by surprise as I usually compile with it included. Maybe I can look at it when I have time so 1o11 can focus on bug fixes.

what was the files size??? I used your brand  diasbled every but
ENABLE_LTO                    := 1
ENABLE_UART                   := 1
this is what I got
text    data     bss     dec     hex filename
47664      12    2608   50284    c46c firmware
Oh and shitty AM quality lol

I don't yet know why your AM sometimes works and the rest of the time don't. I may spot something that triggers my brain into seeing the problem, but so far I just don't know.
It's not as if the AM fix I've done is complicated, it isn't it, it does nothing more than adjust the front end RF gain according to the RSSI level - in order to stop the AM demodulator from saturating. That's it.
Maybe someone will spot something and let me know, who knows.

I don't yet know why your AM sometimes works and the rest of the time don't. I may spot something that triggers my brain into seeing the problem, but so far I just don't know.
It's not as if the AM fix I've done is complicated, it isn't, it does nothing more than adjust the front end RF gain according to the RSSI level (maintains a constant RSSI level of -89dBm) - in order to stop the AM demodulator from saturating (the AM distortion everyone hears without the fix). That's it.
Maybe someone will spot something and let me know, who knows.

Latests builds have been good again...Apologies for not letting you know sooner..

The idea of the original FM Mode is thinking on the convenience of being able to listen to the music while the HT is monitoring the VFOs. If RX happens, the music is stop and you listen to your VFOs. After a while of silence, the system goes back to the music.
I think this is a great approach, and removing such behaviour would mean to have to manually tune the frequency, having the VFO permanently busy (forget about the fake Dual Watch). In my opinion, this is the best approach, as in a certain mode it manages to have a truly Dual VFO, even if it is only for the commercial FM.

But ..
What if they want to use the 88-108 band but in normal FM comms mode ?
I thinking that as the VHF broadcast band is going to be closed down at some point to all broadcast FM (at least here in the UK), it's all going digital (DAB radio) only mode, the FM band will no longer support broadcast radio.

But ..
What if they want to use the 88-108 band but in normal FM comms mode ?
I thinking that as the VHF broadcast band is going to be closed down at some point to all broadcast FM (at least here in the UK), it's all going digital (DAB radio) only mode, the FM band will no longer support broadcast radio.

I thought the gov went back on that as DAB cannot be received everywhere  so would cut off folk as it would not allow them to get information out to them if they could not RX DAB...

oh right :)

Here in my country, DAB is a joke, only working in a few places and not even broadcasting all station. Most likely DAB will never work at 100%, like in other countries, so removing FM mode is quite... a disconsideration for people living in countries in this situation. However I think there is a switch in the Makefile called "ENABLE_FMRADIO", so logic dictates that this should be the trigger that decides if the range 87-108 is to be input as another common frequency or used in the special FM radio mode.

Yes, if needed then radio can be disabled during compilation.

May as well update this.
I integrated the FM radio stuff into the main frequency selection a while back but only saved a couple hundred bytes from the app code removal. Pulling in the BK1080 driver code is where most of the added size comes from.
I lost it when the forks diverged. 👉👈


[Blazor] add support for the onmouseenter and onmouseleave dom events in components

Until preview 7, I could bind an handler to the onmouseenter and onmouseleave events using a syntax similar to onmouseenter="@MouseEnter". Now this syntax is not working anymore and the syntax @onmouseenter="MouseEnter" leads to this runtime error in Blazor Wasm:
Microsoft.JSInterop.JSException: Failed to execute 'setAttribute' on 'Element': '@onmouseenter' is not a valid attribute name.
Describe the solution you'd like
Add the onmouseenter and onmouseleave to the list of the supported mouse events (I don't know if it's just a matter of adding them to the EventHandlers.cs) class.
Additional context
I'm using those events on a div containing a number of other elements. Using the onmouseover and onmouseout events (which are supported) can't be a solution because they're fired when crossing the boudaries of the external div but also whenever the mouse crosses the border of any of the internal elements causing the events to be fired multiple times.

Confirmed that this is not working anymore in Preview 8.

Thanks for contacting us, @ghidello.
You can add a new class called EventHandlers and add define the needed mappings for these event handlers in there (similar to how it's done in the EventHandlers.cs) and it should just work.

I had the same problem and can confirm it works with the workaround of mkArtakMSFT

Thanks a lot @mkArtakMSFT
I've not figured out how but it's working! Thanks again!

Notifying also @miroslavp who asked about the same events on #5501

thanks @mkArtakMSFT , the workaround worked. May I know when can these two events added officially to the EventHandlers.cs?


Some turtle events being fired as wrong user

This issue may be related to #602. On this server, turtles have a 100% chance of duping (+ their inventories) if they are moving up and a player is jumping on top of them. We have been able to reproduce the underlying issue on a fresh server this time.
Production server - MC 1.12.2, SpongeForge (mod list)
We made a claim as Lemmmy (non-admin) and placed a turtle. Lemmmy ran this code on it. Yemmel (admin user, respecting claims, not trusted in the claim) jumped on the turtle as it was moving upwards. The turtle fully duped twice. We reproduced this multiple times, there was a 100% chance of dupe.
Reproduction on test server - MC 1.12.2, SpongeForge
This video demonstrates the dupe

Run forge 2611 installer
Added mods:

cc-tweaked-1.80pr1.4
griefprevention-1.12.2-<IP_ADDRESS>5
LuckPerms-Sponge-4.1.15.jar
spongeforge-1.12.2-2611-7.1.0-BETA-2944

/lp creategroup admin
/lp group admin parent set default
/lp group admin permission set griefprevention.admin
/lp group default permission set griefprevention.user
/lp user ADMIN_USER group set admin
Claim an area of land as TEST_USER
Do not /ignoreclaims as ADMIN_USER
Place a turtle as TEST_USER
Place a block of coal in the turtle's inventory
Run edit dupe
Insert the code:

turtle.refuel()
while true do
    for i = 1, 5 do turtle.up() end
    for i = 1, 5 do turtle.down() end
end

Ctrl, enter to save
Ctrl, right arrow twice, enter to exit
Run dupe, the turtle should begin vertically oscillating
Run /gpdebug on as ADMIN_USER
As ADMIN_USER, jump on top of the turtle. When it is moving upwards, start jumping.
ChangeBlockEvent.Break will be triggered as ADMIN_USER instead of TEST_USER. Turtle will either stop moving, or completely dupe. If it stops moving, restart the server and try again.

OMG XD going to abuse the SHIT out of servers xDDD
Thanks for NOT fixing :D 💃

@Lemmmy First GP cannot cause dupes. It literally just cancels events which causes Sponge to restore the blocks. As for the user, it is technically not wrong as Sponge will always use the notifier over the block owner. However, there probably could be a check to make sure the block actually has physic logic before updating it. In any case, this ticket needs to be placed in SpongeForge as it is not a GP issue.

Thanks blood. I was able to reproduce this on latest SpongeForge (2951) and latest GriefPrevention (559). As such, I have moved the issue to [SpongeForge#2034)(https://github.com/SpongePowered/SpongeForge/issues/2034).


Prepare changelog for new release 2024.2

I think we used to do this everytime for a semesterly release, so LGTM.

That makes sense indeed. Should we then tag that one 2024.1.2 and switch to 2024.2 at the end of this cycle?

Sure, sounds good :+1:


Grails deployment

Hi ,
Can I use eb_deployer, to deploy .war groovy files to Beanstalk?
Is it language agnostic? Or it needs to be a Rails project?
Thanks in advance!

It is language agnostic. eb_deployer support any platform elasticbeanstalk supports. To elasticbeanstalk your groovy war is nothing different with a normal java webapp.

Thank You very much wpc


chore(typo): fix preliminary typo

Description:
fix typo

BREAKING CHANGE:

Related issue (if exists):

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Add negative duration

It would be nice to allow negative timeshift. At leaste the ISO8601 allows a negative notation.

can you please link some reference about how exactly is the standard allowing the negative notation?
I can't find anything here: https://en.wikipedia.org/wiki/ISO_8601#Durations

Yes I can't find it there too. Maybe I was misleaded because the Java documentation https://docs.oracle.com/javase/8/docs/api/java/time/Duration.html#parse-java.lang.CharSequence- implies that it is possible but also says that it is not part of the 8601.
Is there anything wrong with implementing it anyway?

closing since it's not a real issue


[Bug]: Can't mount a writable volume to the docker container

Bug expectation
I expected to be able to mount a volume to the base docker image and write screenshots to it.
What happened instead was that the default pptruser does not have permissions to write anything to the mounted volumes. Not sure if this is my lack of knowledge on using Docker, or is the way the image is built/configured.
Bug behavior

[ ] Flaky
[ ] PDF

Minimal, reproducible example
`docker run -i --init --cap-add=SYS_ADMIN -v screenshots:/home/pptruser/screenshots --rm ghcr.io/puppeteer/puppeteer:latest touch screenshots/some-screenshots.jpg`

Error string
Permission denied
Puppeteer configuration
No response
Puppeteer version
20.0.0
Node version
16.17.1
Package manager
pnpm
Package manager version
8.2.0
Operating system
macOS

This is not a puppeteer issue itself, but the official docker image

Nevermind, I managed to handle this by creating a custom image and running on root user. I had to go with --no-sandbox, but couldn't find any other solution.


Long description malforms markdown

Describe the bug
Saving long descriptions malforms the markdown parts. Tested with a dataset entity.
To Reproduce
Steps to reproduce the behavior:

Go to an entity.
Click on documentation.
Edit the documentation with the following text:

asdfasfd n asdfhajskfhasdf jasdfa asdfjhasfjkd `sahfdafk` nsahdfhjasdfjh `asdfjjhfjhsfdaskdjf` asdfjhkaf sdjhjhkfd fdjjkjhafd dfjjadjak nsajasdjhfda sdafjhsfadjhfdnbfad fadbfd dafbjdfjhfd ndfjhfdajhda djhfadbdsa dfjhdaf

Save the docu.
Click on the right side Read less.
See markdown code blocks are now shown as normal text.

Expected behavior
Markdown format persists even with longer texts
Screenshots

Desktop (please complete the following information):

Browser chrome and firefox
Version v0.12.0

still relevant

still an issue

Issue persists in version 0.13.0

Seems to me with Version 13.1 the Bug is fixed. Read less function was removed?


Add Thanos Grafana dashboards

Changes
Add Thanos Compaction dashboard
Add Thanos Query dashboard
Add Thanos Store dashboard
Add Thanos Sidecar dashboard
Change getting started docs
Add Monitoring readme
Verification
This is how README looks:
https://github.com/povilasv/thanos/blob/monitoring/docs/monitoring/monitoring.md
https://github.com/povilasv/thanos/blob/monitoring/docs/getting_started.md#dashboards
I've tested it on my grafana :)

Not necessarily in this PR, but it would be great if we could provide the dashboards, recording and alerting rules as monitoring mixins[0], that way it's agnostic to the Prometheus configuration and thus labeling of time-series. It's an effort me and other Prometheus upstream team members have introduced. It's catching on great, there are kubernetes mixins, etcd mixins, consul, and the list is only growing.
I recommend everyone to check out the proposal below.
[0] https://docs.google.com/document/d/1A9xvzwqnFVSOZ5fD3blKODXfsat5fg6ZhnKu9LK3lB4/edit

@brancz I agree will look into that. Right now I've sort of solved it via labelselectors /labelvalue grafana constants.

Would be good if we could move this to /examples/graphana/* just so that it is easy to find and that it shows that this is just an example setup for dashboards and not the only way it should be done.

For sure, the idea is also that whereever we have the mixins, we use the mixins to generate the "old"/current json definitions.

Anyone was able to import these? My import ends up with empty dashboard (can see only template vars and +Add Row button). Is my grafana version too old?
t=2018-06-20T09:24:31+0000 lvl=info msg="Starting Grafana" logger=main version=4.4.0 commit=3583057 compiled=2017-07-04T21:42:22+0000

@Bplotka I am using grafana 5.0.4, so my guess that grafana 5.0 exported dashboards aren't compatible with 4

I imported all of them and they seem to be working as expected
There are few things I'd change but maybe it's better to merge first version and iterate on that?
One overview dashboard would be also great to see total number of different node types for generic view. Possibly add CPU usage?
But great work thanks for those it helped me a lot today

@FUSAKLA great to hear it works :)  I agree an overview dashboard would be really useful and we definitely can iterate on those

just beware there are saved different time ranges in default
store: last 15m
compaction: last 15m
sidecar: last 3h
query: from 2018-06-18 12:19:05 to 2018-06-18 13:19:05 (also has no update interval)
that could be unified maybe
at least avoid the exact time range so that no one is mistaken the dashboard doesn't work

Didn't have a chance to try them today, but if we can unify the time ranges and add Grafana 5.+ requirement, I am ok to merge.
Some "birdeye" overall dashboard would be nice, but sure - let's iterate in next PR.

@FUSAKLA thanks for noting. I keep forgetting that grafana exports time ranges.
I've changed dashboards to last 30mins and refresh of 30s.

Hey. This is great @povilasv
However probably plenty of folk not using kubernetes, any chance of throwing up some equivalent dashboards ? Cheers
Pete

Hey @VR6Pete, the plan is to get this merged add some alerts and then work on monitoring mixin, which would be environment agnostic ;)


Allow export of recorded tracks

A list of recorded tracks should be visible somewhere.
There should be a show/hide option for each track.
Download and deletion should be supported, too.

Export as GPX

Plan:

Retrieve objects from IDB
Create GPX and use JSON.stringify (in some order)
Data URL, text format: https://developer.mozilla.org/en-US/docs/Web/API/IDBTransaction

Feature completed.


Upgrade swagger-ui version to 3.38.0 to fix XSS vulnerability

What's this PR do/fix?
Swagger UI XSS vulnerability: https://www.vidocsecurity.com/blog/hacking-swagger-ui-from-xss-to-account-takeovers
Are there unit tests? If not how should this be manually tested?
Manually tested

Looks like this project is currently dead


set default tag sorting order

Changes

update the default sorting order of select_tag for WeldxConverter
use full tag naming schema in all explicit converters

Checks

[x] updated CHANGELOG.rst

please feel free to make further changes and/or merge @marscher
As long as the tests pass we should be good here 👍


OmniSSM restructure/rewrite

[x] Restructure project layout
[x] Rewrite config-subscribe in Go
[x] Add cobra/viper for command-line handling (allows for combining the two agent binaries into one with subcommands)
[x] Use dep for dependency management
[x] Switch back to aws-sdk-go from aws-sdk-go-v2 (developer preview)
[x] Add support for oversized config change events
[x] Add placeholder systemd units
[ ] Testing

[x] Add test(s) for manager (ssm/dynamodb)
[x] Add test(s) for register handler(s)
[ ] Add test(s) for identity package
[ ] Add test(s) for oversized config change event

[ ] Docs

[ ] Add Godoc comments to all exported types
[ ] Add package summary Godoc comments
[ ] Update README

[ ] Logging

[x] Switch to zerolog
[ ] Ensure messages and context provided are clear, consistent and concise

this is a big diff, general good practice / rule of thumb for code changes is to try and break them up for easier review.

lgtm, let's move the remaining items into a separate issue for tracking.


REL: 2024.2 (tiny)

for posterity, my initial commit was misnamed - this is the 2024.2 release for tiny distro, not 2023.9. it's been a long week.


Update README.md to prefer application-default login

This behavior has been added since 2.4.0 and is now the default option, however the documentation did not reflect that:
https://github.com/mitchellh/vagrant-google/pull/213
https://github.com/mitchellh/vagrant-google/pull/251
I believe this method of authenticating is basically the use-case that Vagrant has: developing on your local environment, whereas a service account is for basically every other use-case.

That's a very good contribution, thanks @dalvizu! In general, I think that this project is really good but it needs better docs and usability improvements - and your change fits this exactly.
If I would have the permissions I would just fix the typo myself and merge it.
@Temikus, perhaps you could do that please?


[Android] Stop setting ENABLE_WEBCL=1 in all files.

With chromium-crosswalk commit 64b4265 ("[Blink] WebCL: Only set
ENABLE_WEBCL in the relevant files"), the ENABLE_WEBCL macro is defined
only for the relevant files in Blink.
This means we can now remove it from target_defaults and stop passing
it to all the 15000+ files part of Crosswalk's build since the vast
majority of them do not need to be aware of the macro at all.

@shaoboyan @darktears

LGTM of course

LGTM.