Expose mod input handlers for better integration between mods

Allow mods to:

Get name list of other installed mods
Get info about input handlers, registered for other mods
Call input handlers from lua, as if user presses an input key
Split responsibilities: mods will delegate input management to other specific mods

Current CET implementation of Input & Hotkey functionality provides a good default for mods. But it ignores the game context in which the key was pressed. That means, if the key is bound in CET then it must be unbound in game. That is ok, but with enough mods users will run out of keys they can comfortably press.
My proposal is to allow input mods to monitor in game context and invoke other mods key bindings depending on context. So a single key can do different action, when the player is in a vehicle or while sprinting, for example.
Of course each keybind should be EITHER bound in CET OR managed by separate input mod.

I need to take more closer look at this, these are just some quick remarks.

I added an extra comment and edited some for better clarity (you won't see changes from email, so check them in the PR please)

Also, I have to emphasize, that now this PR introduce a braking change in the GetMod function. Just reiterating, to be clear.

I’m sorry to see my PR gets reverted. Can I help to improve it?
Possible solutions:

Introduce new function and do not introduce a breaking change in the existing function (as it was implemented before the review)
Merge to different branch with breaking changes and release it later after deprecating the current branch (I don’t really like it, as it is too slow and forcing for other mod authors)
Introduce an API version and bind mod to different APIs depending on what version the mod requests. (I like it the most, but it is more work)

Apparently, more mods used GetMod than I expected... So new functions which do that would be better probably, GetModContext+GetModContexts probably.
GetMod could be wired to GetModContext and return rootObject to not break former bindings.

This was only reason this was reverted, no other really... If it was some separate function in the end, it would be fine.


Modify TeamBuilder to create deep copy and add test cases for modelManager

Codecov Report
Base: 60.11% // Head: 61.02% // Increases project coverage by +0.90% :tada:

Coverage data is based on head (af866c1) compared to base (e370946).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #266      +/-   ##
============================================
+ Coverage     60.11%   61.02%   +0.90%
- Complexity      595      606      +11
============================================
  Files           109      109
  Lines          2204     2204
  Branches        267      267
============================================
+ Hits           1325     1345      +20
+ Misses          811      791      -20
  Partials         68       68

Impacted Files
Coverage Δ

src/main/java/seedu/address/model/team/Team.java
86.88% <0.00%> (+1.63%)
:arrow_up:

.../java/seedu/address/model/task/UniqueTaskList.java
60.34% <0.00%> (+1.72%)
:arrow_up:

...a/seedu/address/model/person/UniquePersonList.java
94.44% <0.00%> (+1.85%)
:arrow_up:

...rc/main/java/seedu/address/model/ModelManager.java
57.05% <0.00%> (+2.56%)
:arrow_up:

.../java/seedu/address/model/team/UniqueTeamList.java
78.57% <0.00%> (+3.57%)
:arrow_up:

src/main/java/seedu/address/model/AddressBook.java
75.00% <0.00%> (+5.00%)
:arrow_up:

...a/seedu/address/logic/commands/TaskAddCommand.java
88.23% <0.00%> (+47.05%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Added duplicating property to IBaseIncludeOptions

This property allow a choice to avoid a cartesian product with assocations when using include. The actual Sequelize code is located at the line 487 of model.js. Also, this property in mentioned in multiple PR and Issues on Sequelize repo.

Sry for the delayed reply. Looks good! Thanks for fixing


[System tests] Provide more info when the restartKafkaBroker test case fails

Is your feature request related to a problem? Please describe.
We would need more info when a Kafka Broker is not restarted for some reason during the test run.
Describe the solution you'd like
If that error happens, we should collect the logs of the pod to know the reason of the failure and take some initiative depending on the failure (finish the test, wait more time, restart again...)

In other life, we just used to get the kubernetes state whenever a test failed.  afterAll


feat(badge): add badge component

fixes https://github.com/patternfly/patternfly-next/issues/22
verified in chrome, ff, opera, edge, safari

Deploy preview for pf-next ready!
Built with commit f12bb620d73c74919cea748b18f37bf08eea7577
https://deploy-preview-272--pf-next.netlify.com

Hey @michael-coker this is great! I'd merge it as soon as we address the comments.
Alt styles will probably go out, but let me do that when I figure out the theming

@andresgalante @srambach @matthewcarleton thanks for the feedback! updated the branch with these changes and will be viewable when the build finishes.

I'm fine with read and unread if others are, and everything else looks great! 👍

@michael-coker there is something wrong on the handlebars, since it's not rendering the correct tags
https://deploy-preview-272--pf-next.netlify.com/components/badge/examples/

@andresgalante d'oh! git didn't pick up a case change on one of the files. updated the branch.


Faster statusline

Hello,
With the last updates I've noticed that the get_statusline() function was slowing down my Neovim instance a lot and I think the problem is related with the way the clock gets the currently clocked in headline, which is basically parsing all files and retrieving the first clocked in headline.
Now, on my PC the slow down was barely noticeable, but on my Android phone it became a real issue (despite it being a good smartphone) so I did my best trying to patch it.
The basic idea is that now, with the modifications here, we pay a little upfront cost (it is noticeable on my phone but it is only at startup, I could not test it on my PC yet, will do later this week) but afterwards we store the information of the clocked headline in the instance of the clock itself, making it faster to access the information to display the statusline.
I've been testing this code for a few hours and it is quite stable and reliable, the only problem I see is if someone deletes the LOGBOOK entry manually, the clocked headline won't be reset.. probably we could run an async job every now and then to check if the clock is still valid?
Also, with this modification it is easier for hackers to retrieve the current clocked headline and customise the statusline message, we don't even need a debouncer anymore since we're not modifying or touching the files.
Please let me know if you have better ideas!

It has some drawbacks that you mentioned, but some of them existed even before the refactor

I will most probably do another PR next week to update the clock in the background using an async worker, so to solve the issue I mentioned in my first comment!

Thanks for the PR!

I will most probably do another PR next week to update the clock in the background using an async worker, so to solve the issue I mentioned in my first comment!

Instead of async worker, let's go with listening on BufWritePost on org files and check and re-set the clocked headline. That still leaves the issue if someone edits the file outside Neovim, but it should solve all other cases.


FastReport Designer says "Could not find file or assembly System.Threading.Tasks.Extensions version=<IP_ADDRESS>"

Hello, I am getting this error when I try to add a Postgres datasource in FastReport Designer. It looks like a missing .NET dll. This happens when I click "Test Connection".
I'm using v2021.2.0 of FastReport Designer, which I downloaded here, with Postgres 12.4.

Hello,
Thanks for your message! We will fix it ASAP

Hello, @ProsserCJ . We fixed this problem and FastReport.Community.zip are updated in 2021.2.0 release.

Thank you! It seems to work. I appreciate your quick response.

Hi, same problem in the new 2024.1.3 version

Hello! Same error in 2024.2.0 version.


Deal with fast foreign source

Flow-Efficient Scheduler is a good idea but source-to-sink throughput must be larger than sources' throughput.
Should consider how to deal with super fast foreign source.
E.g.
FS1 ---- (SourceServer A) ----> [1]
                                ^
FS2 ---- (SourceServer B) ------

Both FS1 and FS2 drop data if SourceServers do not fetch them.
Scheduling: SSA, SSB
SSA got row so scheduler waits for B's row
FS1 emits 100rows/sec, while FS2 1rows/sec -> 99rows/sec from FS1 dropped

close by https://github.com/SpringQL/SpringQL/issues/24


Optimization for ctr mode

With this patch, ctr mode runs faster (at least with sbcl x86-64).

Merged, thanks!


Feature/144 model config UI

Expected Differences
WRF configuration editing by GUI is enabled.
Pull Request Testing
Namelist data is represented in three places:

The map on the "Basic" tab
The text fields on the "Basic" tab
In the namelist on the "Advanced" tab

As you change variables listed below in one place, the other two should stay in sync.
Valid namelist files should be generated.
Variables:

wrf.physics.mp_physics
wrf.physics.cu_physics
wrf.physics.ra_lw_physics
wrf.physics.ra_sw_physics
wrf.physics.bl_pbl_physics
wrf.physics.sf_sfclay_physics
wrf.physics.sf_surface_physics
wps.geogrid.e_we
wps.geogrid.e_sn
wps.geogrid.dx
wps.geogrid.dy
wps.geogrid.map_proj
wps.geogrid.ref_lat
wps.geogrid.ref_lon
wps.geogrid.truelat1
wps.geogrid.truelat2
wps.geogrid.stand_lon
wrf.domains.time_step
wrf.domains.e_we
wrf.domains.e_sn
wrf.domains.dx
wrf.domains.dy

Focus on web/src/app/model-config/default-namelists.ts as this file has template/default namelists.
Pull Request Checklist

[ ] Review the source issue metadata (labels, project, and milestone).
[ ] Complete the PR definition above.
[ ] Ensure the PR title matches the feature or bugfix branch name.
[ ] Define the PR metadata, as permissions allow.
Select: Reviewer(s)
Select: Project
Select: Milestone as the version that will include these changes
Select: Development to link to the original development issue.
[ ] After the PR is approved, merge your changes. If permissions do not allow this, request that the reviewer do the merge.
[ ] Close the linked issue and delete your feature or bugfix branch from GitHub.

Closes #144


Page title formatting wrong for Ikhtisar Keamanan Cloud Native

This is a Bug Report

Problem: The page Ikhtisar Keamanan Cloud Native has underscores in the title. (HTML does not let you write formatting in page titles).
Proposed Solution: Fix the page title to remove the underscores.
Page to Update:
https://kubernetes.io/id/docs/concepts/security/overview/

Additional Information:
/kind bug
/language id

/priority backlog
Relevant to #20111

/assign

this issue has been fixed
/close


Link to the wrong ServiceBusTriggerAttribute

Issue:
The Attributes section describes both in-process and isolated model in a single sentence, but the ServiceBusTriggerAttribute link there leads to the Microsoft.Azure.WebJobs.Extensions.ServiceBus libraries source code that is meant for use with in-process model only.
According to Examples section in the same article, isolated model should use Microsoft.Azure.Functions.Worker.Extensions.*  and not Microsoft.Azure.WebJobs.Extensions.*
Suggested fix:
Rewrite the sentence to have separate links to the correct ServiceBusTriggerAttribute for each model, or move the sentence inside the tabs below it for each model with the correct link.
Experience notes:
This issue caused some confusion and wasted time when wrongly using ServiceBusTriggerAttribute from Microsoft.Azure.WebJobs.Extensions.ServiceBus library with isolated model and then wondering why the function gets skipped and not hitting any breakpoints, while the messages get silently consumed.

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

ID: 1d50399c-0873-e903-35ca-1845c84b6c6c
Version Independent ID: 1a3150b4-5eb9-0ac5-ee2b-d1bb6b6fd63e
Content: Azure Service Bus trigger for Azure Functions
Content Source: articles/azure-functions/functions-bindings-service-bus-trigger.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

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


Invenio v2.0.3

bbb1ad3983b7fd44e4e6dd58b0e121bdd087e26c Invenio v2.0.3
04a3a815b8b4f228c2476de58e0f75f3724ffb9d matcher: fix argument types in jaro_fctn
7a122dab6847b1823d3248d2856cc06dd5e2496a legacy: lazy stopwords knowledge base

:shipit: (legacy is not recognised as component name, but not sure what to use instead).

7a122da just amends the commit message of c6478755110d96229bb4726dd423291efb822484 so that the change would appear in the generated release notes.  I have kept the original commit log headline to make the link obvious.  (hence "legacy" as the leading component name in the commit log headline)


[Feature Request] Standard wallpaper options for default background

I'd like to match the M365 login background with the local default background so there can be a design thread maintained between the loginwindows. The M365 background behaves as "Fill Screen", but Xcreds Behaves as "Stretch to Fill Screen". This works without much of an issue with 16x9 and 16x10 screens but does not work well at all on other dimension screens like ultra-wide and portrait.
Could we please get an option for "Fill Screen" along with the default "Fit to Screen"?
You may want to consider mirroring the same default choices that the OS offers: "Fill Screen" (OS Default), "Fit to Screen", "Stretch to Fill Screen" (Xcreds default), and "Center".  I'm sure someone could make a case for "Fit to Screen" and/or "Center".

added new key shouldLoginWindowBackgroundImageFillScreen

Sveet. TY. I'll test soon.


refactor: :bug: client postlogout uris

:white_check_mark: Build TheIdServer 1.0.0-user-ole0058 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/88df705d13 by @aguacongas)


Fixed issue with double domain being displayed

When umbraco uses absolute urls, the domain was diplayed twice in the property editor

Great stuff! Thanks man!


Indicate how a dataset is used (as restraint, for validation, etc.)

Information can be used in integrative/hybrid modeling in several ways:

Restraint; for example, a crosslink can be represented as a harmonic score term that is minimized during computational sampling.
Filter; for example, a pool of generated models can be narrowed by keeping only those that are consistent with one or more experiments.
Validation; the data can be withheld from the modeling and used only to validate the final set of structures.
Representation; the nature of the data can determine how the system is represented, for example an available subunit or interface crystal structure can be used as a starting model, and perhaps as a rigid body.
Sampling; for example a Monte Carlo move set can be chosen to only explore conformations that are consistent with the data.

Currently if we list a dataset in the mmCIF file, the unwritten assumption is that for most experimental data, the information is used as a restraint, while for starting models, the information is used to choose a representation. It may be useful to explicitly say which datasets were used for validation, for example, as that information may be useful to validate the mmCIF model itself.

One proposal: add to the ihm_modeling_post_process category optional struct_assembly_id and dataset_group_id so we can see what is being filtered or rescored and how, respectively. Also add validation to the ihm_modeling_post_process.type enumeration, or maybe add a separate validation table. Datasets for filtering or validation are then clear based on where they are used in this table. Thoughts?

Can you give a specific example of what you are meaning by this?
In HADDOCK terms for example, would this describe our clustering process? I.e. that we only at the end consider models that do cluster (and typically discard isolated ones).

Can you give a specific example of what you are meaning by this?

I thought I'd done that when I opened the issue, but I'll rephrase: given N input experiments, one could convert each of them into restraints and optimize a scoring function that's the sum of all N. This is perfectly captured by the current dictionary. Alternatively, one could convert N-x inputs to restraints and optimize, and rescore the final models against the left-out x inputs, the idea being that if the model satisfies data that wasn't used in its construction, it is more likely to be correct. Put another way, the dictionary doesn't currently state which data we used for the training set and which for the test set. It may be useful for PDB to know this in order to facilitate their own validation of the models.

In HADDOCK terms for example, would this describe our clustering process?

AFAIK the existing ihm_modeling_post_process category would handle that just fine.

@benmwebb This has been addressed in the latest dictionary update.


Compatibility issue between section-index plugin and mkdocs-material blog plugin

Hello!
First, thank you for creating and maintaining the fantastic section-index plugin. It has been a great addition to my documentation workflow!
I recently encountered an issue when trying to use the section-index plugin in combination with the mkdocs-material plugin’s blog feature. I’m not sure if this issue should be addressed here or in the mkdocs-material repository, so I’m reporting it here for now.
Steps to Reproduce:

Create a new MkDocs project.
Use the following mkdocs.yml configuration:

site_name: My Docs
theme: material
plugins:
  - blog
  - section-index

Run the MkDocs development server using:

mkdocs serve

Observed Behavior:
The build fails with the following error message:
(venv) ~/Code/mkdocs-test (main ✘)✖✹✭ ᐅ mkdocs serve
INFO    -  Building documentation...
INFO    -  Cleaning site directory
ERROR   -  Error reading page 'blog/index.md':
Traceback (most recent call last):
  File ".../venv/bin/mkdocs", line 8, in <module>
    sys.exit(cli())
             ^^^^^
  File ".../site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File ".../site-packages/click/core.py", line 1078, in main
    rv = self.invoke(ctx)
         ^^^^^^^^^^^^^^^^
  File ".../site-packages/click/core.py", line 1688, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File ".../site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File ".../site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File ".../site-packages/mkdocs/__main__.py", line 272, in serve_command
    serve.serve(**kwargs)
  File ".../site-packages/mkdocs/commands/serve.py", line 85, in serve
    builder(config)
  File ".../site-packages/mkdocs/commands/serve.py", line 67, in builder
    build(config, serve_url=None if is_clean else serve_url, dirty=is_dirty)
  File ".../site-packages/mkdocs/commands/build.py", line 310, in build
    _populate_page(file.page, config, files, dirty)
  File ".../site-packages/mkdocs/commands/build.py", line 163, in _populate_page
    page.markdown = config.plugins.on_page_markdown(
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File ".../site-packages/mkdocs/plugins.py", line 635, in on_page_markdown
    return self.run_event('page_markdown', markdown, page=page, config=config, files=files)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File ".../site-packages/mkdocs/plugins.py", line 566, in run_event
    result = method(item, **kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^
  File ".../site-packages/material/plugins/blog/plugin.py", line 241, in on_page_markdown
    assert isinstance(view, View)
           ^^^^^^^^^^^^^^^^^^^^^^
AssertionError

Expected Behavior:
The build should complete successfully, and the site should render without errors when using both plugins together.
Environment Details:

MkDocs Version: 1.6.1
Material for MkDocs Version: 9.5.47
section-index Plugin Version: 0.3.9
Python Version: 3.12.5
Operating System: macOS 15.1.1

Additional Notes:

This issue occurs specifically when the blog plugin is active in the configuration.

Please let me know if I can provide further details or assist with debugging!

Closing this as a duplicate of #22


should these 2 links take you to the same page?

both "view  255 genes...."  links go to the same page. Is this intentional?

They don't go to the same page.  The first is a table of genes.  The second link is a table of single allele genotypes and has an Allele column.

Of course. I was not seeing  the "allele". I don't use this link much.
I wonder if it would be better to present the allele column first in this view? (it should not affect anything because  the download options are not available from here).

I wonder if it would be better to present the allele column first in this view?

Sounds sensible.  Shall I go ahead and do it?

Just wait in case @mah11  thinks there is a good reason not to.
I think it would be useful for this page to be a bit more obvious that it's  genotypes (and a bit different from other pages), so I'm for it

no objection; no strong preference

I wonder if it would be better to present the allele column first in this view?

Should we continue order by the product as we do for gene tables?

OK...

All done.


Improve behavior when calling wt.exe to open to a folder

Description of the new feature/enhancement
Currently when you do something like wt.exe -p "Windows PowerShell" -d folder\path and already have a terminal open, it hides and then re-shows instead of just opening a new tab. See video below.
https://github.com/microsoft/terminal/assets/75278/9499ed68-cb3d-4c86-a0ce-719acdfea02f

Proposed technical implementation details (optional)
Open the tab directly instead of doing the hide/show/flash

.... what
I'm definitely not seeing this. What version are you on? Could you share your settings.json file/?
We shouldn't be doing any sort of show/hide dance there 🙃

.... what
I'm definitely not seeing this. What version are you on? Could you share your settings.json file/?
We shouldn't be doing any sort of show/hide dance there 🙃

Here you go! https://gist.github.com/Stanzilla/f4e17a26df66879914e35dbc0fbc3f1b

Can you run that command in the address bar of File Explorer? Does it exhibit the same behavior?
Also, does this happen for the Command Prompt profile?

Can you run that command in the address bar of File Explorer? Does it exhibit the same behavior?
Also, does this happen for the Command Prompt profile?

It does not happen from the address bar, it does happen with the cmd profile (not from progress bar)

Windows Registry Editor Version 5.00

[HKEY_CLASSES_ROOT\Directory\shell\OpenWindowsTerminalProfiles]
"MUIVerb"="Open in Windows Terminal"
"SubCommands"=""

[HKEY_CLASSES_ROOT\Directory\Shell\OpenWindowsTerminalProfiles\shell\01DefaultProfile]
"MUIVerb"="Default Profile"

[HKEY_CLASSES_ROOT\Directory\Shell\OpenWindowsTerminalProfiles\shell\01DefaultProfile\command]
@="cmd.exe /c start wt.exe -d \"%1\""

[HKEY_CLASSES_ROOT\Directory\Shell\OpenWindowsTerminalProfiles\shell\02CommandPromptProfile]
"MUIVerb"="Command Prompt"
"Icon"="imageres.dll,-5323"

[HKEY_CLASSES_ROOT\Directory\Shell\OpenWindowsTerminalProfiles\shell\02CommandPromptProfile\command]
@="cmd.exe /c start wt.exe -p \"Command Prompt\" -d \"%1\""

[HKEY_CLASSES_ROOT\Directory\Shell\OpenWindowsTerminalProfiles\shell\03PowerShellProfile]
"MUIVerb"="PowerShell"
"Icon"="powershell.exe"

[HKEY_CLASSES_ROOT\Directory\Shell\OpenWindowsTerminalProfiles\shell\03PowerShellProfile\command]
@="cmd.exe /c start wt.exe -p \"Windows PowerShell\" -d \"%1\""

[HKEY_CLASSES_ROOT\Directory\Background\shell\OpenWindowsTerminalProfiles]
"MUIVerb"="Open in Windows Terminal"
"SubCommands"=""

[HKEY_CLASSES_ROOT\Directory\Background\Shell\OpenWindowsTerminalProfiles\shell\01DefaultProfile]
"MUIVerb"="Default Profile"

[HKEY_CLASSES_ROOT\Directory\Background\Shell\OpenWindowsTerminalProfiles\shell\01DefaultProfile\command]
@="cmd.exe /c start wt.exe -d \"%V\""

[HKEY_CLASSES_ROOT\Directory\Background\Shell\OpenWindowsTerminalProfiles\shell\02CommandPromptProfile]
"MUIVerb"="Command Prompt"
"Icon"="imageres.dll,-5323"

[HKEY_CLASSES_ROOT\Directory\Background\Shell\OpenWindowsTerminalProfiles\shell\02CommandPromptProfile\command]
@="cmd.exe /c start wt.exe -p \"Command Prompt\" -d \"%V\""

[HKEY_CLASSES_ROOT\Directory\Background\Shell\OpenWindowsTerminalProfiles\shell\03PowerShellProfile]
"MUIVerb"="PowerShell"
"Icon"="powershell.exe"

[HKEY_CLASSES_ROOT\Directory\Background\Shell\OpenWindowsTerminalProfiles\shell\03PowerShellProfile\command]
@="cmd.exe /c start wt.exe -p \"Windows PowerShell\" -d \"%V\""

well, that sorta explains it. It's definitely not just calling wt -d . -p "the profile". It's shelling out to another cmd.exe (which, in this case appears to glom onto your first window). That cmd.exe briefly appears as a tab, then calls wt to open a new tab. Mental.
Wacky that it seems to minimize and restore as the original cmd.exe tab exits, then the new tab opens. I wonder if File Explorer is playing weird games as you launch/close things from context menus?

Windows Registry Editor Version 5.00

[HKEY_CLASSES_ROOT\Directory\shell\OpenWindowsTerminalProfiles]
"MUIVerb"="Open in Windows Terminal"
"SubCommands"=""

[HKEY_CLASSES_ROOT\Directory\Shell\OpenWindowsTerminalProfiles\shell\01DefaultProfile]
"MUIVerb"="Default Profile"

[HKEY_CLASSES_ROOT\Directory\Shell\OpenWindowsTerminalProfiles\shell\01DefaultProfile\command]
@="cmd.exe /c start wt.exe -d \"%1\""

[HKEY_CLASSES_ROOT\Directory\Shell\OpenWindowsTerminalProfiles\shell\02CommandPromptProfile]
"MUIVerb"="Command Prompt"
"Icon"="imageres.dll,-5323"

[HKEY_CLASSES_ROOT\Directory\Shell\OpenWindowsTerminalProfiles\shell\02CommandPromptProfile\command]
@="cmd.exe /c start wt.exe -p \"Command Prompt\" -d \"%1\""

[HKEY_CLASSES_ROOT\Directory\Shell\OpenWindowsTerminalProfiles\shell\03PowerShellProfile]
"MUIVerb"="PowerShell"
"Icon"="powershell.exe"

[HKEY_CLASSES_ROOT\Directory\Shell\OpenWindowsTerminalProfiles\shell\03PowerShellProfile\command]
@="cmd.exe /c start wt.exe -p \"Windows PowerShell\" -d \"%1\""

[HKEY_CLASSES_ROOT\Directory\Background\shell\OpenWindowsTerminalProfiles]
"MUIVerb"="Open in Windows Terminal"
"SubCommands"=""

[HKEY_CLASSES_ROOT\Directory\Background\Shell\OpenWindowsTerminalProfiles\shell\01DefaultProfile]
"MUIVerb"="Default Profile"

[HKEY_CLASSES_ROOT\Directory\Background\Shell\OpenWindowsTerminalProfiles\shell\01DefaultProfile\command]
@="cmd.exe /c start wt.exe -d \"%V\""

[HKEY_CLASSES_ROOT\Directory\Background\Shell\OpenWindowsTerminalProfiles\shell\02CommandPromptProfile]
"MUIVerb"="Command Prompt"
"Icon"="imageres.dll,-5323"

[HKEY_CLASSES_ROOT\Directory\Background\Shell\OpenWindowsTerminalProfiles\shell\02CommandPromptProfile\command]
@="cmd.exe /c start wt.exe -p \"Command Prompt\" -d \"%V\""

[HKEY_CLASSES_ROOT\Directory\Background\Shell\OpenWindowsTerminalProfiles\shell\03PowerShellProfile]
"MUIVerb"="PowerShell"
"Icon"="powershell.exe"

[HKEY_CLASSES_ROOT\Directory\Background\Shell\OpenWindowsTerminalProfiles\shell\03PowerShellProfile\command]
@="cmd.exe /c start wt.exe -p \"Windows PowerShell\" -d \"%V\""

well, that sorta explains it. It's definitely not just calling wt -d . -p "the profile". It's shelling out to another cmd.exe (which, in this case appears to glom onto your first window). That cmd.exe briefly appears as a tab, then calls wt to open a new tab. Mental.
Wacky that it seems to minimize and restore as the original cmd.exe tab exits, then the new tab opens. I wonder if File Explorer is playing weird games as you launch/close things from context menus?

Oh damn, guess that should be mentioned in their docs. From the looks of it, it should just work when removing the cmd call before?

I'd guess as much?

I heard back from Winaero:

The reason why we have there cmd.exe is that WT.EXE is not a "true" binary file. It is an alias. It can point to Windows Terminal or to its Preview version. You cannot call an alias from the Registry. This will fail. The context menu doesn't recognize aliases. This is why I used the start command.

Thanks!
Okay, so, there's a some things to unpack here.
We've not received any reports that Explorer cannot launch app execution aliases directly; that's actually how Win+X->Terminal works (!), and all ShellExecute() (like running wt from the address bar in explorer.)
It might be worth trying pointing to wt directly and seeing how that works out for you. If it's still broken, there might be other things we can do (like launching via shell:appsfolder/....)

At the end of the day though, I don't know that this is something we're entirely in control of. I'm gonna close this bug until we've got some more info :smile:

Specifically, it's shell:AppsFolder\Microsoft.WindowsTerminal_8wekyb3d8bbwe!App. If you put it into quotes, you can run WT from Win+R that way as an alternative to using the alias. Unfortunately, it doesn't work under these registry keys (I've just tested it).
I find it interesting though that wt.exe -d "%1" actually works perfectly fine for me. I can't reproduce any issues that would warrant the use of cmd.exe. (Windows 11, build 25962.)

Specifically, it's shell:AppsFolder\Microsoft.WindowsTerminal_8wekyb3d8bbwe!App. If you put it into quotes, you can run WT from Win+R that way as an alternative to using the alias. Unfortunately, it doesn't work under these registry keys (I've just tested it). I find it interesting though that wt.exe -d "%1" actually works perfectly fine for me. I can't reproduce any issues that would warrant the use of cmd.exe. (Windows 11, build 25962.)

Even from the registry? As far as I understand it, that's the problem?

Yes, exactly. It works fine from the registry for me. If it doesn't work for you, I think it's likely that this was a limitation that has been resolved in an upcoming version of Windows 11 (I mean 25962 is a public version, but it's in the Dev/Canary channel). I haven't tested it on the version that is currently being rolled out ("SV2" / "23H2"), but I hope that whatever got fixed is already in that version.


Move in extension types for 'core.action/provider'

JIRA: HAC-424

[x] core.action/provider
[x] core.action/resource-provider
[x] core.action/group
[x] core.action/filter
[x] core.catalog/item-type
[x] core.catalog/item-provider
[x] core.catalog/item-filter
[x] core.context-provider
[x] core.flag
[x] core.flag/model
[x] core.navigation/href
[x] core.navigation/resource-ns
[x] core.navigation/separator
[x] core.page/resource/list
[x] core.page/resource/details
[x] core.redux-reducer
[x] core.resource/create
[x] core.resource-metadata
this type doesn't exists in console, there's just console.model-metadata I've exported this extension type and also core.resource-metadata, they are both identical. We should consider deprecating one or the other.
[x] core.telemetry/listener
[x] core.yaml-template

Additional extensions:

[x] core.page/route

ping @vojtechszocs

/retitle Move in extension types

/lgtm


Revert "Revert "grpc_cli json support""

Reverts grpc/grpc#16902

Is there a commit adding something to the revert?

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

No. This should be a clean revert.

[trickle] No significant performance differences

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,966,317      Total (=)      1,966,317

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,909,771      Total (=)     10,909,771

 No significant differences in binary sizes

Corrupt JSON data (indicates timeout or crash):
    bm_call_create.BM_IsolatedFilter_ClientChannelFilter_NoOp_.counters.new: 10
    bm_call_create.BM_IsolatedFilter_ClientChannelFilter_NoOp_.counters.old: 10

[microbenchmarks] No significant performance differences

@dgquintas ping~

If this is a clean revert, how is it going to prevent the issues I faced during the import?

Because I fixed it internally :), see 217759704

ah, sneaky...


Add ConfigDocument API

This work came out of the discussion in https://github.com/typesafehub/config/issues/272.
These commits add a new grouping of classes, ConfigNode, to the library, which are used to form an AST. This AST contains all the tokens from which it was parsed, so the original text can be reproduced from them.
I'm opening this up for review now as the implementation is complete, so I think that portion is ready for review. The testing code is rough right now, largely because of the difficulty involved with crafting instances of ConfigNodeComplexValue by hand, so I'm still working on improving that, although any feedback on that front is welcome.
This PR adds the ConfigDocument API in its entirety, including its backend representation with ConfigNodes, the ConfigDocument Parser, and the ConfigDocument Interface.

To avoid crafting complex nodes by hand, one approach would be to go ahead and implement the tokens-to-ConfigNode parser. That would also allow implementing a round-trip test that no tokens or formatting are lost in the parsing process and that the ConfigNode type hierarchy is complete. (Round trip test means:  configNodeParser("any valid config stuff here").render().equals("any valid config stuff here") ... this can be tested for the files that are already in src/test/resources perhaps)
I think you could cut-and-paste Parser.java; the changes I see the need to make right away are:

you can drop all that TokenWithComments / attractsLeading/TrailingComments stuff and instead just generate comments nodes
generate nodes for, rather than discarding, tokens such as ignored whitespace and field separators
don't actually process includes, just make nodes for them
might want to split the path expression parsing stuff out of Parser.java into its own file so you can use it too

I think if your new parser doesn't have the TokenWithComments stuff and shares the path expression parsing stuff, it won't be that many lines of code, and the lines it has will be based on the Parser.java lines. Most of the verbosity in Parser.java is about nice error messages (which would indeed be nice to keep). Don't be afraid of cut-and-paste for the bulk of the parser, since we can clean that up later by making the existing Parser.java go from ConfigNode => ConfigValue instead of using tokens.

@havocp Yeah, I was thinking that implementing the parser would make testing easier, but I felt like there was still some value in directly crafting config nodes for tests.
I have some more tests of config nodes that I haven't pushed up yet. Do you want me to push those up? And would you prefer the Parser be added to this PR or a new PR? I was planning to add in the parser in a new PR after this was merged, but I can add it into this PR if you'd like.

@havocp I've added some more tests and will get to work on addressing your feedback, thanks!

I agree that some direct tests for the nodes are good too. I'm thinking it might be worth adding the parser here because I'm having trouble feeling we have the node API right without also having the parser and roundtrip tests.  What do you think?
What I would like to keep separate is the modification of Parser.java - mostly because that's when we risk breaking existing functionality. It would be nice if this PR is only adding new stuff, and then make it a separate step to unify with the old stuff.
(exception: maybe that path expression parser code can move into its own PathParser file, since that's hopefully just a simple move.)

@havocp Hmm, that makes sense to me. This feature isn't really finished without the Parser, so if they get merged separately, we'll have an incomplete, in-progress feature in the repository. I'll address your feedback, then get to work on the new Parser.  In regards to the path changes, I'm getting to work on that now, and will push that up in a separate commit.

@havocp I've pushed up a new commit that saves the Tokens in Paths produced by Parser.parsePath. It should be ready for review, so any feedback is much appreciated!

Looking better all the time, I appreciate the work on this. I think it's turning out to be fairly simple, maybe more so than I expected... not a common experience in software development :-) yay!

@havocp we owe you a beer for all your help on this! :)

@havocp I've addressed your comments, and have also extracted the Path Parsing into a new class. Let me know what you think!

@havocp Quick question: for the new Parser, should I be keeping the logic that throws errors if the parsed text does not conform to JSON? Or should we ignore that in the ConfigDocumentParser, and let the current Parser handle it when it grabs config nodes from the ConfigDocumentParser?

I think when loading a document you might want to require it to be JSON-conforming, so maybe the doc parser should be able to check JSON conformance too.

@havocp @cprice404 I've just pushed up my implementation of the parser if either of you want to take a look. It can parse a file into a ComplexConfigNode and render that file back out exactly as it was rendered in.
I don't have a ConfigDocument class implemented yet, that's coming next. Once that's implemented, I'm going to add ConfigDocument tests that test replacement and rendering (since ConfigNodeComplexValue does the lion's share of the work, I doubt a ConfigDocument implementation will take me long at all).
I've put this up now as I feel it's ready for review. Hopefully I'll have the ConfigDocument changes up by some point tomorrow, at which point this PR will have all it needs for final review/merge.

@havocp The Parser is finished, and is now ready for review. Any feedback (especially on test coverage) is welcome!

fantastic! I'm at scala days right now and iPhone-only most of the day, so my review may be lagged as late as next week, but I will try to take a first pass this evening or throw in a few impressions via iPhone.
One thing you can do is run jacoco:cover (do "project config" first I think to enter the config subproject). then look at the generated html report under config/target.
I haven't looked at the code yet but another idea would be to share all applicable tests with the old parser, for example they should probably throw most of the same errors.
you can also run findbugs which may turn up a couple small issues, or not. sometimes it does.

Were there any interesting changes or decisions porting the old parser to generate nodes?

Thanks! I'll look into those approaches, probably once I get the ConfigDocument stuff pushed up.
There weren't a ton of super noteworthy changes, it was pretty easy to modify the Parser to create nodes. Mostly I was able to simplify things quite a bit and cut out a number of methods, which was nice, it was a pretty fun exercise.

Okay, got the ConfigDocument stuff written along with some tests. This is ready for final review, I think.
The only problem I know of now is adding a new path in JSON files. This is tricky because of the way JSON works, as all keys need to be quoted. What's happening with CONF files is the path is just being added at the top level, so if you give it path a.b.c, and it doesn't see that path anywhere in the file, it'll just add a.b.c=[value] to the top level. This is not valid in JSON, as all paths need to be quoted, and we can't just add a quote around the path when adding a new path to a JSON input because if we were replacing a value with a multi-element path, we'd add path "a.b.c" instead of a.b.c, and those are different.
I think the path replacement is going to have to be modified somehow. I can think of three approaches:

We don't support adding a new path in JSON if it's not a single-element path.
We support adding all new paths in JSON, but when we add a multi-element path, we quote the entire thing, turning it into a single-element path.
Instead of adding the path at the top level, we move down through the tree until we find the object where only a single-element path remains, and add the value into that object with the necessary key. I definitely think this is the best approach. So if we had the map

{a: b
 c: {
   d: e
 }

And set value 12 at path c.f, we'd get the following map:
{a: b
  c: {
    d: e
    f: 12
  }

In this case, if there does not exist an object wherein the remaining path is single-element, then we'd add the full path to the top level of the map in the case of parsing a CONF file, or throw an exception in the case of a JSON file. So, if in the above example, we wanted to set value 12 at path c.f.g, an exception would be thrown if the file were JSON, but we'd get the following if it were a CONF file:
{a: b
  c: {
    d: e
  c.f.g: 12
}

I'm thinking the third approach would be best, especially now that I've put it down into writing and can see that it probably won't be very hard to accomplish at all.
@havocp How does this sound to you? Would you prefer a different approach?

I like the third approach. To be clear, we would throw if we try to set a field in a non-object right, but if we try to set a field in a path that doesn't exist we could auto create the parent object I would think?

so right now the public ConfigNode is useless right (there's no way to get one?)
That's fine and even good (would be nice to get this basic document with set value thing done, then look at public nodes later, I don't know if you even want that).
But wanted to clarify.

In terms of the value setting, yeah, I think we could definitely create the parent object if it doesn't exist. I think it will be tricky, though, to figure out which level on which to create the parent object and the indentation for the parent object/new setting.
And yeah, right now there's no way to use the public ConfigNode. I'm thinking we can add that in a subsequent PR.

I would think maybe it'd be alright to not worry about indentation for the first pass at this?

@cprice404 That's a good point, that can probably be added in a subsequent PR. If we're just worrying about adding the parent object, this should be quite a bit easier to implement.
I'll work on modifying the value adding so it creates the parent object if it doesn't exist. I think that'll be pretty easy if we're not worrying about indentation, so hopefully I'll have something ready in the next couple hours.

ignoring indentation seems fine to me as a first cut, go for it.

Okay, I've fixed up the field addition. Now, all non-existent objects along the path will be created, and when the ConfigDocument was parsed from a JSON document, the resultant ConfigDocument will render valid JSON.

I've updated the title, since this is now in a state where I feel it is ready for a full review with the intent of being merged.

I'm currently working on a new PR on top of this current one to refactor the current parser to use the new parser. Some internal implementation details will be changed, but the results of the public API as it stands in this PR will not be changing, so this should still be good for merging.

apologies for being slow here, have been traveling and so on.
I'm thinking there might be some work to make the ConfigNode subtypes a little more "semantic," like have nodes for includes and comments and so on. I'd think the public API has stuff like ConfigNodeComment rather than ConfigNodeSingleToken. The old parser then would do very little (drop purely syntactic nodes, process includes, ...)
Is that part of what you're doing now or does it not make sense ?

@havocp Yeah, that's part of what I'm doing now for the subsequent PR. If you'd like I can merge what I've done (minus the Parser changes) into this PR.

Also, no problem on the response time! We know you've been busy with Scala Days and such. :)

It's fine to keep it separate; I'll get this one merged. I don't think I have any more substantive comments on this PR I just wanted to at least read all the code.

@havocp Great! Let me know if you have any feedback, I'm happy to make any more changes you think are necessary (particularly in regards to test coverage).

@havocp I noticed you had given me some feedback around the ConfigDocument that I hadn't addressed yet, I pushed up a new commit that addresses it.

The test coverage looks reasonable to me, I'm also thinking that once you port the existing parser to use this one, we'll get a ton of coverage "for free" from all the existing tests.

the only outright bug I saw was the !=/equals thing, so unless I'm confused there we should probably fix it. otherwise, I've been through this code and it looks good to merge.

@havocp Great! I'll go through and address your last bits of feedback, hopefully it won't take long

Thanks for the review!

throwing on any kind of stuff (comments, whitespace, concatenation not supported by JSON) around the value in setValue makes sense to me, because otherwise it gets into all kinds of thorny issues where we can have multiple trees possible for the same rendered file.

@havocp Alright, I'll get to work now on making a new simple value class that is multi-node to contain whitespace/comments, which will be used solely in the case of setting a value.

I was trying to say don't do that - I don't think we want the whitespace/comments to be part of the value node sometimes, and part of the parent node sometimes. Also, I don't think we want setting a value to be non-idempotent. Instead, setValue could give up and throw if there's any "extras" around the value. That way we can punt on all the semantics of whitespace/comments.

Ah, sorry, misunderstood what you were saying! That should be a fairly quick fix.

@havocp Alright, I've addressed your last few comments!

Merged, thanks very much!


arbitrary_image_stylization code for distillation

Would you be interested in adding the scripts I used for distillation (stylenet inceptionv3->mobilenet) + another thing i'm working on where i replace the transformnet conv2d layers with separable_conv2d?
Right now i haven't uploaded them publicly since they're in pretty messy colab notebooks but I could clean it up and have something like distill_arbitrary_image_stylization_to_mobilenet.py. This way, other people with more resources than me could run them (I can only do around 80K training steps in a few days with Google Colab) and probably get better models. If not, then I'll probably just dump the notebooks as is in my repo.

This would be great to have, thanks!


Thoughts & Experience

This reference serves as a fantastic illustration of developing a phone natively whilst also providing some quick examples around streaming the contact images. Many developers will choose alternative methods for UI... with some interesting design choices to say the least.
Despite ongoing searches, I've struggled to find others adopting a lore-friendly approach for the phone. Thank you!!
Minor Issues:

The Mobile Radio Player app fails to recognize changes when adjusting the radio through another resource. To provide context, I am experimenting with JS Radio to understand the functionality of custom radios. For the record, the Mobile Radio idea is fantastic and has been very popular during testing.

When utilizing a development menu (such as vMenu, EasyAdmin, etc.), conflicts arise as the Phone pops up and interferes with menu navigation, given that both use the same arrow keys. Would it be overly complex to establish a system that communicates with "KGV-Phone" to prevent phone activation when another menu is open? A check of some sort to say, "Hey, don't open the phone!"

Currently, the phone appears abruptly in a static manner. In native GTA V, the phone smoothly transitions onto the screen. Based on the checklist provided, it seems that visual elements like this were deferred until the core functions were completed. If true, this can be safely ignored.

i made this AGES ago as a silly little test and it got out of hand and became a full on phone resource, i have had plans to redo it completely from scratch with modern skill and mindset

mobile radio is a silly little experiment, that was surprisingly useful, that said - updating radio aspects in real time might not be that easy afaik
on my server that i made the resource, i intentionally avoid having easy shortcuts like vMenu or a trainer, to challenge myself to come up with better solutions. as a result - i have no idea how the phone behaves with other menus
the script i started with had animations, but i didnt understand the code so i had to remove it to get anywhere. nowadays i understand code FAR better. if i were to redo it from scratch, as mentioned earlier, i would absolutely have animations

there's been modern attempts at doing the same thing which do a pretty damn good job too:
https://forum.cfx.re/t/scalephone-an-ifruit-phone-framework/4548049
https://github.com/CritteRo/scalePhone

I suppose that most of these issues would resolve themselves in a new hypothetical iteration. Thank you for the prompt response!

i have had plans to redo it completely from scratch with modern skill and mindset

Exciting to hear! It would be even more insightful to compare this old version with a new version, since I've worked with it so much recently, but I totally understand if you aren't interested in that anymore. As a side note, I've got another resource of yours on our server as well - Absolutely love KGV-Hipfire great work ❤

there's been modern attempts at doing the same thing which do a pretty damn good job too

I'll make sure to explore them, thanks again! If they lack Mobile Radio functionality though, we might just stick with your resource. We've also been brainstorming ideas for custom phone variations like Badger or Whiz, so if you ever decide to revive this project, I've got a list of feature requests and ideas ready 😆

you're welcome to submit PRs to the repo, i'll review and pull them without issues. the refactor would directly reference new features added through PRs too, so it's not all in vein


Navigation to internal pages with dots in the URL fails on extension detection

Describe the bug
Navigation to internal pages with dots in the URL fails because the location is not visitable (Controller.locationIsVisitable).
I suspect this is caused by the definition of Location.isHTML.
To Reproduce
Steps to reproduce the behavior:

Have an app with a route that allows dots
Click a link to or call Turbolinks.visit('/websites/example.com')
A full page reload (window.location.href) happens

Expected behavior
Turbolinks navigates to the given location.
Details (please complete the following information):

Operating System: Windows 10
Browser: Chrome
Turbolinks 5.2.0

Additional context
In Controller.locationIsVisitable (https://github.com/turbolinks/turbolinks/blob/271c581/src/controller.ts#L326-L328), the first conditional evaluates to true, since the location is prefixed by the root.
However, the second condition (which I think is there to handle file download links) evaluates to false.
Location.isHTML (https://github.com/turbolinks/turbolinks/blob/master/src/location.ts#L57-L59) checks that, if there is a file extension`, it's on a whitelist.
Since the 'extension' of the location is not on the whitelist, it's not visitable.
I don't have a good suggestion for how to solve this issue, as I understand why the current behaviour is the way it is.
Would it make sense to have a global and/or link-level opt-out for the isHTML check?

Hey @timvdalen,
https://github.com/turbolinks/turbolinks/issues/301 discusses this. The current guidance is to customise Turbolinks.Location.prototype.isHTML to suit your needs.
Closing as duplicate of #301

Thanks, I missed that one in my search!


update travis and appveyor batches, remove codacy and coverall

Changes Unknown when pulling c2182ee6a7dcc7e14de9ace1b60c3eef8afeacf9 on namvd2709:update-readme-batches into ** on CS2103AUG2017-T12-B1:master**.

Coverage remained the same at 91.875% when pulling 88695b317288191edeb3e7391b45a24fad9fcfc5 on namvd2709:update-readme-batches into 26fcfd73bbd951b415cee2b23f631645b69158a5 on CS2103AUG2017-T12-B1:master.

Coverage remained the same at 91.875% when pulling a16061bfb6e018cc03282d3c59a5991575929a66 on namvd2709:update-readme-batches into 26fcfd73bbd951b415cee2b23f631645b69158a5 on CS2103AUG2017-T12-B1:master.


PLRAM Example shows error

The plram example shows error as:
[XRT] ERROR: context is nullptr
[XRT] ERROR: context is nullptr
mullt.ini has the connectivity but still its not able to differentiate banks
The flag XCL_MEM_PLRAM_BANK3 is not applicable to plram.
What flay can be used used for PLRAM

Hi @aeeshms ,
Tried running the plram design and it's working fine without any error.
Can you tell me what exactly is the change you are looking in mmult.ini. If you want to use different PLRAM you can pick from following(PLRAM[0], PLRAM[1] and PLRAM[2]) similarly if different bank is required(DDR[0], DDR[1], DDR[2] and DDR[3]). These PLRAM and bank information is for U200 platform.
Based on platform the resource changes(PLRAM, DDR).

Closing as no response got from filer for last query. Feel free to re-open if this issue still there.


Allow ServiceMonitor CRD for other charts

Related to #115
https://github.com/rstudio/helm/pull/99

This was resolved in #322. We added RSW a while back too


Clean up documentation scripts a tiny bit

Just in case anyone else ever uses them.

+1
On Thu, Apr 5, 2018 at 1:35 PM, Tom Swirly<EMAIL_ADDRESS>wrote:

Just in case anyone else ever uses them.
You can view, comment on, or merge this pull request online at:
https://github.com/ManiacalLabs/BiblioPixel/pull/738
Commit Summary

Move gen_docs.sh to scripts/ (fix #603)
Clean up scripts/

File Changes

M bibliopixel/main/commands.py
https://github.com/ManiacalLabs/BiblioPixel/pull/738/files#diff-0
(2)
D gen_docs.sh
https://github.com/ManiacalLabs/BiblioPixel/pull/738/files#diff-1
(10)
R scripts/make_bp_help_documentation
https://github.com/ManiacalLabs/BiblioPixel/pull/738/files#diff-2
(3)
A scripts/make_documentation
https://github.com/ManiacalLabs/BiblioPixel/pull/738/files#diff-3
(10)
A scripts/make_documentation_live
https://github.com/ManiacalLabs/BiblioPixel/pull/738/files#diff-4
(11)
M scripts/new_version
https://github.com/ManiacalLabs/BiblioPixel/pull/738/files#diff-5
(5)

Patch Links:

https://github.com/ManiacalLabs/BiblioPixel/pull/738.patch
https://github.com/ManiacalLabs/BiblioPixel/pull/738.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/pull/738, or mute the thread
https://github.com/notifications/unsubscribe-auth/AA6a6tYrk09zx6r4bAh2j-uPoZUdAj-Jks5tllXmgaJpZM4TI3kl
.


Add BisSchemas loader performance tests

 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.


avoid corruption when metadata is inconsistent (writePos vs write file size)

…ue was not synced on previous shutdown
Example: if nsqd was forcibly killed before it could complete sync of all the diskqueues' metadata files

LGTM, thanks!


Update SPMe to generalised version

Description
This issue generalises the SPMe formulation. The main contributions are

[x] Upgrade SPMe to the general formulation in #4274.
[x] Remove  Integrated model as it is no longer needed.
[ ] Document SPMe definition
[ ] Implement BasicSPMe

Fixes #4274
Type of change
Please add a line in the relevant section of CHANGELOG.md to document the change (include PR #) - note reverse order of PR #s. If necessary, also add to the list of breaking changes.

[ ] New feature (non-breaking change which adds functionality)
[x] Optimization (back-end change that speeds up the code)
[ ] Bug fix (non-breaking change which fixes an issue)

Key checklist:

[ ] No style issues: $ pre-commit run (or $ nox -s pre-commit) (see CONTRIBUTING.md for how to set this up to run automatically when committing locally, in just two lines of code)
[ ] All tests pass: $ python run-tests.py --all (or $ nox -s tests)
[ ] The documentation builds: $ python run-tests.py --doctest (or $ nox -s doctests)

You can run integration tests, unit tests, and doctests together at once, using $ python run-tests.py --quick (or $ nox -s quick).
Further checks:

[ ] Code is commented, particularly in hard-to-understand areas
[ ] Tests added that prove fix is effective or that feature works

I ran some comparison between all the SPMe models.
Accuracy
I compared the outputs of the models and there are no appreciable differences across SPMe implementations. That might sound a bit disappointing as we are comparing a good approximation with a slightly better approximation. In O'Regan 2022 maybe one can argue that the new model does slightly better, as there the thermodynamic factor and transfer number are also dependent on concentration, which is what the new model captures. I did similar comparison for the half-cell and it also agrees very well.
Marquis 2019

O'Regan 2022

Solving times
Here are the solving times. It is just one occurrence, but I ran the example multiple times and they are representative. Basically, the new SPMe is not significantly slower than the all one so all good. These times are for the Marquis2019 parameter set and the full cell.

Model
Build time [ms]
Solve time [ms]

new SPMe
44.578
11.814

old SPMe
45.518
12.957

integrated SPMe
50.563
11.331

DFN
54.589
113.854


Doc changes

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 91.416%

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
177648

Relevant Lines:
194330

💛  - Coveralls


Update Guild features list

Summary
This updates the guild features list. Discord keeps changing them and not documenting them so I replaced everything with the API docs here and the following are added by me:
ANIMATED_BANNER, ENABLED_DISCOVERABLE_BEFORE, MEMBER_PROFILES, NEW_THREAD_PERMISSIONS, THREADS_ENABLED from discord.py's freatures list.
Checklist

[ ] If code changes were made then they have been tested.

[x] I have updated the documentation to reflect the changes.

[ ] This PR fixes an issue.
[ ] This PR adds something new (e.g. new method or parameters).
[ ] This PR is a breaking change (e.g. methods or parameters removed/renamed)
[x] This PR is not a code change (e.g. documentation, README, ...)

Why are all of these features documented here? Can't we just link to discord's docs for this since it's a straight copy paste?

Why are all of these features documented here? Can't we just link to discord's docs for this since it's a straight copy paste?

that would be nice, discord.py people were against it.


bug: Discrepancy in applied offsets when running protocol through SSH versus Jupyter

Overview
Customer is applying labware offsets directly to protocol according to https://support.opentrons.com/en/articles/5901728-how-to-prepare-jupyter-notebook-and-opentrons_execute-for-5-0-0.
They are reporting that the labware offsets are correctly applied in Jupyter but not with opentrons_execute on the command line.
Steps to reproduce
Note: I could not reproduce
Follow https://support.opentrons.com/en/articles/5901728-how-to-prepare-jupyter-notebook-and-opentrons_execute-for-5-0-0 to download labware offsets and use set.offset to apply it in the protocol, then run in Jupyter and over SSH to see if the offsets are applied.
Current behavior
Customer's behavior: with labware offsets defined in the protocol, opentrons_execute ignores and does not apply the offsets. In Jupyter, it works
Expected behavior
It should work with Jupyter and opentrons_execute.
dispensecover.py.zip

Knowing the code in play, this problem as stated seems implausible. Pretty bad if true, so we'll investigate, but I really don't see how this could happen.
In the attached protocol, there are two different sets of offsets applied according to a USING_SSH flag. Is there a possibility this is the source of the user's observed differences?
    if USING_SSH:
        # ...
        # Apply offests as needed
        tipRack.set_offset(x=0.00, y=9.00, z=0.10)
        suckerRack.set_offset(x=0.00, y=0.00, z=73.00)
        coverRack.set_offset(x=-1.00, y=14.20, z=-8.00)
        chipPlate.set_offset(x=-4.00, y=15.00, z=0.00)
        reservoir.set_offset(x=0.00, y=11.20, z=0.20)

    else:
        # ...
        # Apply offsets
        tipRack.set_offset(x=0.00, y=0.40, z=0.00)
        suckerRack.set_offset(x=0.00, y=0.00, z=0.00)
        coverRack.set_offset(x=-1.00, y=1.00, z=-8.00)
        chipPlate.set_offset(x=-4.00, y=4.20, z=-0.80)
        reservoir.set_offset(x=0.10, y=0.70, z=1.10)

The offset under the “if” is there because those are the offset values that are needed to hit the target areas due to discrepancy.
The offset values under the “else” statement are from the labware position check.

Digging around I found the offset might be caused by not setting the environment variable RUNNING_ON_PI = 1.
My issue now is that applying os.environ['RUNNING_ON_PI'] = '1' in my protocol the robot still seems to think it is not changed

.

Hi @BSNXCody, hope you're doing well! If you are still seeing this issue, I would suggest reaching out to<EMAIL_ADDRESS>to get support for it. This seems like it's more an issue with the protocol than a software bug.

Closing as unable to reproduce


⚠ Immich has degraded performance

In 72f575a, Immich (https://immich.underthere.xyz) experienced degraded performance:

HTTP code: 502
Response time: 18475 ms

Resolved: Immich performance has improved in 748a5bf after 1 minute.


fix: removed storybook warnings

"export" типов заменен на "export type"

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.007%) to 78.705%

Totals

Change from base Build<PHONE_NUMBER>:
0.007%

Covered Lines:
7075

Relevant Lines:
8172

💛  - Coveralls


fix: make the REPL terminal transient

Fixes  #379

The problem did not seem to be that we were launching the terminal too early, but rather that VS Code was automatically saving the state of the terminal and relaunching it on startup, ignoring whether the compiler needed to be downloaded first.
To opt out of this I have marked the terminal as being 'transient'. This of course means that you now have to wait for the terminal to start every time you restart, but oh well.

The problem did not seem to be that we were launching the terminal too early, but rather that VS Code was automatically saving the state of the terminal and relaunching it on startup, ignoring whether the compiler needed to be downloaded first.
To opt out of this I have marked the terminal as being 'transient'. This of course means that you now have to wait for the terminal to start every time you restart, but oh well.

I think that is actually a good solution. I will try it when it get home.

Quick question: What happens if you click run 3 times?

Quick question: What happens if you click run 3 times?

Same as before. Within a single session the flixTerminal variable keeps track of the launched terminal. The searching logic was only used between restarts, but now no REPL will survive that.


How to set the start of cron job?

parser.text('every 4 hours');
Set the frequency of cron job, but what about the start of the job, can we determine when that 4 hour begins and ends?

I have a parser.text('every 30 minutes'); and it starts every round hours. I guess for 4 hours it will start at midnight (00:00) and go on...

It can be found in the later.js documentation at http://bunkat.github.io/later/parsers.html#text:
To start the 4 hours not at midnight (default) but let's say at 6 am:
parser.text('every 4 hours starting on the 6th hour');


[FEATURE] Allow multiple per commit

Is your feature request related to a problem? Please describe.
When committing sometimes you have multiple multiple different things in that commit.
The summary could be fix: I did blah but the git commit body could also have chore: update blah and ci: fix blah
Unfortunately, it seems when committing, cog only picks up the first message in the commit and ignores the rest.
Example:
I made a commit with the fix in the title, and an extra one in the message.

But it only adds the first one, and cog check also rejects it if you don't have fix/feat first

As described in the conventional commits specification a commit can only have one type.
You might end up with squashed commit messages in the commit body but that is not relevant for SemVer bumps.


Localize javascript components in assets/js

The javascript in assets/js contains a few keywords that should be localized as well after #201 is merged.
https://github.com/fortran-lang/fortran-lang.org/blob/8ca4ed7229d90552711def3d8919d7f4ed524bd1/assets/js/gh-contributors.js#L329
https://github.com/fortran-lang/fortran-lang.org/blob/8ca4ed7229d90552711def3d8919d7f4ed524bd1/assets/js/gh-contributors.js#L423
https://github.com/fortran-lang/fortran-lang.org/blob/8ca4ed7229d90552711def3d8919d7f4ed524bd1/assets/js/gh-contributors.js#L425
https://github.com/fortran-lang/fortran-lang.org/blob/8ca4ed7229d90552711def3d8919d7f4ed524bd1/assets/js/gh-contributors.js#L431

This should be resolved with the new webpage (see #415)


Support multiple classes

self.classValue.split(" ").forEach(function (value) {
  el.classList[self.classBehavior](value)
});

See more:https://github.com/qweek/uiscript/blob/master/uiscript.js

I agree, I think this would be an important addition.


Add theoretical drivetrain parameter computation

Probably would be best to prompt the user for it when logging, and then add the data to the JSON so it can compute things like efficiency et al.
To make it less annoying, could store default parameters in a file when starting, and load those so you just have to hit enter.

We could definitely add a small widget to do this - but should it be integrated into the analysis GUI, or be a separate window?


[CHALLENGE SUBMISSION] Day 10

Please provide the following information for your submission:
What is your name? (First, Last)
Barbara Forbes
Where is your GitHub Repo ?(With your challenge solution)
https://github.com/Ba4bes/25daysofserverless/blob/master/Daya10DailyAggregatedDeals/Readme.md
What Challenge is this for? (Challenge number between 1 and 25)
10
(Optional) Anything else we should know? e.g., language used, location, blog post?
LogicApp

From all of us on the Cloud Advocacy team at Microsoft, thank you for creating this issue!
Have a Happy Holiday season!
Feel free to share your solution on Twitter with the tag #25DaysOfServerless

@Ba4bes  👋 Thank you for your submission!! We checked out your repository. We're thrilled to say your solution is now listed in our Hall of Fame
Thank you for your contributions!
This issue is now resolved! 🎉


chore(deps): upgrade configuration

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-configuration-k8s-25-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3723

Questions ?
Please refer to the Backport tool documentation


Flutter web does not support BlendMode.clear or BlendMode.dst using html renderer

Flutter version :2.10.4›
`Doctor summary (to see all details, run flutter doctor -v):

[✓] Flutter (Channel stable, 2.10.4, on macOS 12.3 21E230 darwin-arm, locale en-RO)
[✓] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2021.1)
[✓] IntelliJ IDEA Community Edition (version 2021.3)
[✓] VS Code (version 1.65.2)
[✓] Connected device (1 available)
[✓] HTTP Host Availability

`
Steps to reproduce

import 'dart.ui' as ui
Paint _clearPaint = ui.Paint()..blendMode=BlendMode.clear // it can also be BlendMode.dst
flutter run -d chrome --web-renderer html

 Expected behavior
App should run normally.
 Current Behavior
App crashes with the current error: Flutter web does not support BlendMode.clear
If we decide to use only --web-renderer canvaskit the app will run very glitchy on mobile web

Using below code sample and running on web with html rendering, we get the error as reported:

 runnable code sample

import 'package:flutter/material.dart';

final Color darkBlue = Color.fromARGB(255, 18, 32, 47);

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      theme: ThemeData.dark().copyWith(scaffoldBackgroundColor: darkBlue),
      debugShowCheckedModeBanner: false,
      home: Scaffold(
        body: SafeArea(
          child: CustomPaint(
            painter: MyPainter(),
          ),
        ),
      ),
    );
  }
}

class MyPainter extends CustomPainter {
  MyPainter();

  @override
  void paint(Canvas canvas, Size size){
    Rect rect = Rect.fromLTRB(400, 450, 600, 550);
    Rect bigRect = Rect.fromLTRB(200, 400, 800, 600);
    Paint paint = Paint()..color=Colors.white;

    canvas.saveLayer(bigRect, Paint());
    canvas.drawRect(bigRect, paint);
    canvas.drawRRect(
      RRect.fromRectAndRadius(rect, Radius.circular(10)),
      Paint()..blendMode=BlendMode.dst,  // or use BlendMode.clear
    );

    canvas.restore();
  }

  @override
  bool shouldRepaint(MyPainter oldDelegate) => true;
}

Error in console log:
The following UnimplementedError was thrown during a scheduler callback:
Flutter Web does not support the blend mode: BlendMode.dst

The following UnimplementedError was thrown during a scheduler callback:
Flutter Web does not support the blend mode: BlendMode.clear

This could be due to the fact that web platform does not support these two blendmodes. Other blendmodes are supported though.

 stable, master flutter doctor -v

[✓] Flutter (Channel stable, 2.10.4, on macOS 12.2.1 21D62 darwin-x64, locale
    en-GB)
    • Flutter version 2.10.4 at /Users/dhs/documents/fluttersdk/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision c860cba910 (3 days ago), 2022-03-25 00:23:12 -0500
    • Engine revision 57d3bac3dd
    • Dart version 2.16.2
    • DevTools version 2.9.2

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

[✓] Flutter (Channel master, 2.13.0-0.0.pre.329, on macOS 12.2.1 21D62
    darwin-x64, locale en-GB)
    • Flutter version 2.13.0-0.0.pre.329 at
      /Users/dhs/documents/fluttersdk/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 5c6918933e (77 minutes ago), 2022-03-30 20:30:09 -0700
    • Engine revision a1106daced
    • Dart version 2.17.0 (build 2.17.0-249.0.dev)
    • DevTools version 2.11.4

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

/cc @yjbanov

We can't implement all blend modes in HTML because HTML doesn't support all of them. If you are aware of a way to implement this, please leave a comment. Otherwise, we'll have to close it as not implementable.

Thank you, for the response, I will try to look up a solution. At the moment this is what I was looking into: https://stackoverflow.com/questions/42740839/html5-canvas-blendmode#42741921
I will try to come up with somethibg for BlendMode.clear, this will also fix the package lottie from pub.dev.


Feature/rust tokenizer

What does this changes
Add newmm word tokenization with default thai dictionary which is implemented in Rust.
What was wrong
Word tokenization in python is quite slow for large text.
How this fixes it
This rust module can speedup words tokenization process.
Fixes #...
Your checklist for this pull request
🚨Please review the guidelines for contributing to this repository.

[ ] Passed code styles and structures
[ ] Passed code linting checks and unit test

Move to https://github.com/PyThaiNLP/oxidized-thainlp/


Impersonation issue

The current checks uses author name to ensure that the changes were made by the expected user.
https://github.com/github/privileged-requester/blob/fdd9f579ce88b2494b874bb381b966777473c3c3/src/runner.js#L16-L23
It is however really easy to make a commit with a different name, or email address.
git -c user.name='Linus Torvalds' -c<EMAIL_ADDRESS>commit -m "Wow, this is definitely legit."

This poses a serious risk of impersonation.

The GitHub API, provide commit verification information that is parsed in https://github.com/github/privileged-requester/blob/fdd9f579ce88b2494b874bb381b966777473c3c3/src/runner.js#L16
Format:
      "verification": {
        "verified": false,
        "reason": "unsigned",
        "signature": null,
        "payload": null
      }

E.g. gh api /repos/X/Y/pulls/Z/commits | jq '.[].commit.verification':
{
  "payload": null,
  "reason": "unsigned",
  "signature": null,
  "verified": false
},
{
  "payload": "tree xxx\nparent yyy\nparent 76508bebf310e1152c0d423465a35883b42b306a\nauthor nobe4<EMAIL_ADDRESS>  "reason": "valid",
  "signature": "-----BEGIN PGP SIGNATURE-----\n\nwsFcBA...n-----END PGP SIGNATURE-----\n",
  "verified": true
}

It is possible to add:
    for (const [, commit] of Object.entries(this.pullRequest.listCommits())) {
      let commitAuthor = commit.author.login.toLowerCase();

+     if (!commit.verification.verified) {
+               core.warning("Unexpected unverified commit.");
+       return false.
+     }

      if (commitAuthor !== privileged_requester_username) {
        core.warning(
          `Unexpected commit author found by ${commitAuthor}! Commits should be authored by ${privileged_requester_username} I will not proceed with the privileged reviewer process.`,
        );
        return false;
      }
    }

@nobe4 I'll work on implementing this 🚀


Validate the new installer

Scenarios to cover:
Needs to be verified on both x64 and x86 operating systems.
Standalone scenarios (ANCM not installed on machine)

ANCM msi

[ ] apphost.config file to be updated with aspnetcore section and ANCM module
[ ] aspnetcore.dll dropped in inetsrv

ANCMV2 msi

[ ] apphost.config file to be updated pointing ANCMV2 to ProgramFiles/IIS
[ ] aspnetcorev2.dll in v2 directory
[ ] aspnetcorev2_outofprocess.dll in v2/RH_VERSION directory

ANCM iis express

[ ] apphost.config file to be updated
[ ] aspnetcore.dll dropped in PF\IIS Express

ANCMV2 iis express msi

[ ] apphost.config file to be updated pointing ANCMV2 to ProgramFiles/IIS
[ ] aspnetcorev2.dll in v2 directory
[ ] aspnetcorev2_outofprocess.dll in v2/RH_VERSION directory

Upgrade scenarios:

ANCMV2 msi with RH version higher than currently installed

[ ] aspnetcorev2_outofprocess.dll in v2/higher RH VERSION in directory. Only one RH in directory.

ANCMV2 iis express msi

[ ] aspnetcorev2_outofprocess.dll in v2/higher RH VERSION in directory. Only one RH in directory.

Uninstall scenarios:

ANCM msi

[ ] apphost.config removes ANCM module, but keeps aspnetcore section
[ ] Removes the dlls from inetsrv

ANCMV2 msi

[ ] apphost.config removes ANCMV2 module, but keeps aspnetcore section
[ ] Removes the dlls from PF

ANCM iis express

[ ] apphost.config removes ANCM module, but keeps aspnetcore section
[ ] Removes the dlls from PF

ANCMV2 iis express msi

[ ] apphost.config removes ANCMV2 module, but keeps aspnetcore section
[ ] Removes the dlls from PF

@jhkimnew Please wait for #859 and #883 to be done before getting the CTI folks started. Thanks!

@muratg @shhsu will own ANCM test instead of me.
Assigning this to Peter

I want to do a bit of manual verification too. Will wait for the entire E2E to be done before validating.

First fixing the msis and windows hosting bundle.

@muratg updated with the scenarios to cover.

Thanks @jkotalik!

@pan-wang the aspnetcore section will be removed on uninstalling ANCMV1.
Repro steps:

Install the ANCMv1 and v2 msis.
Uninstall ANCMV1
Check the ApplicationHost.config file, see that  <section name="aspNetCore" overrideModeDefault="Allow" /> is removed.

IISExpress installer uses AspNetCoreModule section rather than AspNetCoreModuleV2 section

Vendor has successfully verified results

Thanks @shhsu & @jkotalik!


mount failure, server: <IP_ADDRESS>, export: /var/nfs, nfs version: 3, returned state: 13

I installed a NFS server on Ubuntu Bionic Server.
Export the /var/nfs which I created.
Created a mount from my Ubuntu Bionic desktop.
The mount on my desktop works (<IP_ADDRESS>:/var/nfs 392287232 210137088 162153472 57% /home/gary/var/nfs) so I made the assumption the Java client should also work.

When I try to connect using EMC NFS Java Client to the same server using the same export I mounted on my desktop with the following code
Nfs3 nfs3 = new Nfs3(hostname, export, new CredentialUnix(), 3);

I get the following error:
SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".
SLF4J: Defaulting to no-operation (NOP) logger implementation
SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.
mount failure, server: <IP_ADDRESS>, export: /var/nfs, nfs version: 3, returned state: 13
com.emc.ecs.nfsclient.mount.MountException: mount failure, server: <IP_ADDRESS>, export: /var/nfs, nfs version: 3, returned state: 13
    at com.emc.ecs.nfsclient.nfs.nfs3.Nfs3.lookupRootHandle(Nfs3.java:342)
    at com.emc.ecs.nfsclient.nfs.nfs3.Nfs3.prepareRootFhAndNfsPort(Nfs3.java:311)
    at com.emc.ecs.nfsclient.nfs.nfs3.Nfs3.<init>(Nfs3.java:258)
    at com.emc.ecs.nfsclient.nfs.nfs3.Nfs3.<init>(Nfs3.java:201)
    at com.emc.ecs.nfsclient.nfs.nfs3.Nfs3.<init>(Nfs3.java:228)
    at com.emc.ecs.nfsclient.nfs.io.FileReadingTest.testReading(FileReadingTest.java:104)
    at com.emc.ecs.nfsclient.nfs.io.FileReadingTest.main(FileReadingTest.java:53)

The error message is not very helpfull and I don't know where to even begin looking for the problem.

I think state 13 === permission denied.

I got the similar problem in Centos. It is fixed by add "insecure" to info of related exported directory in file /etc/exports. More details can be found in https://stackoverflow.com/questions/58186427/emc-nfs-java-client-mountexception.


Needs mobile support

Currently set to disappear below 740px

By increasing support to 720px in https://github.com/LaneCommunityCollege/megamenu/commit/cd437fd6eecc25542ddf4099a8270bdff2a36a7c it's now possible for a site to move to mobile size, but the megamenu doesn't cooperate. Maybe add an option to include a variable for 'mobileView: 680px' and at sizes below that for now it disappears, but in the future it serves as an indicator for when to move to mobile (in which case mobile styles would need to apply based on screen size or class)?

One problem has always been the lack of a good way to point to open the menu, since the 5 categories won't be there. But, if we were to drop the logo from the megamenu, and make it a hamhurger menu, then add a menu option for Lane home, that'd create a pathway for doing a mobile menu.

To support a swipe to close: http://www.javascriptkit.com/javatutors/touchevents2.shtml


'byRef' is not a member of 'Php'

This is what I get when I do "make" on MacOS 10.11.6.
main.cpp: In function 'void* get_module()':
main.cpp:29:17: error: 'byRef' is not a member of 'Php'
                 Php::byRef("array", Php::Type::Array, true),
                 ^~~
main.cpp:30:17: error: 'byVal' is not a member of 'Php'
                 Php::byVal("path", Php::Type::String, true),
                 ^~~
main.cpp:31:17: error: 'byVal' is not a member of 'Php'
                 Php::byVal("value", Php::Type::String, true)

I tried recompiling the whole thing because I had issues while installing the first time. I did the steps as described here: https://github.com/CopernicaMarketingSoftware/PHP-CPP/issues/255
I checked if the necessary files are included but I keep getting this error.
// EDIT:
Ok. I'm getting the exact same error/s on Ubuntu 16.04. I guess I took a wrong turn somewhere.

I think it's 'ByRef'

As @terrywh correctly answered, the correct names are Php::ByVal and Php::ByRef


Add C++ to the list of libp2p implementations

I think our C++ implementation is quite mature now to be included in the list of implementations.
Otherwise please let us know if there is something we should do in order to be added here.
Thanks

Thanks for breaking the libp2p implementation off the Kagome repo into a top level one! Would you like to contribute it to the libp2p org? We’d of course make the developers maintainers in the libp2p org on GitHub!

@raulk Sorry for late reply.
We think it would be really great to have cpp-libp2p in libp2p organization.
What should we do to proceed?

@kamilsa an admin of the repo, or an owner of the Soramitsu org, should initiate a transfer to the libp2p org. libp2p organisation owners will receive a request to accept in their inboxes, and all contributors to the Soramitsu/libp2p repo will become contributors in its new home. Before initiating the transfer, please rename the repo to cpp-libp2p so that it doesn't clash with libp2p/libp2p.
Thank you!


Bug in undo and redo

Three steps:
Firstly, "t-add 1 aaa"
Secondly, "t-remove 1 aaa"
Thirdly, "undo",  it will undo correctly (add tag aaa)
Then "redo", it will delete another tag instead of tag aaa

Noted with thanks! We are working on it now.


[feat] Hard exit or restart of driver when it gets destroyed

Is your feature request related to a problem? Please describe.
I am running Zwavejs2mqtt with a remote TCP serial connection to a Z-Wave adapter. Sometimes the connection seems to get interrupted and I need to restart the container.
2021-07-07 11:41:25.499 INFO ZWAVE: Node 6: value updated: 50-0-value-66561 230.28 => 230.1
2021-07-07 11:41:26.505 INFO ZWAVE: Node 6: value updated: 50-0-value-66817 0 => 0
2021-07-07 11:44:55.340 INFO ZWAVE: Controller status: Driver: read ETIMEDOUT
2021-07-07 14:25:35.072 INFO MQTT: Message received on zwave/nodeID_6/37/0/targetValue/set, <Buffer 54 72 75 65>
2021-07-07 14:25:35.073 INFO ZWAVE: Writing true to 6-37-0-targetValue
2021-07-07 14:25:35.079 ERROR ZWAVE: Error while writing true on 6-37-0-targetValue: The driver is not ready or has been destroyed
2021-07-07 14:25:35.080 ERROR ZWAVE: Unable to write true on 6-37-0-targetValue

Describe the solution you'd like
I see two solutions:

an hard exit on the driver being not ready anymore. Since I am using Kubernetes (but will also work with docker/docker-compose with the restart option set) the container will restart and re-init the driver.
a restart of the driver when it gets destroyed within the zwavejs2mqtt process

I am in favor of the first, it shouldn't be hard to implement and it seems like logical behavior to me, zwavejs2mqtt is nothing without the connection to the adapter so a hard crash seems fine to me.

a restart of the driver when it gets destroyed within the zwavejs2mqtt process

This should already be handled when zwave js throws a driver error. @AlCalzone Could it be possible to throw a driver error when receaving an ETIMEDOUT ?

We added this a while ago: https://github.com/zwave-js/node-zwave-js/pull/2601
With zwave-js v7.9.0, any serial disconnections should cause the driver to be destroyed. AFAIK zwavejs2mqtt picks that up and restarts the driver.
@wilmardo how exactly are you configuring the remote USB connection?

@wilmardo how exactly are you configuring the remote USB connection?

I don't exactly know what you mean by remote USB connection but I have my port defined like this:
tcp://<IP_ADDRESS>:23

And I use esp-link on a ESP8266 with a RaZberry shield connected.
Does that provide the answer you looking for?

tcp://<IP_ADDRESS>:23

Is that the address that's given to the driver? In that case, zwave-js should be able to detect the disconnection and destroy itself.
I've tested with a similar setup with ser2net on a Raspberry Pi. As soon as ser2net was shut down or the network was disconnected, the driver was destroyed.
However, the automatic disconnection only works starting in v7.9.0 - not sure which one you are on.

Closing this for now, I think that an update to latest should fix the errors


If inject_script is not permitted, this tag will not call gtmOnSuccess or gtmOnFailure

https://github.com/bambuser/bambuser-one-to-one-gtm-template/blob/b70dcb1c4f7aa831954d0d57c346fbcaa87a829a/template.tpl#L214
This if statement needs an else block that calls (likely) data.gtmOnFailure().

Makes total sense. Will after review merge this PR
https://github.com/bambuser/bambuser-one-to-one-gtm-template/pull/3
that fixes this issue


return to url of shared healthsite

as a user click a URL of a healthsite shared via social media I expect to return their after signing in.
Normal course for signed out user:

navigate to a healthsite
share site via social media
open link shared in tweet, facebook, google + ect
https://healthsites.io/map#!/locality/c660d7e7b0a24e70b507b11049d78c6a
signin
...at this point the user should be re-direct back to the link above

fixed on staging


🐛  [firebase_dynamic_links] Social Sharing Previews not working on Gettr

i managed to create a dynamic link with social sharing preview (title, description and image) and the details are shown when i share the link as a post on Facebook and as a tweet on Twitter.
but the link does not work in Gettr. when i share the link as a post on Gettr, it shows as a string but without the (title, description and image).
does anyone have any clue on how to show the (title, description and image) of the dynamic link on Gettr?
thank you.

@KennyChan94P
This doesn't seem to be an issue in the plugin since it works on other platforms as you mentioned. You'll need to ask this question on support channels like Stackoverflow for further help.
Closing from here.


Crash if there are multiple services discovered

This caught me off-guard. Apparently, an instance of NsdManager.ResolveListener cannot be re-used. That is to say, for each newly found service, you need to supply a new instance of ResolveListener to the resolveService() method. If you try to reuse the listener, you get a crash with the following trace:
Caused by: java.lang.IllegalArgumentException: listener already in use
            at android.net.nsd.NsdManager.resolveService(NsdManager.java:613)
            at rxbonjour.internal.JBBonjourDiscovery$1$2.onNext(JBBonjourDiscovery.java:99)

The easiest way to solve this is to replace the offending line with:
nsdManager.resolveService(info, createResolveListener());

and then, instead of having a member variable resolveListener, you create a new one in the method:
private NsdManager.ResolveListener createResolveListener() {
    return new NsdManager.ResolveListener() {
        //Rest of the code here
    };
}

I would have sent this change as a PR but I am not sure if this change will impact any of the Rx stuff.

When trying to reproduce the bug, I found that it's triggered quite unreliably, which leads me to believe that this may be a race issue in NsdManager. This is the reason I originally implemented Backlog (since NsdManager can only resolve one service at a time), but supposedly isn't enough. It shouldn't be an issue to have a new instance initialized for each service, but I'll have to test this a little more thoroughly before I can say that for sure.

You're right - I also came across some instances when the crash doesn't occur. But in my experience it does occur 8 out of 10 times. The trick is just to be broadcasting two services simultaneously.
If it helps, here are the commands I used on Linux to advertise two services simultaneously.
$ avahi-publish -s "First" "_http._tcp" 12345 -v
$ avahi-publish -s "Second" "_http._tcp" 12346 -v

Equivalent commands for Mac OSX and Windows would be
$ dns-sd -R "First" _http._tcp . 12345
$ dns-sd -R "Second" _http._tcp . 12346

After running these commands, I connected my Android phone to the same WiFi router that my laptop is connected to. Ran the RxBonjour example (with the RxBonjour.startDiscovery(this, input, true) variant of course). Now I see the crash.
Devices tested:

Moto G (Android 5.0.2)
Micromax Canvas A1 (Android 5.1)
Nexus 7 2012 (Android 5.1)

This has been addressed in 0.1.5 - I haven't been able to reproduce this issue so far. Please let me know if anything comes up again!


Godot enters an infinite loop while manipulating themes

Godot version: 3.2.1

OS/device including version: Windows 10

Issue description: After creating a custom theme based on the editor theme, it is actually changing the editors theme.  When selecting a theme or closing the window in the theme section in the editor options menu, it causes an infinite loop.

Steps to reproduce:

Select a Control Node and create a new Theme
Click "Edit Theme" in the bottom panel
Click "Create from Current Editor Theme"
Change something that's noticeable, like a Button Style colour
Click "Editor" then "Editor Settings" then "Theme"
This is where it hangs.  I had a few things happen here:

Clicking the "revert to default" button and then selecting the "Default" Preset caused the infinite loop
Just selecting the "Default" Preset caused the loop
I just clicked "Theme" and went to close the window and caused the loop

The only message in the log is this (ignore the load_image error, that happens when I load the project).
The Failed method line is where the infinite loop starts.

Minimal reproduction project:

On a side note, while editing a theme and creating a gradient texture, the default width is 2048, so Godot UI resizes to compensate for this inside the current window:

ERROR: texture_get_data: Condition "!texture->active" is true. Returned: Ref<Image>()
   At: drivers/gles3/rasterizer_storage_gles3.cpp:1070
ERROR: texture_get_data: Condition "!texture->active" is true. Returned: Ref<Image>()
   At: drivers/gles3/rasterizer_storage_gles3.cpp:1070

On Linux the infinite loop causes the editor to consume Ram out of control. This randomly happens when editing a theme.

Of Note: Godot Version 3.2.2.-beta4 has yet to crash while manipulating themes.
Even if it did I am on Linux and used a temporary work around of setting
ulimit -v 2000000
in the terminal that i launched Godot from.

Can you reproduce this still in the 3.4-beta releases? This UI was heavily remade as well as the logic behind it.

@pycbouh I will have to re-check when 3.4 hits release.

I'm going to close this due to the lack of updates. The theme editor itself was changed in both 4.x and 3.x since this had been reported.


fedoracoreos: Add FetchStream API

Most callers will want this.  Additionally, if we ever add
more integrity for the stream such as GPG signing, this will
be the place to do it.
xref https://github.com/coreos/fedora-coreos-tracker/issues/774

Rebased :surfer:

Rebased :surfer: and also added error prefixing.


Meet some problem for the first time

root@vpn:~# docker run hello-world
FATA[0000] Cannot connect to the Docker daemon. Is 'docker -d' running on this host?
root@vpn:~# docker -d
INFO[0000] +job serveapi(unix:///var/run/docker.sock)
INFO[0000] Listening for HTTP on unix (/var/run/docker.sock)
INFO[0000] +job init_networkdriver()
INFO[0000] -job init_networkdriver() = OK (0)
INFO[0000] WARNING: Your kernel does not support cgroup swap limit.
INFO[0000] Loading containers: start.
INFO[0000] Loading containers: done.
INFO[0000] docker daemon: 1.5.0 a8a31ef; execdriver: native-0.2; graphdriver: aufs
INFO[0000] +job acceptconnections()
INFO[0000] -job acceptconnections() = OK (0)

@arcticlion could you add more information about the problem you are experiencing? Seems like you tried to run docker without the daemon started, then you started it and then....?

@runcom
OS is Ubuntu 14.04
root@vpn:~# uname -r
3.13.0-37-generic
First, I get the latest Docker package.
wget -qO- https://get.docker.com/ | sh

Then
root@vpn:~# ps aux | grep docker
root     16731  0.0  0.1  11756   932 pts/3    S+   19:35   0:00 grep --color=auto docker
root@vpn:~# docker run hello-world
FATA[0000] Cannot connect to the Docker daemon. Is 'docker -d' running on this host?

@arcticlion from your ps aux output docker is not running at all. Try running service docker start before trying to run hello-world

@runcom
root@vpn:~#  service docker start
docker start/running, process 17131
root@vpn:~# ps aux | grep docker
root     17131  0.6  1.6 125176  8284 ?         Important: image verification is a tech preview feature and should not be relied on to provide security.
Status: Downloaded newer image for hello-world:latest
Hello from Docker.
This message shows that your installation appears to be working correctly.

To generate this message, Docker took the following steps:
 1. The Docker client contacted the Docker daemon.
 2. The Docker daemon pulled the "hello-world" image from the Docker Hub.
    (Assuming it was not already locally available.)
 3. The Docker daemon created a new container from that image which runs the
    executable that produces the output you are currently reading.
 4. The Docker daemon streamed that output to the Docker client, which sent it
    to your terminal.

To try something more ambitious, you can run an Ubuntu container with:
 $ docker run -it ubuntu bash

For more examples and ideas, visit:
 http://docs.docker.com/userguide/
root@vpn:~# docker run -it ubuntu bash
Unable to find image 'ubuntu:latest' locally
f3c84ac3a053: Downloading 7.994 MB/197.2 MB 13m38s
a1a958a24818: Verifying Checksum
9fec74352904: Verifying Checksum
d0955f21bf24: Download complete
511136ea3c5a: Already exists

@arcticlion perfect, it's working then, even if I don't understand why the docker daemon was not started after the installation

@runcom
Is normal?
root@vpn:~# docker run -it ubuntu bash
Unable to find image 'ubuntu:latest' locally
f3c84ac3a053: Pull complete
f3c84ac3a053: Download complete
a1a958a24818: Download complete
9fec74352904: Download complete
d0955f21bf24: Download complete
511136ea3c5a: Download complete
Status: Downloaded newer image for ubuntu:latest
root@0d63996a58d8:/# docker run hello-world
bash: docker: command not found

@arcticlion yes, you are inside a container, you haven't docker installed there.

@runcom I understand what docker is little by little. I continue to read official document. Thank you for your help.

@arcticlion you might wanna close this issue then ;)


Select

Use this issue to discuss the select component in the ONS Design System

We need a select that does not have a set width. The width should be fluid based on its contents.
Similar to languages: https://census-website-respond.netlify.app/end-to-end/languages


PR not merged

https://github.com/Rakesh9100/Beautiify/pull/1321
@sanjay-kv Please check this PR and Merge !
Thank You !

@sanjay-kv please merge this !

@sanjay-kv Please Merge this PR !
Thank You !

PR is getting reviewed one by one and merged, remove the point label from here
@sanjay-kv


support hover and dragTo recorder

If we can recorder the hover and dragTo api, then the crx  must be awesome !
https://seleniumbase.io/demo_page
we can test this demo.

If it gets implemented in playwright codegen, it will be supported in playwright-crx. There's already a feature request for that:
https://github.com/microsoft/playwright/issues/19892


Add filter methods to basic timeseries plotting

Build off of existing functionality used for DB queries/filters and adapt for Pandas DataFrame.
Update existing accessor methods to accommodate user defined queries.

Closing issue for now -- stale, going to focus on #261