himalaya-vim nix package

outdated comment from putting the plugin inside the main derivation

$ nix build .#himalaya-vim
$ tree result/
result
└── share
    └── vim-plugins
        └── himalaya-vim
            ├── autoload
            │   └── himalaya
            │       ├── mbox.vim
            │       ├── msg.vim
            │       └── shared
            │           ├── bindings.vim
            │           ├── cli.vim
            │           ├── log.vim
            │           ├── thread.vim
            │           └── utils.vim
            ├── doc
            │   ├── himalaya.txt
            │   └── tags
            ├── ftplugin
            │   ├── himalaya-msg-list.vim
            │   ├── himalaya-msg-read.vim
            │   └── himalaya-msg-write.vim
            ├── lua
            │   └── himalaya
            │       └── mbox.lua
            ├── plugin
            │   └── himalaya.vim
            ├── README.md
            └── syntax
                └── himalaya-msg-list.vim

12 directories, 16 files

Thanks for your help! The problem is I don't know enough about Nix. It could be great if someone else could have a look?

:100: I need it double-checked just as much

@srid?

It looks generally good to me, and here are my thoughts:

@toastal You might want to undo those indentation changes, to make the diff small
Does the CI pass for this PR? I think it requires @soywod's approval to run.
@toastal How should Nix users use this plugin? Is there a documentation on it somewhere? Perhaps it can be added to the wiki here

If the CI runs, that should be good to merge.

Removed most of the indentation. I think the let being indented 1 must be an issue. I think an EditorConfig would have cleaned up the trailing whitespace issues as generally projects prefer them remove. (:thinking: perhaps @soywod would like to add this config file?)
I'm not sure if there's a test or not. I had some issues trying to get the plugin upstreamed do to the executable("himalaya") Vim line where himalaya was not on the PATH for the configuration step used by the Vim plugin building tools. If I understood how to import a flake in nixpkgs (or I use flakes instead of legacy for my NixOS config), I would just reference this flake.

This seems to be good enough. I got the Vim plugin merged upstream for non-flake. If it's okay with everyone, I think it's mergable.

Thank you :tada:


WCO buttons not zooming properly with titlebar

Enable windows control overlay feature and custom titlebar
Zoom in
The titlebar font becomes larger, but the WCO buttons stay the same size.

I am making the assumption that this refers to the icons not increasing in size since the buttons do increase in height. In most cases, we don't want to increase the width of the buttons, so it makes sense to me that this does not increase the icon size. This is also controlled upstream. My opinion is to close as-designed.


Don't apply scale to autohide theme property

SplitContainer theme property autohide is a boolean and shouldn't be scaled
3.x version #75995

Looked and couldn't find any other boolean valued properties that had scale applied

Thanks!

Thank you!

Cherry-picked for 4.0.3.


Add sanity checks to the graph view controller size calculations.

Closes #432
Added some sanity checks around the size calculation in the graph view controller to prevent crashes. It's not really possible to cause this crash to happen with StatsDemo. You'll need to integrate it into WPiOS with a :path => '' reference to the local pod. Testing that crash is also a bit hard.
I'm okay with this being a code-only PR review.
Needs Review: @kurzee

Thanks much @kurzee!!


[DSSv3] Fix Issue 14250 - add function for reading a password

A new module named getpass was added, which currently exposes a single
homonymous function. This function reads and returns user input from the
current terminal without echoing it.
As suggested by @MartinNowak, the getpass function was placed in a
separate module as its purpose is differs from getopt.

Adding a new module to Phobos is a big commitment. This seems like it would be much better off as a package on code.dlang.org, at least to start with.

Why?


Bacnet Write

Hi
Thanks for the great work. The read is working fine but cannot get write to work correctly. Please see screenshots and error. Any advice is appreciated

I didn't try this version yet, but there are my suggestions:

please test v0.0.14

Thank you for the response. I have updated to v0.0.14 restarted Node Red and re tested, there is now an error "unknown type". I have also tested with other software to ensure that the fault is not with the SB2WEB24 bacnet device and this software can operate this point. I have tried msg.payload in value and also 0/1 and true/false directly in config,also 8 and 15 in priority. Any advice is appreciated

Tried 0.0.14 yesterday my self and getting same error. Tried for all 6 types.
ANALOG OUTPUT, ANALOG INPUT, ANALOG VALUE.
BINARY OUTPUT, BINARY INPUT, BINARY VALUE
Same error "unknown type"

I have raised the issue on Github bacstack and it appears this issue is fixed in the latest release. I have tested basic examples from Node Bacstack and writemultiple works. Could you update the node red modules and retest.
https://github.com/fh1ch/node-bacstack/issues/99

Hi, any update to this?

Thanks, sure does work. Instead of using the bacnet node, you are using the function requesting the bacstack and doing what the bacnet node would be doing except bacnet node isn't working. Function node works fine. I will do additional testing, but looks good.
Sounds like all that needs to happen is @biancode would need to update the bacstack library and all would work.

@triDcontrols 0.0.1-beta.13 is the lastest - did I miss some? @mtoko  what about contributing to the package with your experinces from your functions source code?

@biancode
Apologies, I don't think I have the skills or experience yet to make a contribution, I still have much to learn but this has been good experience. Thank you for all your work, I have used your modbus node on many occasions and the bacnet node would be very useful. I am also monitoring the development of OPC UA

@mtoko I could help you to get more skills to help in development of that package if you want ;)

please test it with v0.0.17

functions like these get also TIMEOUT ERROR - just send the msg from the response and do not return msg at the end of function
`//var msg = {};
const bacnet1 = global.get('bacnet');
// Initialize BACStack
var client = bacnet1({adpuTimeout: 6000});
var state = msg.payload;
client.readProperty('<IP_ADDRESS>', {type: 4, instance: 3342490}, 85, {}, (err, value) => {
if(err) {
node.error(err, msg)
}
msg.payload = {value: value} ;
node.send(msg);

})
`

Thank you for all the assistance
I have tested with v0.0.17 on two different Node Red hardware but error is "unknown type" (see screenshot). I am working to understand where the problem is.
The read node still works and the "hard coded" function node to write also works

remove msg.payload from Value and set it to 1 or 0 msg.payload is wrong here and next try to set msg.payload.values with a function node before and remove all from Value in node

if you set Value and App-Tag it is set to this:
        let defaultValues = [{
          tag: node.valueTag,
          value: node.valueValue
        }]

please read API Docs - maybe you use a old version of bacstack from npm global

I did see that tag/type on line 93 of write.js. I changed this on my local write.js to type but it did not fix the problem

[{"id":"d60cced6.cc4198","type":"function","z":"3345fb9d.6eee74","name":"Yabe Room Simulator and Weather2","func":"const BACnet = global.get('bacnet');\nconst client = new BACnet();\n\n// discover devices\nclient.on('iAm', function(device) {\n  const msg = { payload: {\n      \"address\": device.address,\n      \"deviceId\": device.deviceId,\n      \"maxAdpu\": device.maxAdpu,\n      \"segmentation\": device.segmentation,\n      \"vendorId\": device.vendorId\n    }\n  }\n  node.send(msg);\n});\n\nclient.whoIs();\n \n// Read Device Object\nvar requestArray = [{\n  objectId: {type: 8, instance: 12345},\n  properties: [{id: 8}]\n}];\nclient.readPropertyMultiple('<IP_ADDRESS>', requestArray, function(err, value) {\n    if(err) {\n        node.error(err, msg)\n    }\n     \n    msg.payload = {value: value, error: err} ;\n    node.send(msg);\n});\n\nclient.readProperty('<IP_ADDRESS>', {type: 8, instance: 12345}, 77, (err, value) => {\n    if(err) {\n        node.error(err, msg)\n    }\n     \n    msg.payload = {value: value, error: err} ;\n    node.send(msg);\n})\n\n\nrequestArray = [\n  {objectId: {type: 8, instance: 12345}, properties: [{id: 8}, {id: 77}]},\n  {objectId: {type: 8, instance: 3342490}, properties: [{id: 85}, {id: 8}]}\n];\nclient.readPropertyMultiple('<IP_ADDRESS>', requestArray, (err, value) => {\n    if(err) {\n        node.error(err, msg)\n    }\n    \n    msg.payload = {value: value, error: err} ;\n    node.send(msg);\n});","outputs":1,"noerr":0,"x":470,"y":60,"wires":[["a398b63a.9ebde"]]},{"id":"b29fecfb.5934a","type":"inject","z":"3345fb9d.6eee74","name":"","topic":"","payload":"","payloadType":"date","repeat":"","crontab":"","once":false,"onceDelay":0.1,"x":180,"y":60,"wires":[["d60cced6.cc4198"]]},{"id":"a398b63a.9ebde","type":"debug","z":"3345fb9d.6eee74","name":"","active":true,"tosidebar":true,"console":false,"tostatus":false,"complete":"true","x":730,"y":60,"wires":[]}]

bacstack issues

The flow above with the function node works, but it should do because it is a read and there are no problems with read. The Yabe flow has problems in the function node, I think the problem is BOOLEAN and value of 100? I have only got ENUMERATED to work. See this working write flow
[{"id":"23988557.fae66a","type":"inject","z":"143b8cb5.c27573","name":"","topic":"","payload":"1","payloadType":"num","repeat":"","crontab":"","once":false,"onceDelay":0.1,"x":270,"y":100,"wires":[["58d0ace5.2d73f4"]]},{"id":"ed31364a.5ad958","type":"inject","z":"143b8cb5.c27573","name":"","topic":"","payload":"0","payloadType":"num","repeat":"","crontab":"","once":false,"onceDelay":0.1,"x":270,"y":160,"wires":[["58d0ace5.2d73f4"]]},{"id":"58d0ace5.2d73f4","type":"function","z":"143b8cb5.c27573","name":"Boiler 1","func":"//var msg = {};\nconst bacnet1 = global.get('bacnet');\n// Initialize BACStack\nvar client = bacnet1({adpuTimeout: 6000});\nvar state = msg.payload;\n\nconst values = [\n  {objectId: {type: 4, instance: 0}, values: [\n    {property: {id: 85, index:<PHONE_NUMBER>}, value: [{type: bacnet1.enum.ApplicationTags.BACNET_APPLICATION_TAG_ENUMERATED, value: state}], priority: 15}\n  ]}\n];\nclient.writePropertyMultiple('<IP_ADDRESS>', values, (err, value) => {\n  msg.payload = {value: value} ;\n  \n  if(err) { msg.payload = {error: err} ; }\n  node.send(msg);\n});\n\nclient.readProperty('<IP_ADDRESS>', {type: 4, instance: 0}, 85, (err, value) => {\n  msg.payload = {value: value} ;\n  node.send(msg);\n})\n\n","outputs":1,"noerr":0,"x":440,"y":120,"wires":[[]]}]

v0.0.18 is now type and not tag for write

Upgrade to v0.0.18 error is still unknown type
Sorry
Could you share an example of function node in earlier mesage
set msg.payload.values with a function node

I have tried the flow below and get this bacnet error. I have used values from previous working function node. The client is correct because the reads are still working

Flow
[{"id":"e74b4da5.ee62f","type":"inject","z":"70e6f53f.47c17c","name":"","topic":"","payload":"","payloadType":"date","repeat":"","crontab":"","once":false,"onceDelay":0.1,"x":200,"y":100,"wires":[["a94a766d.a10d18"]]},{"id":"ee1854cb.e0b6e8","type":"inject","z":"70e6f53f.47c17c","name":"","topic":"","payload":"","payloadType":"date","repeat":"","crontab":"","once":false,"onceDelay":0.1,"x":200,"y":160,"wires":[["7d33a68e.a4bc58"]]},{"id":"8d99ef57.02049","type":"BACnet-Write","z":"70e6f53f.47c17c","name":"","deviceIPAddress":"<IP_ADDRESS>","objectType":"","objectInstance":"","valueTag":"","valueValue":"","propertyId":"85","arrayIndex":"4294967295","server":"28e8a4b2.188ccc","multipleWrite":false,"x":680,"y":140,"wires":[[]]},{"id":"a94a766d.a10d18","type":"function","z":"70e6f53f.47c17c","name":"","func":"msg.payload.values = [\n    {\n        "objectId": { type: 4, instance:0 },\n        "values": [\n            {\n                "property": {id: 85, index:<PHONE_NUMBER>},\n                "value": [{type: 9, value: 0}],\n                "priority": 15\n    \n            }\n            ]\n        \n    }\n\n    ];\nreturn msg;","outputs":1,"noerr":0,"x":410,"y":100,"wires":[["8d99ef57.02049"]]},{"id":"7d33a68e.a4bc58","type":"function","z":"70e6f53f.47c17c","name":"","func":"msg.payload.values = [\n    {\n        "objectId": { type: 4, instance:0 },\n        "values": [\n            {\n                "property": {id: 85, index:<PHONE_NUMBER>},\n                "value": [{type: 9, value: 1}],\n                "priority": 15\n    \n            }\n            ]\n        \n    }\n\n    ];\nreturn msg;","outputs":1,"noerr":0,"x":410,"y":160,"wires":[["8d99ef57.02049"]]},{"id":"28e8a4b2.188ccc","type":"BACnet-Client","z":"","name":"","adpuTimeout":"3000","port":"47808","interface":"<IP_ADDRESS>","broadcastAddress":"<IP_ADDRESS>"}]

@fh1ch could you tell us more, please?


updateeachrelease 3.1

To be merged when 3.1 release goes live including API updates

We're still sorting out the assemblies for 3.1. So removed the changes for the API browser index page for now.


problem with TOML format for allowed elements

I have a project with a .markdownlintrc file in YAML format, and I'd like to support some inline HTML markup in it. I tried adding a section for "MD033" as follows, and I get the error below. I've tried several variations, based on YAML documentation, but haven't got this working.
Searching for examples that set this option only brings up JSON variants.
What am I doing wrong? Could a working example for this type of construct be supplied in the documentation?
[MD033]
allowed_elements = ['code', 'p']

and the error:
/home/usr/lib/noe_modules/markdownlint-cli/node_modules/markdownlint/lib/markdownlint.js:636
      throw error;
      ^

TypeError: (params.config.allowed_elements || []).map is not a function
    at Object.MD033 [as function] (/home/usr/lib/noe_modules/markdownlint-cli/node_modules/markdownlint/lib/md033.js:22:8)
    at forRule (/home/usr/lib/noe_modules/markdownlint-cli/node_modules/markdownlint/lib/markdownlint.js:454:20)
    at Array.forEach (<anonymous>)
    at lintContent (/home/usr/lib/noe_modules/markdownlint-cli/node_modules/markdownlint/lib/markdownlint.js:500:14)
    at lintContentWrapper (/home/usr/lib/noe_modules/markdownlint-cli/node_modules/markdownlint/lib/markdownlint.js:525:12)
    at lintFile (/home/usr/lib/noe_modules/markdownlint-cli/node_modules/markdownlint/lib/markdownlint.js:530:5)
    at lintNextItem (/home/usr/lib/noe_modules/markdownlint-cli/node_modules/markdownlint/lib/markdownlint.js:596:9)
    at lintInput (/home/usr/lib/noe_modules/markdownlint-cli/node_modules/markdownlint/lib/markdownlint.js:612:10)
    at Function.markdownlintSync [as sync] (/home/usr/lib/noe_modules/markdownlint-cli/node_modules/markdownlint/lib/markdownlint.js:634:3)
    at lintAndPrint (/home/usr/lib/noe_modules/markdownlint-cli/markdownlint.js:255:35)

Please include the example?

Sorry, I submitted before doing so. I've updated the issue with the example and the error.

I think that formatting is not correct. Here is an example for MD033 that uses JSON from the test cases: https://github.com/DavidAnson/markdownlint/blob/master/test/inline_html-allowed_elements.json
I ran that through one of the online converters to produce the following YAML output:
default: true
MD033:
    allowed_elements: [h1, h3, HR, p]

Try that?

A bit more information, I was mistaken about the file being in YAML format, it's actually TOML. Below is the JSON version that does work for me:
{
    "MD013": {
        "code_blocks": false,
        "headers": false,
        "line_length": 99,
        "tables": false
    },

    "MD024": {
        "siblings_only": true
    },

    "MD026": {
        "punctuation": ".,;:!"
    },

    "MD033": {
        "allowed_elements": ["code", "p"]
    }
}

This site provides a converter that provides this output:
[MD013]
code_blocks = false
headers = false
line_length = 99
tables = false

[MD024]
siblings_only = true

[MD026]
punctuation = ".,;:!"

[MD033]
allowed_elements = [ "code", "p" ]

An example input:
# Example

text

Together, these produce the error I described above.

The CLI does not support TOML-formatted config files. Per the documentation in that project repository, only JSON and YAML formatting is supported.
https://github.com/igorshubovych/markdownlint-cli
TOML is rare enough that it’s not included by default, and the CLI does not expose a way to provide a custom parser to allow it like the library does.

It turns out TOML looks like INI format, which is supported, and the following format works:
[MD033]
allowed_elements[] = "code"
allowed_elements[] = "p"

Thank you for your help, @DavidAnson .


Add tab completion to yotta

Problem
There is no tab completion for yotta, to be a truely useful command line tool it need this.
Suggested tab completions

target names
common options
anything that requires more than 3 characters typed

possible solution
https://argcomplete.readthedocs.org/en/latest/

argcomplete looks cool. Looks like it should be straightforward to integrate at a basic level, and shouldn't be too hard to add completions for things more complex things like yotta target <tab> after that.
pull-reqs welcome!

working on it in spare time, if anyone else manages to do it before me all the better!

Looks like your issues were caused by import order – seems argcomplete has to be before argparse.
I'm doing development on this at https://github.com/autopulated/yotta/tree/autocomplete if you're interested in following / helping out.

will do

Tab completion has been added to yotta, now looking to add support for hitting the yotta registry for completions.

As a MBED developer at MultiTech, one of ARMs partner companies, I'd love to see tab completion built into yotta. That feature would make it a whole lot more useable.

Hi @mfiore02  – thanks for the feedback!
Happy to say that some support is already available: on Linux and Mac yotta supports argcomplete to complete subcommand names and options (unfortunately not module or target names yet though). To enable this, just activate argcomplete following the instructions here, specifically:
pip install argcomplete
activate-global-python-argcomplete


To support [POST] /user/request_withdrawal

Description
To support [POST] /user/request_withdrawal

The package was cut out
https://github.com/bmf-san/bitbankgo


[Java] support auto generic cast for fury deserialization

What do these changes do?
Add support auto generic cast for fury deserialization

Related issue number

Closes #982
Check code requirements

[ ] tests added / passed (if needed)
[ ] Ensure all linting tests pass, see here for how to run them

Hold this PR, the added interface are not elegant enough.


Local builds using Docker

PR https://github.com/electric-sql/pglite/pull/198 addresses this - creates builder images that can be used to run the WASM pg build on both arm and x86


infinite loop when trying to install jax with cuda support on windows

Description
Downloading jax-0.4.12.tar.gz (1.3 MB)
---------------------------------------- 1.3/1.3 MB 13.2 MB/s eta 0:00:00
Installing build dependencies ... done
Getting requirements to build wheel ... done
Preparing metadata (pyproject.toml) ... done
Downloading jax-0.4.11.tar.gz (1.3 MB)
---------------------------------------- 1.3/1.3 MB 13.1 MB/s eta 0:00:00
Installing build dependencies ... done
Getting requirements to build wheel ... done
Preparing metadata (pyproject.toml) ... done
Downloading jax-0.4.10.tar.gz (1.3 MB)
---------------------------------------- 1.3/1.3 MB 16.1 MB/s eta 0:00:00
Installing build dependencies ... done
Getting requirements to build wheel ... done
Preparing metadata (pyproject.toml) ... done
Downloading jax-0.4.9.tar.gz (1.3 MB)
---------------------------------------- 1.3/1.3 MB 21.5 MB/s eta 0:00:00
Installing build dependencies ... done
Getting requirements to build wheel ... done
Preparing metadata (pyproject.toml) ... done
Downloading jax-0.4.8.tar.gz (1.2 MB)
---------------------------------------- 1.2/1.2 MB 15.5 MB/s eta 0:00:00
infinite loop...
when running
pip install -U -r dreamerv3/requirements.txt -f https://storage.googleapis.com/jax-releases/jax_cuda_releases.html
same with
python -m pip install jax[cuda]
System info (python version, jaxlib version, accelerator, etc.)
python 3.10.11

in particular, it works fine on wsl and linux, only windows has this problem..

Hi @nullonesix
As per the documentation, JAX cuda support is not available on Windows.

Please have a look at the documentation: https://jax.readthedocs.io/en/latest/installation.html
It is also mentioned in the documentation that pip install --upgrade "jax[cuda12]" do not work with Windows, and may fail silently.
Thank you.

@nullonesix unfortunately, I have no idea. But I think it's best we close the issue since, as @rajasekharporeddy pointed out, Windows + GPU is officially unsupported. (There may be some community projects though.)


API: add opt flag to flip cwt y axis for plots

Reference issue
#15049
What does this implement/fix?
Add an optional flag to flip y-axis of returned cwt matrix which is helpful for viz.
Additional information
This path seems like the easiest way to resolve #15049 due to the coupling of peak finding to current cwt implementation

Thanks for working on this @seanpquinn.

Thanks for taking a look @rgommers

I'm not sure I understand the solution. The issue suggests that cwt is fine, but only the example in its docstring needs an update to flip the y-axis in the plot.

Perhaps @xiays146 can comment, my view: it was tagged as  and the contention was the returned cwt matrix is not correct for plotting (misleading y-axis)

So why not update just that example? Boolean arguments like the one you are proposing here, to do something that can be done as a one-liner on the output are usually not an optimal choice from an API design perspective.

Sure, happy to go the updating docs route


Snippets in multiple scopes

It should be possible to make snippets that work in multiple scopes and/or an ALL scope option. For example a generic /* */ comment block snippet that could work in JS, CSS, LESS, etc.

Thanks for feedback! I agree that the snippets should be allowed to have multiple scopes, and I'm gonna work on it this weekend.

Thanks. That's awesome, and this is a great extension.

Hi @Lomacar, I've added an Any scope option in 1.8, but multiple scopes may be coming next.

@chuyik Finer control over multiple scopes would be awesome!


Handle 0-downtime deploys with high-read-frequency models

I have a deploy process that runs the migration then rolls out the new app version to our codebase. As a result, there is a period of time where the old version of the model is querying the database after the migration deleting the column has been run. This leads to errors like the following whenever the model with the removed column is read from the DB in the older version of the code:

column TABLE_NAME.COLUMN_NAME does not exist
LINE 1: ...

I think this medium post describes part of a potential solution. I think the solution would involve extending the migration logic to add some kind of noop that allows us to remove the column from the model in the app before removing in the migration, while tricking makemigrations into thinking that everything is in a legal state.

Apologies, after further local testing this appears to handle the case I've described. Much easier than the medium article! Thanks for the project!


Expose infrastructure settings in ansible playbooks

It would be convenient to have more infrastructure settings exposed through the ansible playbooks, such as:

Broker memory limit
Broker queue memory limit
Storage class for persistence
Storage capacity for broker persistence
...

These could go directly to the broker resource definitions as template parameters.

It’s in the infra configuration instead


Initially running causes a lot of 429 errors

Heyo, since you're using the default user agent when fetching the token reddit will 429's a lot. Normally i have to retry around 3 times to get it running.

What do you mean, the default user agent? You can define your own.

When you call PasswordCredentialsToken, the default user agent is used, not the one specified in the agent file.

Thank you for reporting! It appears to be fixed on my end; feel free to re-open if you still experience problems.


fix: make marisa-trie and platformdirs really optional

Closes #140

ahhh just say that you beat me to it 😄
Feel free to close it


CLI to support addition of new source configurations without requirin…

…g user to recreate database from scratch

@AndrewNepogoda I think we might want to do some basic checks to make sure the new JSON config is still compatible with the old, i.e. no steps moved / changed names, as that might mess up recovery.

@stanislav-bedunkevich this is questionable. What if we want to separate some steps (Export, Materialization) and run recovery. There are some benefits from overriding it all the time without any validation.


Feature: onPluginInstalled and onPluginUpdated events

Would it be an idea to have events:

'onPluginInstalled(Event)' for initialisation
'onPluginUpdated(Event)' for migration
'onPluginDeleted(Event)' for cleanup

With Event parameter containing object  with 'name', 'prevVersion' 'currVersion'?
And maybe, 'onThemeInInstalled' and 'onThemeUpdated', 'onThemeDeleted' events, although themes do not seem to have much login going on.

I would say no as the firing of the event is too unreliable -- you can manually install and remove plugins. However, I've done some work to start detecting updates. There's a new config/versions.yaml file which tracks Grav version and makes sure that update scripts are being run. It also supports plugins and themes, but those haven't yet been hooked up. This said, it is easy to use and I already use it in a plugin to detect updates.

For anyone looking for something similar to this: pre/post install/update/remove hooks have been implemented for plugins themselves, I'm not sure why they haven't been documented.
This is the PR with explanation: https://github.com/getgrav/grav/pull/787.


fix buggy test with $ref in older drafts

all keywords adjacent to $ref have no effect in draft7 and earlier.

I don't know how all your implementations were passing with this test! Maybe you have a bug?

Sounds like we should have tests against this scenario if it should have failed.

I don't understand what you're asking. The test as it is in the patch is the test that should fail.

Looking into this more deeply, I don't understand why it should have failed.  The $id has no impact on this test (in neither the original or your change) because the $ref is changing the file name in the base URI anyway, so it doesn't matter whether $ref is a sibling of it.
The only way the $id could have an impact is if it specified a different folder, e.g. http://localhost:1234/subfolder/schema-remote-ref-ref-defs1.json.  This would be ignored in the original case, but it would change the base URI in the second.

When the $id is respected, the $ref resolves to "http://localhost:1234/ref-and-defs.json", which is a file in the resources/ directory. Without $id, the $ref resolves to "ref-and-defs.json" which doesn't exist (and such resolution is undefined; my implementation aborts evaluation and generates a result object with one single error.

Okay, so you're saying that without the $id, the domain (locahost:1234) isn't defined, and so $ref can't resolve.  That makes sense.
Now that you've corrected the schema to operate as the test cases expect, we still need one that ensures $ref can't resolve when it's a sibling of $id.  This is what I'm asking you add.

We already have tests for that, in ref.json.
My goal in this specific PR is to fix a test that should not ever have worked.

There's probably more we can test in here, e.g. with an adjacent $id specifically, or an adjacent definitions, but I'll address that in a separate PR.

We already have tests for that, in ref.json.

Can you link to where we already have it?  The closest I found was this, but it wraps the $ref in an allOf.

My goal in this specific PR is to fix a test that should not ever have worked.

Yes, but in finding this test that should never have worked, you've identified two cases that should be covered: the case that this is attempting to cover and the case of why this should fail.  They each need coverage.

The failure case cannot be covered. There is no way of expressing a uri resolution failure in our current testing infrastructure.
The general case of sibling keywords to $ref being ignored is already covered. The $id keyword specifically is not, but I did say I would handle that in a separate PR.
If there are no objections to the changes I have made here, I'm going to merge this.

$ref being unable to resolve is definitely a car that can be covered.

No it can't, because there's no way to express it with the current testing syntax.
If you can come up with something, then by all means.


Question about 'Body' variable in katex shortcode

The 'body' variable in here
https://github.com/Jieiku/abridge/blob/d45bc26c2250f573a0d83f755831857999fc125f/templates/shortcodes/katex.html#L1
from it's coming from?
I'm asking as I'm making a shortcode for something else, and the expression {{ }} I have in the short code script works fine, but because of the CSP whenever the value changes, the hash changes too making the shortcode unusable. I was looking at katex and it seems there's a way by having the expression {{ }} in the post itself rather than the script, and that might solve it, it's just I don't know the "body" variable is taken from where.
Thanks!

The body variable is a built in feaure of Zola:
https://www.getzola.org/documentation/content/shortcodes/#shortcodes-with-body
I had similar issues when I initially implemented a imgswap shortcode: https://abridge.netlify.app/overview-images/#imgswap-shortcode
Initially I was trying to do it in a way that the src changed, but instead I just include both image sources in the body of the page with different class names. You may have to think outside of the box depending on what your trying to accomplish.


Add USP permutation test from Richard Samworth

https://indico.cern.ch/event/1036527/attachments/2243646/3804631/RichardSamworth_PHYSTAT.pdf

@dsaxton Do you know of this test? I just watched the presentation and it looks very good.

On slide 22 it is shown that the power of the USP test is better than two other popular tests and it also has the property of giving exact type 1 errors.

@HDembinski I hadn't heard of this but looks really interesting (in my experience testing for independence is very useful even for doing basic data exploration / preprocessing for modeling). I'd need to look at this more carefully (and get rid of some mental cobwebs), but nice to see research still being done on this classic problem.


llama.cpp no longer support .bin

thread 'main' panicked at 'Failed to load LLaMA model from "/home/cisco/git/llama.cpp/models/llama-2-13b-chat/ggml-model-q4_0.gguf": invalid magic number 46554747 (GGUF) for "/home/cisco/git/llama.cpp/models/llama-2-13b-chat/ggml-model-q4_0.gguf"', /home/cisco/git/cria/src/lib.rs:54:13
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

You could still find .bin models in huggingface and load them using llm crate...

@AmineDiro Thanks, I totally had a dead brain moment and forgot to check closes issues ))


Verify the publisher for Dart

We are now publishing a Dart package and this is the relevant documentation:
https://dart.dev/tools/pub/publishing
Currently, we don't have a "Verified publisher".
I have attempted to follow it, but I have been hitting a few rough edges:

I have created a free sub-domain here: https://github.com/proj-at/subdomains/pull/23
I have verified the property ( std_uritemplate.proj.sbs ) using the Google Search Console
Trying to create a publisher with the subdomain here: https://pub.dev/create-publisher doesn't work as(seems like) only top level domains are allowed

The next step would be to reach out to the support of pub.dev (email: support@pub.dev).
@ricardoboss would you be up on digging into this one on our behalf?

Sure! Is there a way other than GitHub where I can reach you in case I need further information?

Thanks @ricardoboss ! My private email is public on my GH profile, feel free to write there 👍

I contacted the pub.dev support but haven't heard back yet. I will try again later this week.

Small update: we have heard back from the pub.dev support and it seems that the domain we want to register as a verified published cannot contain underscores.
@andreaTP has already fixed this at the proj-at repository here: https://github.com/proj-at/subdomains/pull/24. Thanks!
Next step would be to retry verifying that domain and then use it to verify the publisher on pub.dev.

Thanks for following up @ricardoboss , I verified the domain again and this time it went through!
You can check here: https://pub.dev/packages/std_uritemplate that the publisher is now certified.
I can do an extra release to verify that everything is still aligned, but it seems unnecessary.

Nice! This can be closed then.


Please consider fixing issues detected by repository checker

Notification from ioBroker Check and Service Bot
Dear adapter developer,
I'm the ioBroker Check and Service Bot. I'm an automated tool processing routine tasks for the ioBroker infrastructure. I have recently checked the repository for your adapter legrand-ecocompteur for common errors and appropiate suggestions to keep this adapter up to date.
Please see the result of the check below.
ioBroker.legrand-ecocompteur
   -

ERRORS:

[ ] :heavy_exclamation_mark: [E033] @iobroker/adapter-core 2.4.0 specified. 3.1.4 is required as minimum, 3.1.6 is recommended. Please update dependencies at package.json
[ ] :heavy_exclamation_mark: [E036] @iobroker/testing 2.2.0 specified. 4.1.3 is required as minimum,  4.1.3 is recommended. Please update devDependencies at package.json
[ ] :heavy_exclamation_mark: [E162] js-controller 1.4.2 listed as dependency but 4.0.24 is required as minimum, 5.0.19 is recommended. Please update dependency at io-package.json.

WARNINGS:

[ ] :eyes: [W028] Minimum node.js version 18 recommended. Please adapt "{'engines' : { 'node' >= '16.0.0' } }" at package.json.
[ ] :eyes: [W040] "keywords" within package.json should contain "ioBroker"
[ ] :eyes: [W044] dependency should not require a specific version. Use "~1.2.3" or "^1.2.3" syntax. Please update "set-interval-async:*"
[ ] :eyes: [W105] Missing suggested translation into uk of "common.titleLang" in io-package.json.
[ ] :eyes: [W109] Missing suggested translation into uk of "common.desc" in io-package.json.
[ ] :eyes: [W115] "common.tier" is required in io-package.json. Please check https://github.com/ioBroker/ioBroker.docs/blob/master/docs/en/dev/objectsschema.md#adapter.
[ ] :eyes: [W145] Missing suggested translation into uk of some "common.news" in io-package.json.
[ ] :eyes: [W181] "common.license" in io-package.json is deprecated. Please define object "common.licenseInformation"
[ ] :eyes: [W184] "common.main" is deprecated and ignored. Please remove from io-package.json. Use "main" at package.json instead.
[ ] :eyes: [W184] "common.materialize" is deprecated for admin >= 5 at io-package.json. Please use property "adminUI".
[ ] :eyes: [W184] "common.title" is deprecated and replaced by "common.titleLang". Please remove from io-package.json.
[ ] :eyes: [W513] "gulpfile.js" found in repo! Think about migrating to @iobroker/adapter-dev package
[ ] :eyes: [W522] Please consider migrating to admin 5 UI (jsonConfig).
[ ] :eyes: [W853] .npmignore found - consider using package.json object "files" instead.

Please review issues reported and consider fixing them as soon as appropiate.
Errors reported by repository checker should be fixed as soon as possible. Some of them require a new release to be considered as fixed. Please note that errors reported by checker might be considered as blocking point for future updates at stable repository.
Warnings reported by repository checker should be reviewed. While some warnings can be considered as a suggestion and be ignored due to good reasons or a dedicated decision of the developer, most warnings should be fixed as soon as appropiate.
Feel free to contact me (@iobroker-bot) if you have any questions or feel that an issue is incorrectly flagged.
And THANKS A LOT for maintaining this adapter from me and all users.
Let's work together for the best user experience.
your
ioBroker Check and Service Bot
@mcm1957 for evidence

This issue has been replaced by new isse #136
This issue can be closed.
your
ioBroker Check and Service Bot


Taking derivative of GridFunction

Hi,
I have a GridFunction and would like to take the derivative.
Thanks,
Nick

Hi, @nickwhitman1993 ,
There are a number of ways derivatives can be obtained in MFEM.  The most appropriate way depends on the situation.  You can use CalcDShape to compute the gradients of the basis functions and then combine these results with the values of the degrees of freedom to compute the gradient at a given location within a single element.  This can also be accomplished by using GridFunction::GetGradient() which does essentially the same thing.  There is even a GridFunction::GetDerivative() method with loops over all elements and performs the above procedure and collects the results in a second GridFunction object although I believe this only computes one component of a derivative at a time (it can compute df/dy for example).
If you want to compute an entire GridFunction which is the derivative of a given GridFunction then you probably want to look at the DiscreteInterpolator classes in bilinteg.hpp.  These can be used to compute the gradient of an H1 field, the curl of an H(Curl) field, or the divergence of a H(Div) field.  If you need something else, like the curl of an H(Div) field for example, then you may need to compute with in a weak sense using a BilinearIntegrator and then solving a linear system to obtain the result.  You can read more about these options on Linear Interpolators and Bilinear Integrators.
Best wishes,
Mark

Hi @mlstowell thanks for the reply.
I do want a "GridFunction" returned that has vector information at every grid point that was on the original "GridFunction", so it sounds to me like the best option is to use the "DiscreteInterpolator". Am I understanding this correctly? The original field is a scalar field.

Hi, @nickwhitman1993 ,
That is a common but problematic request.  The trouble is that the derivative is discontinuous at element boundaries so we need to perform some sort of averaging of the values computed in neighboring  elements. Normally I would recommend using MixedVectorGradientIntegrator to perform the derivative followed by a solve to recover the appropriately weighted GridFunction but, as written, this only supports H(Curl) and H(Div) vector fields.  If this sounds like what you need then we could write a specialized integrator to compute your gradient.  I assume your original GridFunction is an H1 scalar field.  Is this correct?
Another option is to use GridFunction::GetDerivative() (or a similar function) to compute an approximation to the gradient as a simple average of the values from neighboring elements.  This should be fine for visualization or certain types of post-processing but I'd be hesitant to use this as part of a numerically sensitive algorithm.  If this is part of a numerical algorithm you might want to consider using the H(Curl) representation of the gradient (which is my personal favorite) instead.
Best wishes,
Mark

@mlstowell
You are correct, my original GridFunction is indeed an H1 scalar field. It is the output of my solve routine.
I will indeed use this for an additional algorithm, so I need to be as accurate and careful as possible in the calculation of the gradient. It sounds like a H(curl) representation might be the best bet here.
Are there any further stipulations or restrictions with using a H(curl) representation? The end goal is to take the gradient of the GridFunction output from my solve and another vector represented similarly at the GridFunction points and compute a tensor with that.
Essentially the end goal of what I am doing is a Fourier's Law calculation where I divide the heat flux by the temperature gradient to obtain a thermal conductivity tensor.
I also have a few questions on how to best represent and calculate the heat flux, but I can save that until we figure out this portion.
Thanks again Mark.
Nick

Hi, @nickwhitman1993 ,
The H(Curl) space is the most natural space for the gradient of an H1 field.  H(Curl) provides a family of vector fields whose tangential components are continuous across element boundaries but whose normal component is discontinuous.  If you think about how H1 fields are defined as piecewise continuous polynomials their gradients have exactly this type of limited continuity.  What this means is that once you know a field in H1 you can compute its gradient in H(Curl) in a fairly trivial manner (involving only data local to each element).  Specifically, what you do is the following:
GridFunction T(fespace_h1);
GridFunction dT(fespace_nd);

DiscreteLinearOperator grad(fespace_h1, fespace_nd);
grad.AddDomainInterpolator(new GradientInterpolator);
grad.Assemble();
grad.Finalize();

grad.Mult(T, dT);

This will produce an H(Curl) (a.k.a Nedelec or ND) representation of the gradient of T which I called dT.  This multiplication operation should be very fast and as long as you don't alter the topology of the mesh (by refining it for example) the gradient operator will not need to be rebuilt.
Best wishes,
Mark

@mlstowell
Just to clarify, dT would be returned as a GridFunction that contains vector information about the magnitude of the derivative in the x, y, and z directions?

Hi, @nickwhitman1993 ,
Discrete H(Curl) fields don't actually align with the x, y, and z directions.  They align with the edges of the mesh elements.  In fact the degrees of freedom, for a first order basis, are computed as integrals along a mesh edge of the inner product of your vector field with the tangent vector to that edge.
What this means is that the degrees of freedom themselves are generally not that useful unless you're very interested in path integrals of your vector fields.  To get the vector value at some point within an element you can call GridFunction::GetVectorValue(i, ip, v) where i is the element index, ip is the location inside the element in terms of the reference coordinate system, and v is the returned vector value.  The portion of this vector that is tangent to the surface of the element will be continuous across element boundaries, the normal component will be discontinuous.
I'm curious about your computation.  Are you trying to compute point-wise values for the conductivity tensor at each node of a mesh or are you doing homogenization where you're looking for an effective conductivity tensor of a large number of mesh elements?  Also, how do you compute the thermal flux?
Best wishes,
Mark

@mlstowell
Thank you for the explanation.
Really, what it boils down to is that I need to be able to compute the temperature gradient, and the heat flux. The idea I had in mind was to compute the thermal conductivity tensor in general at each node, but the simulations I am running need to estimate a bulk thermal conductivity over the entire length of the problem domain. I am running simulations over nm size widths and trying to extract a bulk thermal conductivity to the macroscale, so the bulk approach might be okay at this point if we are looking at a value(s) over the problem domain.
You ask an interesting question. I am actually simulating heat transport using phonon physics, so I solve for the scalar phonon intensity, call it I. I is related to temperature via I = Cv*v_g*T/4pi where T is temperature and the others are constants. My boundary conditions are just Dirichlet boundary conditions based on a temperature (emission), essentially.
I take the approach of solving this problem using the discrete ordinates method (called Sn) where the angular variable is discretized into a quadrature, and I loop over the angles and solve for each angular intensity and then sum the results into a scalar value. This gives me back the scalar phonon intensity, which can then be related to the temperature by the I = Cv*v_g*T/4pi equation.
I haven't quite figured out exactly how to compute the heat flux yet, but this is just simply the 1st angular moment of the phonon intensity. I already compute the 0th moment fairly easily by just looping over the converged angular values scalar value += weight * angular value, but I haven't quite figured out how to quite format this output yet. This going to be in a follow up question but it looks like this heat flux [x,y.z] += weight * scalar value * direction vector[x,y,z]
Additionally, in some previous conversation we had, you helped me write a linear integrator that I needed. I believe we found what was wrong and actually have a successful code at this point using the linear integrator you came up with. I don't know if that specific integrator would be useful, but it could be something added to the code base if desired.
Let me know what I can clarify. Additionally, it might be beneficial to have a phone call too. Let me know if you want to talk further.
Thanks again for your time @mlstowell!
Nick

Hi, @nickwhitman1993 ,
This sounds like a fascinating project.  I worked on something similar a few years ago trying to compute an effective dielectric tensor for a periodic structure.  This led to problems with spatial dispersion where the tensor depends not only on the frequency of the wave but also its direction of motion.  It's a tricky problem.
It's interesting that you mention computing moments.  I was just experimenting with this in MFEM a couple weeks ago, see branch moment-dev.  You can look at miniapps/common/mesh_extras.cpp in particular to see how I computed 0-th and 1-st moments of a scalar valued field.  If you need moments wrt an angular coordinate you'd need to alter these functions but I think it wouldn't be too difficult.   If you need advice on computing global integrals like these in MFEM I'd be happy to help.
Regarding your linear integrator: this is something MFEM needs more of.  I've been thinking of adding a larger group of integrators for people to work from.  Honestly, I haven't put much thought into it yet.  The hard part is coming up with a comprehensive family of integrators that cover a wide range of needs but are still easy to understand so that people can find what they need.
Best wishes,
Mark

@mlstowell
I would like to try implementing GridFunction::GetDerivative( ) to obtain directional derivatives of my field. Would you be willing to help me step through how to implement this?

Hi, @nickwhitman1993 ,
Yes, of course!  I assume you start with an H1 field and you want an H1 representation of the directional derivative.  Is this correct?
Mark

@mlstowell
This is correct. I start with an H1 field. I just need to be able to obtain the derivatives in the x, y, z directions at the same points as represented in the H1 field. I was able to code up the current fairly easily, so I just need that last part to be able to make my first estimate of the thermal conductivity.
Out of curiosity, what would the L2 implementation entail? Would this implementation be present at the same grid points as the H1 field? Apologies for my ignorance.
Thanks again Mark, you have been extremely helpful throughout this project!
Nick

Hi, @nickwhitman1993 ,
My pleasure.  L2 fields are discontinuous so they would not be suitable for obtaining values at H1 grid points (many of which are on element interfaces).  The advantage of L2 representations is that they can be computed locally using only information from a single element at a time.  You essentially loop over the elements and compute the values you need within each element and you're done.
On the other hand if you need values on element interfaces, like many H1 grid points, you need to compute those values using weighted averages for best results. This is done in two stages; integrate the quantity of interest multiplied by each basis function in turn, divide by the integrals of the basis functions multiplied by each other.  In this context the first step is a matrix-vector product and the second is a matrix solve.  Assume you start with your field of interest u stored as an H1 Gridfunction.
ConstantCoefficient one(1.0);

BilinearForm Dx(h1_fespace);
BilinearForm Dy(h1_fespace);
BilinearForm Dz(h1_fespace);

Dx.AddDomainIntegrator(new DerivativeIntegrator(one, 0));
Dy.AddDomainIntegrator(new DerivativeIntegrator(one, 1));
Dz.AddDomainIntegrator(new DerivativeIntegrator(one, 2));

Dx.Assemble(); Dx.Finalize();
Dy.Assemble(); Dy.Finalize();
Dz.Assemble(); Dz.Finalize();

BilinearForm m(h1_fespace);
m.AddDomainIntegrator(new MassIntegrator(one));
m.Assemble();
m.Finalize();

GridFunction ux(h1_fespace);
GridFunction uy(h1_fespace);
GridFunction uz(h1_fespace);

Vector rhs(h1_fespace.GetVSize());
DSmoother M(m);

Dx.Mult(u, rhs);
PCG(m, M, rhs, ux, 1, 200, 1e-12, 0.0);

Dy.Mult(u, rhs);
PCG(m, M, rhs, uy, 1, 200, 1e-12, 0.0);

Dz.Mult(u, rhs);
PCG(m, M, rhs, uz, 1, 200, 1e-12, 0.0);

I think something like this should work.  Feel free to ask for clarification if any of this is unclear.
Best wishes,
Mark

Hi again @mlstowell,
In the context of what you provided above, I was able to implement this into my code with a few changes:
 //Compute Gradient of phi?
  ConstantCoefficient one(1.0);

  BilinearForm Dx(&fes);
  BilinearForm Dy(&fes);
  //BilinearForm Dz(&fes);

  Dx.AddDomainIntegrator(new DerivativeIntegrator(one, 0));
  Dy.AddDomainIntegrator(new DerivativeIntegrator(one, 1));
  //Dz.AddDomainIntegrator(new DerivativeIntegrator(one, 2));

  Dx.Assemble(); Dx.Finalize();
  Dy.Assemble(); Dy.Finalize();
  //Dz.Assemble(); Dz.Finalize();

  BilinearForm m(&fes);
  m.AddDomainIntegrator(new MassIntegrator(one));
  m.Assemble();
  m.Finalize();

  GridFunction ux(&fes);
  GridFunction uy(&fes);
  //GridFunction uz(&fes);

  Vector rhs(fes.GetVSize());
  DSmoother M(m);

  Dx.Mult(phi, rhs);
  PCG(m, M, rhs, ux, 1, 200, 1e-12, 0.0);

  Dy.Mult(phi, rhs);
  PCG(m, M, rhs, uy, 1, 200, 1e-12, 0.0);

  //Dz.Mult(phi, rhs);
  //PCG(m, M, rhs, uz, 1, 200, 1e-12, 0.0);`

Which seems to work - except one minor thing. It doesn't quite like the way I am asking it to implement DSmoother M(m). I have also tried something like DSmoother M(&m) but also not quite working.
I get something back at compilation like:
/nfs/stak/users/whitmann/broom2/src/Broom.cpp:403:20: note: candidates are: In file included from /nfs/stak/users/whitmann/buildBroomParClang/TPL/include/linalg/linalg.hpp:24:0,                  from /nfs/stak/users/whitmann/buildBroomParClang/TPL/include/mfem.hpp:35,                  from /nfs/stak/users/whitmann/broom2/src/Broom.cpp:15: /nfs/stak/users/whitmann/buildBroomParClang/TPL/include/linalg/sparsesmoothers.hpp:70:4: note: mfem::DSmoother::DSmoother(const mfem::SparseMatrix&, int, double, int)     DSmoother(const SparseMatrix &a, int t = 0, double s = 1., int it = 1);     ^ /nfs/stak/users/whitmann/buildBroomParClang/TPL/include/linalg/sparsesmoothers.hpp:70:4: note:   no known conversion for argument 1 from ‘mfem::BilinearForm’ to ‘const mfem::SparseMatrix&’ /nfs/stak/users/whitmann/buildBroomParClang/TPL/include/linalg/sparsesmoothers.hpp:66:4: note: mfem::DSmoother::DSmoother(int, double, int)     DSmoother(int t = 0, double s = 1., int it = 1)     ^ /nfs/stak/users/whitmann/buildBroomParClang/TPL/include/linalg/sparsesmoothers.hpp:66:4: note:   no known conversion for argument 1 from ‘mfem::BilinearForm’ to ‘int’ /nfs/stak/users/whitmann/buildBroomParClang/TPL/include/linalg/sparsesmoothers.hpp:55:7: note: mfem::DSmoother::DSmoother(const mfem::DSmoother&)  class DSmoother : public SparseSmoother        ^ /nfs/stak/users/whitmann/buildBroomParClang/TPL/include/linalg/sparsesmoothers.hpp:55:7: note:   no known conversion for argument 1 from ‘mfem::BilinearForm’ to ‘const mfem::DSmoother&’ /nfs/stak/users/whitmann/buildBroomParClang/TPL/include/linalg/sparsesmoothers.hpp:55:7: note: mfem::DSmoother::DSmoother(mfem::DSmoother&&) /nfs/stak/users/whitmann/buildBroomParClang/TPL/include/linalg/sparsesmoothers.hpp:55:7: note:   no known conversion for argument 1 from ‘mfem::BilinearForm’ to ‘mfem::DSmoother&&’
From taking a look at some of my other files, it seems like the DSmoother argument needs to be a SparseMatrix. I added this line:       SparseMatrix &Q = m.SpMat();       DSmoother M(Q,1);
which allowed it to compile, but at run-time I get Assertion failed: (width == x.Size()) is false: Input vector size (0) must match matrix width (441)  ... in function: virtual void mfem::SparseMatrix::AddMult(const mfem::Vector&, mfem::Vector&, double) const  ... at line 485 of file: linalg/sparsemat.cpp
Do you have any thoughts on what might need to be done to fix this? Sorry for the lengthy reply.
Thanks again,
Nick

Hi, @nickwhitman1993 ,
The thing that jumps out at made is the declaration of Q as a reference to a SparseMatrix.  Technically m.SpMat() returns a const reference so Q may be a shallow copy (or something worse) of the object that you need.  Try to declare it as:
const SparseMatrix &Q = m.SpMat();

If that doesn't work we may need to dig deeper.
Best wishes,
Mark

Hi @mlstowell, sorry for the delay here.
I did try both the const declaration and not. I believe neither was working. I will test this again to ensure what I am telling you is correct and note back here shortly.
Would you have any further thoughts on where we could dig if so?\
Thanks again,
Nick

@mlstowell
I believe I have corrected the error. Please neglect the above message. I do however receive console output, i.e.
Average reduction factor = 0.44786226    Iteration :   0  (B r, r) = 0.00043460275
Do you know if I can suppress this?
Nick

As another note, I keep getting an error in the PCG subroutine:
Assertion failed: (IsFinite(nom)) is false: nom = -nan
 ... in function: virtual void mfem::CGSolver::Mult(const mfem::Vector&, mfem::Vector&) const
 ... at line 320 of file: linalg/solvers.cpp

Any thoughts?
Nick

Hi, @nickwhitman1993 ,
The "Average reduction..." messages come from the conjugate gradient solver.  They can be controlled by the fifth argument to the PCG function.  Typical values are 0, 1, or 2 with higher values generally yielding more information.  If you want no output at all you can try 0 but -1 might be necessary (I'm afraid this is not well documented).
My guess regarding the NAN is that the solution vectors ux and uy should be initialized to zero before calling PCG.  I believe these are being accessed to compute an initial residual vector (res = rhs - M*ux for example) but since they aren't initialized they could be introducing NANs.
Best wishes,
Mark

Hi again @mlstowell,
I am very delayed in telling you this, but I just wanted to chime in and tell you that I believe everything is working as promised. The issue, I believe, is that I tried to calculate the gradient after every iteration and wasn't clearing the memory somehow. Maybe this helps somewhere down the road.
I wanted to say thank you again for all your help getting this working over the past few months. I sincerely appreciate your time.
Best wishes,
Nick


DRILL-6290: Refactor TestInfoSchemaFilterPushDown tests to use PlanTestBase utility methods

Details in DRILL-6290.

@vdiravka please review.

+1
Thank you for improving the old code


Send insert command using UDP / TCP

Currently scte35-inserter only allows predefined inserts.
We propose to allow to trigger SCTE-35 insertion using TCP/IP or sidecar file, where a call with PTS and SCTE35 encoded string is sent.

Thank you for the proposal. We currently does not have that in the roadmap but unless you intend to provide a pull-request for it we can build it on a time-of-material basis.


Space Inventor: Cleanups

Hi,
I have gathered a lot minor commits into one big pull-request.
Please note the following things:
added csp_hex_dump()
moved csp/crypto headers into include/csp/crypto - I have moved these so drivers written outside the libcsp source tree can benefit from these functions.
added a new clock_get_time64 interface to get nanosecond clock (this is used for better timesync feature comming in future csp versions)
Note: the wscript now uses the freertos instead of the freertos_h include (this may break build for gomspace?)
Note: I have disabled the cflags from the wscript also. I can't remember right now what they were needed for, but they conflicted with our libasf (which defines the cflags for the chip).

@perwulff Can we get some comments on this, in particular:

Note: the wscript now uses the freertos instead of the freertos_h include (this may break build for gomspace?)
Note: I have disabled the cflags from the wscript also. I can't remember right now what they were needed for, but they conflicted with our libasf (which defines the cflags for the chip).

I know you mentioned a way for waf to identify the top level, but maybe that could be added in a future commit?
If i don't hear any objections, i will merge this soon. We need to stabilize and think of our due release.

Woops, didn't mean to approve everything...

I have reverted the wscript, added some return value documentation requested by jeppe, and removed the 64-bit clock type.


Test against new php versions

I think you need to remove PHP 7.4 for now, the tests are not passing.


Blog - A Strategy to handle change in IT

From the premise that the future is unpredictable and that change is a certainty.

open source
open standards
vendor lock-in
loose coupling
technical debt
architecture guardrails instead of gate

nah...


Drop support for building 32-bit Apple docs

Rust's RFC 2837 demoted the i686-apple-darwin target to Tier 3, so that target will not be downloadable anymore through rustup starting from Rust 1.42.0. The change will also land on nightly soon, and that will prevent docs.rs from updating the rustwide toolchain.
This PR removes i686-apple-darwin from the list of targets we build, and adds code to remove it automatically from existing rustwide workspaces.
This is ready for review, but we should hold off deploying it until we announce the demotion on the blog.

We can merge this. Are you fine with the current state of the code @jyn514?

I have a nagging feeling I missed something ... but the code itself looks fine, I'm ok with merging.


Improve config reload

related to https://github.com/prometheus/prometheus/issues/1149

Hey, thanks again! I haven't taken a closer look yet because of weekend and now travel to NYC (VelocityConf), but happy to do so later the upcoming week, when I get to PRs again. If others like @fabxc haven't gotten to it yet by then!

funny, I recall that I fixed it. anyway, something else? what about centralizing the whole thing (from the discussion in the relevant issue)?

@juliusv brought up the valid point that one would have to attach the parsed rule files to the config as they could otherwise be changed after validation and before being loaded again in the rule manager.
So I hit a very similar situation in the Alertmanager and went for the central approach. In retrospect I'm not sure whether passing down a whole config for the sake of having a unified interface was a good idea to begin with as the number of components is reasonably low.
https://github.com/prometheus/alertmanager/blob/fabxc/clean/main.go#L96-L122
Here the validation happens before 113 and everything after that are operations that are guaranteed to succeed.
What do you think about a similar solution for Prometheus itself?
The error-free updating logic could still remain in the components itself. Just with more specialized methods that only receive the portion of config/data they need.

Well, any solution would have his trade-off, but I agree with your approach, it will be more readable and concise. As long as there are no that much components, we can relinquish the interfaces vision.

Is this PR still relevant?

I believe this has all been redone since.


*** TEST Update virtual-networks-arm-asm-s2s.md

Hi @mkarbarz, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.azure.com.

TTYL, AZPRBOT;

@mkarbarz, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, AZPRBOT;

@mkarbarz When you have completed your test, please close this pull request.  Thank you.


Elanzel architecture center

@elanzel : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

Thanks for your review @simonesavi and working through the two PRs on this article @elanzel, merging now.


Use mutable workspace provider for file dependencies

So transforms of file dependencies are not cached between builds in the in-memory cache.
Fixes #15604.

@bot-gradle Test this

Sorry I don't understand what you said, please type @bot-gradle help to get help.

Currently I support the following commands:

@bot-gradle test {BuildStage} to trigger a build

e.g. @bot-gradle test SanityCheck plz
SanityCheck/CompileAll/QuickFeedbackLinux/QuickFeedback/ReadyForMerge/ReadyForNightly/ReadyForRelease are supported
test this means test ReadyForMerge
SanityCheck can be abbreviated as SC, ReadyForMerge as RFM, etc.

@bot-gradle help to display this message
@bot-gradle clean to clear the conversation history

@bot-gradle test this

OK, I've already triggered ReadyForMerge build for you.


Build error: The command powershell exited with code 9009.

When trying to build in Batch mode in Release configuration projects NativeCore x86 and x64, Reclass.NET x86 and x64 and Reclass.NET_Launcher, I get this error:

Severity	Code	Description	File	Line	Suppression State
Error		The command "powershell -Command "((Get-Date).ToUniversalTime()).ToString("yyyy/MM/dd HH:mm:ss") | Out-File 'D:\Projects\Reversing\ReClass.NET\ReClass.NET\Resources\BuildDate.txt'"
" exited with code 9009.

When I remove the pre-build powershell command from the pre-build events, I start to get this error:

Severity	Code	Description	File	Line	Suppression State
Error		Invalid Resx file. Could not find file 'D:\Projects\Reversing\ReClass.NET\ReClass.NET\Resources\BuildDate.txt'. Line 381, position 5.	D:\Projects\Reversing\ReClass.NET\ReClass.NET\Properties\Resources.resx	381

After I manually create empty BuildDate.txt file at the specified path, the solution seems to start to build correctly and I can start the program. Don't know if it will function correctly.
If I manually add the BuildDate.txt file and restore the powershell command, I still get the 9009 error when trying to build.
Powershell is functional:

Windows 10, Visual Studio 2019 and this problem occurs on the last commit of the master branch at the time of writing.

Does powershell work in an old cmd window?

My search engine tells me that 9009 means not found.

Thanks. I was missing the path to the folder that has powershell in the Path environment variable. I tried launching the powershell's command line, it worked and I tried running powershell script from inside Visual Studio but I haven't tried running powershell from the command line before creating the issue.
After I added the folder to the Path environment varialble and restarted Visual Studio, the solution started to build with the powershell script.


"Fewer hooks" error when I deleted a post

When I delete a post by clicking the contextual menu on the post from my feed on the web app, I get a minified React error that when decoded translates to, "Rendered fewer hooks than expected. This may be caused by an accidental early return statement."
To Reproduce
Steps to reproduce the behavior:

Go to the web app
Find one of your posts
Delete it
Get an error

Expected behavior
No error, ideally.
Screenshots

Details

Platform: Desktop Chrome Mac
Platform version: Okay, you got me, it's actually Arc 0.105.3, which is based on Chromium 113.0.5672.126
App version: 1.29.0

Additional context
I assume an alligator ate it. Or a crocodile?

Thanks for reporting this bug! We should get to it pretty soon.

I got #823 in to fix this. Should be part of the next release. @chriszs Thanks for reporting!


[Update Snap] Rarime

Checklist
All items in the list below needs to be satisfied.

[x] Is the summary of the change documented in this ticket?
[x] Has a MetaMask Snaps team member reviewed whether the changes need to be vetted?
[x] If there are changes that need to be vetted, attach a description and the relevant fixes/remediations to this issue.
[ ] The corresponding pull request in this repo has been merged.

This update fixes an issue with saving large Verifiable Credentials to Compose DB(ceramic) by migrating them to another an new database with updated schema.

Most of the changes in the Snap are related to Ceramic and VC Model change. Other than this, no direct functionality of the Snap has been modified. The changes need not be vetted/audited from a Snap point of View.
The full diff between the last published version and the new one is here:
https://github.com/rarimo/rarime/compare/2.0.1...2.0.2
For this update, the previous ver (2.0.1) will be removed as the Snap will be using a new Verifiable Credential Model on Ceramic, and the new version 2.0.2 will be allowListed.

Closed in #417


Transfer jupyterlab-spreadsheet-editor

Target
jupyterlab-spreadsheet-editor enables editing CSV/TSV files using third-party Jspreadsheet editor; in future should also allow to edit MS spreadsheet etc; different from jupytercalpoly/jupyterlab-tabular-data-editor which tries to implement the spreedsheet editor itself using lumino grid; I did not have time to re-implement it from scratch so I went for well tested third-party solution (which is easier in maintenance IMO, but provides less integration with JupyterLab, e.g. the context menus are styled slightly differently).

The code repository: https://github.com/krassowski/jupyterlab-spreadsheet-editor
The npm package on npmjs.com (if available): https://www.npmjs.com/package/spreadsheet-editor
The python package on pypi.org (if available): https://pypi.org/project/jupyterlab-spreadsheet-editor/
The conda-forge recipe on https://github.com/conda-forge (if available): TBD
The documentation website (if available): None

Reason
I want to encourage others to contribute to jupyterlab-spreadsheet-editor and make it possible for others in jupyterlab-contrib to make patch releases in case if I am away on holiday or something. I will continue developing the extension when I get chance to (i.e. the other kind of holidays).
Checklist

[x] You are owner of the repository you want to transfer
Which role do you want to have in the jupyterlab-contrib organization (check one of):

[x] Help with all jupyterlab-contrib repositories
[ ] Help with the transferred repository only
[ ] I don't have time anymore to keep helping the project

The transferred repository (check one of):

[ ] Must keep the extension name
[ ] Cannot use the original name
[x] I don't care

[x] The code is available under a Open Source license
[x] A license file is present in the code repository
[x] The extension is working on at least JupyterLab 2.x
[x] The repository contains a README file describing:

[x] The provided feature(s)
[x] A picture illustrating the feature(s)
[x] The available options/settings (if applicable)
[x] How to install the extension
[x] optional a Binder link to test the feature online

[ ] optional The frontend code (Typescript/Javascript) is tested
[ ] optional The backend code (Python) is tested
[x] I certify the extension is not using exclusively paid-plan of any commercial service
[x] I will help the transfer process on the code repository AND the built packages

You must enable two-factor authentication on GitHub

Thanks @krassowski for proposing your help and this transfer proposal.
You should have received an invitation to join the organization as owner so you can transfer the project to this organization.
Could you please add the jupyterlab-contrib-bot as maintainer to the PyPI project and add the package to the jlab-contrib organization in npm?


HoC home page: "New? Start Here!" is below the fold in 1280x1024

There seems to be room in the header to move the content up.

this has been fixed and can be seen on the https://arcade.makecode.com/hour-of-code-2022


update aws-go-sdk to latest release for support of unpublished govcloud regions with custom endpoints aws/aws-sdk-go#378

updates sdk needed to support some govcloud implementations

i'm not a glide expert, but it looks like you deleted a file. why?

Glide.lock gets created during the make process from the glide.yaml file when glide up gets run (it’s a required step in the Makefile) - it’s just the complete (resolved) dependency tree. If the glide.lock file is already there and doesn’t match the glide.yaml file, the build fails (at least on my local box) because there appears to be a validation in there.
If you want I can check a glide.lock file in that matches the glide.yaml, but it gets generated during the make process so it’s probably not necessary.  Just let me know.
from the docs…:
glide update (aliased to up)
Download or update all of the libraries listed in the glide.yaml file and put them in the vendor directory. It will also recursively walk through the dependency packages to fetch anything that's needed and read in any configuration.
$ glide up
This will recurse over the packages looking for other projects managed by Glide, Godep, gb, gom, and GPM. When one is found those packages will be installed as needed.
A glide.lock file will be created or updated with the dependencies pinned to specific versions. For example, if in the glide.yaml file a version was specified as a range (e.g., ^1.2.3) it will be set to a specific commit id in the glide.lock file. That allows for reproducible installs (see glide install).
To remove any nested vendor/ directories from fetched packages see the -v flag.
glide install
When you want to install the specific versions from the glide.lock file use glide install.
$ glide install
This will read the glide.lock file and install the commit id specific versions there.
When the glide.lock file doesn't tie to the glide.yaml file, such as there being a change, it will provide a warning. Running glide up will recreate the glide.lock file when updating the dependency tree.
If no glide.lock file is present glide install will perform an update and generate a lock file.
To remove any nested vendor/ directories from fetched packages see the -v flag.
--
Keith McClellan
Technical Lead, Federal Programs at Mesosphere
<EMAIL_ADDRESS>(202) 683-8558 direct
On November 16, 2016 at 8:56:32 PM, James DeFelice<EMAIL_ADDRESS>wrote:
i'm not a glide expert, but it looks like you deleted a file. why?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I added in my glide.lock file from my build env in case you want it

/cc @branden

I think you’re right - that was an artifact from when I pulled from master, which was rex ray 0.4 (which requires lib storage).  I’ll fix now.
--
Keith McClellan
Technical Lead, Federal Programs at Mesosphere
<EMAIL_ADDRESS>(202) 683-8558 direct
On November 17, 2016 at 10:38:22 AM, James DeFelice<EMAIL_ADDRESS>wrote:
/cc @branden
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@jdef @branden should be fixed

Thanks @keith-mcclellan , but we are going to need a bit more before we can hope to land this.

Unit test showing that the endpoint URL change works with this new version
Integration test showing that DC/OS upgrades actively using RexRay do not break going from DC/OS 1.8 to 1.9 with this new version
Changelog summary for the DC/OS 1.9 release notes
Separate PR for integrating with DC/OS

@branden what are your thoughts / does this seem like a realistic set of requirements in order for us to land this change?

We'll definitely need a PR against DC/OS that successfully builds and integrates this REX-Ray before it can merge. If it passes the existing external volume test (which uses Amazon EBS) then I'll be comfortable with it.
I don't think a unit test would be appropriate since the unit being changed here is in a third-party lib. Integration testing the specific behavior of using a different AWS URL will be tricky since that belongs to the environment in which we run the tests. I don't think we test against GovCloud or anything similar.
Testing that this doesn't break external volumes after an upgrade would make more sense as part of our other work to make upgrade test clusters more reflective of a production environment. That's best done by someone who's familiar with the tests in dcos/dcos, and it's coverage we could use regardless of this change.

I did the PR against DC/OS (https://github.com/dcos/dcos/pull/965)
As for the rest of it, I don’t have any suggestions - you probably don’t want me writing an integration test anyway!
--
Keith McClellan
Technical Lead, Federal Programs at Mesosphere
<EMAIL_ADDRESS>(202) 683-8558 direct
On November 22, 2016 at 8:46:08 PM, Branden Rolston<EMAIL_ADDRESS>wrote:
We'll definitely need a PR against DC/OS that successfully builds and integrates this REX-Ray before it can merge. If it passes the existing external volume test (which uses Amazon EBS) then I'll be comfortable with it.
I don't think a unit test would be appropriate since the unit being changed here is in a third-party lib. Integration testing the specific behavior of using a different AWS URL will be tricky since that belongs to the environment in which we run the tests. I don't think we test against GovCloud or anything similar.
Testing that this doesn't break external volumes after an upgrade would make more sense as part of our other work to make upgrade test clusters more reflective of a production environment. That's best done by someone who's familiar with the tests in dcos/dcos, and it's coverage we could use regardless of this change.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@keith-mcclellan Tests on https://github.com/dcos/dcos/pull/965 are looking good. Can you squash this branch into a commit with just the changes necessary for the aws-sdk-go update? There's a lot of churn in glide.lock and glide.yaml that doesn't look necessary.

Let's also get a link in the PR description to something that describes the change you're introducing with this update (an aws-go-sdk ticket, issue, PR, commit, etc.) for posterity.

Branden -
Which ones did you think were unnecessary in glide.yaml?
The glide.lock file is generated (can’t modify by hand) so I can’t really change that outside modifying the glide.yaml file and rebuilding.
Best,
Keith
--
Keith McClellan
Technical Lead, Federal Programs at Mesosphere
<EMAIL_ADDRESS>(202) 683-8558 direct
On November 29, 2016 at 4:45:54 PM, Branden Rolston<EMAIL_ADDRESS>wrote:
@keith-mcclellan Tests on dcos/dcos#965 are looking good. Can you squash this branch into a commit with just the changes necessary for the aws-sdk-go update? There's a lot of churn in glide.lock and glide.yaml that doesn't look necessary.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

This PR is related to: https://mesosphere.atlassian.net/browse/DCOS-11335 - RexRay 0.3.3 doesn't support changing the endpoint URL for AWS

It's not clear to me why any of the changes to glide.yaml are necessary, except for the one that bumps aws-sdk-go. Those other libs don't appear to be dependencies of it.
The removal and replacement of glide.lock can be squashed out, as well as the addition and removal of libstorage.
I'd like a link to the specific aws-sdk-go change that this PR is intended to introduce, both for posterity and so that I can verify it's being pulled in. It's especially important absent an automated test that covers this added behavior.

Branden - I went back through and tried to minimize the dependency changes.  FWIW, updating aws-go-sdk required an update to yaml.v2, and gofig needed to be locked to an older version to get rexray to build (was locked to latest/master, which was incompatible with the older rexray), locked to 0.1.4.
Was this everything you needed?

commits squashed and comments updated - was that everything? @branden

Sorry for the delay @keith-mcclellan, trying to track down someone who can review the changes to glide.lock.

Anyone with Go experience could hopefully do it.
Best,
Keith
--
Keith McClellan
Technical Lead, Federal Programs at Mesosphere
<EMAIL_ADDRESS>(202) 683-8558 direct
On Dec 6, 2016 3:54 PM, "Branden Rolston"<EMAIL_ADDRESS>wrote:

Sorry for the delay @keith-mcclellan https://github.com/keith-mcclellan,
trying to track down someone who can review the changes to glide.lock.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dcos/rexray/pull/2#issuecomment-265269167, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEgvLXo2Y1a1P_YLfZmqCAgg9K90oydVks5rFcuegaJpZM4K0twj
.

The changes in glide.lock are too much for what we want. I would just do the minimum change to it (pin the aws-sdk and gofig), but not let it bump all the git shas. That reduces the scope of the change.

cc @alberts can you have a look too please? Basically all we want is bump the sdk and the deps. Nothing else.

@spahl I can't figure out how to squash the updates in glide.lock - the dependency changes are all transient dependencies and not anything I'm explicitly calling out in the glide.yaml file.  As far as I can tell, those updates are upstream dependencies of the changes that were required to update the aws-go-sdk - my guess is they're being derived from the akutz sourced go libraries (goof, gofig, golf, goutil) that I had to unpin (or were already unpinned) to get rexray to build.
Regardless, this will probably require someone who knows Glide better than I do if we want to scope this change down further.

glide.yaml diff I came up with:
albert@localhost ~/repos/gopath/src/github.com/emccode/rexray (v0.3.3-dcos *+)$ git diff glide.yaml
diff --git a/glide.yaml b/glide.yaml
index bb45d13..7449d16 100644
--- a/glide.yaml
+++ b/glide.yaml
@@ -5,7 +5,7 @@ import:
     repo:    https://github.com/akutz/logrus
     vcs:     git
   - package: github.com/akutz/goof
-    ref:     master
+    ref:     v0.1.0
     vcs:     git
   - package: github.com/spf13/pflag
     ref:     b084184666e02084b8ccb9b704bf0d79c466eb1d
@@ -20,7 +20,7 @@ import:
     repo:    https://github.com/akutz/viper.git
     vcs:     git
   - name:    github.com/aws/aws-sdk-go
-    ref:     aac89266a9bc98ecbf440bb675e9dc95a94d8dc6
+    ref:     v1.5.5
     repo:    https://github.com/aws/aws-sdk-go.git
     vcs:     git
   - package: github.com/emccode/goscaleio
@@ -35,10 +35,10 @@ import:
     ref:     v0.1.1
     vcs:     git
   - package: github.com/akutz/gofig
-    ref:     master
+    ref:     v0.1.4
     vcs:     git
   - package: github.com/akutz/gotil
-    ref:     master
+    ref:     v0.1.0
     vcs:     git
   - package: github.com/go-yaml/yaml
     ref:     b4a9f8c4b84c6c4256d669c649837f1441e4b050

Many of the transitive dependencies seem to come from Godeps.jsons in the direct dependencies, so I don't think it makes sense to pin these in the top-level glide.yaml.
The only dependency not pointing to a specific commit in glide.yaml is golang.org/x/net, which points at master.
So my conclusions:

glide.yaml changes in this PR don't seem quite right. I think we do the ones above, then glide up.
It probably makes sense to pull in updated transitive dependencies that this brings in.
DC/OS build rexray with make build-linux-amd64 OFFLINE=1, so we need to add all the changes to the vendor/ folder to this PR after the glide up run.

On the plus side, if rexray works, we still have a reproducible build, since all the vendored deps are checked into the repo.

@alberts Rebuild incorporating your comments, and bumping AWS-GO-SDK to v1.6.2 for latest govcloud endpoints (ref: aws/endpoints: Add endpoint metadata to SDK #961)
Pinned gotil and goof in glide.yaml
checked in vendor dir for offline build
I also bumped aws-go-sdk to 1.6.2

I tested this using make build-linux-amd64 OFFLINE=1 as well, will update my dcos pr to get it to rebuild

dcos build associated with this is: https://github.com/dcos/dcos/pull/965

2.6 million lines of code added. Very nice. @keith-mcclellan can you confirm that https://github.com/dcos/dcos/pull/965 works? Looks like it passed automatic tests.

It definitely worked for my naive test case.
Best,
Keith
--
Keith McClellan
Technical Lead, Federal Programs at Mesosphere
<EMAIL_ADDRESS>(202) 683-8558 direct
On Jan 3, 2017 5:49 PM, "Albert Strasheim"<EMAIL_ADDRESS>wrote:

2.6 million lines of code added. Very nice. @keith-mcclellan
https://github.com/keith-mcclellan can you confirm that dcos/dcos#965
https://github.com/dcos/dcos/pull/965 works? Looks like it passed
automatic tests.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dcos/rexray/pull/2#issuecomment-270248207, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEgvLf9m8FQ0iEbCRkrCmN9kaGKL5hwTks5rOtCNgaJpZM4K0twj
.

There is some discussion from @spahl on https://mesosphere.atlassian.net/browse/DCOS-11335 about what needs to happen next here. In summary, we need two people who are willing to give their 'ship-it' on this... candidates are so far: @jdef @alberts and @branden. If anyone needs govcloud access  to do some testing before their are willing to sign off on this, please let me know and I can provide govcloud creds (Albert it looks like you already have an account).


Brush up Spell Effect: Qi Form

Note that rolling the damage from the effect hover does not currently have its damage changed via the DamageAlteration, but emitting the effect to chat first does work.


FIX: Point the zip-up step to the right binary's name for Mac OS X release

Pull Request Test Coverage Report for Build 569938527

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 79.142%

Totals

Change from base Build 569802705:
0.0%

Covered Lines:
406

Relevant Lines:
513

💛  - Coveralls


Add random sampling functionality (like R's terra::spatSample())

Thanks for a great package! This is a feature request for a function like R’s terra::spatSample() that enables random sampling of locations on a raster, optionally excluding NaN/missing values. Below is some quick-and-dirty example code which demonstrates the functionality, but is inefficient when NaNs should be ignored. Maybe this is already quick & easy to do in Julia? (I am coming from R.)
# Convert Raster to DataFrame, optionally dropping NaNs
function asDataFrame(; x::Rasters.Raster, na_rm::Bool = true)
    # Get DataFrame
    xyz = DataFrame(x)
    rename!(xyz, ["x", "y", "z"])
    # Filter non NA cells
    if na_rm
        xyz = xyz[.!isnan.(xyz.z), :]
    end
    return xyz
end

# terra::spatSample() replacement, via asDataFrame()
function spatSample(; x::Rasters.Raster, size::Int64 = 1, na_rm::Bool = true)
    xyz = asDataFrame(x = x, na_rm = na_rm)
    xyz[sample(1:nrow(xyz), size, replace = true), :]
end

No need for anything that complicated ;)
StatsBase gives us the sample method that will work on any AbstractArray. But we need to remove missing values from a raster first. To do that on pretty much any kind of iterator in Julia we just skipmissing(obj). Then collect turns it into an array again as sampling over a lazy iterator like skipmissing returns cant work efficiently, and sample needs some AbstractArray input anyway. But this is it, here giving you 100 samples:
using StatsBase
sample(collect(skipmissing(raster)), 100)

Thanks a lot for the quick response & detailed explanation! That is really helpful. That is a very nice solution for sampling values--but I was really hoping to sample both coordinates and values for non-NA cells, hence the use of DataFrame() above. Is there a simple way to do this that avoids converting the entire raster into a DataFrame? Thanks again!

It's probably no problem to convert, it's just DataFrame(raster), just use replace_missing first for now.
DataFrame(replace_missing(raster)).
Then probably sample will work too.

Fantastic - thank you!

I've been thinking about adding some sample function that mirrors syntax and outputs generated by extract - including the option to return coordinates. So good to know that more people want this and let's keep this issue open :)

This is now possible by calling Rasters.sample(x, n; skipmissing = true)!


Add: Form to use addPerson function to show usage of useTreeData

Closes #6250
Added usage of addPerson function in collection's useTreeData example
✅ Pull Request Checklist:

[x] Included link to corresponding React Spectrum GitHub Issue.
[ ] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[ ] Filled out test instructions.
[x] Updated documentation (if it already exists for this component).
[x] Looked at the Accessibility Practices for this feature - Aria Practices

You'll need to sign the CLA https://react-spectrum.adobe.com/contribute.html#contributor-license-agreement then close and reopen the PR to get it to take
The tests are failing because main hasn't been merged in, so I've done that for you.

GET_BUILD

GET_BUILD

GET_BUILD

Closing, if you're still interested in making this change, we can reopen it. Thanks!


List formatting Buttons Non-Responsive

Category

[X] Question
[ ] Typo
[ ] Additional article idea

Question
I have a custom list view format applied to a document library. Upon loading the page and clicking the buttons, there is no response. When I click format this view, copy and paste the code into the "format this view window", and preview or save, the buttons will work 1 or 2 times before becoming non-responsive once more. I have attached an image of a list item in the rendered view. Hyperlinks work without issue.

There are five buttons (see pic.):

link the document.
to begin the approval flow.
to delete.
to share.
to run another flow which limits access to named users.

I am at a loss as to why this has begun to occur now.

Your Developer Environment: Windows 10
Target Environment: SharePoint Online, GCC.
Framework: N/A.
Browser(s): Chrome, Edge.
Tooling: VS Code

@szametn We have taken a few fixes in this area over the last few months. Please try now and share if you are still running into this issue.


Handled declined payment or failed payment from cybersource

Handled declined payment or failed payment from cybersource

Summary by Sourcery
Handle declined or failed payments from Cybersource by checking the response status and returning early if the payment is declined. Add error handling for failed payment processing in the ViolationTicketV1DomainApiImpl class.
Bug Fixes:

Handle declined or failed payments from Cybersource by checking the response status and returning early if the payment is declined.

Enhancements:

Add error handling for failed payment processing in the ViolationTicketV1DomainApiImpl class, throwing an error if the payment response is null.

@sourcery-ai review


Implement gosec

According to the Linux Foundation, we should implement common vulnerability static code analysis in our build process.  gosec looks like a good solution.

This would be added in the Dockerfile here.

Thanks @u5surf!


Prevent downloading 2 coppies of a stream at once

[•ิ.•ั] I'm asking a question
[•ิ.•ั] I've looked through the README and FAQ for similar questions
[•ิ.•ั] I've searched the bugtracker for similar questions including closed ones

I wrote a .bat to automatically download all of the twitch channels listed in a file but if I have it scheduled to run every 10min it will open several copies of youtube-dl to download the same stream. My first thought was to check if a window with the stream name in it exists before opening youtube-dl but it turns out when you call youtube-dl from another cmd.exe window with: start cmd.exe /c yt-dlp.exe it doesn't show the stream url in the cmd window. I also considered checking for .part files but that seems to be unreliable because if youtube-dl exits incorrectly it can leave .part files. If there is a way to solve this within youtube-dl I sure can't find it and I can't think of a way to do it externally so any help would be much appreciated :)

Invent a flag file (can be zero-length).
In your batch file:

check whether that file exists and exit if it does
otherwise create the file and continue to run one instance of yt-dl, using the -a .../--batch-file ... option
finally delete the flag file.

@Hfinnell, did that help?


filtering sensor data according to date range - warnings revised

added date-boxes for filtering accordinga defined time period
warnings revised --> displayed in the foreground

debug logging level integrated
renaming variables according to clean-code-agreements
outsourcing convert datetime --> date-strings functionality

integration of load_config() method
method loads data from config file
within the config file is the timedelta (today - 14 days previously)


Added SwiftDataLoader to 'Other implementations'

Thanks for an awesome lib! 🎉
I have made a Swift version of the lib. 🚀
Feel free to come with feedback on the Swift implementation!

Coverage remained the same at 100.0% when pulling 40d01e85def6b8b017f14bbb078c386d8d192545 on kimdv:patch-1 into 420e62f2d424122bfad127d9a7f571d091563894 on facebook:master.


Adds Spring Feature Management to App Configuration

Adds new documents to App Configuration with instructions on how to use Feature Management in a Spring Boot Application.

@mrm9084, Microsoft authors and contributors need to make content contributions through the private repository so the content can be staged and validated by the current validation rules.
After you review staging and fix all validation issues, you can add the sign-off comment to let PR reviewers know that the updates are ready to be merged.
Read about authors working in the public repo for more information.
If you need help getting set up to work in the private repo, see Set up Git repository locally for documentation.
We'll close this PR. Thank you.