Quote the remaining variables

Merge with https://github.com/ChrisTitusTech/linutil/pull/442 https://github.com/ChrisTitusTech/linutil/pull/440 https://github.com/ChrisTitusTech/linutil/pull/443
Type of Change

[x] Bug fix (potential)
[x] Refactoring

Description
Quoting variables is a good practice in shell scripting / bash scripting
NOTICE: Gaming-setup was intentionally left out due to an amazing rewrite in #380 and the aur helper reverts were also left out due to another amazing rewrite in #382
Checklist

[x] My code adheres to the coding and style guidelines of the project.
[x] I have performed a self-review of my own code.
[x] My changes generate no errors/warnings/merge conflicts.

ill fix this for you Chris :heart:

nvm.. ill just close this.


Add windows specific notes, update README.md file

Fixes

Fixes #303  by @TimidRobot
Description

PR resolves problem of windows user being unable to setup docker and pipenv
Technical details

Tests

Screenshots

Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main or master).
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[x] I added or updated tests for the changes I made (if applicable).
[x] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no
visible errors.

Developer Certificate of Origin

For the purposes of this DCO, "license" is equivalent to "license or public domain dedication," and "open source license" is equivalent to "open content license or public domain dedication."

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

@TimidRobot still can't pass all checks due to pre-commit.....
Please how can I resolve this to pass all checks?

Test output showing trailing whitespace:
diff --git a/README.md b/README.md
index e4c11a9..e1f9b33 100644
--- a/README.md
+++ b/README.md
@@ -66,8 +66,8 @@ location.
 Use the following instructions to start the project with Docker compose.

 1. Ensure the [Data Repository](#data-repository), above, is in place
-2. Install Docker
-   - ([Install Docker Engine | Docker Documentation][installdocker]) for linux distributions
+2. Install Docker
+   - ([Install Docker Engine | Docker Documentation][installdocker]) for linux distributions
    - ([Install Docker Engine | Docker Documentation][installdockerwindows]) for windows distributions
    - ([Install Docker Engine | Docker Documentation][installdockermacOS]) for macOs distributions
 3. Ensure you are the top level of directory where you cloned this repository (where `manage.py` is)
@@ -105,7 +105,7 @@ Use the following instructions to start the project with Docker compose.
 [installdocker]: https://docs.docker.com/engine/install/
 [repodata]:https://github.com/creativecommons/cc-legal-tools-data
 [installdockerwindows]:https://docs.docker.com/desktop/install/windows-install/
-[installdockermacOS]:https://docs.docker.com/desktop/install/mac-install/
+[installdockermacOS]:https://docs.docker.com/desktop/install/mac-install/

 ### Manual Setup
@@ -126,7 +126,7 @@ Use the following instructions to start the project with Docker compose.
         ```
       - Windows: [install Python][python-windows] and then use `pip` to install `pipenv`:
         ```
-        pip install pipenv
+        pip install pipenv
         ```
    3. Install Python environment and modules via pipenv to create a
       virtualenv
@@ -174,7 +174,7 @@ Use the following instructions to start the project with Docker compose.
     ```
    - Any changes made to Python will be detected and rebuilt transparently as
      long as the development server is running.
-
+
 [python-windows]:https://www.pythontutorial.net/getting-started/install-python/

Thanks for this it's helped me alot🙏

@TimidRobot I've completed all task please review once more:)

@TimidRobot please review:)

@TimidRobot I'm sure the changes made are okay... Please review:)


Event Forward is not  happening for Chassis and Managers

After subscribing to Chassis and Manager Origin Resources, Chassis and Managers events are not getting forwarded to the destination

closing the issue referring to #77


Changing username could result in failed key lookup

The key used to encrypt the password uses the username as an alias. If the username changes, the alias we will be looking for will be different, and we won't find the key.

Fixed in 7d52c5527767efb7236eedb3582c1a571ace1644


Modified few files for Bootstrap UI

Something went wrong here - you have 3 commits and no net change to any files.

I'd pulled the code from original of the fork (bootstrap of pfsense). Since you'd already modified this file (load_balancer_virtual_server.php), my commit got reverted.

Hi Phil,
I've modified some more files for the bootstrap UI. Please check them.

I am not the one who knows anything about bootstrap!
I will suggest that it will be more confusing for merging having those unwanted commit and reverse things in here. You would be better off to get your bootstrap branch in your fork back to being just a mirror of pfsense:bootstrap. Then making a clean branch off that with just the changes you propose.

I believe it may not be a problem, as the net change has only been in those three files that I'd wished to modify.
I've been syncing my fork branch as mentioned in this link, https://help.github.com/articles/syncing-a-fork/ .
If you still feel that it might be confusing, how shall I mirror my branch? Can you please provide some help links.

Hi Steve,
I'm done with the changes you'd mentioned. But this pull request has been closed! :|

Thank you.
I believe that is because moved the Bootstrap work into the main pfSense master branch yesterday. Your work has not been lost. I will check on it in just a minute.
Steve
Stephen Beaver | +1<PHONE_NUMBER>;239 |<EMAIL_ADDRESS>| www.netgate.com

On Sep 15, 2015, at 8:37 AM, itshari<EMAIL_ADDRESS>wrote:
Hi Steve,
I'm done with the changes you'd mentioned. But this pull request has been closed! :|
—
Reply to this email directly or view it on GitHub https://github.com/pfsense/pfsense/pull/1891#issuecomment-140374158.

Okay. Given you a new request #1896 as this one #1891 doesn't show my recent changes.

Thanks for the changes.
The Bootstrap branch has now been deprecated and all of the Bootstrap work moved into the master. We were able to merge your changes into that without any issues.
Steve
Stephen Beaver | +1<PHONE_NUMBER>;239 |<EMAIL_ADDRESS>| www.netgate.com

On Sep 15, 2015, at 11:49 AM, itshari<EMAIL_ADDRESS>wrote:
Okay. Given you a new request #1896 https://github.com/pfsense/pfsense/pull/1896 as this one #1891 https://github.com/pfsense/pfsense/pull/1891 doesn't show my recent changes.
—
Reply to this email directly or view it on GitHub https://github.com/pfsense/pfsense/pull/1891#issuecomment-140438324.

Okay! :)


vscode-github.login cannot allow

System: MacOS
version: 1.50
vscode-github.com  password is not correct
i've tried thousands of times, but not works

This is a known issue: https://github.com/microsoft/vscode/issues/107480

Merging to https://github.com/microsoft/vscode/issues/108342#issuecomment-706340396, please check that comment for background.


Attack created in character sheet not updating data

Build 2.1.0
When an Attack is created directly on the character sheet, any changes to the blank Attack entry (damage dice, Uses Strength check box, etc) do not get updated.

should be fixed as of v2.1.2-alpha


Problem with running scripts

Hi.
I am trying to run tests but I have got such errors:
npm ERR! code ELIFECYCLE
npm ERR! errno 2
npm ERR<EMAIL_ADDRESS>tdd: `rm -rf dist && tsc && mocha -t 10000 dist/tests`
npm ERR! Exit status 2
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>tdd script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:

I searched google, but I did not find a solution. Any Idea? :)
P.S. Thanks for nice framework

That description doesn’t say much. Is there any other message/log?

No. I write simple test to check out if everything is ok. Chrome is launched, and correct page is loaded but there is no any actions like send keys or clicks. It looks like that this is something with Webdriver.

Full logs:
0 info it worked if it ends with ok 1 verbose cli [<EMAIL_ADDRESS>1 verbose cli   '/usr/local/bin/npm', 1 verbose cli   'run', 1 verbose cli   'tdd' ] 2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'pretdd', 'tdd', 'posttdd' ] 5 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>6 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>7 verbose lifecycle<EMAIL_ADDRESS>unsafe-perm in lifecycle true 8 verbose lifecycle<EMAIL_ADDRESS>PATH: /usr/local/lib/node_modules/npm/node_modules/npm-lifecycle/node-gyp-bin:/Users/flywojt/Desktop/typescript-selenium-example-master/node_modules/.bin:/Library/Frameworks/Python.framework/Versions/2.7/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/share/dotnet:/Library/Frameworks/Mono.framework/Versions/Current/Commands:/Library/Frameworks/Python.framework/Versions/2.7/bin 9 verbose lifecycle<EMAIL_ADDRESS>CWD: /Users/flywojt/Desktop/typescript-selenium-example-master 10 silly lifecycle<EMAIL_ADDRESS>Args: [ '-c', 'rm -rf dist && tsc && mocha -t 20000 dist/tests' ] 11 silly lifecycle<EMAIL_ADDRESS>Returned: code: 1  signal: null 12 info lifecycle<EMAIL_ADDRESS>Failed to exec tdd script 13 verbose stack Error<EMAIL_ADDRESS>tdd: rm -rf dist && tsc && mocha -t 20000 dist/tests13 verbose stack Exit status 1 13 verbose stack     at EventEmitter.<anonymous> (/usr/local/lib/node_modules/npm/node_modules/npm-lifecycle/index.js:285:16) 13 verbose stack     at EventEmitter.emit (events.js:160:13) 13 verbose stack     at ChildProcess.<anonymous> (/usr/local/lib/node_modules/npm/node_modules/npm-lifecycle/lib/spawn.js:55:14) 13 verbose stack     at ChildProcess.emit (events.js:160:13) 13 verbose stack     at maybeClose (internal/child_process.js:943:16) 13 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:220:5) 14 verbose pkgid<EMAIL_ADDRESS>15 verbose cwd /Users/flywojt/Desktop/typescript-selenium-example-master 16 verbose Darwin 17.3.0 17 verbose argv<EMAIL_ADDRESS>"/usr/local/bin/npm" "run" "tdd" 18 verbose node v9.4.0 19 verbose npm  v5.6.0 20 error code ELIFECYCLE 21 error errno 1 22 error<EMAIL_ADDRESS>tdd:rm -rf dist && tsc && mocha -t 20000 dist/tests 22 error Exit status 1 23 error Failed at the<EMAIL_ADDRESS>tdd script. 23 error This is probably not a problem with npm. There is likely additional logging output above. 24 verbose exit [ 1, true ]

It seems that using yarn resolve problems but I have got another issue:
Error: Timeout of 20000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.


Powershell Commands that fail do not fail Cake Task

Hi, thanks for making such a useful addon for Cake!
If a simple command is run, and it fails, the error is not recognized as an error by the Cake Task:
eg, call "Import-Module" with no parameters will throw an exception in powershell:
C:\> Import-Module ""
Import-Module : Cannot bind argument to parameter 'Name' because it is an empty string.
At line:1 char:15
+ Import-Module ""
+               ~~
    + CategoryInfo          : InvalidData: (:) [Import-Module], ParameterBindingValidationException
    + FullyQualifiedErrorId : ParameterArgumentValidationErrorEmptyStringNotAllowed,Microsoft.PowerShell.Commands.ImportModuleCommand

Cake setup:
Task("continue-after-test123")
	.IsDependentOn("test123")
	.Does(() =>
	{
		Verbose("I should not be here. previous method failed.")
	});

Task("test123")
	.Does(() =>
	{
		StartPowershellScript("Import-Module",
			new PowershellSettings()
			{
			 	FormatOutput = true,
			 	LogOutput = true
			}.WithArguments(args =>
			{
			 	args.Append("ErrorAction", "Stop");
				args.Append("-Verbose");
			}));
	});

Output:
C:\> & ".\build.ps1" -Target continue-after-test123
Preparing to run build script...
Running build script...
Analyzing build script...
Processing build script...
Installing addins...
Compiling build script...

========================================
test123
========================================
Executing task: test123
Executing: Import-Module
Import-Module : Cannot process command because of one or more missing mandatory parameters: Name.
At line:1 char:1
+ Import-Module
+ ~~~~~~~~~~~~~
    + CategoryInfo          : InvalidArgument: (:) [Import-Module], ParameterBindingException
    + FullyQualifiedErrorId : MissingMandatoryParameter,Microsoft.PowerShell.Commands.ImportModuleCommand

Finished executing task: test123

========================================
continue-after-test123
========================================
Executing task: continue-after-test123
I should not be here. previous method failed.
Finished executing task: continue-after-test123

Task                          Duration
--------------------------------------------------
test123                       00:00:00.6414215
continue-after-test123        00:00:00.0027986
--------------------------------------------------
Total:                        00:00:00.6442201

Even adding ErrorAction Stop will not aid:
C:\> & ".\build.ps1" -Target continue-after-test123
Preparing to run build script...
Running build script...
Analyzing build script...
Processing build script...
Installing addins...
Compiling build script...

========================================
test123
========================================
Executing task: test123
Executing: Import-Module -ErrorAction Stop -Verbose
Import-Module : Cannot process command because of one or more missing mandatory parameters: Name.
At line:1 char:1
+ Import-Module -ErrorAction Stop -Verbose
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidArgument: (:) [Import-Module], ParameterBindingException
    + FullyQualifiedErrorId : MissingMandatoryParameter,Microsoft.PowerShell.Commands.ImportModuleCommand

Finished executing task: test123

========================================
continue-after-test123
========================================
Executing task: continue-after-test123
I should not be here. previous method failed.
Finished executing task: continue-after-test123

Task                          Duration
--------------------------------------------------
test123                       00:00:00.6469561
continue-after-test123        00:00:00.0022057
--------------------------------------------------
Total:                        00:00:00.6491618

but if you use 'throw' directly from a powershell script it will work as expected:
cake:
Task("continue-after-newtest")
	.IsDependentOn("newtest")
	.Does(() =>
	{
		Verbose("I should not be here. previous method failed.");
	});

Task("newtest")
	.Does(() =>
	{
		StartPowershellScript("throw 'This failed!'",
			new PowershellSettings()
			{
				FormatOutput = true,
        		LogOutput = true
			});
	});

output:
C:\> & ".\build.ps1" -Target continue-after-newtest
Preparing to run build script...
Running build script...
Analyzing build script...
Processing build script...
Installing addins...
Compiling build script...

========================================
newtest
========================================
Executing task: newtest
Executing: throw 'This failed!'
An error occurred when executing task 'newtest'.
Error: Failed to Execute Powershell Script: System.Management.Automation.RuntimeException: This failed! ---> System.Management.Automation.RuntimeException: This failed!
   --- End of inner exception stack trace ---
   at System.Management.Automation.ExceptionHandlingOps.CheckActionPreference(FunctionContext funcContext, Exception exception)
   at System.Management.Automation.Interpreter.ActionCallInstruction`2.Run(InterpretedFrame frame)
   at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame)
   at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame)
   at System.Management.Automation.Interpreter.Interpreter.Run(InterpretedFrame frame)
   at System.Management.Automation.Interpreter.LightLambda.RunVoid1[T0](T0 arg0)
   at System.Management.Automation.DlrScriptCommandProcessor.RunClause(Action`1 clause, Object dollarUnderbar, Object inputToProcess)
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Management.Automation.Internal.PipelineProcessor.SynchronousExecuteEnumerate(Object input)
   at System.Management.Automation.Runspaces.LocalPipeline.InvokeHelper()
   at System.Management.Automation.Runspaces.LocalPipeline.InvokeThreadProc()

But I should not have to have to wrap all of my calls in a try-catch-throw-rethrow:
string script = @"
try
{
	Import-Module -Verbose -ErrorAction Stop
} catch { throw $_ }
";

StartPowershellScript(script,
	new PowershellSettings()
	{
		FormatOutput = true,
		LogOutput = true
	});

Thanks!

Fully agree with you @tcables. Spent quite a time for finding out how I can handle errors and created  #38 to help others.
But, I want actually create an option in PowershellSettings for instance ThrowIfErrorOccured. And method Invoke will throw exception if it's true.
Would @SharpeRAD  accept such PR?

Throwing errors should be the default action, so no need in an extra setting.
Submit a PR that effectively duplicates this (L303 - L310) for "ErrorRecord" as well as Exceptions.

So, it means that there is a bug. Because
 var exceptions = _pipelineResults.Where(result => result.BaseObject is Exception)
                    .Select(e => e.BaseObject as Exception).ToArray();

Will not work Because in case of errors in scripts result.BaseObject will be ErrorRecord not the Exception type. I've checked that yesterday.
So we either should introduce new property or it will break builds someone who do not care about errors (not exceptions).

This thread is talking about being consistent with Powershell and throwing exceptions for errors. I pointed you to those lines as that's the area you would need to change.
Can you name an instance where you would not throw on an error? EG: Cake only has a task method "OnError" it doesn't have a method "OnException" 😄

Add a setting to be safe, as I shouldn't base everything on my own preference.

@SharpeRAD Actually for me always throwing exceptions is also preferred variant.  But I am afraid that it will break builds for guys for whom errors is not critical right now
So we have 3 variants:

Always throw exceptions on errors
Throw exception on errors if ThrowIfErrorOccured option is set.
Throw by default but made it optional via option `SwallowErrorsInsdeadThrowing'

(Names it is subject for discussion)
Can you probably make some mention in gitter chat to make a quick poll here via smiles.
Heart - variant 1
Laugh - variant 2
Hooray - variant 3

Resolved in #46


Question about loading semantic annotation in HM3D Dataset

Habitat-Sim version
vx.x.x
Habitat is under active development, and we advise users to restrict themselves to stable releases. Are you using the latest release version of Habitat-Sim? Your question may already be addressed in the latest version. We may also not be able to help with problems in earlier versions because they sometimes lack the more verbose logging needed for debugging.
Main branch contains 'bleeding edge' code and should be used at your own risk.
Docs and Tutorials
Did you read the docs? https://aihabitat.org/docs/habitat-sim/
yes
Did you check out the tutorials? https://aihabitat.org/tutorial/2020/
yes
Perhaps your question is answered there. If not, carry on!
❓ Questions and Help
I wonder is there any problem loading semantic annotation this way.
According to the example of MP3D dataset
https://colab.research.google.com/github/facebookresearch/habitat-sim/blob/v0.1.6/examples/tutorials/colabs/ECCV_2020_Navigation.ipynb
I loaded config file habitat-lab/data/scene_datasets/hm3d/hm3d_annotated_basis.scene_dataset_config.json and chose test scene habitat-lab/data/scene_datasets/hm3d/minival/00800-TEEsavR23oF/TEEsavR23oF.basis.glb
The output of function print_scene_recur returns
House has 0 levels, 14 regions and 661 objects House center:[0. 0. 0.] dims:[-inf -inf -inf]
and all of the object's names were Unknown_{int}. It assumed scene annotation is not loaded well.
The tutorial MP3D example was worked well.
Is there any point I missed?
-- here is full code of my current state

import habitat_sim
backend_cfg = habitat_sim.SimulatorConfiguration()
backend_cfg.scene_id = "/data/scene_datasets/hm3d/minival/00800-TEEsavR23oF/TEEsavR23oF.basis.glb"
backend_cfg.scene_dataset_config_file = "/data/scene_datasets/hm3d/minival/hm3d_annotated_minival_basis.scene_dataset_config.json"

sem_cfg = habitat_sim.CameraSensorSpec()
sem_cfg.uuid = "semantic"
sem_cfg.sensor_type = habitat_sim.SensorType.SEMANTIC

agent_cfg = habitat_sim.agent.AgentConfiguration()
agent_cfg.action_space = {
    "move_forward": habitat_sim.agent.ActionSpec(
        "move_forward", habitat_sim.agent.ActuationSpec(amount=0.25)
    ),
    "turn_left": habitat_sim.agent.ActionSpec(
        "turn_left", habitat_sim.agent.ActuationSpec(amount=30.0)
    ),
    "turn_right": habitat_sim.agent.ActionSpec(
        "turn_right", habitat_sim.agent.ActuationSpec(amount=30.0)
    ),
}
agent_cfg.sensor_specifications = [sem_cfg]

sim_cfg = habitat_sim.Configuration(backend_cfg, [agent_cfg])
sim = habitat_sim.Simulator(sim_cfg)

def print_scene_recur(scene, limit_output=10):
    print(f"House has {len(scene.levels)} levels, {len(scene.regions)} regions and {len(scene.objects)} objects")
    print(f"House center:{scene.aabb.center} dims:{scene.aabb.sizes}")
    count = 0
    for level in scene.levels:
        print(
            f"Level id:{level.id}, center:{level.aabb.center},"
            f" dims:{level.aabb.sizes}"
        )
        for region in level.regions:
            print(
                f"Region id:{region.id}, category:{region.category.name()},"
                f" center:{region.aabb.center}, dims:{region.aabb.sizes}"
            )
            for obj in region.objects:
                print(
                    f"Object id:{obj.id}, category:{obj.category.name()},"
                    f" center:{obj.aabb.center}, dims:{obj.aabb.sizes}"
                )
                count += 1
                if count >= limit_output:
                    return None
scene = sim.semantic_scene
print_scene_recur(scene)

Hey @jemo-mh
This is working for me locally.
I am seeing a large number of unknown colorings for this scene, but I believe they are isolated vertex groups missed by the paint.
I've attached a file TEEsavR23oF.basis_CC_report.csv which shows the semantic report of the scene.
To get this I ran the viewer.cpp application like:
./build/viewer --dataset data/scene_datasets/hm3d/minival/hm3d_annotated_minival_basis.scene_dataset_config.json -- data/scene_datasets/hm3d/minival/00800-TEEsavR23oF/TEEsavR23oF.basis.glb

and then pressed the 'j' key to generate the report. You should be able to repeat this locally and get the same result.

Same problem for me. Pls post here if any luck.

Issue: Some Scenes in HM3D Cannot Export CSV Files
I am attempting to export CSV files using the following method, but strangely, while some scenes in the HM3D dataset can export CSV, others do not respond when pressing 'J'. "Are all scenes supported for exporting to CSV?" or "is it that operations are not properly executed that would cause the CSV to not export?"
Problematic Scenes:

Nfvxx8J5NCo
VBzV5z6i1WS
mv2HUxq3B53
QaLdnwvtxbs
BAbdmeyTvMZ
ziup5kvtCCR

Command Used:
./build/viewer -- /data/hm3d_scenedata/00813-svBbv1Pavdk/svBbv1Pavdk.basis.glb
Then I press 'J' to trigger the CSV export.
Terminal Output:
![image](https://github.com/user-attachments/assets/525aaad9-90e8-4724-87b7-863f02b54e11)


Readme improvement

Why
README to more clearly.
Summary

Add Node.js version notation

based on package.json

Add link to default config file

clearfy config list

Coverage remained the same at 97.933% when pulling 8df514c81ac1fa581d33b79b6f400121391c3f97 on kubosho:readme-improvement into d3d392ad8222b140f944746e5f7b3f9afc4902cb on t32k:master.


Standalone variable editor

Hello,
I'm looking for a way to use variable editor as a standalone viewer (or editor) for Numpy Arrays and Pandas DataFrame.
Like https://github.com/wavexx/gtabview
Is there a way to do it ?
Kind regards
See also: https://github.com/wavexx/gtabview/issues/7

Not at the moment, sorry.

I think this has been done through #3873 from @dalthviz

Quite right @Prikers!


DatePicker A11y: All content must reside within a WAI-ARIA landmark or labelled region role

When using the DatePicker, DAP reports the violation
"All content must reside within a WAI-ARIA landmark or labelled region role." It associates this violation with the first input element within the calendar, the numInput for the current year.
Looking in the DOM in browser tools, the application code puts a DatePicker within normal content inside a div having role="region", but the elements that make up the calendar appear within a div having class "flatpickr-calendar" immediately below the body, with no labelled role.
Browser: Chrome 73.0.3683.86.
Carbon versions: carbon-components-react 6.71.0 and carbon-components 9.62.3
Offering/product: IBM MDM Express.

It looks like stale bot is threatening to close this issue because of inactivity. We're still interested in a fix. What would need need to happen for that to come about?
Thanks!

Please create a reduced test case in CodeSandbox
https://codesandbox.io/s/github/IBM/carbon-components-react/tree/master/examples/codesandbox


App crashes when changing windows size

When the app is opened in a mobile layout and the window is resized to a larger window the app crashes and the entire browser window becomes unresponsive.
This does not happen everytime, unfortunately.
This happens in Fireforx and not in Chrome.

This is in Firefox on MacOS


After all the hustle and bustle, one gets hungry and just needs something to eat. In Pizza Paradise, there is a wheel of fortune in the middle of the room that you can spin. Each eighth of the wheel of fortune has a slice of pizza with different toppings. But if the wheel of fortune points to the one and only poisonous pizza, the game unfortunately ends.
Possible pizza slices:

Margherita
Salami
Tuna
Broccoli
Salmon
Spinach
Frutti di Mare
Poisonous Pizza

If you have any more ideas to add let me know.

Please grant me permission to assign myself to issues and branches in the ZDD Adventure repository. Thank you!
@florian-huber
@l4nz8

Hey, thank you for assigning yourself to the issue! How's the progress with this room? Do you have any questions so far?


Refactor compose banners CSS to avoid code duplication.

This issue serves as a follow-up to #25551.
Regarding the previous discussion on issue #25580, specifically this comment, our objective is to eliminate any duplicated CSS introduced in that context.

Hi @shameondev, could you mark this issue as complete? I forgot to do that in the PR. Thanks.

Hi @alya, could you mark this issue as complete? I completely forgot to do that in the PR I made in #25693. Thank you!


[WFCORE-3415] Take 2

Reverts and replaces the https://issues.jboss.org/browse/WFCORE-3415 work done in #2930.
That work was incorrectly counting the mbeans because it would count some bogus resources that are in the resource tree but for which there is no type registration. An EE test was detecting this miscount and failing. It would be good to get rid of those bogus resources but that is not simple at all and could be prone to regression.
This PR has the same conceptual solution as the previous one: don't count both JMX domains and drop the RBAC check when counting. It has to iterate the resource tree though. It solves the problem from the other PR by checking each address for validity when iterating.
Since this was reasonably fast but still quite a bit slower than the old PR, I spent quite a bit of effort in optimizing the conversion of PathAddress to ObjectName, particularly when many conversions are expected, i.e. when iterating the whole tree:

Pattern instances that would get repeatedly created for each String.replaceAll call are instead cached and reused.
String.contains calls are dropped in favor of just using the cached Pattern objects.
A contextual object is added that the createObjectName method can use to store escaped/quoted text across different invocations. The methods that iterate the tree pass in one of these, so a string like "subsystem" doesn't need to be escaped over and over and over.

Then, as I wrote this description, I realized that in the getMBeanCount case there is no reason to convert the PathAddress to an ObjectName at all! The code for this use case doesn't use the ObjectName except checking for null. So any old object name can be returned. The 3rd from last commit does that. With that the cost of getMBeanCount() on standalone-full-ha.xml once the jvm optimizes is 3-4ms, just 1 or 2 ms higher than the old PR. It's 30ms on the first invocation though, which is still quite a bit higher than the old PR but a massive improvement over the pre-WFCORE-3415 code.
The last couple commits are some further improvements and cleanup.

Full integration - Windows Build 4639 outcome was FAILURE using a merge of 8cb79535343f9d234b25ac8dab45e080614ca4db
Summary: Tests failed: 1 (1 new), passed: 4175, ignored: 99 Build time: 02:14:26
Failed tests
org.jboss.as.test.clustering.cluster.singleton.SingletonPartitionTestCase(SYNC-tcp).testSingletonService: 	at org.jboss.as.test.clustering.cluster.singleton.SingletonPartitionTestCase.checkSingletonNode(SingletonPartitionTestCase.java:195)
	at org.jboss.as.test.clustering.cluster.singleton.SingletonPartitionTestCase.testSingletonService(SingletonPartitionTestCase.java:167)
------- Stdout: -------
00:51:13,883 INFO  [org.jboss.as.repository] (management-handler-thread - 4) WFLYDR0001: Content added at location C:\BuildAgent\work\5768cf3a0bee5b47\full\testsuite\integration\clustering\target\wildfly-SYNC-tcp-0\standalone\data\content\c2\1d8e4226c55faa93e603b62051d2b8420cd24c\content
00:51:13,885 INFO  [org.jboss.as.server.deployment] (MSC service thread 1-3) WFLYSRV0027: Starting deployment of "SingletonPartitionTestCase.war" (runtime-name: "SingletonPartitionTestCase.war")
00:51:13,989 WARN  [org.jboss.as.dependency.private] (MSC service thread 1-4) WFLYSRV0018: Deployment "deployment.SingletonPartitionTestCase.war" is using a private module ("org.jboss.as.clustering.common") which may be changed or removed in future versions without notice.
00:51:13,989 WARN  [org.jboss.as.dependency.private] (MSC service thread 1-4) WFLYSRV0018: Deployment "deployment.SingletonPartitionTestCase.war" is using a private module ("org.jboss.as.controller") which may be changed or removed in future versions without notice.
00:51:13,989 WARN  [org.jboss.as.dependency.private] (MSC service thread 1-4) WFLYSRV0018: Deployment "deployment.SingletonPartitionTestCase.war" is using a private module ("org.jboss.as.server") which may be changed or removed in future versions without notice.
00:51:13,989 WARN  [org.jboss.as.dependency.private] (MSC service thread 1-4) WFLYSRV0018: Deployment "deployment.SingletonPartitionTestCase.war" is using a private module ("org.infinispan") which may be changed or removed in future versions without notice.
00:51:13,989 WARN  [org.jboss.as.dependency.private] (MSC service thread 1-4) WFLYSRV0018: Deployment "deployment.SingletonPartitionTestCase.war" is using a private module ("org.wildfly.clustering.infinispan.spi") which may be changed or removed in future versions without notice.
00:51:14,022 INFO  [org.jboss.weld.deployer] (MSC service thread 1-4) WFLYWELD0003: Processing weld deployment SingletonPartitionTestCase.war
00:51:14,065 INFO  [org.hibernate.validator.internal.util.Version] (MSC service thread 1-4) HV000001: Hibernate Validator 5.3.5.Final
00:51:14,146 INFO  [org.jboss.as.ejb3.deployment] (MSC service thread 1-4) WFLYEJB0473: JNDI bindings for session bean named 'TopologyChangeListenerBean' in deployment unit 'deployment "SingletonPartitionTestCase.war"' are as follows:

	java:global/SingletonPartitionTestCase/TopologyChangeListenerBean!org.jboss.as.test.clustering.TopologyChangeListener
	java:app/SingletonPartitionTestCase/TopologyChangeListenerBean!org.jboss.as.test.clustering.TopologyChangeListener
	java:module/TopologyChangeListenerBean!org.jboss.as.test.clustering.TopologyChangeListener
	java:jboss/exported/SingletonPartitionTestCase/TopologyChangeListenerBean!org.jboss.as.test.clustering.TopologyChangeListener
	java:global/SingletonPartitionTestCase/TopologyChangeListenerBean
	java:app/SingletonPartitionTestCase/TopologyChangeListenerBean
	java:module/TopologyChangeListenerBean

00:51:14,196 INFO  [org.jboss.as.clustering.infinispan] (ServerService Thread Pool -- 3) WFLYCLINF0002: Started default cache from server container
00:51:14,282 INFO  [org.jboss.weld.Version] (MSC service thread 1-4) WELD-000900: 3.0.2 (Final)
00:51:14,318 INFO  [org.wildfly.clustering.server] (DistributedSingletonService - 1) WFLYCLSV0003: node-0 elected as the singleton provider of the jboss.test2.service.default service
00:51:14,319 INFO  [org.wildfly.clustering.server] (DistributedSingletonService - 1) WFLYCLSV0003: node-0 elected as the singleton provider of the jboss.test1.service.default service
00:51:14,322 INFO  [org.wildfly.clustering.server] (DistributedSingletonService - 1) WFLYCLSV0001: This node will now operate as the singleton provider of the jboss.test1.service.default service
00:51:14,322 INFO  [org.wildfly.clustering.server] (DistributedSingletonService - 1) WFLYCLSV0001: This node will now operate as the singleton provider of the jboss.test2.service.default service
00:51:14,660 WARN  [org.jboss.weld.Bootstrap] (MSC service thread 1-3) WELD-000146: BeforeBeanDiscovery.addAnnotatedType(AnnotatedType<?>) used for class com.sun.faces.flow.FlowDiscoveryCDIHelper is deprecated from CDI 1.1!
00:51:14,681 WARN  [org.jboss.weld.Bootstrap] (MSC service thread 1-3) WELD-000146: BeforeBeanDiscovery.addAnnotatedType(AnnotatedType<?>) used for class com.arjuna.ats.jta.cdi.transactional.TransactionalInterceptorMandatory is deprecated from CDI 1.1!
00:51:14,697 WARN  [org.jboss.weld.Bootstrap] (MSC service thread 1-3) WELD-000146: BeforeBeanDiscovery.addAnnotatedType(AnnotatedType<?>) used for class com.arjuna.ats.jta.cdi.transactional.TransactionalInterceptorNever is deprecated from CDI 1.1!
00:51:14,700 WARN  [org.jboss.weld.Bootstrap] (MSC service thread 1-3) WELD-000146: BeforeBeanDiscovery.addAnnotatedType(AnnotatedType<?>) used for class com.arjuna.ats.jta.cdi.transactional.TransactionalInterceptorNotSupported is deprecated from CDI 1.1!
00:51:14,702 WARN  [org.jboss.weld.Bootstrap] (MSC service thread 1-3) WELD-000146: BeforeBeanDiscovery.addAnnotatedType(AnnotatedType<?>) used for class com.arjuna.ats.jta.cdi.transactional.TransactionalInterceptorRequired is deprecated from CDI 1.1!
00:51:14,704 WARN  [org.jboss.weld.Bootstrap] (MSC service thread 1-3) WELD-000146: BeforeBeanDiscovery.addAnnotatedType(AnnotatedType<?>) used for class com.arjuna.ats.jta.cdi.transactional.TransactionalInterceptorRequiresNew is deprecated from CDI 1.1!
00:51:14,707 WARN  [org.jboss.weld.Bootstrap] (MSC service thread 1-3) WELD-000146: BeforeBeanDiscovery.addAnnotatedType(AnnotatedType<?>) used for class com.arjuna.ats.jta.cdi.transactional.TransactionalInterceptorSupports is deprecated from CDI 1.1!
00:51:14,713 WARN  [org.jboss.weld.Bootstrap] (MSC service thread 1-3) WELD-000146: BeforeBeanDiscovery.addAnnotatedType(AnnotatedType<?>) used for class org.jberet.creation.BatchBeanProducer is deprecated from CDI 1.1!
00:51:14,735 WARN  [org.jboss.weld.Bootstrap] (MSC service thread 1-3) WELD-000146: BeforeBeanDiscovery.addAnnotatedType(AnnotatedType<?>) used for class org.wildfly.extension.messaging.activemq.deployment.injection.RequestedJMSContext is deprecated from CDI 1.1!
00:51:14,739 WARN  [org.jboss.weld.Bootstrap] (MSC service thread 1-3) WELD-000146: BeforeBeanDiscovery.addAnnotatedType(AnnotatedType<?>) used for class org.wildfly.extension.messaging.activemq.deployment.injection.TransactedJMSContext is deprecated from CDI 1.1!
00:51:14,746 WARN  [org.jboss.weld.Bootstrap] (MSC service thread 1-3) WELD-000146: BeforeBeanDiscovery.addAnnotatedType(AnnotatedType<?>) used for class org.wildfly.extension.messaging.activemq.deployment.injection.InjectedJMSContext is deprecated from CDI 1.1!
00:51:14,766 WARN  [org.jboss.weld.Bootstrap] (MSC service thread 1-3) WELD-000146: BeforeBeanDiscovery.addAnnotatedType(AnnotatedType<?>) used for class org.hibernate.validator.internal.cdi.interceptor.ValidationInterceptor is deprecated from CDI 1.1!
00:51:15,364 INFO  [org.wildfly.extension.undertow] (ServerService Thread Pool -- 3) WFLYUT0021: Registered web context: '/SingletonPartitionTestCase' for server 'default-server'
00:51:15,384 INFO  [org.jboss.as.server] (management-handler-thread - 4) WFLYSRV0010: Deployed "SingletonPartitionTestCase.war" (runtime-name : "SingletonPartitionTestCase.war")
00:51:15,420 INFO  [org.jboss.as.repository] (management-handler-thread - 1) WFLYDR0001: Content added at location C:\BuildAgent\work\5768cf3a0bee5b47\full\testsuite\integration\clustering\target\wildfly-SYNC-tcp-1\standalone\data\content\b9\2564869d18077ecdb79b569673cb9b3cde3a16\content
00:51:15,424 INFO  [org.jboss.as.server.deployment] (MSC service thread 1-4) WFLYSRV0027: Starting deployment of "SingletonPartitionTestCase.war" (runtime-name: "SingletonPartitionTestCase.war")
00:51:15,598 WARN  [org.jboss.as.dependency.private] (MSC service thread 1-3) WFLYSRV0018: Deployment "deployment.SingletonPartitionTestCase.war" is using a private module ("org.jboss.as.clustering.common") which may be changed or removed in future versions without notice.
00:51:15,598 WARN  [org.jboss.as.dependency.private] (MSC service thread 1-3) WFLYSRV0018: Deployment "deployment.SingletonPartitionTestCase.war" is using a private module ("org.jboss.as.controller") which may be changed or removed in future versions without notice.
00:51:15,598 WARN  [org.jboss.as.dependency.private] (MSC service thread 1-3) WFLYSRV0018: Deployment "deployment.SingletonPartitionTestCase.war" is using a private module ("org.jboss.as.server") which may be changed or removed in future versions without notice.
00:51:15,598 WARN  [org.jboss.as.dependency.private] (MSC service thread 1-3) WFLYSRV0018: Deployment "deployment.SingletonPartitionTestCase.war" is using a private module ("org.infinispan") which may be changed or removed in future versions without notice.
00:51:15,598 WARN  [org.jboss.as.dependency.private] (MSC service thread 1-3) WFLYSRV0018: Deployment "deployment.SingletonPartitionTestCase.war" is using a private module ("org.wildfly.clustering.infinispan.spi") which may be changed or removed in future versions without notice.
00:51:15,646 INFO  [org.jboss.weld.deployer] (MSC service thread 1-3) WFLYWELD0003: Processing weld deployment SingletonPartitionTestCase.war
00:51:15,697 INFO  [org.hibernate.validator.internal.util.Version] (MSC service thread 1-3) HV000001: Hibernate Validator 5.3.5.Final


Fast click global continuation searching

This is an idea that I think ripgrep now makes possible due to the speed.
Use cases this idea would improve:

Continuous searching finding related symbols:

Search for symbol
See another interesting symbol on a matching result line
Type that symbol into search and search again
Repeat N number times until finding your target information.

Variation of above:

Search for symbol
Result filename is of interest
Search again entering the filename
Repeat any combination of the above until target information found

Proposed improvement for above use cases:

While holding a modifier key in the search results, clicking the mouse over a word will automatically initiate another search using that word.

The user can click on the filename and it will search for that excluding the extension.
The user can click any word in the result text and initiate another immediate search

Also would be useful to be able to do this inside a document

inside a document

Could go with https://github.com/Microsoft/vscode/pull/24015#issuecomment-292196498


Profile: Improve display of ELL status and WIDA level

Who is this PR for?
educators
What problem does this PR fix?
ELL teachers at SHS asked to revise the way ELL and FLEP data is represented to clarify for classroom teachers in particular.
What does this PR do?
It revises the single link about language status and factors that out into <LanguageStatusLink />.  It revises the designations to be either "English Learner", "Former English Learner" or "Fluent English".  For English learners, it removes the text description of the labels in favor of the numeric level only.  For students in Somerville with their program set to "SEIP" or "Wavered SEIP" these are blacklisted and not shown in Insights, based on feedback from SHS ELL teachers noted in comments.
There are related changes made to the <AccessPanel /> component, and several different kinds of tests and refactoring to improve how this data is accessed in language.js.
Screenshot (if adding a client-side feature)
Example from story

Story and test showing combinations across districts and values

Example modal

Checklists
Which features or pages does this PR touch?

[x] Student Profile
[x] Levels page
[x] Included specs for changes
[x] Improved specs for existing code in need of better test coverage

selfie


Need to update your module ASAP

This module is using axios ^0.19.2 which is marked as high security issue (Server-Side Request Forgery).
Please update this module as soon as possible
Sendgrid used by thousand of website & all those sites are effected.
Axios has been released ^0.21.1 with the patch 10 days ago.

@krishnaTORQUE Looks like there's already a PR #1236 and tracking in https://github.com/sendgrid/sendgrid-nodejs/issues/1217.
This is so awesome of the community and thanks for taking initiative @matt-catellier :+1:

@krishnaTORQUE Looks like there's already a PR #1236 and tracking in https://github.com/sendgrid/sendgrid-nodejs/issues/1217.
This is so awesome of the community and thanks for taking initiative @matt-catellier :+1:

@eshanholtz please, do you mind updating this ticket as soon as the fix is released? We're following this topic and are highly waiting for it as it's a high security issue. Thanks.

@eshanholtz please, do you mind updating this ticket as soon as the fix is released? We're following this topic and are highly waiting for it as it's a high security issue. Thanks.

@eshanholtz please, do you mind updating this ticket as soon as the fix is released? We're following this topic and are highly waiting for it as it's a high security issue. Thanks.

@Mikescops Just saw a release come through so figured I'd notify everyone here. I installed 7.4.1 and the npm warning is gone.

@eshanholtz please, do you mind updating this ticket as soon as the fix is released? We're following this topic and are highly waiting for it as it's a high security issue. Thanks.

@Mikescops Just saw a release come through so figured I'd notify everyone here. I installed 7.4.1 and the npm warning is gone.

Fixed with this release: https://github.com/sendgrid/sendgrid-nodejs/releases/tag/7.4.1

Fixed with this release: https://github.com/sendgrid/sendgrid-nodejs/releases/tag/7.4.1

Thanks for the ping @thinkjrs :)

Thanks for the ping @thinkjrs :)


ItemTemplateSelector is broken in all Collection Views (ListView, GridView, TreeView...) that has their default containers specified in the DataTemplate

Describe the bug
What is the Bug
ItemTemplateSelector is in a broken state for collection view controls (ListView, GridView, TreeView, etc) in WinUI3.
When the user scrolls through the views and triggers recycling of containers, the control will constantly choose an incompatible container for the new items that are brought into view. Which will often lead to broken views, binding failing and/or crashing of applications.
It is surprising that Microsoft have this glancing bug to live for so long since WinUI's release.
Workarounds and analysis
There is a Workarounds that can help to you avoid such a crashing: Don't use the default containers in the DataTemplate. 'Default Containers' here means ListViewItem for ListView, GridViewItem for GridViewView.
For Example, don't do
<DataTemplate x:Key="ListViewItemTemplate1" x:DataType="viewmodel:TestVM1">
    <ListViewItem>
        <TextBlock Text="{x:Bind Content1}" />
    </ListViewItem>
</DataTemplate>

Instead, do
<DataTemplate x:Key="ListViewItemTemplate1" x:DataType="viewmodel:TestVM1">
    <TextBlock Text="{x:Bind Content1}" />
</DataTemplate>

My analysis of how this workarounds works: If you directly provide a container corresponding to the view as the root frame of the DataTemplate, the ListViewBase will directly use it as the container of the new items. But if there is no valid container in the DataTemplate, the ListViewBase will create a new container and put your templated stuff into its Content field. In this case, since the Containers will always be the same, the view controls just replaced its contents, so that the container will always be compatible with each other.
Unfortunately, this workarounds cannot work for TreeViews. Because without a TreeViewItem in the DataTemplate, you basically have no way to specify ItemsSource for your nodes. This rendered the TreeViews unusable, as it is basically became a simple ListView.
I had to write my own TreeView control from scratch because of this bug... I really hope Microsoft can have this bug fixed.
Steps to reproduce the bug
WinUIItemTemplateSelectorBug.zip
Attached is a test app for this bug. I added a ListView, a GridView and a TreeView to demonstrate this bug. They use a very simple selector based on some very simple data.

Build and Run this App
You are expected to see a ListView, a GridView, a TreeView in the window, they are displaying the same bunch of data with the help of DataTemplate
Quickly scrolls through any of the views using the scroll bar to trigger container recycling.
->Application Crash. Message = "Source object type is not a projected type and does not inherit from a projected type. (Parameter 'value')"

Expected behavior
No crash, no broken view.
Screenshots

NuGet package version
WinUI 3 - Windows App SDK 1.4.3: 1.4.231115000
Windows version
Windows Insider Build (xxxxx), Windows 11 (22H2): Build 22621, Windows 11 (21H2): Build 22000, Windows 10 (21H2): Build 19044, Windows 10 (21H1): Build 19043
Additional context
This bug has been there since the first release of WinUI3.
For any one that seek to make your  templated TreeView to work without having to write your own treeview control from scratch -  there are an extra workarounds things you can try, but it is not granted that it suit your case:

Create a new TreeView class based on the official TreeViewEx control (Let's call it TreeViewEx)
Create a new TreeViewList class based on the official TreeViewList. (Let's call it TreeViewExList)
Create your own Styles for TreeViewEx, using TreeViewExList to replace the old TreeViewList in the Control Template.
Add a override for IsItemItsOwnContainerOverride() function, and make it always return false. This is so that the control sees nothing as their default containers, and a new TreeViewItem will always be created by the control to contain your templated stuff. This is the key step to avoid the crash.
Create a TreeViewExItem class based on ContentControl. This step is important, that you are not creating a replacement for the TreeViewItem, you are creating a fake hook that is used to provide necessary information for the real TreeViewItem to work.
Create Dependency Properties for the fake TreeViewExItem: ItemsSource, IsExpanded, GlyphBrush, CollapsedGlyph, ExpandedGlyph, GlyphOpacity, GlyphSize, HasUnrealizedChildren, with the same property type of the real TreeViewItem. (ItemsSource is the only one that is nessary to make your TreeView work, and perhaps IsExpanded)
Override OnApplyTemplate() function in the TreeViewExItem, find throught visual tree a TreeViewItem in its ascendants
Hook the changed events of all the Dependency Properties in your TreeViewExItem, and feed the corresponding values to for the real TreeViewItem. Please note that when ItemsSource changes, you may also want to update the value of IsExpaneded.
Change your XAML to use the new TreeViewEx and TreeViewExItem and see if it works for you.

### Tasks

An alternative work around is to comment out the following override in the TestTemplateSelector class:
    protected override DataTemplate SelectTemplateCore(object item)
    {
        return GetTemplate(item) ?? base.SelectTemplateCore(item);
    }

The remarks section of SelectTemplateCore suggests that only the second override should be provided when using virtualising panels

An alternative work around is to comment out the following override in the TestTemplateSelector class:
    protected override DataTemplate SelectTemplateCore(object item)
    {
        return GetTemplate(item) ?? base.SelectTemplateCore(item);
    }

The remarks section of SelectTemplateCore suggests that only the second override should be provided when using virtualising panels

That's very good information. I haven't noticed that part yet.
But still, you get the result of a broken view, as it is the same as putting your Template into an internally created Container. Which leads to your gridview and listview to have doubled layer (if you already put a Container in the DataTemplate), and treeview still have no way to specific a ItemsSource.
Which means we will still have to do similar things to set the ItemsSource value for the hidden real TreeViewItem.

Hello. Thanks for your report. This turns out to be by design.
In a nutshell, taking a ListView as an example, when the application uses a ListView.ItemTemplateSelector instead of a ListView.ItemTemplate, and that selector returns DataTemplate instances with a ListViewItem as its root element, that application must listen and handle the ListView.ChoosingItemContainer and ListView.ContainerContentChanging events.
The ContainerContentChanging handler is in charge of building lists of recyclable containers (for each kind of item template returned by the ItemTemplateSelector).
The ChoosingItemContainer is in charge of returning an appropriate container based on the provided item, if and only if the suggested container is inapplicable to the item. The returned container may be picked from those recyclable container lists or not.
Adding such handlers to your repro application fixes the crashes.
Which changes are required in the repro app, just for the ListView case?

in MainWindow.xaml:
name the Grid: <Grid x:Name="grid" Height="500" ...
add the two handlers:
<ListView
Header="ListView"
ItemTemplateSelector="{StaticResource ListViewSelector}"
ItemsSource="{x:Bind CollectionViewModel}"
ChoosingItemContainer="ListView_ChoosingItemContainer"
ContainerContentChanging="ListView_ContainerContentChanging"/>

in MainWindow.xaml.cs's MainWindow class, add:
private List recyclableVM1Containers = new List();
private List recyclableVM2Containers = new List();
private void ListView_ChoosingItemContainer(object sender, ChoosingItemContainerEventArgs e)
{
if (e.ItemContainer != null)
{
bool itemIsVM1 = e.Item is TestVM1;
      if (itemIsVM1 && e.ItemContainer.ContentTemplateRoot is TextBox)
      {
          if (recyclableVM1Containers.Count > 0)
          {
              e.ItemContainer = recyclableVM1Containers[0];
              recyclableVM1Containers.RemoveAt(0);
          }
          else
          {
              DataTemplate dataTemplate = grid.Resources["ListViewItemTemplate1"] as DataTemplate;
              e.ItemContainer = dataTemplate.LoadContent() as SelectorItem;
          }
      }
      else if (!itemIsVM1 && e.ItemContainer.ContentTemplateRoot is TextBlock)
      {
          if (recyclableVM2Containers.Count > 0)
          {
              e.ItemContainer = recyclableVM2Containers[0];
              recyclableVM2Containers.RemoveAt(0);
          }
          else
          {
              DataTemplate dataTemplate = grid.Resources["ListViewItemTemplate2"] as DataTemplate;
              e.ItemContainer = dataTemplate.LoadContent() as SelectorItem;
          }
      }
      else
      {
          // Suggested container is usable for e.Item. Remove it from the recyclable list if present.
          if (itemIsVM1 && recyclableVM1Containers.Contains(e.ItemContainer))
          {
              recyclableVM1Containers.Remove(e.ItemContainer);
          }
          else if (!itemIsVM1 && recyclableVM2Containers.Contains(e.ItemContainer))
          {
              recyclableVM2Containers.Remove(e.ItemContainer);
          }
      }
  }

}
private void ListView_ContainerContentChanging(ListViewBase sender, ContainerContentChangingEventArgs e)
{
if (e.InRecycleQueue && e.ItemContainer != null)
{
if (e.Item is TestVM1)
recyclableVM1Containers.Add(e.ItemContainer);
else
recyclableVM2Containers.Add(e.ItemContainer);
e.Handled = true;
}
}

We have a couple of potential ideas for improving the developer experience though:

provide a more guiding error message when those ListViewBase events are not hooked up and such a binding fails.
add the ability to set e.ItemContainer to null in the ChoosingItemContainer handler to tell the Xaml framework that it needs to create a brand-new container.
These potential improvements are being considered for a future Xaml release.

Thank you.


prevent CI creating 2 events on each pull request push

While testing FreeBSD job I noticed that each push to a feature branch creates 2 separate events - push and pull request.
This should reduce consecutive commits from running the same job twice and free a lot of CI resources.

sweet, thanks :)


Model Performance is Suboptimal

I observed the training loss (for both long_los and icu_mortality tasks in the MIMIC IV tutorial) is unstable and nonconverging for supervised training. Even after tuning the loss just oscillates over epochs.
I added a synthetic data test in the tune_debug branch that should be easily learnable, the model only has to learn whether or not there is the code "ADMISSION//CARDIAC" or "ADMISSION//ONCOLOGY" in the patient's history. And I trained an LSTM on this and get these metrics over $1000$ epochs and with learning rate $0.1$:

I ran this with the command:
pytest -k test_model_memorization_train[pytorch_dataset-supervised-earlyfusionNone-triplet_encoder-lstm] -s

I also trained a transformer with the same data and task and got these (note that the learning rate is in the legend for the three learning rates I tried):

The command used is:
pytest -k test_model_memorization_train[pytorch_dataset-supervised-earlyfusionNone-triplet_encoder-transformer_encoder] -s

I tried training the transformer encoder with an extremely low learning rate (1e-16). I would expect to get very little changes in metrics over epochs for such a small learning rate, and validation metrics follow this, but training loss is still oscillating:


Check for consistency of scanning parameters across subjects

One of the common problems in acquiring neuroimaging data is accidental changes in scanning parameters (due to manual steps, sequence updates, or scan interruptions). It would be great if QAP could check if all subjects have the same RepetitionTime, EchoTime, SliceTiming, resolution, and scan size (acquisition matrix as well as scan length) and report any abnormalities. Those checks would have to be separated by modalities and runs.

agreed
On Wed, Nov 11, 2015 at 3:22 PM, Chris Filo Gorgolewski <
<EMAIL_ADDRESS>wrote:

One of the common problems in acquiring neuroimaging data is accidental
changes in scanning parameters (due to manual steps, sequence updates, or
scan interruptions). It would be great if QAP could check if all subjects
have the same RepetitionTime, EchoTime, SliceTiming, resolution, and scan
size (acquisition matrix as well as scan length) and report any
abnormalities. Those checks would have to be separated by modalities and
runs.
—
Reply to this email directly or view it on GitHub
https://github.com/preprocessed-connectomes-project/quality-assessment-protocol/issues/32
.

--
Director of Imaging, Center for the Developing Brain, Child Mind Institute
http://childmind.org
Director, Computational NeuroImaging Lab
http://computational-neuroimaging-lab.org/, Nathan S. Kline Institute for
Psychiatric Research http://www.rfmh.org/nki/
Co-Founder, The Neuro Bureau http://neurobureau.org
Impact Story https://impactstory.org/CameronCraddock
Google Scholar Citations
http://scholar.google.com/citations?user=lZ9hwSsAAAAJ&hl=en
GitHub https://github.com/ccraddock
Share that Brain @ the International Neuroimaging Datasharing Initiative
http://fcon_1000.projects.nitrc.org/!
Join me 2015-10-23 @ Brainhack Americas http://brainhack.org/Americas/!
The GigaScience Brainhack Thematic Series
http://brainhack.org/brainhack-thematic-series-call-for-papers/ is open
for your submission.

i am thinking that this would be functionality that would happen after QAP? currently we output many of the scanning parameters in the QAP output, by plotting these values outliers can be quickly identified. Would this be a suitable solution?


Missing speaker file

For the inference on the lora-svc-for-pretrain branch, we need the singer's speakers file,
python svc_inference.py --config config/maxgan.yaml --model maxgan_g.pth --spk ./config/singers/singer0001.npy --wave test.wav
How we can generate this files? (./config/singers/singer0001.npy )

https://github.com/PlayVoice/lora-svc/tree/main/config/singers
this speaker files are from mulit-singer data, is the average of singer's audio timbres
https://github.com/PlayVoice/lora-svc/blob/main/svc_preprocess_speaker_lora.py#L13~L25


Add Monochrome icon (to fit theme template) on macos.

Closes #18.
Thanks https://github.com/iiiGerardoiii for the icon files themselves.

Nice!
One thing I noticed is that the icon looks slightly blurry in the screenshots as if it was resized, quite weird since the icon is 16x16px.

Yeah, we could increase it. Do we have any full-scale icons to be resized from?

I think that even if we had a high-res image of the icon it would still look blurry because resampling.
The icon is 16x16 which is the size of a menubar icon, right? dunno why it looks blurry.

It looks like the images themselves are blurry even in github. What's the highest res image we have that could be scaled down?


Cli crashes after ExportDatabase when command-recording is on

Describe the bug
I turned on command recording, then attempted to export a database (I used Xmdv which has an export option).
The cli crashes.
(Export succeeds).
To Reproduce

Open globe.silo and create a plot
Turn on Command recording
Open the ExportDatabase window, choose Xmdv, click Export and change the export option when prompted.
Stop the command recording
The cli crashes

the B.cli.vlog file shows this at the bottom:
Logging: SetStateLoggingRPC
Xfer::Process: Opcode=47, len=167, type=ViewerRPC
Logging: ExportDBRPC
signalhandler_core: SIGSEGV!

I added some print statements, and the failure is occuring in this call:
https://github.com/visit-dav/visit/blob/6d06dfc07138d0214acd9647303967a788c85d79/src/visitpy/visitpy/PyDBOptionsAttributes_Helpers.C#L36
Expected behavior
Cli shouldn't crash when logging DBOptions.
Desktop

OS and version: linux
VisIt Version: 3.2
Server info (if applicable):

@cyrush Any ideas on this?

Perhaps logging of ExportDBRPC should be disabled until this is fixed ? (simple change to visitpy/common/Logging.C)

Yes, disabling this until it is fixed is better than a crash. Perhaps add a message to the logging that says it is temporarily disabled.

@cyrush PyDBOptionsAttributes_CreateDictionaryFromDBOptions is almost completely identical to the one in visitmodule.C, except the  const-ness of the DBOptionsAttributes object. Could that be a factor?


Missing Functions page on readthedocs.io

https://pode.readthedocs.io/ does not have the "Functions" page that https://badgerati.github.io/ has. I couldn't figure out what was going on for the longest until I realized they were two different websites. They appear to be synchronized otherwise having receive the 2.41 changes.

I officially host the documentation on https://badgerati.github.io/Pode, and that's where everything links to, and where the README takes you to as well; including Pode.Web.
Logging into readthedocs it looks like somehow it's been doing the docs for just Pode, and I don't even remember ever setting it up! From what I can see it's just running mkdocs and that's it, and the Functions pages are generated using PlatyPS which is why they won't be there.
The official docs are here: https://badgerati.github.io/Pode. I would disable the readthedocs one, but I've no idea who else is using it :/
I believe mkdocs now has support for a news-ticker across the top, so I could look into adding one of those to inform people for a while and then close it down 🤔

I think that'd be best if anyone is like me I got dissuaded from Pode initially when I couldn't find the functions page. Not entirely sure how I found the readthedocs versioon first.


New package: WykoOPDReader v0.1.3

Registering package: WykoOPDReader
Repository: https://github.com/jaakkor2/WykoOPDReader.jl
Created by: @jaakkor2
Version: v0.1.3
Commit: 220ee93bce8aecdb53515354f7627dd8faf583db
Reviewed by: @jaakkor2
Reference: https://github.com/jaakkor2/WykoOPDReader.jl/issues/1#issuecomment-2440036854
Description: Reader for Wyko OPD datasets (.opd)

Close this request without merging.  There are more than 100 surface profiler file formats, not a good idea to have individual packages.  I would like to rename the package.


When scene is empty an "instance scene" icon, should show a hint on how to create inherited scene

When empty scene is opened, some people would generally want to use inheritance scene feature but they are not aware of the fact that it's under Scene->New Inherited scene. They are trying to click + sign above the scene tree tab, and because it's dimmed, it looks like it's impossible to create new scene with root from the different scene.

I think that the '+' should not be dimmed so much in this case, and after click there should be dialog popup with text something like: "To create inherited scene you need to click Scene->New inherited scene"
As a side note: Some might think it would be good to remove Scene->New inherited scene, and put this functionality under '+' sign when scene tree is empty. I don't think it's a good idea since Inherited Scenes works slightly different then regular scene instancing, and the fact that it's creation is putted somewhere else might be good indicator of that (and hint for users to check documentation on how exactly it's working).
It would be ideal to have this in 2.0 since this is first stable with this functionality.

I'm not for adding a custom behaviour to the '+' button that would be enabled only in empty scenes, that would be more confusing than the current situation IMO. If the '+' button is normally used only to instance scenes, it should stay like that. It is also in the scene tree panel which is specific to members of the current scene tree. Creating an inherited scene is a behaviour which is clearly separated from the currently opened scene (and also independent of whether we want to use this opened scene as base for the inherited scene), so it's normal that it is not in the scene tree of the current scene.
Now for the question of making the inherited scene creation more evident, I don't have a strong opinion. It's a quite advanced feature after all, and also not one that would be used very often by most devs (for example in @KOBUGE-Games' current Project Kumquat game we used it only four times so far to create four tower scenes based on a common base scene), so I don't find it overly complicated to use in Scene->New inherited scene. Creating an inherited scene is like creating a normal scene, but you get to select a base scene to use for some shared parts, so IMO it's normal to have it next to Scene->New (empty) scene.

@akien-mga well I don't have problem eighter but quite a few times people on irc were thinking that it's impossible to do that because + sign was dimmed. Yesterday I had one of this situations again that's why I decided to open this issue.

junior job :)))))))))

It makes sense I guess
On Jan 14, 2016 13:03, "kubecz3k"<EMAIL_ADDRESS>wrote:

@akien-mga https://github.com/akien-mga well I don't have problem
eighter but quite a few times people on irc were thinking that it's
impossible to do that because + sign was dimmed. Yesterday I had one of
this situations again that's why I decided to open this issue.
—
Reply to this email directly or view it on GitHub
https://github.com/godotengine/godot/issues/3352#issuecomment-171683234.

quite a few times people on irc were thinking that it's impossible to do that because + sign was dimmed.

In this case it's more that they misunderstood what inherited scenes are IMO, as creating them is not related to instancing. Once they are created, they can of course be instanced using '+' like any other subscene, but we should not accomate these user's misunderstand by modifying the subscene instancing button IMO.
We just need to find a better way to make the concept of inherited scenes easier to understand.

I plan on removing those buttons and replacing them with a contextenu for
2.1
There are two reasons for this:

Can't add more node related features (no room)
text should make it more clear what each function is
On Jan 14, 2016 13:16, "Rémi Verschelde"<EMAIL_ADDRESS>wrote:

quite a few times people on irc were thinking that it's impossible to do
that because + sign was dimmed.
In this case it's more that they misunderstood what inherited scenes are
IMO, as creating them is not related to instancing. Once they are created,
they can of course be instanced using '+' like any other subscene, but we
should not accomate these user's misunderstand by modifying the subscene
instancing button IMO.
We just need to find a better way to make the concept of inherited scenes
easier to understand.
—
Reply to this email directly or view it on GitHub
https://github.com/godotengine/godot/issues/3352#issuecomment-171686899.

@reduz not sure what to do with the issue, close and open new one with your context menu idea as a reminder?

Closing this issue since it's not valid anymore after recent context menu changes in current 2.1 alpha branch.


Ubuntu 14.04 js-error in the app

I successfully compiled .deb package without any errors. Then I installed it.
The Atom opens but I receive the error at start:
TypeError: Unable to watch path
at HandleWatcher.start (/usr/share/atom/resources/app/node_modules/pathwatcher/lib/main.js:65:29)
at new HandleWatcher (/usr/share/atom/resources/app/node_modules/pathwatcher/lib/main.js:29:12)
at new PathWatcher (/usr/share/atom/resources/app/node_modules/pathwatcher/lib/main.js:119:30)
at Object.exports.watch (/usr/share/atom/resources/app/node_modules/pathwatcher/lib/main.js:178:12)
at Config.module.exports.Config.observeUserConfig (/usr/share/atom/resources/app/src/config.js:90:109)
at Config.module.exports.Config.load (/usr/share/atom/resources/app/src/config.js:67:19)
at Atom.module.exports.Atom.startEditorWindow (/usr/share/atom/resources/app/src/atom.js:342:19)
at Object. (/usr/share/atom/resources/app/src/window-bootstrap.js:14:8)
at Object. (/usr/share/atom/resources/app/src/window-bootstrap.js:20:4)
at Module._compile (module.js:455:26)
at Object.Module._extensions..js (module.js:473:10)
at Module.load (/usr/share/atom/resources/app/node_modules/coffee-script/lib/coffee-script/register.js:45:36)
at Function.Module._load (module.js:311:12)
at Module.require (module.js:363:17)
at require (module.js:379:17)
at window.onload (file:///usr/share/atom/resources/app/static/index.js:20:5)
What can cause the problem?

@envygeeks : The problem, as shown above, is not that there are too many "watch instances" (which is what would happen if the programmer created individual watches as you suggested), but instead that there are too many "watches" - which are watched files.
Also, from a programmer's PoV, creating individual watches for specific files, when I need to watch more than a couple of files in the same directory is significantly more annoying and cumbersome write and debug then just watching the entire directory and filtering for only the files I'm interested in - and this is where the problem lies.
I'm using another electron based application, and running that through strace, I can see it is setting up 5 different watch instances just on /etc. As /etc/ on my system has more than 250 files, electron just ate up more than 1000 "watches" just for the heck of it - I'm not even sure why to setup watches for /etc in the first place - it sets up individual watches for /etc/hosts and a few other /etc/ files that it makes sense to watch.
To me this looks like a bug, and a serious one - until operating systems start to ship with a higher default limit in fs.inotify.max_user_watches, with modern desktop environments consuming their fair amount of watches, and with /etc growing in size to accommodate more modern operating systems, add (multiple) watches for /etc is not a responsible approach.

Using Ansible, in some are interested in, one can simply do:
- name: Atom: Put higher number for fs.inotify.max_user_watches
  become: yes
  lineinfile: dest=/etc/sysctl.conf line="fs.inotify.max_user_watches=32768" insertafter=EOF

HTH

@batmat FYI, Ansible has a sysctl module.
- name: "Atom: Set higher value for fs.inotify.max_user_watches"
  sysctl:
    name: "fs.inotify.max_user_watches"
    value: 65536
    state: present


update core to 2.3.1 and sync to 1.0.4

Core changelog here: https://github.com/realm/realm-core/blame/v2.3.1/CHANGELOG.md#L1-L31
I don't think anything else from core applies to our end-users, nor does anything from realm-sync: https://github.com/realm/realm-sync/compare/v1.0.3...v1.0.4

The async commit daemon change cuts the number of files we open per thread in half on macOS (but not any other cocoa platforms).

Filed rdar://30446115 about clang crash from the ios-static debug Xcode 8.2 job.

I couldn't reproduce the clang crash locally. The other CI failures are unrelated.

Response from Apple on the radar I had filed:

We can’t reproduce the reported issue and the provided crashlogs look fairly random. We would advise testing your machine for faulty memory, the compiler is often one of the first victims of bad RAM.


fix: 修复前端界面Bug

Coverage remained the same at 93.339% when pulling e4f8c54c85d00ff29239bc85ad15d931b2e517ae on develop into 73a418b906f22435734f33bab95a4f0818616258 on master.


Refactor Derivation interface

Move implementation out of Derivation for easier reuse in case a different interface is needed

@OlegYch sorry for the delay. I'm hesitant as this would increase the surface area of binary compatibility that we have to maintain quite significantly (as it would include the inline method implementations).

@adamw i agree, but these are methods already exposed in public api, just packaged in a more a reusable way

You are right, it's already public :) I've adjusted the naming abit, let me know what you think

@adamw naming changes look good, but promoting type member to type parameter makes it less flexible and wouldn't work for my specific usecase (sample here https://github.com/softwaremill/magnolia/issues/242 )

@OlegYch now? :)

@adamw LGTM

Should be released soon :)


MUIC-406: [Android] User sees blank view instead of picture from his …

…camera
closes:
MOB-801: [Android] Upgrading from chat to video call in visitor side is shown blank withe screen

I ran it on device and it is working fine. However you can see (hopefully) that there are some things after the rounded corners, it's as if the background of the container is not fully transparent. Could that also be fixed here?

https://salemove.atlassian.net/browse/MUIC-433 created issue to handle that


remove long long from union param_value_u

The long long was mistakenly added when debugging an alignment issue
on x86_64.
Signed-off-by: Mark Charlebois<EMAIL_ADDRESS>

@davids5 Here is the fix


how do one use the componentHandler.upgradeAllRegistered() please provide a solution with ajax involved

** PLEASE READ THIS BEFORE FILING AN ISSUE **
Feature Requests / Breaking Changes in mdl-1.x
The MDL core team has actively begun work on the next major version of MDL, dubbed MDL v2. Because we are a small team that's hyper-focused on delivering the best Material Design Library possible for the web, it is highly unlikely that we will be actively working on new features or making backwards-incompatible changes for MDL as it currently exists.
If there is a non-breaking feature you would like to see implemented in mdl-1.x and are willing to contribute, we'd be happy to offer assistance with you doing so. But we will not personally be actively working on said features.
While we are just getting started with our next version of MDL, you can see our current progress on master as well as an overview of the direction we're headed in our (WIP) developer guide as well as our initial POC branch for our new architecture.
If you're interested in information for a specific MDL v2 component, check out our v2-component issues to see which v2 milestone it's associated with and feel free to subscribe to that issue for updates.
Bugs
Please include the following information with your bug report:

What MDL Version are you using? (please be specific, e.g. major.minor.patch)

What browser(s) is this bug affecting (including version)?

What OS (and version) are you using?

What are the steps to reproduce the bug? Can you create a plunker/codepen/jsfiddle which reproduces it?

What is the expected behavior?

What is the actual behavior?

Any other information you believe would be useful?

Please use StackOverflow for support and integration assistance. This tracker is only for problems with the code provided by Material Design Lite.
I notice this is now the second issue you've filed where you haven't provided any information. Please follow the instructions in the issue template and provide the required information when submitting issues. This way we have something to work with in understanding the actual problem if there is one.
Thank you.


Bug: App crashes on startup

Description
i start files, its load older tabs and crashed after second
Steps To Reproduce
open  2 One drive folder (privat and Business)
open 1 folder (Downloads)
open 1 folder Icloud Drive
step 2:
close files.
step 3:
reopen files
Requirements

e.g. Fix crash that occurs when...
fix chrashing files occurs when open older folder tabs

Files Version
990068
Windows Version
Windows 11 Pro Version	23H2 1000.22677.1000.0
Log File
debug.log

Which version of Files do you have?

Is this still an issue in v3.0.2?


New Port: Biomass Growth

New Port for Biomass Growth
Game Information

Title: Biomass Growth
URL: https://brainos.itch.io/biomassgrowth

Submission Requirements
CFW Tests
Ensure your game has been tested on all major CFWs:

[x] AmberELEC
[x] ArkOS
[x] ROCKNIX
[x] muOS

Resolution Tests
Test all major resolutions:

[x] 480x320
[x] 640x480
[x] 720x720 (RGB30)
[x] Higher resolutions (e.g., 1280x720)

I have accidentally closed it too soon


Added test cases in AuditingServiceTest

Implemented test cases for AuditingServiceTest.java class and increased the code coverage to 60%. I have disabled some test cases in the same class due to errors similar to AuditingServiceTest.writeForMissingEndPoint. I also have disabled DatabaseServiceTest.java. To increase the code coverage, we either have to either create a local immudb database or create mocks using PowerMockito for the objects of the private method writeInDatabase.

Can one of the admins verify this patch?

ok to test


Feature/update from content

This simplifies applying updates from a remote source.
Most of the time the updated executable comes from an outside source, in my case a database.
It's more efficient to be able to call self_replace_with(content) instead of writing the new file content to a random temporary file on the filesystem and then call self_replace(tmp_filename) which  then copies the file to yet another temporary.
I took the liberty to refactor a bit to avoid code duplication due to my additions.

I’m surprised that you have a case where this is more efficient. It requires buffering in memory whereas writing directly to the disk and to then move has the advantage that you can stream bytes in.
I don’t feel compelled to add a way to dump bytes in directly for the most part. Can you share more about why you have that need?

Good point, there isn't much benefit other than being more ergonomic. We're still having to store 3 executables during this process. It is good that you push back on this because my attempt here wouldn't be enough for my use case.
My use case is OTA on embedded (Linux) with limited resources (including NAND flash that we should try to not wear out unnecessarily). It should also handle any failures such as sporadic resets without human intervention, so the overall update process needs to be idempotent. I believe writing directly to a file with a unique and persistent name would be a part of a solution towards that. Ideally we want to be able to successfully update as many devices as possible in the shortest time possible using as little data traffic as possible (because of LTE data costs). To that end I also want to use a signed hashtree to both validate the new executable and to enable continuing a partial update process (outside of this crate) - increasing the success rate and lowering the risk of wearing out flaky devices.
In summary our focus is, in order:

Low resource usage
Robustness
Autonomy

Here are some suggestions:

How about replacing the &[u8] argument with a closure providing a BufWriter to the created temporary file? That would give us a 2 executables solution while also limiting the required buffer size. It also makes it easy to build a more sophisticated stack on top.
Add possibility to provide a name (not necessarily path) of the temporary file.
Add possibility to choose whether to overwrite an existing temporary file or to append to it.

So far I have seen the process of managing the temporary file outside the scope of this project. Here is for instance how Rye does it: https://github.com/astral-sh/rye/blob/ab8d5b433d5c4342c2bb125583c6bff4d29f5fbc/rye/src/cli/rye.rs#L303-L318
While it's possible to move some of this logic into this crate, there are so many subtle problems with tempfiles that I'm not sure if it's a great idea to try to have a one-size fits all kind of solution.
I would propose you try to make it work with self_replace today and if you learn something from your experience, we can look if we move some of the code you write around into this crate.


IGListIndexSetResult and IGListIndexPathResult docs updates use old index

Changes in this pull request
Updated documentation to fix #408
Pull request checklist

[X] All tests pass. Demo project builds and runs.
[X] I added tests, an experiment, or detailed why my change isn't tested.
[ ] I added an entry to the CHANGELOG.md for any breaking changes, enhancements, or bug fixes.
[X] I have reviewed the contributing guide

Is it necessary to add anything to CHANGELOG.md?

Coverage remained the same at 98.065% when pulling c07f0b8ed72252f99e5c3b4acde17c5ccf3391b3 on diogot:update-list-index-documentation into 279f348a4a5bde7f12c23bd33efdbc8281c0397b on Instagram:master.

Coverage remained the same at 98.065% when pulling c07f0b8ed72252f99e5c3b4acde17c5ccf3391b3 on diogot:update-list-index-documentation into 279f348a4a5bde7f12c23bd33efdbc8281c0397b on Instagram:master.

Thanks @diogot! No don't worry about the changelog 😄


cudnn problem

For CUDA 7 and cudnn 4. I get:
 libcudnn (R5) not found in library path.
Please install CuDNN from https://developer.nvidia.com/cuDNN
Then make sure files named as libcudnn.so.5 or libcudnn.5.dylib are placed in your library load path (for example /usr/local/lib , or manually add a path to LD_LIBRARY_PATH)

When I run that echo $LD_LIBRARY_PATH that I read in other issues, I get /usr/local/cuda-7.0/lib64:/home/ubuntu/torch/install/lib:/home/ubuntu/torch/install/lib:/usr/local/cuda-7.0/lib64/:
What should I do now? (Please, I am new to Ubuntu, so I would appreciate easy-to-follow instructions :))
Thanks!

did you figure it out? it seems your system expects cudnn 5 instead of 4

also have a read here: https://github.com/jcjohnson/neural-style/issues/319


Search: GEO Work Programme Activities on Search Result Items

To make it easier to identify the records, we should add a badge to the search result items.

This work can be done on the user dashboard and in the general search page.


Binary operators ignore surrounding brackets

Right now, all binary operators ignore possible surrounding brackets, even if they are significant for the code logic.
For example the following
a = k * (qx + qw)
is incorrectly treated as equivalent to
a = k * qx + qw
and actually converted into the latter by the parser.
This is, obviously, very bad and very wrong

Mostly fixed this in commits 8f741b5c2422ead7909e6642272977f8cc2e7f7b and 2e419dc48500d0bd62e82851a4bda867b78c7216
it now adds unecessary parenthesis sometimes, but only when it doesn't change the code logic. See e.g. the currently skipped test_arithmetic_operations_with_parenthesis_1


Notify about ghost connection is terminated

Right now it's impossible to know about if the system GC the ghost connections (for example if the client went to airplane mode or disconnected from wifi we collect these connections).
I've added an event where we notify as a normal disconnect call that can be listened from outside and clean up custom business logic code.

@grosmar Thanks for fix.


export

How to export a textured glb file

https://github.com/eliphatfs/zerorf/issues/3#issuecomment-1865346818

You may try to bake the colors on vertices in view dir in normals


EVMOS client.js

Hello. I want to discuss basic requirements for a client.js.
My suggestions for the cosmos network part:

connect EVMOS RPC node
method to get transactions list
method to get balances

What are the basic features it should have? Should it also support the ETH?
Please, share yours thoughts.

Cosmjs client: https://github.com/cosmos/cosmjs/tree/main/packages/stargate
Tendermint RPC client: https://github.com/cosmos/cosmjs/tree/main/packages/tendermint-rpc

I started in that repository. I will try to provide a PR in which you can at least initiate a client with connection to an EVMOS node.

For the EVMOS part everything is included to the package @cosmjs/tendermint-rpc, Tendermint34Client class.

The class Tendermint34Client (@cosmjs/tendermint-rpc) can't be extended because of private constructor and it's already contains balance method.


PO file generator issues

I want to ask two questions. How can I define headers when using the PO file generator? And after I translate, when I produce a file, all the translated words disappear. How should I deal with this problem?

Headers are added in the Translations instance.
$translations->getHeaders()->set('key', 'value');

And then you have to pass the translations instance to the Po generator:
$poGenerator = new PoGenerator();

$poGenerator->generateFile($translations, 'locales.po');

Headers are added in the Translations instance.
$translations->getHeaders()->set('key', 'value');

And then you have to pass the translations instance to the Po generator:
$poGenerator = new PoGenerator();

$poGenerator->generateFile($translations, 'locales.po');

Thanks. Thanks to you I can make it work like this.
`foreach ($phpScanner->getTranslations() as $domain => $translations) {
//seems to work, but po file does not seem to have the "language" header
$translations->setLanguage($domain);
$dosya_yolu = 'locales/' . $domain . '.po';

$translations->getHeaders()->set('Last-Translator', 'Mehmet Bozkurt');

if (file_exists($dosya_yolu)) {
    $mevcut_translations = $loader->loadFile($dosya_yolu);
    $updatedEntries = $translations->mergeWith($mevcut_translations);
    $generator->generateFile($updatedEntries, "locales/$domain.po");
} else {
    $generator->generateFile($translations, "locales/$domain.po");
}

}`


Fixed #25938 -- Factored out common base class for GEOSBase and GDALBase.

https://code.djangoproject.com/ticket/25938

I have not much time currently to review in details, but overall it looks very good. Thanks for this work.


Add string format function

add 'format' function for GDScript,
Usage:
format("{0} {1} {2}", "1111", "2222", "3333")
Output:
1111 2222 3333
Format "{?}", ? = argument index, argument can pass any type(string/int/Vector/etc...)

This PR has merge conflicts and apparently performance issues according to the above comment.
I also think we should use the exact same formatting as in Python3, it would be much more consistent than a custom one.
Closing for now as OP is not really active anymore so there are little changes to see this PR rebased.

Actually this was implemented already in #1176.


Bridge-iOS-SDK build 8 compatibility

SBBUserProfile no longer has a "username" property
Activity Manager get completion handler now passes the items array instead of the whole SBBResourceList object

looks good.


Add progress output to NBNco lookup. Rename function for clarify, and fix arg type.

Sample output:
/Users/simonroberts/src/github/lyricnz/nbn-upgrade-map/.venv/bin/python /Users/simonroberts/src/github/lyricnz/nbn-upgrade-map/code/main.py somerville vic
INFO:root:Fetching all addresses for Somerville, VIC
INFO:root:Fetched 5957 addresses from database
INFO:root:Submitting 5957 requests to add NBNco data...
INFO:root:Completed 100 tasks
INFO:root:Completed 200 tasks
...
INFO:root:Completed 5800 tasks
INFO:root:Completed 5900 tasks
INFO:root:All threads completed
INFO:root:Tally of tech types: Counter({'FTTN': 4484, None: 946, 'FTTP': 323, 'WIRELESS': 163, 'SATELLITE': 37, 'FTTC': 2, 'FTTB': 1, 'NULL': 1})
INFO:root:Location ID starting with "LOC": Counter({True: 5011, False: 946})
INFO:root:Writing results to results/VIC/somerville.geojson

(the locid without "LOC" are from other work, not included in this patch)


Lesson 11: Cannot get the prefix | Linux

Hi! I am on ubuntu linux, and I am having trouble getting the prefix from lesson 11 onto my system, can someone help me?

export PREFIX="/usr/local/i386elfgcc"
This is the path to the GCC that you're going to build in the following steps will be installed. That directory wouldn't exist by default in /usr/local/.

I have gcc preinstalled, so how do I get it there

thanks


Tiering: SHS prototype for "levels formula" and levels of support

Who is this PR for?
HS dept. heads, housemasters, principal, part of https://github.com/studentinsights/studentinsights/issues/1802.
What problem does this PR fix?
Ships prototype-quality feature for end-of-year conversations with HS staff, and for further conversations and research over the summer.  This is more intended as a conversation piece and aid for user research than as a product feature.  In other words, the design here isn't super intentional towards solving any real problem.  The most intention choices are to minimize the "negative" aspects of tiering or the "if-then" aspect, and call out the "what supports are being provided" more directly.
The intention is to have the "Experimental" banner on this indefinitely, until after the school year starts next year.  Depending on other planning, we may revisit this again in the fall but there's no commitment yet.
This will be open to all educators who have the link, with the authorization standard and filtered further by school.  The expectation is that this might show some useful things for MS or K8 students, so we can explore that, but the only intentional focus here is the HS prototype.
What does this PR do?
Adds a new /levels/shs page, intended for HS admin team educators.  This PR makes that only visible to members of the project team first, and doesn't add a link anywhere.  We may white list for release to particular educators later this week.
The UI is essentially a table, with the full list of active students in the school.  There are filters for house/grade and a textbox to search by name (since the table is virtualized, Ctrl+F in the browser won't work).
This also implements an iteration on the "tiering" formula from back in https://github.com/studentinsights/studentinsights/issues/1179, from some conversations with JR.  This is called "levels" in this iteration to distinguish that this is a different concept than "tiers" in MTSS language and that they mean different things.
Finally, there's a rough heuristic to map student courses to "subjects" in an effort to both simplify the presentation of grades and to expand it beyond just ELA/Math as in the Aspen report.
Screenshot (if adding a client-side feature)
Page:

Clicking the "summary" link in the upper right:

Checklists

[ ] Author checked latest in IE - HS Levels
[x] Author included specs for new code

This doesn't have amazing test coverage relative to what it is; this is just experimental and the prototype banner will stay for on here for the next few months at least.

selfie


Optimize compile times

I noticed that we are spending multiple seconds compiling even some trivial changes. We need to investigate and see if something can be done about this. E.g. we could hide gltf stuff behind a (default) feature, or some derivatives, or use less generics, etc.

glTF is certainly not the reason of slow compile times on my machine. Non-gltf examples take 25 sec, while gltf-enabled ones take 30 sec. I'm going to rename this issue and leave #183 for follow-ups.


[FEATURE] Support SamplerWorker in GPU

Sampling should be along with the last layer. We can implement part of it in python.

And to support top_k, sampler should wait for k tensors of a token.

And to support top_k, sampler should wait for k tensors of a token.

This might be a non-trivial case if we put the sampler in Python; we may need a separate Python pool.
Plan to support GPU Sampler after #32


Change in Attribute class in ESMF 8.1 snapshot breaks MAPL/Model

In chasing another bug we tried running the GEOS gcm we tried using the latest ESMF snapshot  (ESMF 8.1 beta 23) for the upcoming 8.1 release. It looks like changes being made to the ESMF_AttributeClass have issues with GEOS. It appears the ESMF_Attribute class is being superseded by the ESMF_Info class, and the legacy ESMF_Attribute class will be a wrapper on top of the ESMF_Info class.
However when running GEOSgcm.x with this snapshot the model immediately crash in MAPL_FieldCopyAttributes as the typekind of what had been added as a 32-bit integer, was coming being returned by the ESMF_AttributeGet call as being a 64-bit integer. The subsequent if block fails as it expects either a 32-bit integer, 32-bit real, 64-bit real, string or logical. If not one of those it fails. Just change the if to expect a 64-bit integer gets you a little further until the next crash.
The documentation says the ESMF_Info class JSON compatible and as such:
"Only 64-bit data types for integers and reals are supported ([3], [2]). 32-bit data is converted to 64-bit data and
vice versa. An error will occur if truncation is triggered during a conversion."
It seams as though this conversion on the get call is not happening. Communication with Gerhard said 8.1 should be backwards compatible with 8.0 so perhaps this is a "hiccup" along the way that may be fixed in a future snapshot but we should be keeping an eye on this going forward.
In the longer term the ESMF documentation suggest transitioning to the ESMF_Info class.

Found what seems to be the cause and have sent a reproducer to ESMF. Seems to be that MAPL_StateCreateFromSpec, if the field is a copy of an existing field, then all the attributes we set have already been added to the field, via a call to MAPL_FieldCopyAttributes. In this case the ESMF_AttributeSet fails now, where as before it did not in ESMF 8.0 and before. It seems to be HOW we are doing the copy the cases the problem. For example, I can create a simple program, create a field, and do
integer :: i
i=2
ESMF_AttributeSet(field,name=attname,value=i,rc=status)
i=3
ESMF_AttributeSet(field,name=attname,value=i,rc=status)
etc...
and the return status is always 0.
the point is that it is not the fact that the attribute has already been set that is the issue but rather, how it was set the first time.

Ok, ESMF seems to think it is related to the fact that in the copy we add the attribute as an array, but then try to set it again as a scalar.

Making sure @bekozi has his eyes on this one.

@bena-nasa @rsdunlapiv Do you know if this has been taken care of in ESMF 8.1.0 development? Or is it something we need to press for ESMF 8.2.0?

Reexamined on 2021-Apr-12 with @bena-nasa and @tclune
We are running with 8.1.0, so must work.