TPF factory overhaul with Spitzer demo

Altering the workflow to base off of origin/spitzer_tpfs rather than gully/spitzer_tpfs to get the GitHub actions to work.

I migrated from_fits_images to factory.py.  Here is a path towards an overhaul:

(optionally) Migrate factory related tests to a new path test_factory.py
Update the tests with backwards-breaking behavior

Check for and flag calls to heritage from_fits_images, raise deprecation warning/error
Test for this warning/error

Remove static method decorator from from_fits_images:

replace with regular method
Update from_fits_images internal references to factory with self
De-deplicate common arguments and key-word arguments between factory and from_fits_images.
Make new tests that look for the new behavior

Break up the large function into more manageable functions
Identify mission-specific functionality and replace with custom mission-specific dataloaders:

Make a Spitzer data loader
Make a Kepler/TESS data loader?

Verify that the delivered TPFs have standardized metadata mimicking Kepler/TESS, and potentially introducing new metadata header keywords if necessary

RA_obj,Dec_obj, time_format, time_scale

Specify how Spitzer WCS will map to poscorr

Should we use WCS differential or the diff_x, diff_y header columns?


Fix bug where instructions were not printed when doing TRACE_EVAL

It just always printed that instruction was not part of the program. This also moves print_instruction to the instruction class and removed the duplicate function.

Looks  like we might still be having some network issues...


Rules on the same domain as the auth domain break the redirect

When I add this rule --rules.auth.action=allow --rules.auth.rule="Host(`auth.domain.com`)" to allow my auth domain the redirect back to the service breaks and traefik gets stuck on auth.domain.com.

Why do you want to do this?
Auth host style redirects should work without this if that's what you're try to achieve :)

Cause I set DEFAULT_ACTION=auth and I thought that it otherwise wants to authenticate the auth domain.
We should probably add this to the readme.


[widgets] Allow case insensitive #diff

Addressing issue https://github.com/mojotech/dill/issues/17 and, potentially, https://github.com/mojotech/dill/issues/46
Reopened from: https://github.com/mojotech/dill/pull/115

@david All great points. I've removed the values classes and implemented the private methods instead. You had one comment I did not address, regarding downcasing the Hashes separately:
"Would it be possible to skip this and always convert hashes to arrays?"
I played with that, but when a table comes back as an array of arrays when an array of hashes is expected, I couldn't get it to work:
# this works
[{"name"=>"guybrush threepwood", "protagonist"=>"yes", "selected_item"=>"wimpy little idol"}]

# this does not
[["name", "guybrush threepwood"], ["protagonist", "yes"], ["selected_item", "wimpy little idol"]]

@adsteel could you tell me why the second case doesn't work? When is an array of hashes expected?

@david A form's #to_table method returns an array of hashes, one for each row. I could replace the #downcase_hash method with something like this, but I'm not sure it's much better:
      def downcase_to_table(to_table)
        headers = to_table.first.keys
        rows = to_table.map { |row| row.values.map(&:downcase) }

        rows.prepend(headers)
      end

Alright, I think it's fine to leave it as is for now. :shipit:


leverage WaitForExitAsync from polyfill

/azp run

@SimonCropp some other part of the code will need to be fixed:
src/Platform/Microsoft.Testing.Extensions.HangDump/HangDumpProcessLifetimeHandler.cs(384,17): error VSTHRD103: (NETCORE_ENGINEERING_TELEMETRY=Build) WaitForExit synchronously blocks. Await WaitForExitAsync instead. (https://github.com/Microsoft/vs-threading/blob/main/doc/analyzers/VSTHRD103.md)


Fix track elements selector

this.el.querySelector returns wrong element if there is nested .simplebar-track exist. This cause issues with nested simplebars in some cases (e.g. simplebar-react which creates full dom tree including tracks). Searching for track element inside immediate children solves this issue.

Could you give me more context on what actual bug this is solving? We have nested SimpleBar on the example page and it seems to work fine http://grsmto.github.io/simplebar/examples

Could you give me more context on what actual bug this is solving? We have nested SimpleBar on the example page and it seems to work fine http://grsmto.github.io/simplebar/examples

There is no problems in demo because you don't use simplebar-react for nested scrollbars, but initialise them one by one from parent to child. But simplebar-react construct all DOM before initialisation, that's why querySelector selects wrong (child's) .simplebar-track element.

@Grsmto here is an example: https://jsfiddle.net/hduoc68L/4/

Hey, forgot to thank you for your contribution! I just left some comments, let me know what you think :)
Thanks again great work.


DDS FAILED [mipmaps] (80070057)

(80070057: ???????? ????? ???????.) What code page using? (console app)
texconv.exe -w 256 -h 256 -if linear -sepalpha -f BC2_UNORM -srgb -y "test.dds"  FAILED [mipmaps] (80070057) (should generate max mipmaps possible for current dds size)
texconv.exe -w 256 -h 256 -m ANY NUM -if linear -sepalpha -f BC2_UNORM -srgb -y "test.dds"  SUCCESS!
I remember early versions doesn't have some issue
TEST DDS:
https://drive.google.com/file/d/1ryfz08wi8iZBskW2dXffCyr_hdTa3Qxi/view?usp=sharing
UPD:
-if fant while resize also  cause error

The ???????? ????? ??????? text is coming from FormatMessage and the text in English is "The parameter is incorrect". I'm using wprintf and I'm not modifying the code-page, so I'm not sure why it didn't print correctly for your locale.
The problem is a bug in texconv in this particular case.
-m 0 for images is 'generate all mips', but in the case of a DDS file I use what's in the original file. Per below, I set my target mips to 12.
DDS c:\temp\test.dds
Flags 00021007: CAPS HEIGHT WIDTH PIXELFORMAT MIPMAPCOUNT
2048 x 2048
Pitch: 0
Mipmap count: 12
Surface flags: 00401008: COMPLEX TEXTURE MIPMAP
PIXELFORMAT: 00000004: FOURCC
FourCC = 861165636 'DXT3'
DXGI_FORMAT_BC2_UNORM

Then after the resize, the max possible mips is only 9.
Using -m 9 would be a good workaround for now. I'll fix up the logic there.
Thanks for the report!

Hi,
You should add this
const wchar_t* GetErrorDesc(HRESULT hr)
    {
        static wchar_t desc[1024] = {};
        LPWSTR errorText = nullptr;
        DWORD result = FormatMessageW(FORMAT_MESSAGE_FROM_SYSTEM | FORMAT_MESSAGE_ALLOCATE_BUFFER, nullptr,
            static_cast<DWORD>(hr),
            MAKELANGID(LANG_NEUTRAL, SUBLANG_DEFAULT), reinterpret_cast<LPWSTR>(&errorText), 0, nullptr);
        *desc = 0;
        if (result > 0 && errorText)
        {
            setlocale(LC_ALL, ".1251");  // <----------------THIS!
            swprintf_s(desc, L": %ls", errorText);
            size_t len = wcslen(desc);
            if (len >= 2)
            {
                desc[len - 2] = 0;
                desc[len - 1] = 0;
            }
            if (errorText)
                LocalFree(errorText);
        }
        return desc;
    }

Now:

Notice:
Need test on other systems (tested - win10x64 enterprise russian)
(But i think 1251 will work fine on any system)

the mips issue is fixed in this commit.

Thx
What about -if fant ?
it not work with resize...

It looks like I just need to change the C++ locale on startup: std::locale::global(std::locale(""));

As variant...

Locale issue should be fixed in this commit
Can you verify it works?
Thanks!

Can you verify it works?
Yep, all fine
Thx


Maintainership

Hey Austin - would you be open to adding me as a collaborator here w/ write access to help maintain?

I just did. I appreciate you taking charge the past couple months. I should have more time for open source work in the coming weeks, and will look into how I can contribute to this project, and the broader netlify cms ecosystem.

Awesome, sounds great!


fix node fetch vulnerability

Update isomorphic-fetch to eliminate a node-fetch vulnerability.
refs: #31
test
yarn test
yarn run v1.22.10
$ mocha

  English TTS
    ✓ Hello (101ms)
    ✓ hello
    ✓ hello world
    ✓ 123

  Long Characters
    ✓ English: 180 characters
    ✓ English: 200 characters
    ✓ English: 268 characters throw RangeError
    ✓ Chinese: 193 characters
    ✓ Chinese: 200 characters
    ✓ Chinese: 211 characters throw RangeError

  parameters
    ✓ text = null (566ms)
    ✓ text = '' (201ms)
    ✓ text = 123 (186ms)
    ✓ lang = null (191ms)
    ✓ lang = '' (198ms)
    ✓ lang = 123 (number) (193ms)
    ✓ speed = null (188ms)
    ✓ speed = '123' (226ms)
    ✓ timeout = 10 ms (too short to success)

  Chinese TTS
    ✓ 你好
    ✓ 你好世界
    ✓ 123

  22 passing (2s)

✨  Done in 4.75s.

Coverage remained the same at 96.364% when pulling 7be0b40628cd8ce885e34aac28604d45bd9af906 on okhiroyuki:fix/node-fetch-vulnerability into b55689fffcdcc2445ad30a65d51899f18bfb8e6f on zlargon:master.

@okhiroyuki, thanks for helping solve the vulnerability. The vulnerability issue has been fixed in the latest version 0.0.5.
Also, the test framework was changing from Mocha to Jest for better supporting with latest node version.
Thanks.


added pregenerating downsampled x4 images when running run.py

Pregenerates x4 downsampled images when running run.py . Also, sorry about "596 additions and 576 deletions". I was writing locally, and then scp-ed my files over to nersc to run, and I wasn't aware that replacing your files would cause git to assume all the lines were replaced.

Thanks for the update but ouch about the many fake line changes.  To avoid massive merge conflicts and to keep the history cleaner, let's start over on this.  Please identify the (presumably small) lines of code needed for adding these pre-generated preproc*.html files, and re-add just those to a new branch and open a new PR.
The many differences likely are due to Windows CRLF vs. Unix LF which git normally autoconverts for us when checking in and out, but if you scp directly git gets confused about a Windows-looking file being committed on a Unix-looking OS (or vice-a-versa).
Please do not merge this PR as-is.

Sure thing! The new branch is located on #23


Better content disposition attr extract_from

The current implementation of the ContentDispositionAttr header extraction too strict and does not cover the full standard. Moreover it has limitations with escaped quotation marks characters.
With this pull request I

improved flexibility, allowing header value extraction even if the name header is not in the first position;
fixed parsing of values containing escaped quotation marks
extended the tests to prevent regression

The proposed implementation should not introduce additional allocations (as the original one) but may require performance evaluation.
All tests are passing.

Closed in favor of #53


Support dark mode in the UI

As a UI user I want to be able to change the colors to a light-on-dark theme (dark mode). As an option the color scheme (light or dark mode) should automatically be chosen based on the system/browser setting.
Earlier designs:

assigning to Sunglow for now to have on our backlog, but this is not currently prioritised for this quarter, cc @JamWils

For any people following this issue, design is working on all of the pieces for this support.

@JamWils Do we have an update on whether this will make it into one of the next releases?

Background

Impacted users: All users of Weave GitOps
Mission/Objective: Allow users to set their color schemes based on their preferences.
Scope: This issue focuses on what is required to release the Dark Mode MVP. We'll describe future iterations in separate GH issues.

Acceptance Criteria for V 1.0

[ ]  As a Weave GitOps user, I will be able to choose my UI color settings and select between light mode, dark mode and system mode.
[ ]  As a Weave GitOps user, I can only select one color mode at a time.
[ ]  As a Weave GitOps user, I should be able to revert any changes made by simply choosing a different color mode.

Additional Reqts

[ ]  There should be a brief description for system mode (maybe via a tooltip that works on hover)

Relevant PRs and Issues

[ ] #3670
[ ] #3677
- [ ]  Adds dark mode switch that remembers your selection plus dark mode design.
- [ ]  Footer JSX and styling changed to have background inside content.
- [ ]  Sign in background changed and animation removed.
- [ ]  Makes ton of small changes to accommodate dark mode where we weren't prepared
[ ] #3681
[ ] #3682
[ ] #3683

@yiannistri @joshri Closing per our refinement. Great work!


C++/WinRT needs Windows builds to build with Clang as well as Visual C++

We currently only build with Clang on non-Windows platforms. That means that C++/WinRT itself is never built with Clang as part of the build. We have a growing number of teams that need to use C++/WinRT with Clang on Windows so this is becoming very important.

Is there much work required to get the Windows builds working with Clang? From my perspective I would be interested in support for both clang and clang-cl.

You can manually build with Clang. I describe how to do it here. VS is expected to support Clang natively in a future update. At that point, you'll be able to pick the compiler directly inside Visual Studio.

Thanks Kenny. What about when building via CMake? Is that (or will that be) a supported scenario?

That should work as well, although we don't really use cmake for C++/WinRT development. You are of course free to use whatever build system you wish for your own projects. C++/WinRT is used inside Clang/CMake projects. Typically projects will a custom CMake command to run cppwinrt.exe to generate the headers, but I don't know much about that.

Moved to https://github.com/microsoft/cppwinrt/issues/368


Use batch_alter_table for column rename

SQLite doesn't support column renames until v3.25.0, so use Alembic's
batch_alter_table support to make it work the same everywhere.

I tried this one but it doesn't work for me with sqlite 3.22 and sqlite databases removed:
# /usr/local/bin/keylime_verifier
Using config file /etc/keylime.conf
2021-03-15 13:43:04.715 - keylime.keylime_db - WARNING - database_url is not set, using deprecated database configuration options
2021-03-15 13:43:04.770 - keylime.keylime_db - WARNING - database_url is not set, using deprecated database configuration options
2021-03-15 13:43:04.772 - alembic.env - INFO - Migrating database cloud_verifier
2021-03-15 13:43:04.772 - alembic.runtime.migration - INFO - Context impl SQLiteImpl.
2021-03-15 13:43:04.772 - alembic.runtime.migration - INFO - Will assume non-transactional DDL.
2021-03-15 13:43:04.777 - alembic.runtime.migration - INFO - Running upgrade  -> 8a44a4364f5a, Initial database migration
2021-03-15 13:43:04.781 - alembic.runtime.migration - INFO - Running upgrade 8a44a4364f5a -> eeb702f77d7d, allowlist rename
Traceback (most recent call last):
  File "/usr/local/bin/keylime_verifier", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.6/dist-packages/keylime/cmd/verifier.py", line 19, in main
    keylime.cmd.migrations_apply.apply('cloud_verifier')
  File "/usr/local/lib/python3.6/dist-packages/keylime/cmd/migrations_apply.py", line 31, in apply
    alembic.config.main(argv=alembic_args)
  File "/usr/local/lib/python3.6/dist-packages/alembic/config.py", line 581, in main
    CommandLine(prog=prog).main(argv=argv)
  File "/usr/local/lib/python3.6/dist-packages/alembic/config.py", line 575, in main
    self.run_cmd(cfg, options)
  File "/usr/local/lib/python3.6/dist-packages/alembic/config.py", line 555, in run_cmd
    **dict((k, getattr(options, k, None)) for k in kwarg)
  File "/usr/local/lib/python3.6/dist-packages/alembic/command.py", line 298, in upgrade
    script.run_env()
  File "/usr/local/lib/python3.6/dist-packages/alembic/script/base.py", line 489, in run_env
    util.load_python_file(self.dir, "env.py")
  File "/usr/local/lib/python3.6/dist-packages/alembic/util/pyfiles.py", line 98, in load_python_file
    module = load_module_py(module_id, path)
  File "/usr/local/lib/python3.6/dist-packages/alembic/util/compat.py", line 184, in load_module_py
    spec.loader.exec_module(module)
  File "<frozen importlib._bootstrap_external>", line 678, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/usr/local/lib/python3.6/dist-packages/keylime/migrations/env.py", line 147, in <module>
    run_migrations_online()
  File "/usr/local/lib/python3.6/dist-packages/keylime/migrations/env.py", line 127, in run_migrations_online
    context.run_migrations(engine_name=name)
  File "<string>", line 8, in run_migrations
  File "/usr/local/lib/python3.6/dist-packages/alembic/runtime/environment.py", line 846, in run_migrations
    self.get_context().run_migrations(**kw)
  File "/usr/local/lib/python3.6/dist-packages/alembic/runtime/migration.py", line 522, in run_migrations
    step.migration_fn(**kw)
  File "/usr/local/lib/python3.6/dist-packages/keylime/migrations/versions/eeb702f77d7d_allowlist_rename.py", line 20, in upgrade
    globals()["upgrade_%s" % engine_name]()
  File "/usr/local/lib/python3.6/dist-packages/keylime/migrations/versions/eeb702f77d7d_allowlist_rename.py", line 42, in upgrade_cloud_verifier
    batch_op.alter_column('verifiermain', 'ima_whitelist', new_column_name='allowlist', existing_type=sa.Text(length=429400000), existing_nullable=True)
  File "/usr/lib/python3.6/contextlib.py", line 88, in __exit__
    next(self.gen)
  File "/usr/local/lib/python3.6/dist-packages/alembic/operations/base.py", line 353, in batch_alter_table
    impl.flush()
  File "/usr/local/lib/python3.6/dist-packages/alembic/operations/batch.py", line 114, in flush
    fn(*arg, **kw)
  File "/usr/local/lib/python3.6/dist-packages/alembic/operations/batch.py", line 403, in alter_column
    existing = self.columns[column_name]
KeyError: 'verifiermain'

@stefanberger this should be fixed now

Looks good, LGTM

Maybe leave a message in the python code why you do it this way....

Good point, and done.

+1

LGTM


Set network-specific thresholds for release 1.3

@fassadlr , please review the chosen thresholds for each network type. Specifically the mainnet threshold is set to 95%.


ValueError: Default process group has not been initialized, please make sure to call init_process_group.

When I use a single GPU for fine-tune, I encounter the following error. How can I solve it?
Traceback (most recent call last):
File "/OmniGen/train.py", line 368, in
main(args)
File "/OmniGen/train.py", line 236, in main
dist.all_reduce(avg_loss, op=dist.ReduceOp.SUM)
File "/anaconda3/envs/omnigen/lib/python3.10/site-packages/torch/distributed/c10d_logger.py", line 75, in wrapper
return func(*args, **kwargs)
File "/anaconda3/envs/omnigen/lib/python3.10/site-packages/torch/distributed/distributed_c10d.py", line 2208, in all_reduce
group = _get_default_group()
File "/anaconda3/envs/omnigen/lib/python3.10/site-packages/torch/distributed/distributed_c10d.py", line 1008, in _get_default_group
raise ValueError(
ValueError: Default process group has not been initialized, please make sure to call init_process_group.

Add the code "if dist.is_available() and dist.is_initialized():" before the "dist.all_reduce(avg_loss, op=dist.ReduceOp.SUM)" and dist.barrier(). And change "model.module.save_pretrained(checkpoint_path)" to
if dist.is_available() and dist.is_initialized():
model.module.save_pretrained(checkpoint_path)
else:
model.save_pretrained(checkpoint_path)
And then you can run train.py use single GPU.

Can you show me your command? I want to reproduce the error

Can you show me your command? I want to reproduce the error

CUDA_VISIBLE_DEVICES=5 accelerate launch
--num_processes=1
train.py
--model_name_or_path Shitao/OmniGen-v1
--batch_size_per_device 2
--condition_dropout_prob 0.01
--lr 1e-3
--use_lora
--lora_rank 8
--json_file ./toy_data/toy_subject_data.jsonl
--image_path ./toy_data/images
--max_input_length_limit 18000
--keep_raw_resolution
--max_image_size 1024
--gradient_accumulation_steps 1
--ckpt_every 50
--epochs 200
--log_every 1
--results_dir ./results/toy_finetune_lora

Add the code "if dist.is_available() and dist.is_initialized():" before the "dist.all_reduce(avg_loss, op=dist.ReduceOp.SUM)" and "dist.barrier()". And change "model.module.save_pretrained(checkpoint_path)" to if dist.is_available() and dist.is_initialized(): model.module.save_pretrained(checkpoint_path) else: model.save_pretrained(checkpoint_path) And then you can run train.py use single GPU.

Thanks for your suggestions! I have updated these code.


CODE RUB: LocalFiles Upgrade V2.10.0

Upgrade all LocalFiles Exceptions to v2.10.0 of The Standard
See the video explaining the process here
Anyone wanting to take this, please leave a comment letting others know you are working on this.

Hey @glhays , I am working on this code rub issue. Thanks.

@sammy-timalsina if you would please make all the access modifiers of the test methods in LocalFiles private. Thanks

@glhays  Sure no problem, I made the changes, I will shortly commit it. Regards.


Provide a way of directly using images in me.image(src=

You can use data URLs but it's not super obvious to do this.
Example: https://colab.research.google.com/drive/1xIQ3eGTc7eDTdCurBas9by1K5ACjCIZj?usp=sharing

once we have static assets #960, it will alleviate the need to pass in images directly (which would be less efficient)


Removing layer throws error and makes plugin unusable

As there is currently no way to abort a long request (#8), I removed the layer in which data was being pumped. This threw an error (window), which I closed. When I reopened the plugin, it was no longer responsive.
Restarting QGIS solved the issue.

Hey @peterdesmet, the plugin is now interruptible (Stop button)!
Do you think that solves the issue?

Yeah, works great.


[Scala 3] expression evaluation doesn't evaluate

When attempting to evaluate an expression while stopped at a breakpoint:
1+1
Cannot evaluate because of failed compilation:
-- Error: /var/folders/71/ynbmyt9d3nz_pqr93020rp7r0000gn/T/scala-debug-adapter-8b9d92d83c274012a52576709f7adac017798932612006910083/StepSequencer.scala:190:36
190 |      state.keyScrollOffset = guardY(state.keyScrollOffset + (pageMul * inc))
    |                              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |                              expression already inserted.

(the "already inserted" expression is the line at the stopped breakpoint)
Build server currently being used is Bloop v1.5.2.
Metals Java: 17.0.2 from Eclipse Adoptium
Metals Server version: 0.11.7
Scala 3.1.1

I should add this happens very rarely, I can't even remember how to reproduce this. Most of the time it's still
1+1
Cannot evaluate because of ch.epfl.scala.debugadapter.internal.evaluator.MethodInvocationFailed: "java.lang.NoClassDefFoundError: scala/runtime/Nothing$".

Thanks for opening the issue. I think I know how to fix the  expression already inserted.
The second error "java.lang.NoClassDefFoundError: scala/runtime/Nothing$" is unexpected. I would be curious to know how to reproduce this. Could you give more details?

@unthingable I tried to reproduce the second error but with no success. There must be some problem with the classpath used by Bloop to start the debuggee. Could you share your repository or reproduce in another repository?

@adpi2 so this one is a little weird to explain: the project in question is a Bitwig controller extension (https://github.com/unthingable/monsterjam - haven't published the new Scala3 version yet), that's built against a public API, available as interfaces only. The runtime is Bitwig's own JVM, which contains the API implementation. It doesn't contain Scala, instead I bundle the Scala runtime as a shadow dependency into the extension JAR.
When debugging it's not Bloop that starts the debuggee, the process is already running with an open remote debug port. I launch the debugger with a remote debug configuration:
    "version": "0.2.0",
    "configurations": [
        {
            "type": "scala",
            "name": "Bitwig Debug (Attach)",
            // "projectName": "monsterjam",
            "buildTarget": "monsterjam",
            "request": "attach",
            "hostName": "localhost",
            "port": 9898
        }
    ]

Seems that the debugger is unable to locate the bundled and loaded Scala runtime.
Probably not reproducible without actually running Bitwig, but if anyone is up for it, there is a free trial version (https://www.bitwig.com/download/) and then % BITWIG_DEBUG_PORT=9898 /Applications/Bitwig\ Studio\ 4.3.app/Contents/MacOS/BitwigStudio :)

Thanks for the details. It seems your process manages several class loaders and the expression evaluator does not know which one to use, so it picks one randomly, but not the right one. I think I can fix this by calling getClass().getClassLoader() on the current this.

Actually if the scala lib is shaded then its package names are changed and the evaluator cannot load them any more. So I don't think there is a general solution to this problem. Shaded jars are not compatible with debuggers.

So I don't think there is a general solution to this problem. Shaded jars are not compatible with debuggers.

Thank you for looking into this. It seems there is at least one solution and IntelliJ has found it, but I don't know what it is.


Featue: Playlist management

Featue: Playlist management
Epic: Playlist management #11
Changes made

[x] Created a common modal
[x] Created playlist modal
[x] Updated playlist context
[x] Added reducers for playlist management
[x] Added services for playlist management
[x] Added playlist page for showing all playlist
[x] Added a single playlist page for showing videos in a particular playlist

How does the solution address the problem
This PR will add the playlist management feature
Live preview: https://video-4k3kggv66-pratham82.vercel.app/
Linked issues
Resolves #11

Feature is Working.


authentication

First step to adding authentication.
Doesn't change server.js much, all of the action happens in /api/auth/index.js
There's already a /login route stashed somewhere, so I'm using /logger as the login page for now as a placeholder.
So far:

/admin now redirects to /logger if you havent logged in
/logger accepts an email and password defined manually in /api/auth/index.js which also carry a scope parameter
After logged in under an account with the proper scope, your cookie is stored for 2mins and you are redirected to /admin and can now access it any time (if you're quick, or feel like changing the cookie stored time).

Todo:
Hook up accounts to db
Play with cookie settings
Probably will need a lot of testing once the db is connected and there are more cases.
Edit - more todo, make /admin redirect to a login before authenticated. I tried doing this with a function in the handler but I don't know how to set view: 'Default' if my handler is a function so the page would redirect but wouldnt authenticate

Also need to add joi, bluebird, hoek and boom to dependencies

if you're following<EMAIL_ADDRESS>it should be good
since this is using user session auth we probably need to add a user model unless we want to just tie user auth to google or facebook oauth. anyway looking good!


Can the CRL path of the DC cert contain LDAP end points if using windows 10 AAD joined machines (not AD joined)

I had terrible issues with a CRL server offline error
a combination of removing the LDAP end points from the cert, using HTTP endpoint AND making sure the DC cert subject field was populated with a DN plus in the other people store finally fixed this.
It would be good to have clarification if the LDAP points would be inaccessible to non-AD joined device.

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

ID: c47de28c-e44b-ca73-efde-fae303e0d3f8
Version Independent ID: 71202a24-5b2f-844a-acce-d48c46873f2b
Content: Hybrid Key trust Windows Hello for Business Prerequisites (Windows Hello for Business) - Microsoft 365 Security
Content Source: windows/security/identity-protection/hello-for-business/hello-hybrid-key-trust-prereqs.md
Product: w10
Technology: windows
GitHub Login: @mapalko
Microsoft Alias: mapalko

This clarification is important because in the cert guidelines for cert trust it says 'should' is this a should or must?

@scyto An independent contributor will follow-up with necessary clarifications. Thank you.

Hello @scyto,
For sure, ldap cannot be used in the described scenario, as ldap is not accessible by non-AD joined machines.
a combination of removing the LDAP end points from the cert, using HTTP endpoint AND making sure the DC cert subject field was populated with a DN plus in the other people store finally fixed this. - from my experience, using HTTP endpoint and adding cert to trusted root store is enough, are you sure it didn't work without adding DN as a DC cert subject ?
It would be good to have clarification if the LDAP points would be inaccessible to non-AD joined device. - it is stated in the "important" section in the same part of the article:
For Azure AD joined device to authenticate to and use on-premises resources, ensure you:
Install the root certificate authority certificate for your organization in the user's trusted root certificate store.
Publish your certificate revocation list to a location that is available to Azure AD joined devices, such as a web-based url.
Thank you


use Map is Error

var  map = json.decode('{"a":1,"b":2}');
map['a'] = 7;
it error !!
TypeError: Cannot use variable of type String as list index at unknown
print(map['a']);
Receiver: null

Fixed in v0.6.1


RAD Link fails to find relative library paths: Warning(042): /LIBPATH: path doesn't exist ......<something>

Might be related that they are in a response file? Can't really test it as our linker call is huge so it needs to be in a response file. But radlinker seems to fail to find the relative paths. Same command line on lld-link.exe and link.exe work

This warning is specific to radlink and AFAIK other linkers don't warn about invalid paths. Can you please double check if path really exists? I also changed linker in https://github.com/EpicGamesExt/raddebugger/commit/af57e321de659edd6416d9e03f6f297b4a2f1999 to print out full path to help with debugging these warnings.


Instructions to download / upload a grid profile file or alternative advice

Thank you for building this HA integration.
Describe the bug
On a previous Envoy firmware (7.6) I had the ability to select and set specific custom grid profiles that Enphase support had put onto my Envoy.  However, since an upgrade (to 8.2) these custom grid profiles are not persistent.  I can get Enphase support to enable one of them, but then the other is no longer available in the grid profiles via local access (or via enphase_envoy.get_grid_profiles).
What is the best manner to use set_grid_profile in this case?  Is there a way to download the grid profile (after getting Enphase support to enable it for me) and then use upload_grid_profile when I next want to use that profile ?
Account type

[ ] Installer
[x] DIY / DHZ
[ ] Home Owner (This integration is not (fully) functional with a home owner account!)

Envoy

[x] Metered with CTs installed

[ ] Metered without CTs

[ ] Standard

FW version: D8.2.127

Amount of micro inverters connected: 40

Home Assistant

Version: 2024.8.3

Additional context
The first grid profile has no export limit, and is generally in use.  The second grid profile has a 0.2kW export limit and is used when feed in tariffs go negative.  The reason I need the 0.2 kW export is that with 0 kW export at times the battery would service home load and be drained, rather than the curtailment of the solar production being reduced.
Relevant snippet of Home Assistant error log.

Hi @fhopley, unfortunately I didn’t have time to look into the grid profiles changes and firmware 8 yet. So currently I do not have an advise for you. I will also look into grid profiles itself at some point, see if I can extract one at some point. But I wonder how feasible it is to re-upload one every time.

But I wonder how feasible it is to re-upload one every time.

It is very feasible to re-upload one every time.
Since my Envoy system starts with 8.x, the grid profile never persists for me (one gets deleted after system switching away from it). I'm using upload_grid_profile just before calling set_grid_profile, and it has been working reliably ever since I added it to this integration.
But I agree that extracting the grid profile is a tricky one. I have extracted grid profile files, but I feel that it is not appropriate to share my approach publicly. If you need some help, we could chat privately.

Would be good to see if there is a way to extract grid profiles. Working through with someone who has everything ready, installer access etc but now needing some guidance on setting grid profiles.
Cheers

(Feel free to send an email to me if anyone is interested in my approach.)


PERF: Speed up distance computations

Use Pandas vectorized operations to speed up the validation and
computation of first distances and distances to baseline.
In a dataset with ~6K samples runtime went from >1 minute to under a
second.

All tests pass except for test_first_distances_ecam which fails because of replicate_handling='drop'. As far as I can tell when there's a repeated state the repeated state and the next state are dropped. Which seems odd, but wanted to check if this is expected or if I am misunderstanding something. Seems to me that only the repeated state should be dropped.
For example, when I go to ecam_map_maturity.txt, I noticed that sample 10249.C001.07SS is not in ecam-first-distances even though the state for that sample isn't repeated. Same is true for 10249.C001.17SS which should be included since state 10 is repeated but state 11 isn't.  Any help would be great here.

Fair enough, thanks for the explanation! I'll fix the code in that case. Would you also be OK if I changed the documentation in plugin_setup.py? Seems like "will discard all replicated samples" is ambiguous on the fact that t and t+1 are dropped.

Would you also be OK if I changed the documentation in plugin_setup.py? Seems like "will discard all replicated samples" is ambiguous on the fact that t and t+1 are dropped.

documentation updates are very welcome 😄
but not that line — a bunch of actions in this plugin share the same parameter descriptions. So this action should either get a unique parameter description, or better describe this behavior in the action description (which IMO would be best, since this is the behavior from any missing samples, not specifically replicate_handling='drop')

Cheers thanks so much.
Would you be able to help me diagnose one more case. I've updated my code, but now I am seeing that other samples are currently being excluded from the expected output. For example: 10249.C002.09SS, 10249.C002.15SS, 10249.C002.18SS, 10249.C005.07SS, 10249.C008.04SS, 10249.C008.12SS, 10249.C009.16SS, 10249.C011.04SS, 10249.C014.12SS, 10249.C018.03SS, 10249.C018.17SS, 10249.C021.04SS, 10249.C021.05SS, 10249.C021.07SS, 10249.C021.08SS, etc. When I look at the mapping file for a few of these, I don't see them being associated with any of the repeated sample pairs that are dropped. Any hints as to why these are dropped from the expected output?
I've highlighted the first few in the context of the mapping file here:
10249.C002.09SS, 10249.C002.15SS, 10249.C002.18SS

10249.C005.07SS

10249.C008.04SS, 10249.C008.12SS

Hey @ElDeveloper ,
Those first distances are not being calculated because they are missing the immediately preceding timepoint (i.e., it's not being dropped, rather that timepoint is just missing to begin with).
The first distances intervals are meant to be measured across the same time intervals in each subject. So if you have a table like this (rows are samples, cols are months, cells indicate if a sample was collected at that timepoint):

subject-id
0
1
2
3
4
5
10

A
+
+
+
+
+
+
+

B
+

+
+
+
+

A has all first distances calculated, but B only has distances starting at timepoint 4 (i.e., samples were collected at 0 and 3, but the immediately preceding timepoint is missing). On the other hand, one could drop those columns and all distances would be calculated for each sample in a table that looks like this:

subject-id
0
3
4
5
10

A
+
+
+
+
+

B
+
+
+
+
+

This is the case above, where 10249.C002.09SS is a sample at month 8, but month 7 is missing. If month 7 were missing in all subjects, the distance interval would be from months 6->8. But that's not the case here, as the interval is based on all samples in the table.
Does that make sense?

Thanks so much for the explanation. I didn't realize all this was going on under the hood.  I like the idea of auto vs shared? or something like that. Let me rework this current version and I'll see where I get. 👍🏽

@nbokulich all tests are passing and should be ready for review now. I've left out the interval selection out for this pass but it is something we could naturally add in a follow-up PR. Looking forward to hearing your thoughts, I know this sort of code can sometimes be hard to read.

Hey @ElDeveloper, thanks for all of your hard work here! We are currently doing some PR triage and review right now - are you still in the process of addressing @nbokulich's review comments?

Hi @lizgehret thanks for the heads up. I have almost replied to all the comments (although I haven't posted the review yet), but there's a few where I still need to do some additional work. I'll respond ASAP.


Add support for the mail envelope from

To handle bounced emails with a dedicated email address, it is required to set the envelope from mail address: https://stackoverflow.com/questions/1782659/how-to-set-the-return-path-to-an-email-address-other-than-sender-address-using-j
Currently Simple Java Mail enables to set the SMTP parameter mail.smtp.from via the configuration of the Session of the Mailer.
It would be great if Simple Java Mail could:

handle a property to manage the mail.smtp.from option directly (so users won't have to deal with the Session object),
handle the "envelope from" property per mail (if the solution with SMTPMessage in the stackoverflow issue works: I have not tested it yet).

If that's ok, I can help implementing these features (and also test the solution with SMTPMessage).

Isn't this the replyTo you are referring to?

Actually the replyTo enables to set an address that will be used when the recipient will answer the mail.
Whereas in the case of the mail envelope from field, the address will be used by MTA and SMTP servers to notify the sender that the email could not be sent.
Usually the default behavior is correct: we want the sender to be directly notified when an email could not be sent.
However, in some cases we want to be able to customize the address that will be used in the Return-Path header (that will be set by the SMTP server). In my current case, I want to use the Return-Path field for technical reasons: my client wants to use a special address for sending mails, but he cannot see the received emails in this address (big corporation, slow processes...) and he also wants to be notified when emails could not be sent.
But I think other people can benefit from this feature: you may want to have a nice email address to send mails and use a dedicated address like<EMAIL_ADDRESS>to automatically handle bounced back emails by a reporting software.

It's also written that the from address should be filled by the receiving party, not the sending party. I'm still looking into it, but it's a little confusing.

It seems that this is the Return-Path header field that is filled by the receiving party.
However this field should be based on the mail envelope from value that is sent to the mail server before the mail object.
Currently I made some tests with Google accounts and non existing email providers: each time the SMTP server I am connecting to (a postfix server on my network, not the Google mail server) is notifying me on the correct address (not the one I sent the e-mail from).
Also the tests were currently only made with the mail.smtp.from parameter in the Session class.

Excellent, I'll add it in this weekend. Thanks for the tests!

Thank you for your help!
If you want, I can also make a test with the SMTPMessage solution to set the mail envelope from value. This other solution would enable to use this feature per Mail instead of using it per Mailer.

Currently working on these changes. Going to name it bounceToRecipient
email.setBounceToRecipient(..)
emailBuilder.bounceTo(..)

The JavaDoc will explain it will optionally set the Return-Path / Envelope FROM as a hint for the SMTP server to return bouncing emails to...

That sounds perfect, thank you!!


Make crop scriptable

This PR makes crop transform scriptable as discussed in #1375.

Codecov Report

Merging #1379 into master will increase coverage by 0.02%.
The diff coverage is 25%.

@@            Coverage Diff             @@
##           master    #1379      +/-   ##
==========================================
+ Coverage    63.9%   63.92%   +0.02%
==========================================
  Files          78       78
  Lines        6147     6149       +2
  Branches      940      941       +1
==========================================
+ Hits         3928     3931       +3
  Misses       1941     1941
+ Partials      278      277       -1

Impacted Files
Coverage Δ

torchvision/transforms/functional.py
70.85% <25%> (-0.41%)
:arrow_down:

torchvision/transforms/transforms.py
81.04% <0%> (+0.58%)
:arrow_up:

Continue to review full report at Codecov.

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

Hi @fmassa ! Made the changes you requested.

@ekagra-ranjan @fmassa, since the transforms are happening on the tensors and we would ideally don't want them to be in-place by default, should we add a boolean like we have in normalize ?
If not, I think then we should add an in-place checking test to prevent any such behaviour due to some sneaky code-err.

@surgan12 the indexing that we perform in here indeed shares memory with the original tensors, but that's the same behavior that we already have in PIL.Image.crop, so I think this is fine.
import numpy as np, PIL

a = (np.random.rand(2, 2) > 0.5).astype(np.uint8)
i = PIL.Image.fromarray(a)
j = i.crop((0, 0, 1, 1))
aa = np.asarray(j)

print(a.flags.owndata)  # True
print(aa.flags.owndata)  # False, is a view

@fmassa, yes makes sense.


fix: symbol upload moduleName

Description
Changes to support uploading with bsv1 protocol via symbol-upload
Checklist

[x] Tested manually
[x] Unit tests pass with no errors or warnings
[x] Documentation updated (if applicable)
[x] Reviewed by at least 1 other contributor

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Is this your job?

I saw the work of Sam in https://github.com/wuwen1994/omni-shadow. After comparing the code structure and details, I found that they are exactly the same as those you released. Is this a sister article?

I saw the work of Sam in https://github.com/wuwen1994/omni-shadow. After comparing the code structure and details, I found that they are exactly the same as those you released. Is this a sister article?

No, this paper has nothing to do with us

The article of that code even copied the images from SAMUS. amazing
https://www.sciencedirect.com/science/article/pii/S1047320324001019

The article of that code even copied the images from SAMUS. amazing https://www.sciencedirect.com/science/article/pii/S1047320324001019

Amazing ∑(;°Д°)


🛠 Repo: HTML reporter needs tests

We have decent coverage in most places now.  The glaring exception is the HTML reporter, which is completely untested.
Add tests for the HTML reporter.
This will necessitate likely some unit tests and browser-based functional tests.  The latter of which we have none of, so we'd need to pull in some extra dependencies to do so.  This won't be trivial.

I have branches that do this with Karma that I need to rebase (and see if the issues on Sauce went away or not). Bonus: the test coverage for the browser, while I implemented it for the reporter, theoretically opens the door to browser-specific units tests if we still have anything that would need it (I dunno if all of those were shims we've now gotten rid of) or browser integration tests (e.g. for https://github.com/mochajs/mocha/pull/1800#issuecomment-319479473) with alternate Karma settings (something I still need to explore to ensure they're set up right).

@ScottFreeCode You are still assigned to this even though a PR is there in https://github.com/mochajs/mocha/pull/3102.

@rugk seems like the owner has lost interest, perhaps checkout branch, rebase and re-open PR if you are still interested. It sounded close if passes IE11 testing.

@giltayar do you have any suggestions how to start with this one?
We also have some funds available we could invest.

There is playwright of Microsoft.
They also offer their own test runner for Node. Folio, "a highly customizable test framework", available in preview.
This is going to be a very competitive player in test frameworks.

There's no lack of browser automation tools, although not sure this is needed. I would go with a simpler snapshot test (i.e. save the HTML baseline file, and always compare against that). Usually, I'm not a fan of these, but given that we're looking at one or two tests, I think it's the easiest solution.
If you do want a browser automation test. Given that there aren't a lot of tests here, I would go with the tried and true: Selenium WebDriver.

I don't believe this

Marking this as one I'm interested in, I use Playwright as a Microsoft employee and it's excellent (I'm biased ;) ) but I agree it may not be necessary. Happy to work with whoever wants to get started on this, as I'm currently focusing on some other issues.


internal/chanworker: staticcheck

Closing this, see my comment here:

https://github.com/perkeep/perkeep/pull/1405#issuecomment-894727096


Added Time on Debug Messages

Added timestamp on debug log messages

@trap98 regarding your comment during standup meeting, here is what is displayed before we start I²C with the current PR. As you see, we start by displaying 2000-00-00T00:00:00Z, which seems acceptable behavior to me.
[2000-00-00T00:00:00Z][HEAP:33360]DEBUG: Reading configuration file as JSON: /coolBoardLedConfig.json
[2000-00-00T00:00:00Z][HEAP:33360]DEBUG: Configuration JSON:{"ledActive":true}
[2000-00-00T00:00:00Z][HEAP:33488]INFO:  LED configuration loaded
[2000-00-00T00:00:00Z][HEAP:33760]INFO:  MAC address is: 2C:3A:E8:4F:BE:B5
[2000-00-00T00:00:00Z][HEAP:33760]INFO:  Firmware version is: v0.2.5-6-g65262d07
[2000-00-00T00:00:00Z][HEAP:33296]DEBUG: Reading configuration file as JSON: /wifiConfig.json
[2000-00-00T00:00:00Z][HEAP:33296]DEBUG: Configuration JSON:{"wifiCount":1,"timeOut":180,"Wifi0":{"ssid":"LaPaillasse","pass":"biohacking"}}
[2000-00-00T00:00:00Z][HEAP:33392]INFO:  Wifi configuration loaded
[2000-00-00T00:00:00Z][HEAP:33392]INFO:  Wifi configuration
[2000-00-00T00:00:00Z][HEAP:33392]INFO:    Wifi count  = 1
[2000-00-00T00:00:00Z][HEAP:33392]INFO:    Timeout     = 180
[2000-00-00T00:00:00Z][HEAP:33392]INFO:   Network # 0
[2000-00-00T00:00:00Z][HEAP:33392]INFO:      SSID    = LaPaillasse
[2000-00-00T00:00:00Z][HEAP:33712]INFO:  No firmware update pending
[2000-00-00T00:00:00Z][HEAP:32640]DEBUG: Reading configuration file as JSON: /coolBoardConfig.json
[2000-00-00T00:00:00Z][HEAP:32640]DEBUG: Configuration JSON:{"logInterval":3600,"ireneActive":false,"jetpackActive":false,"externalSensorsActive":false,"sleepActive":true,"manual":false,"mqttServer":"mqtts.lacoolboard.io"}
[2000-00-00T00:00:00Z][HEAP:32896]INFO:  Main configuration loaded
[2000-00-00T00:00:00Z][HEAP:32624]DEBUG: Reading configuration file as JSON: /coolBoardSensorsConfig.json
[2000-00-00T00:00:00Z][HEAP:32624]DEBUG: Configuration JSON:{"BME280":{"temperature":true,"humidity":true,"pressure":true},"SI114X":{"visible":true,"ir":true,"uv":true},"vbat":true,"soilMoisture":true,"wallMoisture":false}
[2000-00-00T00:00:00Z][HEAP:32896]DEBUG: Builtin sensors configuration loaded
[2018-06-25T10:23:14Z][HEAP:33696]DEBUG: BME280 status after begin is: 96


about multi instance inpainting

Does the model support to generate multi objects in one pass?

hi @zhangh0920, thanks for your interest. Currently, we do not consider generating multi objects in one pass, which can be achieved by multiple passes. If you have any exciting ideas or want to make a pull request for this feature, we're happy to review and merge it :)


Licence details should be in a drop down

The content for the details looks good but we would like this to be viewable via a dropdown only.
Similar to the "Show more", "Show less" expand and collapse option for the summary below the data set title on this page, we should have "See details", "Hide details" for the licence information.

@sabahfromlondon how about somethink like this:

and

@yolile thanks for sharing, The alignment is slightly off if you can fix that, otherwise looks good.

Thank you, @sabahfromlondon, it looks like this now

all good?

Ah ok, looks good apart from the slight alignment issue


Package details page for package with multiple versions is slow

Example: https://www.nuget.org/packages/FAKE/
takes ~15 seconds to load

@skofman1 Is this a duplicate?
https://github.com/NuGet/Engineering/issues/1453

@chenriksson , you are right. Closing! 😄


Cloning error

Hey All,
I get an error cloning this repo as follows:
error: invalid path 'Log-Sources/LogScale-Internal/src/alerts/LogScaleLogCollectorErrors>5in30minutewindow.yaml'
error: invalid path 'Log-Sources/Microsoft/AzureAD:EntraID/build.sh'
error: invalid path 'Log-Sources/Microsoft/AzureAD:EntraID/microsoft--azuread--0.1.0.zip'
error: invalid path 'Log-Sources/Microsoft/AzureAD:EntraID/src/README.md'
error: invalid path 'Log-Sources/Microsoft/AzureAD:EntraID/src/dashboards/ApplicationManagement.yaml'
error: invalid path 'Log-Sources/Microsoft/AzureAD:EntraID/src/dashboards/DirectoryManagement.yaml'
error: invalid path 'Log-Sources/Microsoft/AzureAD:EntraID/src/dashboards/FailedSignIns.yaml'
I'm not sure what causes this error but I think it is related to > being an invalid character in the file path. In which case the first file path would need to have its name changed. But I might be wrong, are others able to download this successfully?
thanks,
Andrew

No issues here, I just cloned it to check. Still having a problem cloning it?

It seems to be an issue cloning using Git for Windows. My workaround was cloning into a Linux distro on WSL2.


Problems upgrading docker env from 0.13.6 to 0.14.2

Deployment Method

[ ] Installer
[ ] Docker Development
[X] Docker Production
[ ] Bare metal Development
[ ] Bare metal Production
[ ] Digital Ocean image
[ ] Other (please provide a link Steps to Reproduce

Describe the problem*
I have issues updating my env from 0.13.6 to 0.14.2.
I tried it several times now.
If I do it with the 0.13.6 docker-compose environment and just pull the new images and run "invoke upgrade" in the server container it first seems to work but I get authorization error messages when I try to do certain things, when I then clear my authorization cookie to log in again I am not able to.
If I update my .env and compose files to the new versions using Caddy I am not able to acces the server but everything seems fine, no real error messages.
Actually Caddy is making me some troubles because I run everything already behind a reverse HAProxy using HTTPS, Caddy in the default config tries to get a cert but is not able to. I deactivated the auto HTTPS, I tried default entries for server and also my real server name but nothing works.
Is there an upgrade path for docker env from 0.13.6 to 0.14.2? What do I have to change and what not?
Steps to Reproduce

Relevant log output
No response

because I run everything already behind a reverse HAProxy using HTTPS,

This is likely the issue. Check out the "breaking changes" section in the 0.14.0 release notes:

https://inventree.org/blog/2024/03/02/0.14.0
https://github.com/inventree/InvenTree/releases/tag/0.14.0

Site access has become more "locked down" in django 4.
You will need to read these docs to configure - https://docs.inventree.org/en/stable/start/config/#server-access

but I get authorization error messages when I try to do certain things, when I then clear my authorization cookie to log in again I am not able to.

What error messages are you getting?

What error messages are you getting?

Can't tell for sure at the moment, will update when I do the next try if it happens again...
Thanks for the links, I missed the  blog entry completely...
Still I am not absolutely sure how to handle the Caddy stuff, I used Caddy in a different project but there all the automation it provides was useful, not in this particular case so.
I think for the moment I'll stick with Nginx and try to get the other problems sorted...

Ok got it working with Nginx still, played with the CORS settings and site URL and now it seems to work.
Post-poning the switch to Caddy...


filebot: 5.1.4 -> 5.1.5

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for filebot is: Ultimate TV and Movie Renamer
meta.homepage for filebot is: https://filebot.net
meta.changelog for filebot is: https://www.filebot.net/forums/viewforum.php?f=7
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 5.1.5 with grep in /nix/store/0h6mngph0kw1wn5xn7if4nq5g2ss5ync-filebot-5.1.5
found 5.1.5 in filename of file in /nix/store/0h6mngph0kw1wn5xn7if4nq5g2ss5ync-filebot-5.1.5

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

filebot

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/0h6mngph0kw1wn5xn7if4nq5g2ss5ync-filebot-5.1.5 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A filebot https://github.com/r-ryantm/nixpkgs/archive/8c36f34e027074bd5c5488008fa0464c4f879429.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/8c36f34e027074bd5c5488008fa0464c4f879429#filebot

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/0h6mngph0kw1wn5xn7if4nq5g2ss5ync-filebot-5.1.5
ls -la /nix/store/0h6mngph0kw1wn5xn7if4nq5g2ss5ync-filebot-5.1.5/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

filebot

Maintainer pings
cc @gleber @felschr for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 336827 run on x86_64-linux 1

1 package built:

filebot

Superseded by https://github.com/NixOS/nixpkgs/pull/336870, which also migrates it to pkgs/by-name.


Fix create org tests

@satbir121 @devrk96 Please fix these tests so that we can clear up release for further merging


Display language settings in 2 languages

Languages in Language setting should not be translated according to local/browser language. So "English" should be "English" even if my local language of browser is "French" or something else.

Why?
Suppose I change my local language to "French", languages list in setting also changed, and I am no longer be able to change language back to "English" as I couldn't understand which one of the option was "English" as all options were translated to "French".

I don't think it's a problem. English is at first and name of languages are understandable in most languages.
I never seen this sort of double translation elsewhere. Did you?

I was not sure whether double translation is the most suitable solution. But I do think that there should be some way that allows user to see languages names without translated to local language.
I have seen this double-translated language list setting on Twitter, not checked other applications.

What could be the appropriate solution to this problem, according to you?

Fixed in https://github.com/llaske/sugarizer/commit/0cd4903a18c6c35a81589e2ac916530a0710ed87

Closing the issue as it is fixed!

I don't close issue before release of a new version.

Closed in https://github.com/llaske/sugarizer/commit/dd3c58d56981e4dfa6d94601b95d7117a3bc9241


Question: Remove BottomBarTab during Runtime

Can I remove some tab during the runtime?
Like some
mBottomBar.removeTabAt(3); //or mBottomBar.removeTabWithId(R.id.my_id);
Thanks!

What's the use case? Do you want to disable some functionality?
Just thinking why would that be needed.

For example: In my application I have 2 ways of authentication (user&pass/free) - This information is returned by an API.
At the bottomBar I added a logout button. I want to remove this button if the authentication comes with tag "free".
I solved my actual problem creating 2 .xml, one with logout and another without it and calling
if(auth.equals("free")){
mBottomBar.setItems(R.xml.bottombar_normal);
}
else{
mBottomBar.setItems(R.xml.bottombar_logout);
}
My problem has been solved with the example above, but my suggestion is to have any way to change the items bar dynamically, in case of the developer have some parameters that depend on external factors.
Thanks for the quick response!

I'm thinking of including possibility of disabling tabs. Maybe removing them at runtime too, but disabling them is really easy to implement and I will make that available sooner.

@roughike we should have a talk in the board about this, so that we don't overlap work, cause the branch i have for custom tabs already allows this behavior


Package published on NPM is not the latest code from GitHub

Your source code on NPM and GitHub are not the same.
Seems like the NPM package is older, because the NPM package is missing MultiStreamRecorder.addStream, while it is not missing in your GitHub source code.
See attached screenshot of a diff.

also needed got #112


[PE-D][Tester B] Help window should contain a quick reference for commands

As a user, I might want to view all commands at a glance by running the help command, instead of being directed to the User Guide website.
Moreover, as a new user, if I use CampusConnect without connecting to the the internet, I will not be able to access the User Guide website entirely and may not know how to use the commands.

Labels: severity.Medium type.FeatureFlaw
original: Vatinius/ped#5

Closed, might implement this as a future feature


The org >GUIDES>Hot Module Replacement,when I try the chapter HMR with Stylesheets....

Do you want to request a feature or report a bug?
bug

What is the current behavior?
It is OK now. Because I follow all the chapters before this chapter,I find it necessary to add the behave "1.end first 2.do as your chapter content 'npm install --save-dev style-loader css-loader' 3.you should' npm start' "
If the current behavior is a bug, please provide the steps to reproduce.

What is the expected behavior?
I expected that I don‘t need to restart the serve and the program will automatically find the new loader and the module.rule that I just added in webpack.config.js .That is all .
If this is a feature request, what is motivation or use case for changing the behavior?
Please mention other relevant information such as the browser version, Node.js version, webpack version and Operating System.
{
"name": "webpack-demo",
"version": "1.0.0",
"description": "webpack 2.0 demo",
"main": "webpack.config.js",
"scripts": {
"test": "echo "Error: no test specified" && exit 1",
"watch": "webpack --progress --watch",
"start": "webpack-dev-server --open",
"server": "node server.js",
"build": "webpack"
},
"keywords": [
"webpack 2.0 ",
"Jet12138",
"学习版"
],
"author": "Jet12138",
"license": "ISC",
"devDependencies": {
"clean-webpack-plugin": "^0.1.17",
"css-loader": "^0.28.7",
"csv-loader": "^2.1.1",
"express": "^4.16.2",
"file-loader": "^1.1.5",
"gulp": "^3.9.1",
"html-webpack-plugin": "^2.30.1",
"image-webpack-loader": "^3.4.2",
"style-loader": "^0.18.2",
"url-loader": "^0.6.2",
"webpack": "^2.6.1",
"webpack-dev-middleware": "^1.12.0",
"webpack-dev-server": "^2.9.2",
"xml-loader": "^1.2.1"
},
"dependencies": {
"lodash": "^4.17.4"
}
}
win7  Chrome59.0.3071.115（正式版本）<EMAIL_ADDRESS>

We decided to not watch the webpack configuration. It would be too difficult/impossible to get reliable in all cases. So we better go for reliable not-watching.


Enhancement - A Dedicated page showing all configured/active shortcut key mappings

Description of the new feature / enhancement
Can be used to quickly check the mapped keys, make sure there is no duplication with other tools or conflicts with windows default mapped keys. Perhaps colouring those mappings that conflict with each other.
Scenario when this would be used?
Changing a PowerToys tool key mapping requires checking all other tools to make sure it does not conflict. It lets users map the same key combinations without checking if it's already in use.
Supporting information
No response

Many people have actually been asking for this. Most point to Shortcut guide, but with PowerToys shortcuts. My idea is to change the system tray fly-out: make it bigger and include the hotkeys. Have a look at #24880
FYI @niels9001

Many people have actually been asking for this. Most point to Shortcut guide, but with PowerToys shortcuts. My idea is to change the system tray fly-out: make it bigger and include the hotkeys. Have a look at #24880
FYI @niels9001

With the amount of new features and shortcuts cluttering the quicklaunch view in the systray might not be the best solution. An easy win is to atleast add them to the on/off list in the systray.
Still I wonder if we'd need something more dedicated in Settings or Shortcut Guide (@crutkas @jaimecbernardo thoughts?).

/dup #26816 and #15405


Armorstand are always visible (even with Invisible:1b NBT tag)

since 2.2.0 armor stands can be set invis the same as item frames.
to stop them from being visable go to .minecraft/config/InvisFrames.conf
change the line above Armor Stand has item check to none
if you also want to prevent them toggling visability in survival
set the line above Enable invis armor stands to false
i probably should have had these as the defaults...

Oh, I tried to do a shift-punch and it didn't change anything... :') And as I didn't see any custom settings in the modmenu I thought it was a bug... Thank you !
And thanks for the compliment, I worked hard on it ;)

changed config to ini in 2.2.1 (has some advantages and looks better in text editors)
and defaulted invis item frames to off (if you already changed it, it'll figure it out no need todo it again)
and yeee i really need to get around to modmenu stuff, just very indecicive about how to handle it.

That is and would be awesome :))


Command Line usage seemed not working

If I start nanomaven I get a list of commands like:
19:13:36.027 INFO | NanoMaven 2.2.2 Commands:
19:13:36.027 INFO |   help - List available commands
19:13:36.027 INFO |   status - Display metrics
19:13:36.027 INFO |   tokens - List all generated tokens
19:13:36.027 INFO |   keygen <path> <alias> - Generate a new access token for the given path
19:13:36.027 INFO |   purge - Clear cache
19:13:36.027 INFO |   stop - Shutdown server
19:13:36.027 INFO |

But unfortunately it does not change a thing If I give them on command line like this:
java -jar nanomaven-2.2.2.jar help

If I correctly understand the code. The command line args are not given to NanoMaven:
NanoMaven nanoMaven = new NanoMaven();
nanoMaven.launch()

which means from my understanding that could not work that way. Maybe I misunderstand a thing here. Maybe you can enlighten that a little bit?

Commands are a part of the CLI - it means they can be only invoked in a working instance of application.
java -jar nanomaven-2.2.2.jar help
Values provided like that are called program arguments and in fact, they're not commands.

Hm...we can discuss about if arguments are called commands or not.
If I call NanoMaven like given:
java -jar nanomaven-2.2.2.jar help

It will simply startup the repo server like this:
$ ~/tools/jdk-11.0.7+10/Contents/Home/bin/java -jar nanomaven-2.2.2.jar help
19:35:46.887 INFO |
19:35:46.892 INFO | NanoMaven 2.2.2
19:35:46.892 INFO |
19:35:46.892 INFO | --- Preparing workspace
19:35:46.892 INFO | Using an existing configuration file
19:35:46.892 INFO | Using an existing repositories directory
19:35:46.893 INFO | Using an existing tokens data file
19:35:46.893 INFO |
19:35:46.980 INFO | --- Loading data
19:35:47.003 INFO | Tokens: 0
19:35:47.004 INFO |
19:35:47.005 INFO | --- Scanning to find repositories
19:35:47.005 INFO | + releases
19:35:47.005 INFO | + snapshots
19:35:47.005 INFO | 2 repositories have been found
19:35:47.005 INFO |
19:35:47.005 INFO | Binding server at *::9100
19:35:47.033 INFO | Done (0.022s)!
19:35:47.033 INFO |
19:35:47.033 INFO | NanoMaven 2.2.2 Commands:
19:35:47.033 INFO |   help - List available commands
19:35:47.033 INFO |   status - Display metrics
19:35:47.033 INFO |   tokens - List all generated tokens
19:35:47.033 INFO |   keygen <path> <alias> - Generate a new access token for the given path
19:35:47.033 INFO |   purge - Clear cache
19:35:47.033 INFO |   stop - Shutdown server
19:35:47.033 INFO |

and it does not change anything if I give: status instead help or whatever...always the server will be started. That's based on the analysis of the code?

After taking a deeper look into the code I found out that I have to type the commands directly into the running console. This is simply not clear from the output nor from the documentation. I would suggest to improve that.

I see your point, I've made this division to explain the problem :) It is a good idea to support commands like version through the parameters.

it means they can be only invoked in a working instance of application

I wanted to emphasize that invoking commands like stop, status or purge does not make sense if NanoMaven is not working and it is why I didn't support mapping program arguments into commands.

#57

I understand that point to divide it that way. I think it might be good way to make a special rest call to your server and support the usage of the commands via a second instance which calls the rest api... cause claming System.in might be an issue. In particular if you run it inside a container etc.

I'll need a dedicated rest api in the future to hide nanomaven behind reposilite, but it's not a priority for these days I think 🤔

Hey ... the priority is of course up to you. Just a thought of mine.


Adding Activity fragment to support runtime permission check.

Resubmitting PR 105 after resolution of conflicts.
This creates an Activity Fragment that handles the flow of checking the runtime permissions. The native code calls this with a callback which returns the result of the permission check.

For this case, app is audio playback / recording and will access disk, we could ask for the access at the startup of the app[that would be purely on Java side]; it is a fair statement that if user does not grant the access, the app will not enable record/play back UI button.  That way we save the trip to native and back to java UI traffic. If people are interested in how to use run-time permission, we could route them to samples that shows run-time permission; this way we keep native sample's jni side for purely audio purpose.

I think this change is more in line with being a native sample.  It moves more of the overall logic into C++; keeping on the the necessary code in Java (the fragment and permission lifecycle).    Is there a specific piece of code you are concerned about?

tried to load on Nexus6 phone, after I denied the right, the app crashes. maybe have more look at it?

CMake support is released today, branch changed ( master is cmake code, gradle-experimental is previous master ). Feel free to re-submit to the new master branch after re-based, thanks.


Direct Routing

Not clear how to setup a non-hybrid Direct Routing endpoint for an auto attendant resource account. New-CsHybridApplicationEndpoint appears to not be part of the CsOnlineSession imported functions and not clear how to use this to create the resource account without skype for business.
Can you please add a example of setting up a direct routing resource account that doesn't use Skype for Business. We are using an SBC that is certified but has no Skype for Business connection.

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

ID: 81fa6268-92b2-546f-9200-f323bb48e90e
Version Independent ID: 6ce89bc0-ae69-b6f1-df50-532e46b2e67b
Content: Manage resource accounts in Teams
Content Source: Teams/manage-resource-accounts.md
Service: msteams
GitHub Login: @jambirk
Microsoft Alias: jambirk

#2004 this is a linked question

@officedocsbot assign @scanum

Hello @radderz how are you?
I can see that currently that feature is in development, I recommend looking into this resources for more information:
https://www.microsoft.com/en-us/microsoft-365/roadmap?filters=&searchterms=queue
https://techcommunity.microsoft.com/t5/Microsoft-Teams/Auto-Attendant-and-Call-Queues-with-Direct-Routing/td-p/269038
As soon as it is released, documentation will be updated.
Thank you

@radderz We appreciate your feedback, we encourage you to keep making the docs better.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.

@officedocsbot close

@scanum @get-itips  This feature is released now (Direct routing for Auto Attendant), but the docs aren't clear how to setup a direct routing (from SBC directly) resource account. Can an example be done now?

@waseemhashem  Please contact me so we can resolve this together.

@waseemhashem @jambirk Any updates on this? Thanks

@waseemhashem @jambirk Any updates on this? Thanks

This is now the final example in the "Create a resource account in Powershell" section. If more is needed please let us know.


Noisy Circuits Improvements

Improving and documenting pretrajectories and related functionality
If you want to submit an unfinished piece of work in order to get comments and discuss, please mark the pull request as a draft and ping the repository maintainer.
Please address only one topic or issue per pull request! Many small PRs are much easier to review and merge than one large PR.
Before merging, all changes and new functionality should be marked in the CHANGELOG file, but feel free to just leave your CHANGELOG notes in the PR description, to avoid merge conflicts with other requests modifying that file. The maintainer will add these CHANGELOG notes for you if you do so.
Before considering your pull request ready for review and merging make sure that all of the following are completed (please keep the clecklist as part of your PR):

[ ] The code is properly formatted and commented.
[ ] Substantial new functionality is documented within the docs.
[ ] All new functionality is tested.
[ ] All of the automated tests on github pass.
[ ] We recently started enforcing formatting checks. If formatting issues are reported in the new code you have written, please correct them. There will be plenty of old code that is flagged as we are slowly transitioning to enforced formatting. Please do not worry about or address older formatting issues -- keep your PR just focused on your planned contribution.

If possible, keep your git history not too wild (rebase and squash commits, keep commits small and semantically separated) so that review is easier.

Hi Isaac,
It seems the merge conflicts might have arisen because commits from your previous PR have resurfaced.

Stefan's note: fix the merge conflicts here without doing cherrypicks or rebases or anything like that. The merge conflicts seem reasonably small, so this solution might be easiest.

With regards to your mermaid schematic link:
https://mermaid.ink/img/pako:eNq1lltvmzAUgP-K5T60k6DiFrogbS_r9raLljxUCXkwxiRWDI58WRc1_e91MVlIyzRteH4JHH_n41g6jv0AMS8JzGDeVIzf4w0SCsxv8waYIRVSZHl3BxaLFfD994cZrTVDivIGRIe1mQzR1fLDl69zEHrx6s2fs2KbVfxdVmiz8OssW4PlNcZEykMbi451RV5yxhYDbPEbFgP_2sAfheCiQ_EwGp2XIIjUTEXL7-3vqscVfaXF4gEMD-jCc64LGk7q-mo503UXkgA1JZD7uuCMYrATvEAFZVQReSy5q8_m9kNxP2QezOsPxDSRy289jZ0-jbnQBHTFAvAOBNfTIJ4k6UvuE6JMCwIsE8TpJEinryEmT7YWnCQ3aZCsjn2yZ8R2C6goY9lF1Q4PVLxRfoVqyvYZuJwRQatLTyrBtyS7CNrRV3StM95ROHDg8Y7IwVoiB2uJRq6la2wXksiFJB4nMbtonMBuwH91MNpsZ60nAOfzHe7f01JtsnD308OccZEVDOGtN_SBl8bQuTFyboydGxPnxolzY-rceOPc-Na5ceq-w__DpnG_a8Lx2wZ6sCaiRrQ0V8GH5w_kUG1ITXKYmceSVMj8FebmlvhoUKQVn-0bDDNljn4PCq7XG5hVz0e3B_WuNMfALUVrgepf0R1qFpyf3klJFRef7eWzvYM-PgGgdnFL?type=png)](https://mermaid-js.github.io/mermaid-live-editor/edit#pako:eNq1lltvmzAUgP-K5T60k6DiFrogbS_r9raLljxUCXkwxiRWDI58WRc1_e91MVlIyzRteH4JHH_n41g6jv0AMS8JzGDeVIzf4w0SCsxv8waYIRVSZHl3BxaLFfD994cZrTVDivIGRIe1mQzR1fLDl69zEHrx6s2fs2KbVfxdVmiz8OssW4PlNcZEykMbi451RV5yxhYDbPEbFgP_2sAfheCiQ_EwGp2XIIjUTEXL7-3vqscVfaXF4gEMD-jCc64LGk7q-mo503UXkgA1JZD7uuCMYrATvEAFZVQReSy5q8_m9kNxP2QezOsPxDSRy289jZ0-jbnQBHTFAvAOBNfTIJ4k6UvuE6JMCwIsE8TpJEinryEmT7YWnCQ3aZCsjn2yZ8R2C6goY9lF1Q4PVLxRfoVqyvYZuJwRQatLTyrBtyS7CNrRV3StM95ROHDg8Y7IwVoiB2uJRq6la2wXksiFJB4nMbtonMBuwH91MNpsZ60nAOfzHe7f01JtsnD308OccZEVDOGtN_SBl8bQuTFyboydGxPnxolzY-rceOPc-Na5ceq-w__DpnG_a8Lx2wZ6sCaiRrQ0V8GH5w_kUG1ITXKYmceSVMj8FebmlvhoUKQVn-0bDDNljn4PCq7XG5hVz0e3B_WuNMfALUVrgepf0R1qFpyf3klJFRef7eWzvYM-PgGgdnFL

For implementation, embed the Mermaid chart within

 tags. You can also render it directly in the browser using some HTML, allowing you to verify its appearance—particularly if you're using a custom color palette. Here's an example of the syntax:
```@raw html

graph TB

It’s just my observation, but using the default color palette in Mermaid allows it to render nicely in both dark mode and light mode without any additional adjustments. However, if you choose to apply your own custom color scheme, you’ll need to verify how it appears in both light and dark modes to ensure it looks good across both themes.

Regarding the addition of a Mermaid schematic, you can refer to QuantumSavory for inspiration: https://quantumsavory.github.io/QuantumSavory.jl/stable/register_interface/. On a related note, click the ⚙ button, select "documenter-dark" if you are not using dark mode and let me know if all the boxes in the QS Mermaid charts are clearly readable. I am unable to read the text in white for Mermaid charts in the aforementioned link in dark mode. I've switched browsers and operating systems, but this issue still occurs when using dark mode or documenter-dark mode. Apologies for the slight digression.

hi, Isaac! As mentioned in the meeting, this needs to be synchronized with current master. There are a few options to do so, but I suspect it would be easiest just to get synced master branch and to cherry pick the necessary changes. I will close this PR -- when you submit the fixed PR, please reference this one so it is easy to find in the future.
Please submit separate PRs for the initial fixes/documentation you did and for the upcoming examples with decision gates.

replaced by #456


[WIP] Small enhancements

- Summary

add -m shorthand flag for init command
add loading message to make sites:list feel faster

- Test plan

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)
🐎
closes #73

Coverage remained the same at 100.0% when pulling c4c0250b8fdd5b51b3b1655ee752be4e52420c68 on satyarohith:small-enhancements into 6cc6fc1cb8ca4b7fe46a96479106d8994f6833a8 on netlify:master.

Coverage remained the same at 100.0% when pulling c4c0250b8fdd5b51b3b1655ee752be4e52420c68 on satyarohith:small-enhancements into 6cc6fc1cb8ca4b7fe46a96479106d8994f6833a8 on netlify:master.

Updated! :tada:

I've had it on my wishlist to audit every async step to add spinners to indicate network activity.

I will update to this PR.

@bcomnes cli-ux persists the loading message which is not so appealing in few cases,
I would like to know your opinion on using ora  for spinners and also on using opn(great cross-platform support) to open links in the browser because cli-ux fails to all the time in Linux.

@satyarohith Mainly consistency with oclif recommendations, but I wasn't aware of those issues.  Let me look into it some more.

Triaging this onto my todo again

I can review this again tomorrow.

Thank you @satyarohith I'll it.


TinyB documentation is misleading (BlueZ version)

Please update your README.md file with a correct supported version of BlueZ. It mentions BlueZ 5.37, but tinyb 0.5.0 does not work with it. It fails on BluetoothGattCharacteristic.readValue(), something to do with DBus method could not be found. Bluez 5.43 works fine. I spent a fair amount of time to figure out what's wrong.

yes you have reason. I have the same problem and i have to look for, and you have update 5.42


Deprecation warnings with pip loops

Ehi @benyanke, it took long but v2.0.0 is now fixing all these deprecation warnings adding full support to Ansible >= 2.7.0


Parsing array that contains elements whose contain encoded pipe char (%7C instead of '|')

Hello.
I am trying to parse an array of elements separated by the pipe character '|'.
The elements themselves might contain the pipe char, hence  those should be encoded first (to %7C).
With array of 2 elements or above it works just fine.
However with array of one element it doesn't. My guess is that parse doesn't see explicitly pipe char, it decodes %7C into pipe char and creates array of 2 elements.
(A side note: this method with other special characters works just fine.)
please refer the following code example:
// begin of code
function TestQueryString(): void {
  const value1: string = encodeURIComponent('a|b');
  const twoElements = parse(`foo=${value1}|2`, {
    arrayFormat: 'separator',
    arrayFormatSeparator: '|',
  });
  console.log(twoElements); // Works as expected got: [ 'a|b', '2']

  const oneElement = parse(`bar=${value1}`, {
    arrayFormat: 'separator',
    arrayFormatSeparator: '|',
  });
  console.log(oneElement); // Here is the issue. Expected: 'a|b' but got [ 'a', 'b']
}
// end of code

Screenshot with '|'. Not expected behavior in one element case:

Screenshot with '%'. Expected behavior as another special characters:

Thanks,

// @jpoehnelt


Update size calculation in extractGroupMetadata function

This change corrects the retrieval of the total number of participants within the group.

Hi


adding numdiff lines to RANS2P.h and RANS2P2D.h, reverting rdls toler…

…ance back to 0.1*he in TwoPhaseFlow.py
Mandatory Checklist
Please ensure that the following criteria are met:

[ ] Title of pull request describes the changes/features
[ x] Request at least 2 reviewers
[ ] If new files are being added, the files are no larger than 100kB. Post the file sizes.
[ ] Code coverage did not decrease. If this is a bug fix, a test should cover that bug fix. If a new feature is added, a test should be made to cover that feature.
[ ] New features have appropriate documentation strings (readable by sphinx)
[x ] Contributor has read and agreed with CONTRIBUTING.md and has added themselves to CONTRIBUTORS.md

As a general rule of thumb, try to follow PEP8 guidelines.
Description

Let me try duplicating the result on a docker container.

Just kidding, I don't think it's  possible

@cekees It looks like restarting the build is insufficient. Considering the small number of changes, I'm quite alright with simply skipping that test and having someone with osx actually take a closer look to understand the effects of the diff.

OK with me. @jhcollins do you want to just put in a line in .travis.yaml to exclude that test on osx? @ejtovar whenever you get a chance, could you please run that test and see if it fails on your machine?

Going to go ahead and merge this. @ejtovar feel free to add your observations in this chain even if the PR is closed.

Thanks alvin. I was holding off until eric could check it out, but im fine with merging 👍
Sent from my iPhone

On Jun 26, 2020, at 10:16 PM, Alvin Zhang<EMAIL_ADDRESS>wrote:

Going to go ahead and merge this. @ejtovar feel free to add your observations in this chain even if the PR is closed.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


ControlledPawn parameter in custom utility decorator is always invalid

[x] build a non-utility custom decorator and see if the same issue occurs

it does (overriding CalculateUtility function

[x] if it does, fix it in the source

Issue was CalculateUtility was being called with the AIController being passed as the pawn, instead of its controlled pawn

Fixed by #3


ModuleNotFoundError: No module named 'robopal.robots'

When I run "ros2 run gptserver gptserver", I will receive an error message "ModuleNotFoundError: No module named 'robopal. robots'", but I have already installed robopal. How can I resolve this issue

It seems you have not installed robopal correctly. Make sure you install robopal from the source instead of the pip.

maybe i got the same problem ，which states no module named“langchain” and “robopal” in gptserver and gptclient .but i have downloaded both of them🤔

我也是  你解决了吗？me too，did you sovle？

Maybe i got the same problem, which states no module named“langchain” and “robopal” in gptserver and gptclient. But i have downloaded both of them🤔

你解决了吗 我和你一样


If we don't find a tsconfig.json consider the file its own project

Doesn't seem relevant anymore.


WIP: Custom subtitles renderer

Hey, I'm back!
I was just reviewing your PR and...

Are you planning to add a way of adding subtitles dynamically? (Like a loadSubtitle method).
Nowadays a lot of players let users load their own .vtt/.srt files (Openload, for example).
I'm working on a personal project where I want to display 2 different subtitles at the same time (1 at the top and another at the bottom). I would also like to make them stylable but, from what I'm seeing in the code, your implementation doesn't take into account this feature. Any thoughts on this?

Cheers!

Hello,

The only way to specify subtitles is to include them in the config (https://matvp91.github.io/indigo-player/#/Configuration?id=subtitles). It is up to the integrator to craft the config and get the subtitles before player initialization.
The code is written in a way that 1 subtitle can be selected and displayed in the custom renderer, I don't have any plans to show multiple, sorry.


Handle Temporal activity/workflow cancelations in code

Right now, we don't handle cancelations in activity code. We should handle them so that we don't keep working on the activity when it's already been canceled.
We should also have the code just stop importing data / running INSERT INTO ... UPDATE ON CONFLICT if it takes too long (set timeouts).

Migrated to internal project management tool.


missing/null Resolution in Demo Asset in Safari on iPad

Have you read the FAQ and checked for duplicate open issues?
Yes
What version of Shaka Player are you using?
4.3.1
Can you reproduce the issue with our latest release version?
Yes
Can you reproduce the issue with the latest code from main?
Yes
Are you using the demo app or your own custom app?
Yes, Demo
If custom app, can you reproduce the issue using our demo app?
n/a
What browser and OS are you using?
Safari v16.2 on iPad (9th Gen)
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
n/a
What are the manifest and license server URIs?

Big Buck Bunny: the Dark Truths of a Video Dev Cartoon (HLS)
https://shaka-player-demo.appspot.com/demo/#audiolang=en;textlang=en;uilang=en;asset=https://storage.googleapis.com/shaka-demo-assets/bbb-dark-truths-hls/hls.m3u8;panel=ALL_CONTENT;build=uncompiled
What configuration are you using?  What is the output of player.getConfiguration()?
default
What did you do?
Play the demo asset on latest version
What did you expect to happen?
Resolutions were missing in resolution menu. Only one track was listed and that too with nullp value.
What actually happened?

After attaching my debugger, I found that the below statement just returns us 1 track
https://github.com/shaka-project/shaka-player/blob/a61c08433dab654edd224b2dc930c6d257460ec9/ui/resolution_selection.js#L70
Here below is the track info:
{"id":5,"active":true,"type":"variant","bandwidth":0,"language":"en","label":"stream_0","kind":"main","width":null,"height":null,"frameRate":null,"pixelAspectRatio":null,"hdr":null,"mimeType":null,"audioMimeType":null,"videoMimeType":null,"codecs":null,"audioCodec":null,"videoCodec":null,"primary":true,"roles":["main"],"forced":false,"audioRoles":["main"],"videoId":null,"audioId":null,"channelsCount":null,"audioSamplingRate":null,"spatialAudio":false,"tilesLayout":null,"audioBandwidth":null,"videoBandwidth":null,"originalVideoId":null,"originalAudioId":"0","originalTextId":null,"originalImageId":null}
On parsing the above track we see that both height and width are null, Hence causing that one single track to be nullp.

On iOS/ipadOS, the preference is to use Safari's native HLS, but it does not return height/width information, so this information is not returned at this point. Thanks!


Rework Dockerfile

This rewrites the Dockerfile and moves all dependencies into the docker image, where dependencies are compiled modularly on their respective toolchains.
Note that this moves the base image from ubuntu:latest to debian:bullseye.

bors try

bors try

bors try

bors try

bors try

bors r+


fixed typo from "base/model.py" to "base/models.py"

Was doing the tutorial and found a typo.

Nice catch @DamBor05! If you have the time, could you please make a similar change in the docs/tutorial/customize_homepage.md file. For consistency,
Now, modify your home/models file to include the following:
should be
Now, modify your home/models.py file to include the following:

Thanks @DamBor05! Merged in 952416cfbc399bfa5c9eb797890bd880fb5cd5ed. I've added you to the contributors list as "Damian" - let me know if you'd like this updating to a full name.
For future reference, when working on the Wagtail codebase I'd recommend starting a new branch before making any changes - it looks like your original fix got overwritten when you made the follow-up commit with @activus-d's suggestion.

@gasman "Damian Borneman" is fine. Thank you for catching the push mistake!


Handling backend changes and removal of organization from api-config json file in one command

APIM-CLI version
1.14.1
API-Management version
v7.7.20230830
Bug description
In a scenario, where customer wants to delete the organization from the 'access tab' in APIM, through apim cli.:
( command: apim api import -s APIM -c v1\api-config.json -clientOrgsMode replace -force)
It was observed that when changes were made in backend application code and api-config json file (here, deleting 'clientOrganizations') together, both the changes are not getting reflected  at once by running the above command. What it does is:

First it creates a new API
after running the same command again, it updates API with organization deletion.

Expected behaviour: the apimcli command should import the API after changes in backend application as well as delete the organization from APIM as mentioned in api-config json file together at once.
Note: the command "apim api import -s APIM -c v1\api-config.json -clientOrgsMode replace"  works in 'add' mode(when trying to add an organization in the api-config.json file along with the backend code changes) but not
in 'replace' mode (when trying to remove an organization from the api-config.json file along with the backend code changes). If organization deleted or not can be checked in API manager 'access tab'.

Steps to reproduce
Step1: consider a frontend API with 2 organization mentioned in "clientOrganizations" in api-config file.
Step2: remove one organization from "clientOrganizations"
Step3: make some changes(change "summary" or "name") in backend code file
Step4: Run "apim api import -s APIM -c v1\api-config.json -clientOrgsMode replace" (in this format : apim api import -s ${ENVIRONMENT} -c ${WORKSPACE}/api-config.json -clientOrgsMode replace -force)
Step5: Check the API manager, on the 'Frontend' tab, open the API and check the organization deleted or not in 'access' tab by running the command only once.
api-config.json
APIMCLI_yamini.json
Relevant log output
No response

Fixed as part of 1.14.3 release


tink c#

will it support C# later?

We're discussing this on https://github.com/google/tink/issues/109.


Document a few types

This is a neighbouring PR to a larger one I made to brod.

Thank you @xxdavid


when it come to support compatibilty with react 18

react-dates version
e.g<EMAIL_ADDRESS>Describe the bug
A clear and concise description of what the bug is.
Source code (including props configuration)
Steps to reproduce the behavior:
<DateRangePicker
  startDate={this.state.startDate}
  startDateId="your_unique_start_date_id"
  endDate={this.state.endDate}
  endDateId="your_unique_end_date_id"
  onDatesChange={({ startDate, endDate }) => this.setState({ startDate, endDate })}
  focusedInput={this.state.focusedInput}
  onFocusChange={focusedInput => this.setState({ focusedInput })}
/>

If you have custom methods that you are passing into a react-dates component, e.g. onDatesChange, onFocusChange, renderMonth, isDayBlocked, etc., please include the source for those as well.
Screenshots/Gifs
If applicable, add screenshots or gifs to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Is the issue reproducible in Storybook?
Please link to the relevant storybook example
Additional context
Add any other context about the problem here.

Via peer deps, we explicitly do not yet support react > 16, so you shouldn’t be trying to use them together.
Duplicate of #2199.


TypeError: add_marker() missing 1 required positional argument: 'id'

Thanks a lot for this great project!
I've been trying to create a Mosaic like the one put up on the Readme but I'm facing a frustrating error
Aruns-MacBook-Air:scripts arun$ python3 explorer.py avon-park/images
Notice: cache dir doesn't exist: avon-park/images/ImageAnalysis/cache
Notice: state dir doesn't exist: avon-park/images/ImageAnalysis/state
state: path missing: avon-park/images/ImageAnalysis/state
no histogram templates found...
Known pipe types:
  CocoaGraphicsPipe
(all display modules loaded.)
Loading saved annotations: avon-park/images/ImageAnalysis/annotations.json
Traceback (most recent call last):
  File "explorer.py", line 691, in <module>
    app = MyApp()
  File "explorer.py", line 114, in __init__
    proj, ned_ref, tk_root)
  File "scripts/explore/annotations.py", line 28, in __init__
    self.load()
  File "scripts/explore/annotations.py", line 109, in load
    self.add_marker( ned, "" )
TypeError: add_marker() missing 1 required positional argument: 'id'

I have used the avon-park dataset and I'm running the code on macOS Big Sur with Python 3.7.0
Please Advise.

It turns out the avon-park data set had not been updated to match some of the more recent code changes.  I just did that, and changed to the new file link in the README-DataSets file.  Here is the new avon-park data set download.  Hopefully this addresses the issue you are ssing.  As a very short term fix, you could try simply removing the annotations.json file.
http://www.all-aero.com/index.php/component/content/article/126-planes-c/20214-cheranovski-bich-7
I am going to close this issue, but if you continue to have concerns or notice something I missed, please let me know.

It turns out the avon-park data set had not been updated to match some of the more recent code changes.  I just did that, and changed to the new file link in the README-DataSets file.  Here is the new avon-park data set download.  Hopefully this addresses the issue you are ssing.  As a very short term fix, you could try simply removing the annotations.json file.
http://www.all-aero.com/index.php/component/content/article/126-planes-c/20214-cheranovski-bich-7
I am going to close this issue, but if you continue to have concerns or notice something I missed, please let me know.


Revert "Update version"

Reverts wso2/ballerina-integrator#426

 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.


Docker - pip support is not available at the moment

https://github.com/jorgebastida/gordon/blob/master/examples/docker/helloworld/settings.yml
Just wondering if you have a issue opened already regarding still needing pip support?
I'm interested in using this tool and having ability to build the Python lambdas in environment that is near identical to the lambda env that it runs (especially if a pip install requires some dependent machine specific install which may need machine dependent library support outside of pip - for example, the pymssql package (so lambda can read/write to SQL Server based RDS database) is good use case that depends on FreeTDS which is pre-requisite machine specific installation needed before doing pip install pymssql - http://pymssql.org/en/latest/intro.html.

You might want to retry this. The current lambci/lambda:build-python2.7 Docker image includes pip and virtualenv.

FYI, I tried the instructions for using pip in the docker image (here), but it looks like these build instructions do not actually override the defaults.
From my jenkins log:
Building project...
  0001_p.json
      cp -Rf * /var/lib/jenkins/.gordon/tmp4Kl5h1
      pip install --install-option="--prefix=" -r requirements.txt -q -t /var/lib/jenkins/.gordon/tmp4Kl5h1
      /usr/local/lib/python2.7/dist-packages/pip/commands/install.py:194: UserWarning: Disabling all use of wheels due to the use of --build-options / --global-options / --install-options.
        cmdoptions.check_install_build_global(options)

      cd /var/lib/jenkins/.gordon/tmp4Kl5h1 && find . -name "*.pyc" -delete
      cp -Rf * /var/lib/jenkins/.gordon/tmpnrqvNQ
      echo "[install]\nprefix=" > /var/lib/jenkins/.gordon/tmpnrqvNQ/setup.cfg
      docker run --rm -v $HOME/.pip/pip.conf:/root/.pip/pip.conf -v $HOME/.ssh:/root/.ssh -v /var/lib/jenkins/.gordon/tmpnrqvNQ:/var/task lambci/lambda:build-python2.7 pip install -r requirements.txt -q -t /var/lib/jenkins/.gordon/tmpnrqvNQ

  0002_pr_r.json
  0003_r.json

@astewart-twist Could you try renaming requirements.txt to requirements-docker.txt so that gordon does not automatically pick it up.


Add support for topic/partition in KeyRecordGrouper

This adds support for including topic/partition in the file template when using KeyRecordGrouper.
The wider context is the desire to have a connector ingesting multiple compacted topics, and wanting to write records within the same topic and with the same key, to the same location
Fix #178

Thanks @jeqo, I've made the requested changes.
One thing I did notice (and I think this can be resolved by a separate PR), but validateKeyFilenameTemplate, which was added here, isn't executed (nor is it invoked by the mentioned S3 & GCS connectors).
I'm not sure if we'd want to invoke that method in AivenCommonConfig::validate or whether it should be invoked in S3SinkConfig::validate. Happy to create separate PR to resolve that one way or the other.

@stephen-harris thanks! Good catch! Agree, it looks like either dead-code or we are missing to call it from the storage connector configs. Could you create an issue first to discuss this further?
Could you also rebase this PR to get the latest codeql workflow executed?

@jeqo Done :)

@stephen-harris thanks! But seems git history got a bit messed up. Could you take a look?
From my side, seems that if you move one step back from the last merge commit, it should be fine:

So, reset to the previous commit and force push. I tested it here and looks better: https://github.com/Aiven-Open/commons-for-apache-kafka-connect/compare/master...jeqo:commons-for-apache-kafka-connect:feat/support-topic-partition-in-key-record-group

Yup, I accidentally merged in the remote branch rather than force-pushing the rebased local branch. Fixed now.

Thanks @jeqo, are you able to publish the update and I'll get a PR together for the S3-connector.

@stephen-harris sure! will do a release later this week. Will ping you here once available

@stephen-harris see https://github.com/Aiven-Open/commons-for-apache-kafka-connect/releases/tag/v0.11.0 -- looking forward to more contributions! Many thanks!


How to remove Command key as meta key

How to remove Command key as a meta key. I am happy with the only option key to be my meta key and want to remove command key as meta.

By changing ns-command-modifier, which I've fixed in https://github.com/hlissner/doom-emacs/commit/85b5103cd9c1405b80d0fb5f690588f013a86f98. Let me know if that isn't the case and I'll reopen the issue.