Older versions not coming up on NPMJS registry

When I try to install older versions of this module, I get file not found errors from NPM.
The following url should have the older version 0.8.1, for example: https://registry.npmjs.org/xlsx/-/xlsx-0.8.1.tgz
however I get back json:

{"error":"package could not be found."}

@deathgaze most of the older versions didn't pin dependency version numbers in package.json, causing problems in NPM 5.  See https://github.com/npm/npm/issues/17227 -- you should use a version after 0.9.2.
If you are using node, there was no breaking change since 0.8.  The major version jumped because of some browser-related interface issues (for example, there used to be a separate ods.js script, but that was causing issues with webpack so it was ultimately folded into the main xlsx.js and that triggered a version jump)


Allow "=" in autolinked urls.

I think it makes sense to support "=" in urls.
Without fix it doesn't work e.g. in this url (link stops after "comment_id"):
"https://www.facebook.com/yaroslav/posts/10152268742977405?comment_id=10152269203352405&offset=0&total_comments=38"

I think you're right! GitHub seems to allow =s too. Would you mind adding a test by the other autolinking tests?


Mongo DB Storage

Data Storage
We've changed from a postgres rdb to a mongo doc store! :tada:
Each pid record is a single document, identified by its handle. Each field in the pid record needs some metadata, which is now represented in a json:

index
type
data

for regular fields: format (always "string"), and value
for HS_ADMIN: format="admin", index=200, value=0.NA/{prefix}, permissions =<PHONE_NUMBER>11

ttl
timestamp

An example document is found below:
{
  "_id": "20.5000.1025/ABC",
  "normalisedSpecimenObjectId": "06211:BIRD-COLLECTION-1:RMS-123",
  "values": [
    {
      "index": 1,
      "type": "digitalObjectType",
      "data": {
        "format": "string",
        "value": "https://hdl.handle.net/21.T11148/532ce6796e2828dd2be6"
      },
      "ttl": 86400,
      "timestamp": "2024-06-28 04:32:04Z"
    },
    {
      "index": 203,
      "type": "normalisedPrimarySpecimenObjectId",
      "data": {
        "format": "string",
        "value": "06211:BIRD-COLLECTION-1:RMS-123"
      },
      "ttl": 86400,
      "timestamp": "2024-06-28 04:32:04Z"
    }
  ]
}

Indexing Local Identifiers
3 kinds of objects have local identifiers we can use to check if a handle has already been created for that kind of objects:

Annotations: annotationHash
specimens: normalisedSpecimenId
Media: mediaUrl (called primaryMediaId in the handle API)

When we create a specimen, we first search on normalisedSpecimenId to see if a handle has already been created for that object. By putting normalisedSpecimenId on the top-level of the mongo document, we can index the field and quickly find if the handle exists.
We don't do this for annotations or media, but I've included the functionality, if we want to make the check.
Changes to Updates
We no longer support partial updates. Now, the request must contain all fields required to build a new

We keep a few fields from the previous version: pidIssueDate, fdoType, license, PID, PidStatus. These fields will have their original timestamp
We increment pidIssueNumber
We replace all other fields, even if they are unchanged, updating the timestamp

Changes to Tombstone
Tombstoned records are not truncated anymore. Now, they have all the fields they had before they were tombstoned, but with additional tombstone metadata.
JsonSchemaValidator

When we allowed partial updates, we had separate validation protocols for updates and creates because we didn't require the user to provide all the fields for an update. Now, all fields are required for both kinds of requests, so the validator got simplified
Pretty sure we can remove this class honestly. We can the validator hassle by just accepting a specific object instead of a raw json. trying to keep this PR within its original scope though

Other

Cleaned up testsUtils
Refactored some names

Haven't tested large scale, but small tests are performing well. While we can't use the psql batch insert function anymore, i don't think handle creation will be a bottleneck. We now only insert one document instead of up to 30 rows per specimen. Updates are a lot better too, since we had something like update (ATTRIBUTE, where HANDLE = xyz and INDEX = 111) for EACH field we wanted to update. Now we just replace the document on the _id. And adding the local ids to the top-level is really good.
:partying_face:


Route schedule statistics

This PR:

[x] Closes #715.
[ ] Adds and passes relevant tests.
[ ] Has well-formatted code and documentation.

Notes:
Please read our contributing guidelines first.
In particular:

You must add tests when making code changes.
This keeps the code coverage level up, and helps ensure the changes work as intended.
When fixing a bug, you must add a test that would produce the bug in the master branch, and then show that it is fixed with the new code.
New code additions must be well formatted. Changes should pass the pre-commit workflow, which you can set up locally using pre-commit.
Docstring additions must render correctly, including escapes and LaTeX.
Finally, it is essential that all contributions in this PR are license-compatible with PyVRP's MIT license.
Please check that this PR can be included into PyVRP under the MIT license.

This needs to be cleaned up, added to a notebook, and I need to write tests. But I think the basic design is just about there now.

needs to be cleaned up

For this we'll probably want to revert to what we did before #241.

@leonlan could you have a look at this? I'm not sure about naming (e.g., VisitDatum- ehhhh) and I don't like the extra loop inside Route's constructor, but I also don't immediately see how we can get this data from the DurationSegment we're constructing there.


Development of switch component

Description
The switch component is part of the MVP of the SL Design System. A toggle switch is a digital on/off switch. Toggle switches are used for changing the state of system functionalities and preferences.
### Acceptence criteria
- [x] keyboard nav
- [x] default state
- [x] focus state
- [x] disabled state
- [x] disabled checked
- [x] icons in the thumb; default an x and a v, but should be configurable
- [ ] vertical layout
- [ ] cutom sized labels

examples
https://web.dev/building-a-switch-component/
https://gui-challenges.web.app/switch/dist/
Design
See issue #220
base component:
https://www.figma.com/file/SKFTFiiz7YK9E2TPfCan9o/Base-Components?type=design&node-id=1215-5016&t=2jHZV2tSrJ3EaE8U-0
in SL light theme:
https://www.figma.com/file/1oIVcD9VMbOtLXYp48ehsA/Sanoma-Learning-Light-mode?node-id=2180-7430

https://web.dev/building-a-switch-component/

Put on hold for now until we figure out the issue with the sizing of the labels; should they match with the size of the toggle or should they be left up to the user (like they are now for all other inputs).
If they are flexible; how is the alignment going to be?

Waiting till the new vertical layout is added to Figma:
#220

to do:

make sure no-label also gets added on load, not just on slot change.


[README] Fix import statement in example code

react-dazzl -> react-dazzle

[Current coverage][cc-pull] is 81.74%

No coverage report found for master at 73edb19.

Powered by Codecov. Last updated by [73edb19...c7eb16a][cc-compare]
[cc-base-branch]: https://codecov.io/gh/Raathigesh/Dazzle/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/Raathigesh/Dazzle/compare/73edb198a225dd9fbc27fd09e43c01bd0bf038a4...c7eb16a6f56137a2d62fdb2dd47a206be6e4bd8f
[cc-pull]: https://codecov.io/gh/Raathigesh/Dazzle/pull/5?src=pr

Merged. Thank you for the PR!


[#461] Move user_email in publish_wiki workflow to a secret

Close #461

What happened 👀
Move user_email in the publish_wiki workflow to a secret instead of publicizing it.
Insight 📝
n/a
Proof Of Work 📹
Just changed the usage, nothing special. 🚀

Should we consider using this action? Then we don't need to care about those values 🤔
More context: https://nimble-co.slack.com/archives/CSQG2QXFX/p1698199200680339

Closed this PR since the change is against this RFC.


Ethereum

Describe the requested icon
The Ethereum logo
Screenshots

Added currency-eth in v1.4.0!


App Configuration GA updates

When creating the resource in the Azure portal, I noticed the latest App Configuration create pane includes a pricing tier. That’s missing from the screenshot in this doc. Also, this quickstart never shows how to add or toggle the feature flag setting.

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

ID: 3de19743-3ef7-893b-d63f-d352cc4c105d
Version Independent ID: 30fe6283-c6a6-a571-2edc-60961d46d350
Content: Quickstart for adding feature flags to .NET Framework apps
Content Source: articles/azure-app-configuration/quickstart-feature-flag-dotnet.md
Service: azure-app-configuration
GitHub Login: @lisaguthrie
Microsoft Alias: lcozzens

@cecilphillip, Thanks for the feedback! We are taking a look into this and will get back to you soon.

@jpconnock screenshot update request FYI

Sorry -- should've updated here that I made these changes (updating the screen shot/surrounding text, and also adding the missing step 6) and they are now live. Thanks for reporting @cecilphillip and @GilbertZhangPfd!

#please-close


Warning: Failed prop type: The prop column.text is marked as required in HeaderCell, but its value is undefined.

Hi folks,
I recently started using your component and so for it worked like a charm and was very easy to integrate into my code. Thanks and keep up the good work!
Now, today I faced the following issue: I'm using react-bootstrap-table2 along with reactstrap, inserting a table into a modal to edit some stuff, in a master-slave fashion. The edits are made in the table itself, and in a field that should be numeric, greater than 0 and smaller or equal to 1 I've added a validator. Problem is: if I try to edit a cell where this validator is applied and set an invalid value  (for example, 2, or 'a'), it closes the modal just after showing the error message, and gives me the following error in the console:

Warning: Failed prop type: The prop column.text is marked as required in HeaderCell, but its value is undefined.
in HeaderCell (created by Header)
in Header (created by BootstrapTable)
in table (created by BootstrapTable)
in div (created by BootstrapTable)
in BootstrapTable (created by Context.Consumer)
in PaginationDataProvider (created by Context.Consumer)
in SortProvider (created by Context.Consumer)
in SearchProvider (created by Context.Consumer)
in DataProvider (created by BootstrapTableContainer)
in BootstrapTableContainer (at Forms.js:171)
in div (at Forms.js:166)
in ToolkitProvider (created by Toolkitprovider)
in Toolkitprovider (at Forms.js:159)
in div (at Forms.js:155)
in Forms (created by Route)
in Route (at App.js:18)
in div (at Layout.js:13)
in div (at Layout.js:11)
in Layout (at App.js:16)
in App (at src/index.js:18)
in Router (created by BrowserRouter)
in BrowserRouter (at src/index.js:17)
Here's the relevant code:

const weightValidator = (newValue, row, column) => {
    if (isNaN(newValue)) {
        return {
            valid: false,
            message: 'Peso deve ser numérico'
        };
    }
    if (newValue <= 0) {
        return {
            valid: false,
            message: 'Peso deve ser maior do que 0'
        };
    }
    if (newValue > 1) {
        return {
            valid: false,
            message: 'Peso deve ser menor ou igual a 1'
        };
    }

    return true;
}

const formsItemsColumns = [
    { dataField: 'number', text: '#', sort: true, editable: false },
    { dataField: 'description', text: 'Descrição' },
    { dataField: 'weight', text: 'Peso', validator: weightValidator },
    { dataField: 'actions', formatter: formsItemsActionsFormatter, editable: false },
];

<Modal centered={true} size="lg" isOpen={this.state.modalAddEdit} className={this.props.className}>
    <ModalHeader toggle={this.toggleAddEdit} close={closeBtn}>Formulário</ModalHeader>
    <ModalBody>
        <Form>
            <FormGroup row>
                Nome
                <Col sm={10}>
                    <Input name="form" id="formName" />
                </Col>
            </FormGroup>
            <FormGroup row>
                Código
                <Col sm={10}>
                    <Input name="code" id="formCode" />
                </Col>
            </FormGroup>

Items

            <Button className="float-right btn btn-success btn-circle btn-sm mb-1">

            </Button>
            <BootstrapTable
                BootstrapTable
                bootstrap4
                bordered={false}
                hover
                condensed
                keyField='id'
                columns={formsItemsColumns}
                data={forms[0].items}
                expandRow={expandFormsItemsRow}
                noDataIndication="Não há dados"
                cellEdit={cellEditFactory({ mode: 'dbclick' })} />
        </Form>
    </ModalBody>
    <ModalFooter>
        <Button color="success">Salvar</Button>
        <Button color="danger" onClick={this.toggleAddEdit}>Cancelar</Button>
    </ModalFooter>
</Modal>

I tried to create a custom input field but got the same error (just without the error message from the validator).
Any help would be greatly appreciated!
Cheers!

@liordino sorry for lately reply,
const formsItemsColumns = [
    { dataField: 'number', text: '#', sort: true, editable: false },
    { dataField: 'description', text: 'Descrição' },
    { dataField: 'weight', text: 'Peso', validator: weightValidator },
    { dataField: 'actions', formatter: formsItemsActionsFormatter, editable: false },
];

i think this issue is irrelevant to cell editing, but is just missing column.text for last column, because it is required.
Do let me know if above answer can not help you
BTW, the last column in your table is just a action column, so I encourage to set column. isDummyField as true
https://react-bootstrap-table.github.io/react-bootstrap-table2/docs/column-props.html#columnisdummyfield-bool
online example

Hi @AllenFang ! Thanks for your reply!
I tried your suggestion, and while the error that was being logged in the console doesn't occur anymore (in fact no errors are displayed in the console as off now) the behaviour is still the same: after setting an invalid value in a field that has an associated validator it shows the error message and shortly after that closes the modal. If I set a valid value, the model keeps opened. Here's a GIF showing what happens (first the behaviour with a valid value, and them with an invalid one):

Notice that as soon as the error appears bellow the input, the modal in which the table resides is closed.
I'm new to web development, but could this be happening because of a form submission anywhere?
Thanks again!

@liordino I think it's not a bug directly with react-bootstrap-table but if ok, would you please give me a simple and minimal example on codesandbox so that I can help you to debug, thanks.
here is an example you can start, but just remember to change the dependency version, thanks

Hi @AllenFang !
It was indeed a form submission, so nothing directly related to react-bootstrap-trable. Thank you so much for your help!


include clone commands in setup steps

Alternative to #61 that's a little more general

Done: f1ca461


Reverse sequence for LSTM model

Within LSTMEncoder, when attempting to compute the reverse outputs though the Bidirectional LSTM (aka the reverse_lstm layers), reversing the sequence does not return anything when input_mask=None (see below). This is because of incorrect indentation on L127.
https://github.com/songlab-cal/tape/blob/2e5bfa2249274392baf045df7de1490e92d47ed1/tape/models/modeling_lstm.py#L114-L127

Yup looks like a bug!


Bump PHP version to 5.6; auto-adjust array syntax

Changes

Update PHP version to 5.6
Update all long array format to short
Adjust scanners to for new PHP version

Note to reviewers: 99% of the changes here were completed automatically with a code formatter

@lbalmaceda - Moved the changes out of the changelog


Use execve instead of std::system

Currently, this pull request creates a class called Command that runs the command as input.
An example: Command("cd").arg("poac").output()
Closes #970

Closing as https://github.com/poac-dev/poac/pull/986 implemented the same issue. Thank you for your interest and time!


Adding link to dwdweather2

Project name: dwdweather2
Project website/repository: https://github.com/panodata/dwdweather2
License: MIT License

Checklist for new projects:

[ ] Project is certified awesome

[x] The project is open-source and accessible.
[x] Searched the existing entries to make sure this is not a duplicate.
[x] Contains only a single addition (make separate PRs if adding more than one).
[x] Added the link to the bottom of the relevant category.
[ ] Created a new category only if necessary.

[x] Link the repo rather than the website/documentation
[x] Installation instructions present
[ ] Documentation looks great
[ ] Examples or Tutorials to follow
[ ] Tests and Travis CI running
[x] Development status of the project is communicated

[ ] Easily Accessible
[ ] Comprehensive and Appealing

[x] Add icon from the media/icon/ folder if applicable (like ![Python](media/icon/python.png) ).
[x] Checked spelling and grammar.
[x] Removed trailing whitespace and periods.
[x] Confirm the dash – is not a minus -.
[ ] Used title-casing (AP style) for project name.

This package is really helpful for downloading open data from the german weather service. However, I see various caveats to placing an awesome label as of now. The maintainers are, however, working on some of them:

Most important to me: Their Python3 compatibility isssue is still open (still Py2.7)
docs are too weak to me
so do the unit tests

Further, I am not sure about the category. It's really closely related to Datasets but is, however, not a dataset. Would fit, it the category would be called 'Datasets & Data acquisition'

I like the idea of renaming to 'Datasets & Data acquisition'. I think we have to wait for automated tests, documentation and python 3 before approving this PR.

Wrong button, did not mean to close it..

This one is very close to awesome, but we do have not (yet) awesome links in the list, marked by the absence of the "awesome" icon, just like this one. Therefore I would be happy to include it and add the awesome icon once the maintainers implemented tests.

I agree. I will drop an issue at dwdweather2 informing them about this list suggest improvements, once the PR is approved and the link is actually there. @schymans, what is your opinion on renaming 'datasets' to 'datasets & data aquisition' and push the link there?

I am not sure about mixing data sets and data download scripts in the same category, as it would be nice to have an overview of independent datasets. How about adding another category, "Data mining"?

I am not sure about mixing data sets and data download scripts in the same category, as it would be nice to have an overview of independent datasets. How about adding another category, "Data mining"?

Yeah, that's better. Just didn't feel right to put everything that is not a model under miscellaneous. I'll update.

As a quick update: The authors just closed the issue to move to Python 3, so dwdweather2 is now supporting Python > 3.3. Thus I think we can merge the link into the list.


This provider is not recognised by eodag: wekeo

Hi,
When I do: dag.set_preferred_provider(PROVIDER.lower())
I got the following error: This provider is not recognised by eodag: wekeo
I would like to know if the wekeo provider has been removed from version 2.3.1?

Hello @carolinepacheco , yes wekeo has been completely removed from the providers list with #295. This was released in 2.3.0b1.
This provider was deprecated from a long time ago, and never got more than a partial support in eodag.
See also #57: even using the latest HDA API, too few metadata were available (lacks of product geometry, ...)

Hi @sbrunato
Ok. Thank you so much for your reply.


changes() error when creating new db

We're using rethindbdash with Feathers JS service and all is peachy except when we create a new DB and table. If we do this we get:
Unhandled rejection ReqlOpFailedError: Table `taco.messages` does not exist in:
r.db("taco").table("messages").changes()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

    at Connection._processResponse (C:\projects\vue\server\node_modules\rethinkd
bdash\lib\connection.js:395:15)
    at TLSSocket.<anonymous> (C:\projects\vue\server\node_modules\rethinkdbdash\
lib\connection.js:201:14)
    at emitOne (events.js:96:13)
    at TLSSocket.emit (events.js:188:7)
    at readableAddChunk (_stream_readable.js:176:18)
    at TLSSocket.Readable.push (_stream_readable.js:134:10)
    at TLSWrap.onread (net.js:548:20)

After that everything is fine (because the table and db exist) . So it looks like something is awry with not waiting for a newly created db. If we add a new table to an existing DB, we do not get the error. Here is our creation code:
...
init (opts = {}) {

    return this.r.dbList().contains(this.options.db) // create db if not exists
      .do(dbExists => this.r.branch(dbExists, {created: 0}, this.r.dbCreate(this.options.db)))
      .run().then(() => {
        return this.db(this.options.db).tableList().contains(this.options.name) // create table if not exists
          .do(tableExists => this.r.branch(
            tableExists, {created: 0},
            this.db(this.options.db).tableCreate(this.options.name, opts))
          ).run();
      });
  }
...

Even if we execute this in isolation and do not make any additional service calls, we still get the error. So it seems the issue is not waiting for DB creation to finish before trying to call changes() on it.

It's because a table can be listed but not ready yet. I think you have to look at the table_status table to be able to safely create a feed.

It would seem that is correct. It we add r.db(db).table(t).wait({waitFor: 'ready_for_writes'}) to the creation code, it works without the error. We're still getting used to rethink, but it seems odd to have to do this in app code instead of the driver or ideally rethinkdb itself handling something like this behind the curtain. I guess with great power comes great responsibility. Anyway it works so closing this and thanks.

Actually we we're mistaken. We did not get the error because the db existed, which falsely led us to believe this worked. If the DB does not exist, calling wait on the table makes the error change from the table not existing to the DB not existing. Calling wait on the DB does not have any effect and the error persists stating the db does not exist. So it looks like there is still something odd here somewhere:
Unhandled rejection ReqlOpFailedError: Database `taco` does not exist in:
r.db("taco").table("messages").changes()
^^^^^^^^^^^^

   at Connection._processResponse (C:\projects\vue\server\node_modules\rethinkd
bdash\lib\connection.js:395:15)
   at TLSSocket.<anonymous> (C:\projects\vue\server\node_modules\rethinkdbdash\
lib\connection.js:201:14)
   at emitOne (events.js:96:13)
   at TLSSocket.emit (events.js:188:7)
   at readableAddChunk (_stream_readable.js:176:18)
   at TLSSocket.Readable.push (_stream_readable.js:134:10)
   at TLSWrap.onread (net.js:548:20)

This seems fixed using the latest Feathers JS service provider. From what we can tell, it looks like queries were getting executed before service initialization occurred in older versions of the service provider. That seems to be addressed now in the new service structure, so this issue no longer occurs.


Add a playbook to decommission a compute node

this playbook is a draft!
Closes /osism/issues/issues/414
Signed-off-by: Mathias Fechner<EMAIL_ADDRESS>

I'm closing this. We'll do it again in the next release cycle.


ESM: remove a trace of AMD

This pr would like to remove a trace of AMD, performs a cleaning work!

@jrieken, I have fixed the "Compile & Hygiene" job of pipeline, could you please have time for taking a look at this?

Sorry for bother you..


IndexError on Image when switching to 3D in async and hovering over canvas

🐛 Bug Report
When switching to 3D view in async, the viewer displays the already-loaded 2D array in 3D space before replacing the data with a real 3D node, asynchronously. Before the full 3D texture is loaded, and since #7126, hovering the mouse over the canvas triggers get_value, which gets evaluated to _get_value_3d, which fails because the array is 2D at this moment.
Edit: link to the traceback 😅: https://gist.github.com/jni/a244d3931f8eadd8fc7e64887feff2ca
Summary:
File ~/projects/napari/napari/components/viewer_model.py:571, in ViewerModel._calc_status_from_cursor
--> 571     status = active.get_status(
File ~/projects/napari/napari/layers/base/base.py:2142, in Layer.get_status
-> 2142     value = self.get_value(
File ~/projects/napari/napari/layers/base/base.py:1404, in Layer.get_value
-> 1404         value = self._get_value_3d(
File ~/projects/napari/napari/layers/_scalar_field/scalar_field.py:687, in ScalarFieldBase._get_value_3d
--> 687     return self._get_value_ray(
File ~/projects/napari/napari/layers/_scalar_field/scalar_field.py:655, in ScalarFieldBase._get_value_ray
--> 655     values = im_slice[tuple(clamped.T)]
IndexError: too many indices for array: array is 2-dimensional, but 3 were indexed

💡 Steps to Reproduce

run NAPARI_ASYNC=1 napari https://uk1s3.embassy.ebi.ac.uk/idr/zarr/v0.4/idr0001A/2551.zarr/A/1/0
select a visible image layer
switch to 3D mode
hover the mouse over the canvas

💡 Expected Behavior
No error should be raised.
🌎 Environment
napari: 0.5.4rc4.dev0+gc73e9d5ec.d20241014
Platform: macOS-14.5-arm64-arm-64bit
System: MacOS 14.5
Python: 3.11.10 | packaged by conda-forge | (main, Sep 30 2024, 17:58:42) [Clang 17.0.6 ]
Qt: 5.15.8
PyQt5: 5.15.9
NumPy: 2.0.2
SciPy: 1.14.1
Dask: 2024.9.1
VisPy: 0.14.3
magicgui: 0.8.3
superqt: 0.6.7
in-n-out: 0.2.1
app-model: 0.3.0
psygnal: 0.11.1
npe2: 0.7.7
pydantic: 2.9.2

OpenGL:
  - GL version:  2.1 Metal - 88.1
  - MAX_TEXTURE_SIZE: 16384
  - GL_MAX_3D_TEXTURE_SIZE: 2048

Screens:
  - screen 1: resolution 1800x1169, scale 2.0

Optional:
  - numba: 0.60.0
  - triangle not installed
  - napari-plugin-manager: 0.1.3

Settings path:
  - /Users/jni/Library/Application Support/napari/all_f206df4881e2999baa22e6df448393db299c70d6/settings.yaml
Plugins:
  - midi-app-controller: 0.0.13.dev6+g180a6ba (2 contributions)
  - napari: 0.5.4rc4.dev0+gc73e9d5ec.d20241014 (81 contributions)
  - napari-console: 0.1.1 (0 contributions)
  - napari-ome-zarr: 0.6.1 (2 contributions)
  - napari-signal-selector: 0.0.6 (10 contributions)
  - napari-svg: 0.2.0 (2 contributions)
  - skan: 0.12.1.dev4+g3d5da88 (4 contributions)

💡 Additional Context
We can probably check the ._loaded state on the layer and return early, or we can make reshape the 2D array to make it very thin 3D before displaying it.

Somewhat related to #3391

CC @brisvag @andy-sweet

We can probably check the ._loaded state on the layer and return early, or we can make reshape the 2D array to make it very thin 3D before displaying it.

My first instinct was to return early if Layer.loaded is False. I can't work on this right now (i.e. with a test and this fix), but might be able to find some time this evening or over the long weekend here.

FYI @jni example doesn't work gracefully with 0.5.4/main
File "/Users/sobolp/micromamba/envs/napari-054-qt6/lib/python3.12/site-packages/napari/components/viewer_model.py", line 1297, in _open_or_raise_error
    raise NoAvailableReaderError(
napari.errors.reader_errors.NoAvailableReaderError: No plugin found capable of reading https://uk1s3.embassy.ebi.ac.uk/idr/zarr/v0.4/idr0001A/2551.zarr/A/1/0.
WARNING: QThread: Destroyed while thread is still running
09:54:55 : WARNING : MainThread : QThread: Destroyed while thread is still run

Needs napari-ome-zarr plugin.

Needs  napari-ome-zarr  plugin.
(I hate that I have to install this into every env all the time btw)

I'm not opposed to adding it to [all], or making some other extra, pip install "napari[peters-version]" 😃

Your link is to /0 within the zarr "container?"--is that correct?--which should be an array.

I got caught out by this as well but Juan was saying no, it's just one of the image sequences, but it's a full ome zarr dataset

Sorry, I didn't really get to this yet...
I did try to reproduce with the steps described above, and was unable to. But I think I understand the bug enough to write a deterministic reproducing test, given enough time. Will try to take another look later.

I did try to reproduce with the steps described above, and was unable to.

Oh interesting, I wonder if it's because you have lower latency to that dataset in NYC so you don't have time to trigger the error. 😅

Why does NAPARI_ASYNC=1 napari https://uk1s3.embassy.ebi.ac.uk/idr/zarr/v0.4/idr0001A/2551.zarr/A/1/0 result in a long beachball before anything with the layers starts happening? has that always been the case? I seem to recall seeing the layers with their loading things and not beachballs.

It's a fantastic question that probably should go in its own issue. I can reproduce it with napari v0.4.19 so this goes back a long way.


Pdf

Coverage decreased (-1.2%) to 43.984% when pulling 56ad4aaa22da97db1a12ebb7b5dd788f0f3724dc on pdf into 0a2238a727a5658d26299d4d092b62d29c7e33b5 on master.


Add Adapt VQE to tutorials

Closes https://github.com/Qiskit/documentation/issues/1167

I added in some more explanations to the text descriptions.
I commented the code a bit more.
I added in refactored code for real hardware in step 3.

But I also came into some issues.
I only have the open plan and this one tutorial exceeds an entire month's worth of usage limits so I could not actually run it on the hardware
At the start when defining the molecule, mol.energy_elec and mol.energy_tot don't work for me and I can't find source code where they would work in the same way as energy_nuc.
Maybe I am just missing something but could be worth checking.
Any suggestions or insights would be appreciated. Thanks!

hi @iheuze thank you for your interest in this piece but it has already been assigned to another docs maintainer. In future could you please ask to be assigned an issue before starting work on it


Crash in FIRInstanceIDCheckinService.m on iOS 13 when app is a new install

[REQUIRED] Step 2: Describe your environment

Xcode version: 11 GM
Firebase SDK version: 6.8.1

Installing Firebase (6.8.1)
Installing FirebaseABTesting (3.1.1)
Installing FirebaseAnalytics (6.1.1)
Installing FirebaseAnalyticsInterop (1.4.0)
Installing FirebaseCore (6.2.3)
Installing FirebaseCoreDiagnostics (1.0.1)
Installing FirebaseCoreDiagnosticsInterop (1.0.0)
Installing FirebaseInstanceID (4.2.5)
Installing FirebaseMessaging (4.1.4)
Installing FirebaseRemoteConfig (4.4.0)
Installing Google-Mobile-Ads-SDK (7.49.0)
Installing GoogleAppMeasurement (6.1.1)
Installing GoogleDataTransport (1.2.0)
Installing GoogleDataTransportCCTSupport (1.0.4)
Installing GoogleMobileAdsMediationFacebook (<IP_ADDRESS>)
Installing GoogleUtilities (6.2.5)

[REQUIRED] Step 3: Describe the problem
Crashes on Launch

Relevant Code:
// TODO(you): code here to reproduce the problem

Just a note, I'm able build and run using the iOS 12 SDK on Xcode 10.3. It works fine there.

@kevinchau Is it only happening on real devices? I can't reproduce on simulators.

@chliangGoogle both. It's only happening on new installs though.

@morganchen12  FCM with InstanceID works without crash, unlikely a regression. Seems like a complication with the new remote config and perf, I will do more testing on that.

I'm not able to reproduce yet, can you share with me your sample code? especially the way you call remote config during app start?

I'm not sure this is a InstanceID issue, from the stack trace, looks like issues are happening inside remote config thread, although @kevinchau manually highlighted the IID thread.
I only see errors occur inside RC call here:

thread #4, queue = 'com.google.GoogleConfigService.FIRRemoteConfig', stop reason = EXC_BAD_INSTRUCTION (code=EXC_I386_INVOP, subcode=0x0)

@chliangGoogle I can send you some code samples, I'd prefer not posting it to the public.

SG, feel free to DM me.

@kevinchau is it possible to repro without the Perf SDK?

@chliangGoogle looks like Github doesn't have  DM feature anymore, have an email?

I can't repro it anymore - I was having a build issue with Xcode 11 and had to Pod deintegrate and reinstall. It appears to have went away.
Thanks for the help, and sorry it was not an bug on your end.

Glad it's gone.
Might have been a variation of #3344 if you had old pods before deintegrating.


docs: create pre/post-deploy getting started

Replaces https://github.com/keptn/lifecycle-toolkit/pull/1337, https://github.com/keptn/lifecycle-toolkit/pull/1336, and other corrupted PRs.
Satisfies https://github.com/keptn/lifecycle-toolkit/issues/1302
Getting started guide for the orchestrate pre/post-deployment checks use case, based on Andi's demo video
Comments welcome but this still needs some writing

@aepfli markdown checks seem to be failing (from what I interpret) because /docs/readme.md contains a link to localhost.

This PR doesn't touch docs/readme.md - is there a way for MDLint to only run on changed files during PRs? This might speed up checks too.
The link to localhost in docs/readme is actually necessary and thus valid. Can we ignore that file + link combination in MDLint?

@aepfli markdown checks seem to be failing (from what I interpret) because /docs/readme.md contains a link to localhost.
1. This PR doesn't touch [docs/readme.md ](https://github.com/keptn/lifecycle-toolkit/blob/main/docs/README.md)- is there a way for MDLint to only run on changed files during PRs? This might speed up checks too.

2. The link to localhost in `docs/readme` is actually necessary and thus valid. Can we ignore that file + link combination in MDLint?

the markdown link check fails due to:
 ERROR: 1 dead links found!
[✖] ../../implementing/integrate.md → Status: 400

in: FILE: docs/content/en/docs/getting-started/observability/_index.md
There is a bad markdown link within the file referencing another. localhost already has an exception as seen here https://github.com/keptn/lifecycle-toolkit/blob/2c51231fd700009c1588259de1974e1dfa80e8b8/.github/mlc_config.json#L2-L9

Rebased and fixed issues, pr is green.
Additional note regarding running changes only for the files changed. Especially a link checker is a good example for such a test because we also want to check the "incoming" links from other readmes. Detecting if an internal link changed, or a file is suddenly not available anymore etc. is not such a trivial topic.


Fix leftover function name

The function name change doom-num-cpus to doom-system-cpus was missed.

Already done in #4718, but thanks for the help!


Setting to control the terminal's buffer size

Test: https://github.com/Microsoft/vscode/issues/19071

VSCode Version: 1.9
OS Version: Windows

An option to control the buffer size of the terminal would be helpful.

This is configurable by the user via terminal.integrated.scrollback.

Interesting name :-)

Seemed to be the common term at least in Unix-like environments which is why I chose it over something like bufferSize.


Add scope service Command

closes #464

See notes in #464

I've reviewed this PR, and don't have any comments.  I've asked to see if @seanvaleo wants to review as well before we merge...

Looks good to me.  Merging.


Vat crashing the world

As soon as I placed a vat (next to a killer joe, not sure if it is related) my world crashed. Here is the log:
http://pastebin.com/nxRAjQPL

That has been fixed in 2.3.0_beta build 373.


reset count of affected rows (sqlite3)

when reusing prepared statements, the count was not reset, causing subsequent calls to execute to set get_affected_rows wrong.
Here is sample code that was failing:
session << "CREATE TABLE IF NOT EXISTS kv("
        "k   CHARACTER(32) PRIMARY KEY,"
        "v       TEXT);";

    session
        << "INSERT INTO kv (k, v) VALUES ('a', '2')";

    soci::statement st(session);
    st.alloc();

    std::string k = "a";
    std::string v = "1";

    st.clean_up(false);

    st.prepare("UPDATE kv SET v = :v WHERE k = :k;");
    st.exchange(soci::use(v));
    st.exchange(soci::use(k));
    st.define_and_bind();
    st.execute(true);

    if (st.get_affected_rows() != 1)
    {
        throw std::runtime_error("could not update");
    }

    std::string k2 = "b";

    st.clean_up(false);

    st.exchange(soci::use(v));
    st.exchange(soci::use(k2));
    st.define_and_bind();
    st.execute(true);

    // update should not do anything as 'b' does not exist
    if (st.get_affected_rows() == 1)
    {
        throw std::runtime_error("should not happen");
    }

failures seems to be unrelated to this change: I see
4/14 Test  #4: soci_mysql_test_static ..............***Exception: SegFault

C:\projects\soci\tests\odbc\test-odbc-mssql.cpp(23): FAILED:
due to unexpected exception with message:
  Error binding input parameter #1: [Microsoft][SQL Server Native Client 10
  .0]Invalid precision value (SQL state HY104) while executing "insert into
  soci_test(long_text) values(:str)" with :str="Line #0

Yes, these are long standing failures on Windows platform.

AFAICT, the patch is sound, isn't it? :+1: thanks!

@mloskot AFAIK, there is common test for get_affected_rows(...) here https://github.com/SOCI/soci/blob/master/tests/common-tests.h#L3813 which is not failed on master.
Perhaps, it should be added another test to expose issue described.

@snikulov Yes, good point. @MonsieurNicolas could you update the PR adding new test for your use case?

I added the test, but it looks like db2 has a different bug in there: an update that doesn't update any row throws

Ok, lets get it in.
Thanks @MonsieurNicolas

It seems, this also reveales a subtle bug in DB2. To be checked

Changes Unknown when pulling 8cd3cf5844d07eb9ce177a05d966f873c5f20913 on MonsieurNicolas:affectedRowFixSqlite into ** on SOCI:master**.


Euchre: Suggestion for defending when holding left bower

From a customer:

When defending and the bot defender holds the left bower and another small trump, the bot will, when able, trump small leaving the left bower alone. I've been playing Euchre for over 60 years and the first thing you learn is to never bare your left bower. You will surely lose it and that is what happens in your game. You get over trumped and out comes the right bower. When my bot partner bares the left bower and then loses it giving the opponents 2 or 4 points instead of 1, I want to teach the bot but there is no way to do it except to write you. This happens very frequently.

Duplicate of https://github.com/TricksterCards/TricksterBots/issues/30

Thanks for catching this @drittich!


workspace.exclude disregarded for crate in subdir of workspace.member

Problem
It seems as if cargo does not always notice that a package is part of workspace.exclude . This happens when the excluded package is in a subdir of a package which is in workspace.members, and e.g. cargo check is invoked directly on the excluded package.
Step-by-step

Cherry-pick this commit to the cargo repo which adds the following test:

#[cargo_test]
fn excluded_subcrate_of_workspace_member() {
    let p = project()
        .file(
            "Cargo.toml",
            r#"
                [workspace]
                members = [
                    "member",
                ]
                exclude = [
                    "member/excluded",
                ]
            "#,
        )
        .file("member/src/main.rs", "fn main() {}")
        .file("member/Cargo.toml", &basic_manifest("member", "0.1.0"))
        .file("member/excluded/src/main.rs", "fn main() {}")
        .file(
            "member/excluded/Cargo.toml",
            &basic_manifest("excluded", "0.1.0"),
        );
    let p = p.build();

    p.cargo("check").cwd("member/excluded").run();
}

cargo test excluded_subcrate_of_workspace_member

Expected
Test passes
Actual
The test fails with
--- stderr
error: current package believes it's in a workspace when it's not:
current:   /home/martin/src/cargo/target/tmp/cit/t0/foo/member/excluded/Cargo.toml
workspace: /home/martin/src/cargo/target/tmp/cit/t0/foo/Cargo.toml

this may be fixable by adding `member/excluded` to the `workspace.members` array of the manifest located at: /home/martin/src/cargo/target/tmp/cit/t0/foo/Cargo.toml
Alternatively, to keep it out of the workspace, add the package to the `workspace.exclude` array, or add an empty `[workspace]` table to the package's manifest.

which is strange, since the package is already in the workspace.exclude array. But it does not seem to be noticed by cargo.
Version
git master branch of cargo

Thanks for the report! I believe this is a duplicate of #6745, so closing in favor of that.


Usage without CLI?

How i can use phpDocumentor without CLI on an main application? The usage is limited to the CLI (see Readme.md, no informations about that in the docs)
Expected Behavior
Including phpDocumentor in existing projects without CLI usage.
Actual Behavior
Can't start the phpDoc process. I had try to use it with (ugly) tricks like:
<?php
	require_once('../vendor/include.php');

	use phpDocumentor\Bootstrap;

	$_SERVER['argv'] = [ 'phpdoc', 'run', '-d', 'test', '-t', 'lol' ];

	$app = Bootstrap::createInstance()->registerProfiler()->initialize();
	$app->run();
?>

But i want a concrete API to call phpDoc directly.

phpdocumentor is an application that should not be part of an application but can be used as a tool during development. Therefor we will not provide an api to integrate phpdocumentor in an api. The internals will change somehow and we won't provide any feedback on this.
If you want to use phpdocumentor this way the best solution is the way you already did it. Maybe future development will move us in another direction but for now it will stay an cli only application.


DROL-BAN

hello！
There is a problem when I introduced the classifier and template update strategy into SiamBAN.
The speed is slow, such as 4.5 FPS, and the  GPU utilization is 100%. But DROL-RPN is 35 FPS tested on my workstation. I don't know the reason.
It would be great if you could give me some suggestions！
Thanks in advance!

Hello, the author. I transplanted DROL to SiamBAN. After debugging, it is also successful, and the accuracy has increased. However, there is a problem that this part of code must be commented out：
        # Train filter         if hard_negative:             self.filter_optimizer.run(cfg.TRACK.HARD_NEGATIVE_CG_ITER)         elif (self.frame_num-1) % cfg.TRACK.TRAIN_SKIPPING == 0:             self.filter_optimizer.run(cfg.TRACK.CG_ITER)
I don't know why, can you give me some suggestion?
thanks in advance!

hello！ There is a problem when I introduced the classifier and template update strategy into SiamBAN. The speed is slow, such as 4.5 FPS, and the GPU utilization is 100%. But DROL-RPN is 35 FPS tested on my workstation. I don't know the reason. It would be great if you could give me some suggestions！ Thanks in advance!

Hi, I have met the same question, Have you solved it?
It would be great if you could give me some suggestions！
Thanks in advance!

Sorry, I still haven't solved it.
I still need to comment out those two pieces of code
 # Train filter         if hard_negative:               self.filter_optimizer.run(cfg.TRACK.HARD_NEGATIVE_CG_ITER)        elif (self.frame_num-1) % cfg.TRACK.TRAIN_SKIPPING == 0:          self.filter_optimizer.run(cfg.TRACK.CG_ITER)


Prefix Namespace

How about adding a namespace prefix for controller?

Hi,
I've written a package that provides this feature among other appreciable stuff: https://github.com/Xocotlah/SmartController#default-controllers-namespace
(sorry for advertising :p )


Prepare Release

Prepare the GitHub release to roll out on September 1st 2021.

Release is prepared for rollout on September 1st.


Can't scrape HAProxy: Requested canceled while waiting for connection.

Hi there, I'm using the haproxy_exporter to export metrics the metrics of an haproxy container that's running on multiple openstack controllers in the form of a cluster, I have a dedicated endpoint from which I run the exporter (as a service) and two separate prometheus endpoints which request the exporter to scrape at a specified interval, problem is that while prometheus_server_1 runs without errors I tend to get a communication error on prometheus_server_2 about every other minute or so.
The error:
Can't scrape HAProxy: Get http://***:***@controller:1984/?stats;csv: net/http: request cancelled while waiting for connection (Client.Timeout exceeded while awaiting headers)" source="haproxy_exporter.go:305"
And another error that follows:
Unsolicited response received on idle HTTP channel starting with "HTTP/1.0 408 Request Time-Out Cache-Control: no-cache Connection: close Content-Type: text/html
408 Request Time-Out
 Your browser didn't send a complete request in time.

It appears the exporter wasn't able to scrape your HAProxy instance. I hope you have found the issue in the meantime.


Bump rustls to 0.21

Hey there !
I hope everything's fine.
To give you some context: I'm currently working on gremlin-rs and I'm currently removing native-tls.
Doing so, I had some dependency version issues regarding rustls. Do you think that's possible to accept this PR and create a new version of async-tls and publish it to crates.io ?
That'd be amazing !
Thanks for your consideration

Hey there, is it possible to have a look please ?
It is a blocking point for me to move further into my work on gremlin-rs. Help would be much appreciated !

Hello !
I apologize for insisting so much, but this is currently blocking my work on gremlin-rs.
Help on this PR would be much appreciated ! If I can be of any help, please let me know

@dignifiedquire I suggest getting this out to crates.io soonish.

@dignifiedquire I suggest getting this out to crates.io soonish.

Hey @djc, I would like that a lot ! This is currently blocking some pending work I have

Reason for suggesting this is now public:
https://rustsec.org/advisories/RUSTSEC-2023-0052.html


Type.Void / Type.Undefined throws FST_ERR_SCH_SERIALIZATION_BUILD since typebox 0.30.0

Prerequisites

[X] I have written a descriptive issue title
[X] I have searched existing issues to ensure the bug has not already been reported

Fastify version
4.23.2
Plugin version
3.5.0
Node.js version
18.15.0
Operating system
macOS
Operating system version (i.e. 20.04, 11.3, 10)
13.5.2
Description
https://github.com/sinclairzx81/typebox/blob/master/changelog/0.30.0.md#Extended-Type-Representation-Change might have produced this regression.
Below example will produce an error since 0.30.0 (0.29.6 was fine, i'm using 0.31.16) :
FastifyError [Error]: Failed building the serialization schema for GET: /test, due to error schema is invalid: data/type must be equal to one of the allowed values
Steps to Reproduce
import { type TypeBoxTypeProvider, Type } from '@fastify/type-provider-typebox'
import fastify from 'fastify'

async function main () {
  const webServer = fastify().withTypeProvider<TypeBoxTypeProvider>()

  webServer.route({
    method: 'GET',
    url: '/test',
    handler: () => { },
    schema: {
      response: {
        200: Type.Void()
      }
    }
  })

  //
  await webServer.ready()
}

//
main()
  .catch((error) => {
    console.error(error)
    process.exit(1)
  })

Expected Behavior
void should still be considered a valid validation value by ajv, as it describes an absence of returned value, which is not very user-friendly API-wise, but still useful for mocking.

Just noticed Type.Undefined() has the same problem.

@Amphaal Hiya,
Unfortunately, Ajv never actually supported these types as Json can't represent void or undefined as values. Because these values can't be represented, Json Schema doesn't support validation of them (and it's difficult to implement work arounds in Ajv to make these types validate at all)
Previous to 0.30.0, TypeBox made some concessions to try and support these types through addKeyword configuration (and custom AJv schematics), however these representations were only partially workable, and a bit too tied to some of the behaviors of Ajv (when other validators exist), so the decision was made to simplify the TB schematics and draw a greater distinction between Json serializable data, and JavaScript native data types.
Essentially, the general recommendation moving forward is to use only types prefixed with the [Json] doc comment, and reserve those types prefixed with [JavaScript] for TypeBox's TypeCompiler and Value modules (as these are written to validate data structures above and beyond Json Schema).
You can configure Fastify to use the TypeCompiler here which would support validation of these types, but would recommend keeping on specification if you can (so using Ajv with Json supported types)
Hope this brings some insights.
Cheers
S

@Amphaal Hiya,
Unfortunately, Ajv never actually supported these types as Json can't represent void or undefined as values. Because these values can't be represented, Json Schema doesn't support validation of them (and it's difficult to implement work arounds in Ajv to make these types validate at all)
Previous to 0.30.0, TypeBox made some concessions to try and support these types through addKeyword configuration (and custom AJv schematics), however these representations were only partially workable, and a bit too tied to some of the behaviors of Ajv (when other validators exist), so the decision was made to simplify the TB schematics and draw a greater distinction between Json serializable data, and JavaScript native data types.
Essentially, the general recommendation moving forward is to use only types prefixed with the [Json] doc comment, and reserve those types prefixed with [JavaScript] for TypeBox's TypeCompiler and Value modules (as these are written to validate data structures above and beyond Json Schema).
You can configure Fastify to use the TypeCompiler here which would support validation of these types, but would recommend keeping on specification if you can (so using Ajv with Json supported types)
Hope this brings some insights. Cheers S

Many thanks for the detailed explanation, much clearer to me 🙏


feat: adds hook for node-fetch

Signed-off-by: iamskp99<EMAIL_ADDRESS>There is no hook for node-fetch. Therefore, we can't use require for it.
This PR adds hook for node-fetch.

Converting PR to the draft because there is a problem in the simulate function of Keploy's SDK.


index: fetch: validate localfs meta using md5

Codecov Report
Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (05afa6f) 63.08% compared to head (6826db4) 63.03%.

Files
Patch %
Lines

src/dvc_data/index/fetch.py
20.00%
4 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #507      +/-   ##
==========================================
- Coverage   63.08%   63.03%   -0.05%
==========================================
  Files          62       62
  Lines        4321     4326       +5
  Branches      731      732       +1
==========================================
+ Hits         2726     2727       +1
- Misses       1439     1443       +4
  Partials      156      156

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


how to read the messages past week/month like that? i need to reed past 10/15days messages only...

how to read the messages past week/month like that? i need to reed past 10/15days messages only... @msmakhlouf @briankabiro

Hi, @logesssh. It might not be possible to do this using the module but there is a way you can do some data manipulation after getting the messages using the package.
You can get specify the messages you want to be returned using the maxCount parameter e.g maxCount: 15.
Since the object for each message has a date value, you can then use it to sort the messages.


Ping osidb-bindings==3.6.0 and change default search to fallback to --search-latest

Updated osidb-bindings to 3.6.0 to match prod.
Rearranged search-provides so if we find no results then we fallback to --search-latest
Also light refactor to set some options germane to all search types in one place.

osidb prod is now mismatched with griffon, need to push this out.


ELQ cannot detect covid 19

This seems really really odd. "covid 19" is resolved as multiple things like 'Severe acute respiratory syndrome-related coronavirus', 'Indiana vesiculovirus', 'Coombs test', 'Caprine arthritis encephalitis virus', 'CORC'. But never 'COVID-19' as on wikipedia. Any way to fix this? Given the importance of the entity

This has a simple answer thankfully. The knowledge base (Wikipedia) that they used to generate their pre-trained models is based on an Aug. 2019 view of the world, as they explain in their papers. The solution is to then retrain the models on a more recent Wikipedia dump. Of course, they don't make dataloading or training on fresh Wikipedia dumps very straightforward, and there's little documentation, but it can be done with the building blocks they have provided throughout the repo.

You can follow this tutorial for getting a new fresh Wikipedia dump and train your model using directly blink code or following this tutorial

Hi all,
You can refer to my answer https://github.com/facebookresearch/BLINK/issues/106#issuecomment-1014507351 regarding adding new entities and generating embeddings for them.
Thanks!


Integrate Svelte as front-end framework with TypeScript

Intro
Due to the nature of bare-bones JavaScript, scalability across multiple components is pretty difficult for an application that get's bigger in terms of features. Also, designing good looking components and pages is really time-consuming without proper front-end frameworks and CSS or UI libraries and frameworks.
Regarding Beta status
Svelte and SvelteKit look like fun tools to check out for a portfolio project like that. Though SvelteKit has not left Beta state yet, for a project of this kind it won't matter. The core of the application will remain in Rust obviously, with only the front-end being reimplemented in something that makes it easier to work with down the line when we're trying to add new features.
Branch
It's appropriate to start developing on the transfers to Svelte and SvelteKit within a new branch. This feature represents a big change in how we're integrating JavaScript/TypeScript with the Rust-to-WASM method core of our application, and things could break. So the suggested branch is feat/svelte-front-end.

Current state of issue
After choosing to go with SvelteKit as the new way-to-go for creating Svelte apps (according to the documentation), we ran into an issue with Vice currently not supporting the import of WASM as ES6 modules. Having fiddled around a decent amount with the bundler did not yield any success with the import capabilities.
We then jumped straight to the community plugin solution of vite-plugin-wasm, which, together with this explanation, did eventually lead us to the following error:
The request url ... is outside of Vite serving allow list. Refer to docs for configurations and more details.
What now?
We'll try moving the compiled WASM package (pkg folder and content) somewhere inside the src directory for now. Two possible solutions going forward are:

Integrate the move from the crate pkg directory to src inside the svelte directory with the build pipeline
Extend the Vice serving allow list

Option 1 seems like a low hanging fruit due to the rather easy extension of the package.json build scripts and possible integration of some shell scripting to automate away everything together (build and move process would then be "hidden" behind a shell script of some sorts). Option 2 might have some performance implications or makes it a lot harder to debug when there's a major change to the API or folder structure if we ever moved to split repo instead of this mono repo. If the crate could be considered stand-alone, we would not run into this issue, because it would be imported via cargo etc. So, moving the crate to a repo itself could be considered too down the road.

Update
Well, a super fast update to this comment:

Current state of issue
After choosing to go with SvelteKit as the new way-to-go for creating Svelte apps (according to the documentation), we ran into an issue with Vice currently not supporting the import of WASM as ES6 modules. Having fiddled around a decent amount with the bundler did not yield any success with the import capabilities.
We then jumped straight to the community plugin solution of vite-plugin-wasm, which, together with this explanation, did eventually lead us to the following error:
The request url ... is outside of Vite serving allow list. Refer to docs for configurations and more details.
What now?
We'll try moving the compiled WASM package (pkg folder and content) somewhere inside the src directory for now. Two possible solutions going forward are:

Integrate the move from the crate pkg directory to src inside the svelte directory with the build pipeline
Extend the Vice serving allow list

Option 1 seems like a low hanging fruit due to the rather easy extension of the package.json build scripts and possible integration of some shell scripting to automate away everything together (build and move process would then be "hidden" behind a shell script of some sorts). Option 2 might have some performance implications or makes it a lot harder to debug when there's a major change to the API or folder structure if we ever moved to split repo instead of this mono repo. If the crate could be considered stand-alone, we would not run into this issue, because it would be imported via cargo etc. So, moving the crate to a repo itself could be considered too down the road.

Properly configuring the vite.config.js in terms of the serving allow list did actually yield the intended behavior. The WASM module (compiled via wasm-pack --target web) can be served as an ES6 module via Vite now and is consumable after calling the init method onMount in the front-end application. This has to be performed as async though. All functions are then callable via buttons etc. The Universe itself will probably be initialized via the onMount call once.
Next steps
We can now start porting over the content from www/index.js and the adjacent files (front-end app logic) to the Svelte app.
This also entails encapsulating all components and adding UI elements, either via an UI framework (the current state of UI frameworks and reusability of ones for React like Mantine remain unclear right now) or TailwindCSS and some custom component logic.


⚠ AVS (kty) has degraded performance

In ad8916b, AVS (kty) (https://avus.com.br) experienced degraded performance:

HTTP code: 200
Response time: 8067 ms

Resolved: AVS (kty) performance has improved in 5fbecff.


Add PFM validation

Closes: #4731
What is the purpose of the change
Add a registry to check if a chain supports PFM

To propose validation for a chain, user can send a token from that chain.
Once it has properly been unwrapped, we store the chain as supporting PFM

Brief Changelog

Add PFM validation
The registry checks that the chains support PFM before sending the tokens
[TODO] Add validation for channels as well? (maybe in a separate PR)

Testing and Verifying
Added tests specific to setting forwarding
All tests should pass
Documentation and Release Note

Does this pull request introduce a new feature or user-facing behavior changes? (yes / no)
Is a relevant changelog entry added to the Unreleased section in CHANGELOG.md? (yes / no)
How is the feature or change documented? (not applicable   /   specification (x/<module>/spec/)  /  Osmosis docs repo   /   not documented)

Thanks for the feedback! I'll fix the nits/typos/comments

Removing the stale label


[deck] deterministic scroll-to-plank on navigation

right now we don't always or don't fully scroll to a navigated plank, this is apparently due to a problematic scrolling API

The root cause has remained elusive after nearly half a day of searching. I will pause this for now and may return to it.

This is most likely a reactivity issue which may resolve with improvements occurring now. We should try to reproduce the issue next week.

Probably blocked by #7424.

Blocking this as needing a new repro given recent changes to DeckLayout.


Documentation: Correlation plot can also show Spearman's correlation.

Added link to the Spearman's correlation coefficient which can also be selected by an user.

Also, please, consider #220


Bump version to 1.4.5 to release #518

I noticed 1-4-stable contains the feature added in #518, but it's not released in v1.4.4 right now. So here's a PR to bump it.
I also noticed a different fix, from #243, is in master, which is supposed to include v2 features? Shouldn't #518 override this?
Updated README since bin/setup and bin/console were deleted.

Thanks @mattboldt !

@mattboldt the answer to your question is:

Yes, #518 must be ported to master, and hasn't been thus far.  There was an attempt at a similar solution merged to master, but it was later reverted.

ref: https://github.com/oauth-xx/oauth2/pull/516#issuecomment-662783939

@mattboldt 1.4.5 is released

@pboling Thank you so much! Do y'all plan to merge 1-4-stable -> master for v2, or will you be cherry picking certain changes? I could look into getting #518 ported to master. I think the default proc should support what v1.4.5 does now.

There will be no merges from 1-4-stable to master.  Generally fixes will not be back-ported unless someone wants to do the work.  Maintenance effort will go into preparing 2.0 for release.


Added possibility to load Image from a buffer

I'm developing a small indie game using your library and I found it really handy. However I need to load images from my custom archives without knowing their dimensions, so I had to add some code. Others may find this useful too.

Thanks. Looking back at this project, I really wonder why I didn't juse use something stb_image or DevIL instead of rolling my own magic byte detection and BMP/TGA implementations.

I would have done the same just for the fun :) Thanks for merging it.


Color and price for skins

It would be beneficial to have price and color information included in the skin details. Currently, these attributes are missing, which limits the usability and completeness of the repository. Adding price and color fields would greatly enhance the repository by providing users with easy access to essential information about each item.

@AmarSmajlovic  For getting the prices I recomend you https://prices.csgotrader.app/latest/prices_v6.json, to implement the prices in this API i would need to scrap the steam market periodically and that is a lot of work for something that you can already have with the last link.
When you say colors you mean the rarities color? That is something doable and also something that you could implement as you have the rarities of every skin.

I'm not sure if you meant rarities color.
If somehow the rarity color is important reopen this issue.


Check whether graylog2/gelf-php is installed  in GelfMessageFormatter constructor

To make it obvious, the package is required to use the formatter.

Thanks


NetDriver sometimes isn't decoding the correct color value

Describe the bug
NetDriver doesn't use the correct color the selected menu bar on Windows and Linux. WindowsDriver and CursesDriver doesn't have this issue.
To Reproduce
Steps to reproduce the behavior:

Open UICatalog
Open ASCIICustomButtonTest scenario
Click on the Window Size menu bar.
The selected menu bar is showing with the Blue/White color

Expected behavior
It should showing with the Black/White
Screenshots

@tig do you have any idea why this is happening?

Odd. FWIW, adding a hotkey to Window Size changes things:

It's not unique to the ASCIIBUttonText scenario.

It's not unique to the ASCIIBUttonText scenario.

Yes I know. That also happened when I added a menu bar on a toplevel that also has an AdornmentEditor.


Namespace conflict when both shvdn and shvdn2 asis are installed

Today I installed ScriptHookVDotNet2.asi and changed my VAutodrive mod to reference this assembly, which was no problem. However, there are older mods which reference ScripthookVDotNet.asi (V2.9.3) and so I keep this as well. Unfortunately at least some of those older mods cease to function. I think this is because both shvdn versions share the same namespace.

Download v2.9.4 again. The rename to ScriptHookVDotNet2.asi was a mistake that was fixed soon after.

Thanks :)

Oh and I hope there will be no namespace conflict once we get shvdn 3. You we have to keep both versions  or older mods which aren't updated anymore will cease to run and that would be a bummer because there are some real good ones out there. Old but great.


Fix race condition ch. 5 counter

Fix #12
trying to change the code as little as possible

Thank you for this, unfortunately I am not allowed to accept code from others. I will fix the race condition for the next edition.

Right. Hope 2nd ed will be a terrific success :-)


Update 'Remove' button functionality to remove all selected rows

Currently the 'Remove' button removes only the last selected row; it should be updated to remove all selected rows.

Task completed in 3ccce0d - closing.


mruby: 3.1.0 -> 3.2.0

Description of changes
https://github.com/mruby/mruby/blob/3.2.0/NEWS
Fixes:

CVE-2022-0080
CVE-2022-0240
CVE-2022-0326
CVE-2022-0631
CVE-2022-0481
CVE-2022-0525
CVE-2022-0570
CVE-2022-0614
CVE-2022-0623
CVE-2022-0630
CVE-2022-0631
CVE-2022-0632
CVE-2022-0717
CVE-2022-0890
CVE-2022-1106
CVE-2022-1212
CVE-2022-1276
CVE-2022-1286
CVE-2022-1934

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review run on x86_64-linux 1

1 package built:

mruby

A lot of these CVEs are covered by  #160679, #168676, #160679 et al - mruby have a habit of putting out CVEs for issues that are only ever present for a short time in development versions (perhaps they are put out by a third party that doesn't understand).

No reason not to upgrade of course, but a backport probably isn't needed.


[Bug] hasLabel() cannot be queried after inE()

Bug Type (问题类型)
No response
Before submit

[X] 我已经确认现有的 Issues 与 FAQ 中没有相同 / 重复问题

Environment (环境信息)

Server Version: v0.12.0
Backend: memory
OS: CentOS
Data Size:  100 vertices, 200 edges

Expected & Actual behavior (期望与实际表现)
We execute the query g.V().inE('el0').hasLabel('el0','el1') and expect it returns some edges.
But it throws an exception java.lang.IllegalStateException: Illegal key 'LABEL' with more than one value.
Reproduce
The schema of the simplify graph as follows.
connection.getHugespecial().schema().propertyKey("ep0").asBoolean().ifNotExist().create();
connection.getHugespecial().schema().propertyKey("ep1").asBoolean().ifNotExist().create();
connection.getHugespecial().schema().edgeLabel("el0").sourceLabel("vl0").targetLabel("vl0").properties("ep0").ifNotExist().create();
connection.getHugespecial().schema().edgeLabel("el1").sourceLabel("vl0").targetLabel("vl0").properties("ep1").ifNotExist().create();
connection.getHugespecial().schema().indexLabel("el0Byep0").onE("el0").by("ep0").shard().ifNotExist().create();
connection.getHugespecial().schema().indexLabel("el1Byep0").onE("el1").by("ep1").shard().ifNotExist().create();

Vertex vertex1 = new Vertex("vl0").property("vp0", true);
Vertex vertex2 = new Vertex("vl0").property("vp0", false);
Vertex vertex3 = new Vertex("vl0").property("vp0", true);
graph.addVertices(Arrays.asList(vertex1, vertex2, vertex3));

Edge edge1 = new Edge("el0").source(vertex1).target(vertex2).property("ep0", true);
Edge edge2 = new Edge("el1").source(vertex1).target(vertex3).property("ep1", false);
graph.addEdges(Arrays.asList(edge1, edge2));

GremlinManager gremlin = connection.getHugespecial().gremlin();

String query = "g.V().inE('el0').hasLabel('el0','el1')";

### Vertex/Edge example (问题点 / 边数据举例)

_No response_

### Schema [VertexLabel, EdgeLabel, IndexLabel] (元数据结构)

_No response_

The exception can also be triggered when we remove the label el0 in hasLabel('el0','el1'). Maybe the reason is not the multiple labels.
In addition, I find that if I execute this query as the following way, it does not throw exception but return the correct result.
g.V().inE('el0');   -- {e:{1}}
g.E(1).hasLabel('el0','el1');  -- {e:{1}}

@imbajin
I have opened a new issue https://github.com/apache/incubator-hugegraph/issues/1969 because the incorrect return results when executing outE().hasLabel() with multiple edge labels.
It seems to be similar with this issue. Maybe the reasons for these two issues are the same, and you can investigate these two issues together.

@imbajin
I find this issue is similar to https://github.com/apache/incubator-hugegraph/issues/1951. Maybe the reasons for these two issues are the same, and you can investigate them together.
I also find that a closed issue (https://github.com/apache/incubator-hugegraph/issues/1735) reported the similar bug, and it seems to be fixed. But we still found it. Maybe it deserves further investigation?

@zhengyingying Yes it looks fixed, but due to v0.12.0 version does not contain the latest code, you can try cherry-pick that commit and execute mvn package manually to get a bugfix tar.
how to cherry-pick and package manually:
git checkout master
git pull
git checkout -b release-0.12 origin/release-0.12
git cherry-pick cadeade476e5e9773471cbc67b1a65ed505eba77
mvn package -DskipTests

@javeme Thanks! I will try to do it.
In addition, may I ask the cause of this bug？

@javeme
Hi, I have tried to build the project. But an exception was thrown on the package hugegraph-dist.
org.codehaus.plexus.component.repository.exception.ComponentLookupException: java.util.NoSuchElementException
      role: org.apache.maven.artifact.resolver.ArtifactResolver
  roleHint: project-cache-aware

How can I solve it?


Can tf.sparse_reduce_sum infer shape while keepdims=True?

Please make sure that this is a feature request. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:feature_template
System information

TensorFlow version (you are using): 1.11
Are you willing to contribute it (Yes/No):
No. Sorry, I'm not familiar with the TF codebase.

Describe the feature and the current behavior/state.
tf.sparse_reduce_sum should be able to infer the shape if the input has known shape.

In [2]: x = tf.SparseTensor(indices=[[0,0],[1,1]], values=[1,2], dense_shape=[2,2])

In [3]: tf.sparse_reduce_sum(x)
Out[3]: <tf.Tensor 'SparseReduceSum:0' shape=<unknown> dtype=int32>

In [4]: tf.sparse_reduce_sum(x, axis=1, keepdims=True)
Out[4]: <tf.Tensor 'SparseReduceSum_1:0' shape=<unknown> dtype=int32>

Will this change the current api? How?
No.
Who will benefit with this feature?
Users who use TensorFlow to do graph learning
Any Other info.
Hi, please check the following information, thanks!
Have I written custom code
N/A
OS Platform and Distribution
Windows 10
TensorFlow installed from
conda -c anaconda tensorflow-gpu
Bazel version
N/A
CUDA/cuDNN version
9.0/7.1.4
GPU model and memory
Quadro M520 2GB memory
Exact command to reproduce
describe in the original post
Mobile device
N/A

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce
Mobile device

Hi, please check the following information, thanks!
Have I written custom code
N/A
OS Platform and Distribution
Windows 10
TensorFlow installed from
conda -c anaconda tensorflow-gpu
Bazel version
N/A
CUDA/cuDNN version
9.0/7.1.4
GPU model and memory
Quadro M520 2GB memory
Exact command to reproduce
describe in the original post
Mobile device
N/A

Hi, please check the following information, thanks!
Have I written custom code
N/A
OS Platform and Distribution
Windows 10
TensorFlow installed from
conda -c anaconda tensorflow-gpu
Bazel version
N/A
CUDA/cuDNN version
9.0/7.1.4
GPU model and memory
Quadro M520 2GB memory
Exact command to reproduce
describe in the original post
Mobile device
N/A

Added a PR #23271 for the fix.

Thanks for the PR @yongtang


Series File Tombstone Tracking

Overview
This pull request adds series & tombstone statistic tracking to the series file compactor. These are calculated on a per-segment basis to later determine if they are past a threshold for compacting.
Fixes #9514.

During a run of megacheck the following issues were discovered:
/tmp/210506072/src/github.com/influxdata/influxdb/tsdb/index/tsi1/log_file.go:139:2: 'if err != nil { return err }; return nil' can be simplified to 'return err' (S1013)

During a run of megacheck the following issues were discovered:

During a run of megacheck the following issues were discovered:

@benbjohnson hercules-influx with an empty comment is a bug in the static checker. Sometimes it means that the branch doesn't build.

@benbjohnson I don't know why it's not emitting the error. I will have to look at it. The issue anyway is:
⇒  megacheck github.com/influxdata/influxdb/...
query/monitor_test.go:49:11: this value of err is never used (SA4006)

@benbjohnson what is the status of this PR? Is there an advantage to getting it into the next release? /cc @rbetts

@e-dard I need to update based on the feedback but it's been low priority compared to EARs. There's no benefit to adding this to the next release.

don't close.

@benbjohnson working on clearing our old PR queue, is this something that would make sense to re-target at 2.x?


Resurrect custom 404 page (aka nein.html)

404 Not Found

Were you trying to create a commitment?

Right now the preferred format is like

<tt>alice.commits.to/do_the_thing</tt>

Please avoid any other special characters right now.
Things are a bit in flux with the URL format!
And there are a couple special characters that will cause things to break horribly.

@chrisbutler suggested doing this with a handlebars template. But maybe this should be postponed till after reactifying?


Thin lines around some divs on BlackBerry 9900

I got a BB Bold 9900 here and have the issue, that on certain divs a border or outline is shown. Using bradbirdsall cool SwipeJS Slider (https://github.com/bradbirdsall/Swipe), and I narrowed it down to this plugin.
Dont know how this happens, but the lower part of the testpage is not even part of the slider (in my original page it is), but shows also the lines/margins/spaces when the slider gets initialized. Could not narrow it down furthermore, I thought it is a problem about margins, because it looked good once when i removed the margin from #contentbereich and/or the inner divs, but later on it was same as before.

(there should be no lines where yellow markings on that pic, i made a spare on the right side so u can see how they look).
Here is a test link: below the orange box you will see thin lines around the "enter your data", but only in blackberry bold 9900 at the first link.
silder enabled http://maxmuc.tipido.net/bbswipejs/z_test.php
silder disabled / not loaded http://maxmuc.tipido.net/bbswipejs/z_test_noslider.php
Any ideas from your side please?
Thanks, Max
PS: this is a duplicate of http://stackoverflow.com/questions/17153065/blackberry-9900-swipejs-thin-lines-around-divs

Guess there are none of these BBs around anymore.


HAC-1684:  implement Workspaces getting started card

https://user-images.githubusercontent.com/895728/179043991-9453c8bc-f2f8-4754-955f-f72f134b5e5b.mov
TODOs:

[x] bump @redhat-cloud-services/frontend-components once it has been updated to make use of @redhat-cloud-services/types >= v0.0.10 in order to pick up ChromeAPI.helpTopics type.  See https://github.com/RedHatInsights/frontend-components/issues/1611
[x] switch to forthcoming get started upstream component
[ ] correct topicId once new help topic is created per https://github.com/RedHatInsights/quickstarts/blob/main/docs/help-topics/README.md

/retest

@florkbr, @kdoberst, this is ready for review!

Tested locally using the hosted version of HAC-Core using the local proxy and works perfectly. Had a couple of very minor things otherwise looks great. I'm going to let @florkbr take a look and give the final "lgtm" as he'll be more connected to the project at least in the near future.

Thanks, @kdoberst.  Feedback incorporated.  https://github.com/openshift/hac-infra/compare/a10b08b32701d1a3d50f541532a04e0f2198621c..2e082f75bedc6a43f89e8f577be75d531ff5062c and https://github.com/openshift/hac-infra/compare/2e082f75bedc6a43f89e8f577be75d531ff5062c..3fd24ee9d64a05ad246fe54a16586f5654c9cf99 (because I don't always pay attention to what I'm doing when committing).

@rhamilto Hi Robb, I tested against the pr, and the getting started card displays on the workspaces page, I want to confirm that the getting started card will always there no matter the workspace list is empty or not, right?

@rhamilto Hi Robb, I tested against the pr, and the getting started card displays on the workspaces page, I want to confirm that the getting started card will always be there no matter the workspace list is empty or not, right?

That is correct.  The card will always be present.

The getting started card works as designed.
/label qe-approved

I don't see any reason to hold this up - we can have discussions on my points above offline. Thanks @rhamilto!


simple JSONArray search using JsonPath

Here is a sample JSONArray in which I am trying to search:
["ح م د","ر ب ب","ع ل م","ر ح م","م ل ک","ی و م","د ی ن","ع ب د","ع ون","ع ون","س ف ہ","س ف ہ"]
I am trying to search array with first character "س" using following XPath expression which is returning 0 instead of 2:
JsonPath.read(myJSONArray.toString(), "$[*][?(@ =~ /^س/i)]")
My array is of Arabic characters, so its in RTL and I am trying to use ^ in regex to find first character but its returning 0, if I omit ^, still its returning 0. Any idea whats going wrong?

Have you tried with out Arabic RTL?

Yes, just checked and it doesn't work with that too. Probably some problem with XPath expression?

I did further test and it seems that XPath only works with JSONObjects type Array, for simple JSONArray like above the XPath regex doesn't work. It will works with $[*] which displays entire list.

You must match the whole word not only first char.
String json = "[\"abc\",\"def\"]";

Assertions.assertThat(
  JsonPath.read(json, "$[?(@ =~ /^a.*/i)]").equals(Collections.singletonList("abc"))
);


Propose: ng-animate like functionality in Mithril?

Coming from Angular, the only thing I miss is the easiest and intuitive way one could animate elements being added, removed and moved. I know there are lots of ways one could achieve this in Mithril but all of them has to be hard coded into the component. I believe animations and transitions for changes in DOM are essential for pleasant UX and it has to be part of any modern web frameworks.
It would be great if Mithril adds appropriate classes to following DOM changes:

add new element
remove an element
move an element
data changes in an element e.g. could be used to flash an element when its content changes, useful for real time applications

Hi @ludbek i believe that the idea of mithril is keep the framework simple with a minimalistic API. But this kind of functionality can be implemented as an external module.
A few weeks ago i made something similar to ng-animate but instead of add classes in some changes on the DOM my library give us a callback function where you can do the animation as you want...using css3, velocityjs, etc
Is not documented ``yet```, but is easy to use and you have an example in the repository.
https://github.com/geut/mithril-transition

I may write a module for this for the rewrite. For the current v0.2.x stream, this is out of scope

@tinchoz49 @lhorie I have been working on mithril-ui. Its a work in progress. I could not help to show you guys the animation module I wrote there. You can clone and test the animation.
Everything seems to work fine except for animating element being removed from DOM. I did manage to pull it off but the code is just too messy(not included in the module above). My approach was to hack around .onunload.

Hi @ludbek, this has been a desirable feature for a long time. I came up with my own implementation on the train this morning, mithril.exitable.
Having explored and hacked around the limitations of Mithril components & virtual DOM statefulness for a while, I figured that (component controller instance : root element) was the most reliable contract, but I shied away from touching onunload, and instead opted for a new exit method where I could afford to take ownership of function signature without worrying about user expectations in that area.
I think I managed to keep code complexity to a minimum while providing a pretty simple and solid end use API. I'd be interested in your thoughts.

An application needs to have transitions for showing and hiding. Just adding a class is not sufficient, because you need to know when the transition has ended. And some components need to do more than fade in or out.
In Polythene I've solved it like this:

Component has functions show and hide.
These functions call a generic transition function with custom options (for instance the CSS 'show' class for that component)
Transition options can be set in a theme too, for example to specify beforeShow or afterHide
Controller gets transitioning value
When the transition is finished, the promise.then function is called to update the controller with the new state (and do some cleanup if needed)

@ArthurClemens interesting! So by your system you register an intent to modify the model, communicate that to the component, then make the actual change to the model that results in the component's destruction?
What does the application code for a full destruction sequence look like?

@barneycarroll Its a cleaver hack, loved it. I was having hard time animating nodes and components using single APIs. I can now see how it could be achieved. Thanx for the insight. Also I would love to incorporate your approach for handling exiting elements.

Glad you like the idea @ludbek, however I must admit I spent a couple of hours trying to get a functional demo and it doesn't quite work — I think the reason is that the last-minute subtree retain corrupts Mithril's internal snapshot and breaks subsequent diffs. I also seem to remember from previous exercises in this area that this is where you end up having to write a load of horrible code to second-guess and duplicate Mithril's internal model, and then hope you can trick it into behaving as if it wasn't performing all this logic we want to avoid. I'm afraid that this is one of those cases where the last 10% of the problem is 1000% of the code, and this might be a dead end after all :)

@barneycarroll If you mean with "model" the view model, yes; at least the Polythene components only contain view states, if any. But of course you could have view states in a data model and use that to direct view components.
The behaviour of transitioning components varies a bit, dependent where they are drawn.
Dialog and Notification/Snackbar are shown in 1 container and generally only one at a time. They use helper component "Multiple" that maintains a queue (based on ids) and calls the next queued item (more useful for notifications than for dialogs). For instance m.component(dialog) creates a Multiple instance, and as long as it does not get any instructions, only a stub span is drawn. When an instance needs to be shown, Multiple gives it the property show. As long as the instance exists in the queue has this property, the instance component will be called. That is: during fade in transition; when it is shown; during fade out transition.
So let's say that a dialog instance is already shown (with dialog.show(options, id)) and I want to remove it.

A button click invokes dialog.hide(id).
Dialog is a Multiple, and it checks which instances need to be shown. Nothing has changed yet (hide transition is still about to start), so the instance is returned in its view function.
Dialog instance view function checks instanceData.hide && !ctrl.isTransitioning. This is true, so call module function hide.
Hide sets ctrl.isTransitioning to true, calls the transition.hide helper function with default options.
transition.hide sets the transition duration and delay, removes the "show" class, and waits until the timeout.
After the timeout, Multiple function "remove" is called with dialog.remove(id). It is removed from the queue, so on the next redraw Multiple will no longer find this instance to show.
m.redraw is called to clear the view and show the stub instead

Menu is not using Multiple because it does not use 1 stub container. Instead it is shown locally near the buttons where the menu should appear. I first made the mistake to handle menus like dialogs, but found out of course that they would never scroll with the rest of the page, so the menu would get disconnected.
Probably it is also better to have the menu in a local component where the view controller can store which menu row is selected and should be highlighted.
So a menu is instantiated with m.component(menu, options) where one of the options is show which will likely be a controller driven value. When the invoke button is clicked, that controller value will be set to true, and passed to the menu component. The menu will only be shown when the view function checks true for opts.show && !ctrl.visible.
Assuming this has happened, and the user clicked inside or outside of the menu to close it.

The hide event is passed to a function with the component view.
Module function hide is called
Hide sets ctrl.isTransitioning to true, calls the transition.hide helper function with default options.
Again transition.hide sets the transition duration and delay, removes the "show" class, and waits until the timeout.
After the timeout, ctrl.visible is set to false, so the menu will no longer be shown at the next redraw.
m.redraw is called to clear the view and show the stub instead

So I've managed to get mithril.exitable.js working. The toughest problem was that an exit animation trigger requires us to diff and patch the virtual DOM before swapping in a subtree retainer at the last moment. which corrupts Mithril's internal virtual DOM records, leading it to believe the patch has taken place. The solution is neater than I thought, though: by storing the view output of every component, we can do some quick redraw API ninja work that patches Mithril's internal model back to its previous state before resuming normal lifecycle logic.
In the end, the ability to provide an arbitrary exit animation function to a component provides a tiny and simple API surface, requires far less convoluted application code, and offers maximum flexibility.

@barneycarroll That solution looks cool IMHO. It basically shows how simple it is to make Mithril correct itself when it gets too eager in its redraws.

@isiahmeadows yeah, the crucial logic is all here, it's basically judicious use of memoization and the existing redraw API.
This was definitely more a product of thinking about how Mithril worked (away from the computer) rather than hacking away.
Knowing it's possible to do last minute subtree retainment and restore Mithril to a coherent internal state like this really extends the boundaries of what's possible with the 0.2 API. For instance, granular redraws seem within much closer reach now without any need to modify core.

@barneycarroll :+1:


make the second SIGINT exit the program

It was not possible to stop boom easily after realising you made a
configuration flag mistake. The only options currently are to manually
go kill it or to type CTRL+\ (SIGQUIT) in the terminal.
With this change we now unregister the SIGINT handler after it runs the
first (and currently only) time, making subsequent SIGINT signals fall
back to the default of exiting the program.

Looks like signal.Reset() was introduced in go 1.5 and the travis builder is using 1.3. Should it remain 1.3 compatible?

Let me know if you prefer I change the travis config to require 1.5 or implement a solution that does not require signal.Reset().

The project has moved to https://github.com/rakyll/hey. Closing the PRs on this legacy repo.


ERROR after running make install-server

Describe the issue
There's an ERROR related to the version of pyflakes displayed after running make install-server.
ERROR: pip's legacy dependency resolver does not consider dependency conflicts when selecting packages. This behaviour is the source of the following dependency conflicts.
flake8 3.8.4 requires pyflakes<2.3.0,>=2.2.0, but you'll have pyflakes 2.3.1 which is incompatible.

Everything still seems to work as intended (ie. local updates to the CLI work as intended), but I would think this shouldn't be happening.
To Reproduce
Run make install-server in the command line.
Deployment information

Operating system: Linux

No longer reported in 3.7+


Random text box appearing in event creation screen

Describe the bug
Another odd issue. When creating an event there's a textbox that is not editable. That textbox is not present on production. See this screenshot:
Expected behavior
A clear and concise description of what you expected to happen.
Context
Does this bug on production, development, or both?

[x] Tested on development instance
[x] Occurs on development instance
[x] Tested on production
[ ] Occurs on productions

Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

The 'textbox' was actually a broken Google maps embed. Can be disabled in Events Organizer settings.


Please create a partial for ToC

Is your feature request related to a problem? Please describe.
Not having a partial for ToC it is impossible to customize the markup on my posts
Describe the solution you'd like
Change single.html file to use a partial template file for .Params.toc, as it's doing with series or related posts.

@jaestevan It is already implemented. You can activate by setting .Params.toc to true.

@jaestevan It is already implemented. You can activate by setting .Params.toc to true.

What I mean is at file https://github.com/lxndrblz/anatole/blob/v1.9.0/layouts/_default/single.html, line 45
  {{- if (eq .Params.toc true) -}}

Table of Contents

    {{ .TableOfContents }}

There is no use of a partial template, like for example at line 54
  {{- if isset .Params "series" -}}
    {{- partial "series.html" . -}}

And then there is no option to create a partial template file to customize the ToC output without modifying the theme files. For example to remove or translate this  tag hardcoded in the single.html file.
Hope that helps clarify.
Thanks!
JA.

@lxndrblz I sent a pull request with a very small change to fix this issue.
I think I also included what I was expecting to be a different PR. Sorry about that, still new with GitHub. Still I think this second change is also simple but useful.
Thanks,
JA.

@jaestevan Hi there, I've opened a separate PR for your other adjustment. Please only do one topic at a time.


Visual corruption in Trine 4 DXVK with both amdvlk-open and -pro

With amdgpu-pro vulkan 20.45.1188099, effects like fire in the chimney of the main menu flicker or are otherwise broken:

With amdvlk-open v-2021.Q1.3, these effects look correct. However, the rendering is still incorrect (more aliased and shadowy) vs. native D3D11.
amdvlk-open:

native d3d11:

radv looks the same as native d3d11. This is with latest dxvk git-master on 5700 XT.

amdvlk-open v-2021.Q2.1 contains a fix, amdvlk-pro 20.50 not.


OCaml implementation of depth first search, topological sort and Kosaraju's algoritm

Added the Dfs module

Added Topological sort

Added Kosaraju's algorithm

I added the changes you requested.
I kept the order function as it is for me the most universal description of an order. Here it is not used but it could be useful to find if a node is a predecessor of another without writing this function yourself.


Specs for provider and resourcs

I don't know if this is useful, but I thought I would try adding some rspec tests for the providers and resources here, starting with aws_subnet as a random example.
A lot of this is cargo-culting, but I wanted to update some of the providers and add others, and needed a framework to do so safely, and I think this will help.

Love this. I've been thinking about this myself, because I really don't want to be running manual tests against AWS just to see if new stuff works.

Definitely a good idea!

@pburkholder Great work!  I've already been implementing your patten in my branches for ec2 volumes and ec2 network interfaces.

@patrick-wright #101 is better organized spec-wise since I used it to build a new provider.

@tyler-ball This and #101 are what I've been working on. I wrote this to see if I could write specs around an existing, working resource, and #101 to implement a new resource, internet_gateway, with the tests to help me in the process.
Regarding data_bags, see https://github.com/chef/chef-provisioning-aws/blob/master/lib/chef/provider/aws_provider.rb#L25
and
https://github.com/chef/chef-provisioning-aws/blob/master/lib/chef/resource/aws_resource.rb#L2

For what you are doing here, I actually think cheffish might work even better--it lets you test the resource and provider in the context of a recipe: https://github.com/chef/cheffish/blob/master/spec/integration/chef_user_spec.rb#L17-L21
(Note: this is not a C/R requirement, more a suggestion :)
with_converge do
aws_subnet ... do
end
end

@jkeiser I saw that code in Cheffish this weekend, but wasn't sure how to apply in this case, as I don't want to make calls out to AWS to test the code while it's under development. Would that approach work as an integration test while we use mocked API calls, as in this PR, to do unit tests?
Also, what's a "C/R requirement" ?
Thanks,
Peter

I like these tests!  I have the same comment here about allow_any_instance_of vs expect_any_instance_of that I made in https://github.com/chef/chef-provisioning-aws/pull/101#discussion_r25173814

@pburkholder oh, "C/R" was just an abbreviation for Code Review.  Just saying the suggestion to switch it to Cheffish certainly won't block a merge, but would probably be a better pattern for the tests to follow.
And yep, the "run the chef client" tests are compatible with mocked API calls :)  The approach of mocking the AWS API is a good one.

I like the idea of using the Cheffish spec_helper because it removes a lot of the boilerplate around defining node, run_context, etc.

Given that "please try and use Cheffish instead" isn't a blocker, this seems OK to merge, :+1:

Merged as 48bc0816d80dcdc70c337b3bf77aafa54f23aad8 (I added a "bundle exec rake spec" and .travis.yml as well).

:+1: