GridItem Destroys the Wrapped Child Element

Thanks for submitting an issue to RGL!
Please mark the type of this issue:

[X] Bug
[ ] Feature Request
[X] Question

If you have a question or bug report, please use the WebpackBin Template
to demonstrate. It is much easier for us to help you if you do.
NOTE: I can't get the bin working. The app constantly reports that some error occurs in the template... I'd like to create a simple demo if the error can be fixed...
For now, I'll describe the issue, expectations and possible solution with code snippets and references...
The code this issue discusses is here.
Summary
In my case, I use this library to create an editable dashboard (refer to sample code blow). The UI provides a button toggling the editable state of the dashboard via setting isDraggable and isResizable props of ReactGridLayout...But, due to the aforementioned implementation, every time the values of these two props change, the custom element/component wrapped in GridItem will be recreated by cloning. Hence, the original wrapped element is destroyed and its internal state is lost...
This seems to imply that every custom item/component passed into the grid layout has to be stateless. If this is true, its not very feasible, since it's not always true for a custom component to be stateless...
// dashboard code
// A modification borrowed from https://github.com/STRML/react-grid-layout/blob/master/test/examples/1-basic.jsx

class Dashboard extends React.PureComponent {
  static defaultProps = {
    items: 20,
    rowHeight: 30,
    cols: 12
  }

  constructor(props) {
    super(props);

    const layout = this.generateLayout();

    this.state = {
      editable: false,
      layout
    };
  }

  toggleEditMode = () => {
    this.setState(({editable}) => {
      editable: !editable
    });
  }

  generateLayout() {
    const p = this.props;
    return _.map(new Array(p.items), function(item, i) {
      const y = _.result(p, "y") || Math.ceil(Math.random() * 4) + 1;
      return {
        x: (i * 2) % 12,
        y: Math.floor(i / 6) * y,
        w: 2,
        h: y,
        i: i.toString()
      };
    });
  }

  generateDOM() {
    return _.map(_.range(this.props.items), (i) => (<CustomItem key={i} />));
  }

  render() {
    const {
      editable,
      layout
    } = this.state;

    return (

          <button onClick={this.toggleEditMode}>editable toggle</button>

          <ReactGridLayout
            {...this.props}
            layout={layout}
            isDraggable={editable}
            isResizable={editable}
          >
            {this.generateDOM()}
          </ReactGridLayout>

    );
  }
}

class CustomItem extends React.Component {
  componentWillUnmount() {
    console.info("Oh no, I'm destroyed!!!!");
  }

  render() {
    return (

        Some custom item/component might have internal state.

    );
  }
}

Current behavior:
changing isResizable or isDraggable destroys the custom element/component always.
Expect behavior:
the custom element/component should be destroyed only when necessary by the React itself.
Possible solution
After reading this comment,
https://github.com/STRML/react-grid-layout/blob/aacba5fcb466860573284d8d081d71e1965c2f8c/lib/GridItem.jsx#L471
I think it's more useful to use hoc (a function that wraps GridItem around the custom element/component) to achieve the same goal. A draft skeleton looks like below:
function connectToGridItem(Component) {
  class GridItem extends React.Component {
    ...
    render() {
      const newClassNames = classNames(
        "react-grid-item",
        this.props.className,
        {
          static: this.props.static,
          resizing: Boolean(this.state.resizing),
          "react-draggable": isDraggable,
          "react-draggable-dragging": Boolean(this.state.dragging),
          cssTransforms: useCSSTransforms
        }
      );

      return (
        <Component
          classNames={newClassNames}
          style={
            ...this.props.style,
            ...child.props.style,
            ...this.createStyle(pos)
          }
        />
      );
    }
    ...
  }

  return GridItem;
}

For any custom component being rendered into the grid, it should be wrapped by the above hoc (this will give React a chance to reuse the rendered component which can result in more performant code):
class CustomComp extends React.Component {
  ...
}

export default connectToGridItem(CustomComp);

May I see the output ? :)

@michaelroevievictoria Is there other online IDEs (other than WebpackBin) where I can put up a sample for this issue? That might be more helpful. As of the output in console, it's N times of "Oh no, I'm destroyed!!!!" on every editing mode toggling, where N is the number of rendered items.

@ZheyangSong Actually I have a problem to you can see my issue.  Well I just create a GIF to see my output. :)

I had this exact same problem. If you need a quick solution to work around it here is what I did:
isDraggable and isResizable is always set to true. draggableHandle set to '.dragHandle'
I wrapped all children of the grid layout with my own GridItemEditable component. The editable prop is also passed down to all the children.
GridItemEditable:
for draggable - when editable is true, add 'dragHandle' class otherwise leave it out.
for resizable - when editable is false, remove the last child
e.g.
    const childrenCount = React.Children.count(this.props.children);
    const childrenArray = React.Children.toArray(this.props.children);
    const resizeHandle = childrenArray.slice(childrenCount - 1);
    const otherChildren = childrenArray.slice(0, childrenCount - 1);

    return (

        {otherChildren}
        {editMode && resizeHandle}

    );

Ugly but it works.

I'm having the same exact issue as well.  @foodforarabbit can you elaborate more on how you handle removing the resize handle?  Are you manually removing the one that react-grid-layout adds?

@michaelroevievictoria was the issue deleted?

@foodforarabbit Thank you. That's quite a solution before the library resolves it, I hope the author will be agree on improving the library itself though.

@jackie-benowitz Yes I am manually removing the resize handle that react-grid-layout generates inside the ReactGridLayout children. The resize handle they generate is ALWAYS the last child so you can simply count the children inside your GridItemEditable object, and the last one is the handle. Then I simply output the other children as usual, and do a conditional render of the handle. Probably better to wrap the handle in a div and show/hide it with css classes.
Here is my ReactGridLayout:
<ReactGridLayout
  dragApiRef={dragApi}
  verticalCompact={true}
  onDragStop={layoutOnChange}
  onResizeStop={layoutOnChange}
  isDraggable={true}
  isResizable={true}
  draggableHandle=".dragHandle"
  layout={layout}
  margin={[32, 32]}
>
  <GridItemEditable>
    {your_content}
  </GridItemEditable>
  <GridItemEditable>
    {more content}
  </GridItemEditable>
</ReactGridLayout>

with GridItemEditable looking something like this:
class GridItemEditable extends React.Component {

  render() {
    const {
      classes,
      editMode = false,
    } = this.props;

    const isDragging = (className || '').indexOf('react-draggable-dragging') !== -1;
    const isResizing = (className || '').indexOf('resizing') !== -1;

    const childrenCount = React.Children.count(children);
    const childrenArray = React.Children.toArray(children);
    const resizeHandle = childrenArray.slice(childrenCount - 1);
    const otherChildren = childrenArray.slice(0, childrenCount - 1);

    return (

        {/* your "normal" content here*/}
        {otherChildren}
        {/* other custom editMode things you want to add... e.g. floating absolute positioned delete button */}
        {editMode &&  <Button
          variant="fab"
          style={{position: 'absolute', top: '8px', right: '8px'}}
          onClick={deleteItem}
        >
          <FontAwesome name="trash" />
        </Button>}
        {/* only render resizeHandle if editMode is true */}
        {editMode && resizeHandle}

    );
  }
}

@foodforarabbit why my Issue has been deleted ? My problem is I try to install React-grid-layout and I Just follow the usage and importing the '/node_modules/react-grid-layout/css/styles.css'
and '/node_modules/react-resizable/css/styles.css ' and I don't know why the behaviour is not like this demo
I like to achieve the resizable and the background  when the component is being drag to the grid background..
NOTE: My React Version: "react": "^16.2.0"
my code is look like this
import React, {Component, Fragment} from 'react'
import {Grid, Row, Col} from 'react-flexbox-grid'

// Material imports
import Paper from 'material-ui/Paper'
import  rg, {Responsive, WidthProvider} from 'react-grid-layout'
const ResponsiveReactGridLayout = WidthProvider(Responsive)

const ReactGridLayout = WidthProvider(rg);

export default class extends Component {

  constructor(props) {
    super(props)

  }

  render() {

  return <ReactGridLayout className="layout" cols={12} rowHeight={30} width={1200} >

a

b

c

          </ReactGridLayout>
  }
}

and the preview looks like this

@michaelroevievictoria I tried out your code (minus the react-flexbox-grid import) and it works fine. Are you sure you're importing the css correctly? I have this in my code:
import 'react-grid-layout/css/styles.css';
import 'react-resizable/css/styles.css';

I get somewhat similar behaviour to yours when I remove the css imports...

@foodforarabbit yes,
i actually
import the import 'react-grid-layout/css/styles.css';
import 'react-resizable/css/styles.css';
from my parent component and also to my component, actually I made different solution,, Like copy paste the css style from the node_mudules and create my own css style, And the output is actually the same.

@michaelroevievictoria It doesn't look like CSS problem to me. Could you try to use your rg without wrapping it in WidthProvider? The WidthProvide expects the Responsive one as input.

@STRML hi, any thoughts on the issue and propose?

@foodforarabbit Thank you very much for your workaround. Works like a charm.
I now use a stateless DragItem component which sets the dragHandler class and that solves the issue.
The DragItem just wraps around the actual children.

Is this an issue you will think about addressing soon? I created a minimum reproducible sandbox demonstrating the issue. In my case, I am trying to allow the users to pin a card in place so it is static. https://codesandbox.io/s/x7kyrly7rz?codemirror=1&fontsize=14 I will try the workarounds to manually remove the resize/draggable classes

We are also encountering this issue, just wondering if there's any plans to git this fixed in master?

Relates to #892


Console fix

Should fix character handling in lua console. Still more work to do with key combination and non printable characters.

Thanks, I'll check to improve this later.


Hide sort button when there's only table headers/no table content

It's just a small visual issue I encounter when using tables like this...

I decided to remove the "up-down" arrows for all tables. This is the easiest the cleanest solution.
So now, all tables, in their default state, have the headers unchanged (by CSS). The cycle is like this:

1st click on a header: the up arrow appears (due to sorting in ascending order).
2nd click on a header: the down arrow appears (due to sorting in descending order).
3rd click: reset to default state, no up-down arrows.

Done in https://github.com/alexandru-dinu/obsidian-sortable/releases/tag/0.2.6.

I like this implementation, clean and simple!


Error creating RCU schema in database 18c

Hi,
I'm facing this issue when executing rcu script in a database oracle XE 18c installed in a VM in azure.

Later, when i try to execute create-soa-domain.sh script. i get the following error, i suppose that it´s because the previous script didn't finish well.

PD: I'm using release/21.2.3 because branch release/21.3.2 doesn't exist.
Could you help me?
Regards,

The connection string in create domain screen shot shows that you are using XE and not PDB ( for example XEPDB1). Hence the RCU error seems to be because you are not using the PDB (pluggable database) in connection string.
And regarding create domain error, you cannot proceed with creating domain unless RCU schema creation is success.
Please use PDB (pluggable database) in connection string. In case you still face similar error during RCU creation, provide the rcu logs. Also the release/21.3.2 is available now.
Note:  XE DB is not supported configuration for a production environment.


404 at /desktop/mac/release-notes/2.x/undefined

I found a broken link : https://docs.docker.com/desktop/mac/release-notes/2.x/undefined
I arrived on this page through: https://docs.docker.com/desktop/mac/release-notes/2.x/

Hey @shijunjie007 Can I solve this?


Unable to find logged in user name from a visual

powerbi.session.userInfo is null inside a visual, is there any other way to get information regarding the user?

Guys, any update on this?

When will this issue is fixed? Is this working on V1.6

I am trying to get powerbi.session.userInfo but still getting null. By what release are you planning to have this fix?

Hello, I am also interested in a solution to this.

any update on this??


Unexpectedly (bad) predictions?

I'm just trying out the pretrained models accompanying this repo via HF Spaces and I'm seeing some weird results.

For some models there's a huge difference in quality when I include/exclude a period. E.g. the difference between the two sentences "This is James" vs "This is James**.**" sometimes causes models to not be able to recognise "James" as a person.
The multilingual model was not able to recognise major cities in a straightforward Dutch sentence ("Ik woon in Leuven." - also tried it with "Amsterdam", and "Parijs").

Am I using the pretrained models wrong? Is it expecting different kinds of inputs?
I can elaborate and test more, but I figured I'd post this first. 😊
Thanks!

Hello! Thanks for your report.
As for your first comment, I've previously encountered issues that the model simply could not include the last token as an entity (#1), but since fixing that one, most of the models still tend not to include the last token as an entity, even if it's an obvious one like Paris. I think this issue originates from the training data always including punctuation at the end. For example, the CoNLL03 dataset, which does not always have a dot, does not seem to have this issue: https://huggingface.co/tomaarsen/span-marker-xlm-roberta-large-conll03?text=This+is+James
As for the second issue: The multilingual model was trained on English data exclusively, meaning that the multilingual behaviour must originate from the underlying XLM-RoBERTa-large encoder.
I've just done some testing, and it seems that there is some odd behaviour here: if punctuation is attached to the words, then it cannot detect it. But if punctuation is separate, then it works well:

This may be related to how Amsterdam  and Amsterdam tokenize differently. If the underlying encoder is only familiar with one of the two ways, then it makes sense that it doesn't really know how to deal with the other one. I think I should be able to resolve this with pre-processing of prediction sentences.
When using a model in practice, you can also provide a list of words, e.g. ["Ik", "woon", "in", "Amsterdam", ",", "in", "Nederland", "."]. This should always work as intended - this is also how the model is trained.
I'll see if there are things I can do to improve the performance under these two circumstances.

Tom Aarsen

Interesting! So, the pretrained models are then all intended for English NER, right? I'm going to try out cleaning/tokenising myself and then passing the processed input to the model. Thanks for the feedback 👍

The xlm models should work with other languages, too. but I didn't explicitly train them for anything other than English. These models seem to suffer from bad performance when the punctuation is attached, as mentioned. I think you'll find that they perform well when they're pre-tokenized (or rather, converted to words).
I just remembered, the spaCy integration automatically does this conversion to words.
import spacy

# Load the spaCy model with the span_marker pipeline component
nlp = spacy.load("en_core_web_sm", exclude=["ner"])
nlp.add_pipe("span_marker",
    config={"model": "tomaarsen/span-marker-xlm-roberta-base-fewnerd-fine-super"}
)

# Feed some text through the model to get a spacy Doc
text = "Ik woon in Amsterdam, in Nederland."
doc = nlp(text)

# And look at the entities
print([(entity, entity.label_) for entity in doc.ents])
"""
[(Amsterdam, 'location-GPE'), (Nederland, 'location-GPE')]
"""

from spacy import displacy
displacy.serve(doc, style="ent")

I've narrowed this down to the roberta and xlm-roberta models - they tokenize my training data differently than how real sentences are tokenized. So, for the roberta models you'd have to make sure that there's spaces before all punctuation. I'm trying to figure out if there's a convenient solution.

I'll close this in favor of #23, as I've narrowed down the issue.


GNSS: ublox surpport needed

ublox is added at config.proto, And some pesudo code is listed in howto Docs,but Maybe completed&official code surpport will be very helpful .

Closed due to inactivity. If the problem persists, pls feel free to reopen it or create a new one and refer to it.


#564 Add notes on square brackets syntax

Addition of notes to improve clarity on GUIDE.md and FEATURES.md regarding the square brackets syntax difference of lists and maps on Bend.
Fix #564.

Thank you, we also wanted to add this clarification since other users have had trouble with this


Apollo new

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.lixiaotao seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@zhxf94 Please explain the function of submitting code and format the commit. You can refer to the community contribution guidelines. Thank you.


lightning 2.3.3

lightning 2.3.3
Destination channel: Defaults
Links

PKG-6298
dev_url:        https://github.com/Lightning-AI/lightning/tree/2.3.3
conda_forge:    https://github.com/conda-forge/lightning-feedstock/blob/main/recipe/meta.yaml
pypi:           https://pypi.org/project/lightning/2.3.3
pypi inspector: https://inspector.pypi.io/project/lightning/2.3.3

Explanation of changes:

new version number

there may be some missing dependencies based off this

@Jrice1317  As a dirty cheat I ran grayskull against the current version on got something pretty similar to the current recipe and then ran grayskull against this version and got something pretty similar to this recipe.  That may not be perfect but at least confirms that the bulk of the changes are genuine.

If I install lightning using the staging channel with commit 06530f5:

lightning --help yields:

Traceback (most recent call last):
  File "/Users/msentissi/miniconda3/envs/lightning-test/bin/lightning", line 7, in <module>
    from lightning.fabric.cli import _legacy_main
ImportError: cannot import name '_legacy_main' from 'lightning.fabric.cli' (/Users/msentissi/miniconda3/envs/lightning-test/lib/python3.12/site-packages/lightning/fabric/cli.py)

fabric --help yields:

Traceback (most recent call last):
  File "/Users/msentissi/miniconda3/envs/lightning-test/bin/fabric", line 7, in <module>
    from lightning.fabric.cli import _main
ImportError: cannot import name '_main' from 'lightning.fabric.cli' (/Users/msentissi/miniconda3/envs/lightning-test/lib/python3.12/site-packages/lightning/fabric/cli.py). Did you mean: 'main'?

The tests pass because click is available in the test environment, but the entry_points are not available with the distributed lightning.


License code in documentation under the MIT-0 license

Prerequisites

[X] I have searched for duplicate or closed feature requests
[X] I have read the contributing guidelines

Proposal
License code in the documentation under the MIT-0 license.
Motivation and context
The term "substantial" seems ambiguous to me. By licensing the code in the documentation under the MIT-0 license, it's always fine to copy any amount of code without requiring to include the copyright notice and permission notice in "substantial" portions. This removes the necessity to discern if the copied code is a substantial and non-substantial portion.

When I wrote the initial post it seemed to me that the code in the documentation is licensed under the MIT license (because it's code). But that the intention was that it is licensed under the CC BY 3.0 (because it's in the documentation) seems also possible to me now. ;-)
Also, I have noticed that the MIT license text also covers the documentation.


JavaScript Framework

Recently there has been some discussion on which JavaScript framework we should really be using. So far, our options are: AngularJS (the one we are currently using) and ReactJS.
Which one should we use? Why? Do you know of any other possible options?

#118 - Why we should switch to React.

For me the most important thing to solve is #88 - Different HTML files for different mods. As long as we can do that, I'm fine with any framework. I'm not sure if AngularJS supports this at all, or if it is just webpack that is currently restricting us.
Also, as I mentioned in #118:

The most important question, for #88, is: Can you check if a file exists before you include it, and if it doesn't can you then include a different file?

Closing as no longer needed #144 as we have chosen to go with Vue.js and have implemented the new client.


Not really an issue, but are there any plans for manipulating the "forward" and "back" buttons in the media player?

Great job on the plug-in by the way, it's working great in iOS 7.

Will it be difficult to add a close button on the video play mode? The only way to close a video stream on android is to hit the back button. Thanks for the great plugin.

No plans to support this. If someone wants to submit a PR, great!


[BugFix] Using UTC timezone when read time which less then unix epoch time in orc timestamp type

If orc's type is Timestamp Instant, we should add a timezone offset.
If orc's type is Timestamp, we should use UTC timezone directly.
What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.1
[ ] 3.0
[ ] 2.5
[ ] 2.4

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

add a UT?

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

@mergify backport branch-2.5


Missing CONTRIBUTING.md file in the repository

Already the content of the contributing was included in readme

Duplicate issue. Already raised  #100


Shadow Priest sim

This may apply for most classes, but spell coeffients/talent resources can be found https://wotlk.evowow.com/

Adding a few items to work through here with notes from beta testing. We may break these down into more PRs when the time comes or combine several pending how integration works out.
Priest Baseline:

[ ] Base stats for 80
[ ] Talents Proto Update (Going to go with full talent tree in case any healers / other use cases come up)
[ ] Removal of priest specific racials (e.g. Shadowguard)
[ ] (Shadow) Presets for Sim (Will require non-priest updates for consumes, party buffs, etc)
[ ] (Smite) Presets for Sim (Should we remove in relation to #11 ?)

Abilities To Migrate (Notes / Updates):

[ ] General Updates (Spell ids updated to new max rank, mana is based on base mana cost for all ranks)
[ ] Mind Blast (Base mana, talent integration)
[ ] Shadow Word: Death (Base mana, talent integration)
[ ] Vampiric Touch (Base mana, base tick damage update, talent integration)
[ ] Shadow Word: Pain (Base mana, talent integration, need testing on snap shot numbers for MF refresh)
[ ] Devouring Plague (Baseline now for all priests. Improved Devouring Plague (talent) is 150% crit damage from testing)
[ ] Shadowfiend (No mana now, need to integrate the new Shadowcrawl ability as well as get testing numbers)
[ ] Mind Flay (Base mana, base tick damage update, talent integration, no longer considered a "dot" as a channeled spell)
[ ] Dispersion (New. May need to consider breaking out of dispersion early in a later implementation)
[ ] Mind Sear (New. Probably will be integrated at a later time if we can get multi target integration)

Talents to Update / Integrate (Sim damage usage only):

[ ] Improved Spirit Tap
[ ] Improved Shadow Word Pain (Moves from increased time to damage done)
[ ] Veiled Shadows (Was Improved Fade, reduces Shadowfiend CD)
[ ] Mind Melt (New. Crit chance on most spells)
[ ] Shadowform (Allows all dots to crit and DP and VT to benefit from haste)
[ ] Misery (Now grants 3% hit to raid instead of 5% spell damage debuff. Also buffs MB, MF, and sear SP gain)
[ ] Vampiric Touch (Reworks mana gain to give replenishment buff on MB cast)
[ ] Twisted Faith (New. Adds 20% of current spirit to spell power. Adds 10% more MB / MF damage if SWP is applied to target)
[ ] Dispersion (See abilities above)

Rotations:

[ ] TBD - Pending beta testing. Will need to consider several factors here including if/when MB and/or SWD are dropped from rotation based on gear as well as cast priorities from snap shot testing.

Potential Cross Over Work:

[ ] Dot snap shot logic (Crit and SP % buffs)
[ ] Misery as a preset / setting on other classes

Leaving a note to checkout the wotlk-beta-findings channel in priest discord.

Notes on spirit from talents with no gear:
Base spirit: 155
Twisted Faith (20% of spirit): 31 SP
Glyph of Shadow (30% of spirit): 77 SP (Talent + Glyph are additive)
Improved Spirit Tap (10% more spirit): 85 SP (170 spirit)

This sim is in alpha, remaining issues can be tracked in separate threads.


steps and dag exhibit different behavior when using continueOn failed true

Checklist:

[x] I've included the version.
[x] I've included reproduction steps.
[x] I've included the workflow YAML.
[x] I've included the logs.

What happened:
When setting continuneOn.failed to true on a given task, whether it's in a step or dag, and if there are subsequent tasks depending on said task, then I expect the Workflow to have a consistent status after-the-fact.
What you expected to happen:
For both to either succeed or fail.
How to reproduce it (as minimally and precisely as possible):
argo submit https://raw.githubusercontent.com/argoproj/argo/master/examples/status-reference.yaml
argo submit https://raw.githubusercontent.com/argoproj/argo/master/examples/dag-continue-on-fail.yaml

Anything else we need to know?:
Environment:

Argo version:

$ argo version
argo: vv2.5.2+4b25e2a.dirty
  BuildDate: 2020-02-24T22:49:11Z
  GitCommit: 4b25e2ac1d495991261e97c86d211d658423ab7f
  GitTreeState: dirty
  GoVersion: go1.13.4
  Compiler: gc
  Platform: linux/amd64

Kubernetes version :

$ kubectl version -o yaml
clientVersion:
  buildDate: 2019-02-13T11:15:10Z
  compiler: gc
  gitCommit: 954ff68d59e9dc62fa8252ffa9023a90ff8a358c
  gitTreeState: clean
  gitVersion: v1.10.13
  goVersion: go1.9.3
  major: "1"
  minor: "10"
  platform: linux/amd64
serverVersion:
  buildDate: 2018-11-13T11:33:04Z
  compiler: gc
  gitCommit: be1a908c6aa47e0ae1b1dc861a1de6ccfe963aa2
  gitTreeState: clean
  gitVersion: v1.10.10
  goVersion: go1.9.3
  major: "1"
  minor: "10"
  platform: linux/amd64

Other debugging information (if applicable):

workflow result:

Logs for status-reference:
Name:                status-reference-sslm2
Namespace:           anomaly-detection
ServiceAccount:      review-feature-fa-1ro31a-anomaly-detection-backend
Status:              Succeeded
Created:             Thu Apr 09 13:19:44 -0500 (13 seconds ago)
Started:             Thu Apr 09 13:19:44 -0500 (13 seconds ago)
Finished:            Thu Apr 09 13:19:57 -0500 (now)
Duration:            13 seconds

STEP                                          PODNAME                            DURATION  MESSAGE
 ✔ status-reference-sslm2 (status-reference)
 ├---✖ flakey-container (flakey-container)    status-reference-sslm2-141736205   8s        failed with exit code 1
 └-·-✔ failed (failed)                        status-reference-sslm2-1903185625  3s
   └-○ succeeded (succeeded)                                                               when 'Failed == Succeeded' evaluated false

Logs for dag-continue-on-fail:
Name:                dag-contiue-on-fail-7jgvr
Namespace:           anomaly-detection
ServiceAccount:      review-feature-fa-1ro31a-anomaly-detection-backend
Status:              Failed
Created:             Thu Apr 09 13:06:01 -0500 (11 minutes ago)
Started:             Thu Apr 09 13:06:01 -0500 (11 minutes ago)
Finished:            Thu Apr 09 13:07:20 -0500 (10 minutes ago)
Duration:            1 minute 19 seconds

STEP                                     PODNAME                               DURATION  MESSAGE
 ✖ dag-contiue-on-fail-7jgvr (workflow)
 ├-✔ A (whalesay)                        dag-contiue-on-fail-7jgvr-3913376868  49s
 ├-✖ B (intentional-fail)                dag-contiue-on-fail-7jgvr-3963709725  3s        failed with exit code 1
 ├-✔ C (whalesay)                        dag-contiue-on-fail-7jgvr-3946932106  18s
 ├-✖ E (intentional-fail)                dag-contiue-on-fail-7jgvr-3846266392  2s        failed with exit code 1
 ├-✔ F (whalesay)                        dag-contiue-on-fail-7jgvr-3896599249  19s
 └-✔ D (whalesay)                        dag-contiue-on-fail-7jgvr-3863044011  5s

Logs

executor logs:
For status-reference:

kubectl logs <failedpodname> -c init

time="2020-04-09T18:19:47Z" level=info msg="Creating a docker executor"
time="2020-04-09T18:19:47Z" level=info msg="Executor (version: vHEAD+unknown, build_date: 2020-03-04T21:31:18Z) initialized (pod: anomaly-detection/status-reference-sslm2-141736205) with template:\n{\"name\":\"flakey-container\",\"inputs\":{},\"outputs\":{},\"metadata\":{},\"script\":{\"name\":\"\",\"image\":\"alpine:3.6\",\"command\":[\"sh\",\"-c\"],\"args\":[\"exit 1\"],\"resources\":{},\"source\":\"\"}}"
time="2020-04-09T18:19:47Z" level=info msg="Loading script source to /argo/staging/script"
time="2020-04-09T18:19:47Z" level=info msg="Start loading input artifacts..."
time="2020-04-09T18:19:47Z" level=info msg="Alloc=2357 TotalAlloc=3498 Sys=68610 NumGC=1 Goroutines=3"

kubectl logs <failedpodname> -c wait

time="2020-04-09T18:19:49Z" level=info msg="Creating a docker executor"
time="2020-04-09T18:19:49Z" level=info msg="Executor (version: vHEAD+unknown, build_date: 2020-03-04T21:31:18Z) initialized (pod: anomaly-detection/status-reference-sslm2-141736205) with template:\n{\"name\":\"flakey-container\",\"inputs\":{},\"outputs\":{},\"metadata\":{},\"script\":{\"name\":\"\",\"image\":\"alpine:3.6\",\"command\":[\"sh\",\"-c\"],\"args\":[\"exit 1\"],\"resources\":{},\"source\":\"\"}}"
time="2020-04-09T18:19:49Z" level=info msg="Waiting on main container"
time="2020-04-09T18:19:51Z" level=info msg="main container started with container ID: a1fbba5c9be4976b61b031a6dee81288c44ee37c2f97147d0d78e4a0f67a4e1f"
time="2020-04-09T18:19:51Z" level=info msg="Starting annotations monitor"
time="2020-04-09T18:19:51Z" level=info msg="docker wait a1fbba5c9be4976b61b031a6dee81288c44ee37c2f97147d0d78e4a0f67a4e1f"
time="2020-04-09T18:19:51Z" level=info msg="Starting deadline monitor"
time="2020-04-09T18:19:51Z" level=info msg="Main container completed"
time="2020-04-09T18:19:51Z" level=info msg="No output parameters"
time="2020-04-09T18:19:51Z" level=info msg="No output artifacts"
time="2020-04-09T18:19:51Z" level=info msg="No Script output reference in workflow. Capturing script output ignored"
time="2020-04-09T18:19:51Z" level=info msg="Killing sidecars"
time="2020-04-09T18:19:51Z" level=info msg="Annotations monitor stopped"
time="2020-04-09T18:19:52Z" level=info msg="Alloc=3055 TotalAlloc=5359 Sys=69442 NumGC=2 Goroutines=8"

For dag-continue-on-fail:
kubectl logs <failedpodname> -c init
# invalid
kubectl logs <failedpodname> -c wait
time="2020-04-09T18:31:09Z" level=info msg="Creating a docker executor"
time="2020-04-09T18:31:09Z" level=info msg="Executor (version: vHEAD+unknown, build_date: 2020-03-04T21:31:18Z) initialized (pod: anomaly-detection/dag-contiue-on-fail-dgxtt-1052672616) with template:\n{\"name\":\"intentional-fail\",\"inputs\":{},\"outputs\":{},\"metadata\":{},\"container\":{\"name\":\"\",\"image\":\"alpine:latest\",\"command\":[\"sh\",\"-c\"],\"args\":[\"echo intentional failure; exit 1\"],\"resources\":{}}}"
time="2020-04-09T18:31:09Z" level=info msg="Waiting on main container"
time="2020-04-09T18:31:33Z" level=info msg="main container started with container ID: c4bfe034ea2fa7375b2c7ad9a64a04b767a6b87d78944e256ed46fc5fdd28725"
time="2020-04-09T18:31:33Z" level=info msg="Starting annotations monitor"
time="2020-04-09T18:31:33Z" level=info msg="docker wait c4bfe034ea2fa7375b2c7ad9a64a04b767a6b87d78944e256ed46fc5fdd28725"
time="2020-04-09T18:31:33Z" level=info msg="Starting deadline monitor"
time="2020-04-09T18:31:33Z" level=info msg="Main container completed"
time="2020-04-09T18:31:33Z" level=info msg="No output parameters"
time="2020-04-09T18:31:33Z" level=info msg="No output artifacts"
time="2020-04-09T18:31:33Z" level=info msg="No Script output reference in workflow. Capturing script output ignored"
time="2020-04-09T18:31:33Z" level=info msg="Killing sidecars"
time="2020-04-09T18:31:33Z" level=info msg="Annotations monitor stopped"
time="2020-04-09T18:31:33Z" level=info msg="Alloc=3367 TotalAlloc=5377 Sys=70848 NumGC=2 Goroutines=8"

workflow-controller logs:

kubectl logs -n argo $(kubectl get pods -l app=workflow-controller -n argo -o name)

kubectl logs -n argo $(kubectl get pods -l app=workflow-controller -n argo -o name)
# no access :(

Message from the maintainers:
If you are impacted by this bug please add a 👍 reaction to this issue! We often sort issues this way to know what to prioritize.

Opened https://github.com/argoproj/argo/pull/2656 to fix this.
Note that dag-continue-on-fail actually has two failed tasks, A and E, but only A has continueOn set. The bug in this issue is still valid (dag-continue-on-fail still fails if E is removed), but note that dag-continue-on-fail should still not succeed as-is even after this bug is fixed.
Also, we kindly ask that only one 👍 is added per organization for issues. It helps keep things fair 🙂


How to inject css into a application

Homework

[ ] I looked at nativefier --help and https://github.com/jiahaog/nativefier/blob/master/docs/api.md
[ ] I searched existing issues, open & closed. Yes, my question is new.
[ ] I'm using the latest version available at https://github.com/jiahaog/nativefier/releases

See so I made a game website called krunker.io into a client, but i want to inject my own css into the application. How do I accomplish this? Thank you

https://github.com/jiahaog/nativefier/blob/master/docs/api.md#inject

Thanks @mattruzzi !
@idgafidc as pointed out by @mattruzzi , this is already documented. Please give it a try, and if documentation is unclear, chime in here and we'll improve the documentation.


remove all uses of check-missing: false

I've been doing this wrong.
in 2.12.x, this has been resulting in binaries being retrieved from Maven Central instead of dbuild-built ones being used (e.g. for kind-projector)
in 2.13.x, it just doesn't work at all (because Maven Central doesn't have matching binaries), which is how the problem turned up
this will take some work to make the change build-wide

closing in favor of #914

JFYI you can change an issue into a pull-request by using the -i flag in hub's pull-request. https://hub.github.com/hub-pull-request.1.html

oh, does that still work? I had thought they removed that capability

Yeah, it was gone but it was brought back (a few months ago).  Not sure why.


Do not break app when experiment folder is missing

Do not break the whole app as soon as an experiment folder is missing.
Closes
https://github.com/h2oai/h2o-llmstudio/issues/757

I find it personally this way better. Just keep it as-is. It is fine that the tabs fail, but then the whole app does not fail like it did now. It is a very edge case that this happens anyways.


Portal view position changes, Blocks cut into view

I'm using 1.14.4 Fabric build 12, Loader 0.4.8 build 159 with the following mods:
Fabric API 0.3.1+build.208
Immersive Portals 0.12

As I move around, the point from which the portal sees from moves as well, causing it to clip through blocks and see things it shouldn't from it's intended perspective.

I fixed culling with shaders but accidentally broke the culling functionality without shaders. It's fixed. It will be in 0.13 version.


TDD + Biz

We are using TDD and we are trying to find out a way to mock dates in a way that works with biz. Do you know any solution that would work?
In particular we have a "expire date" set in working hours in our task model and we need to test our methods to see if a task is expired or not and all the things that are done if so.
Thanks!

Hey there! I'd be happy to help, but I'm having a little trouble envisioning how you're using biz in this case.
Do you have some example code (production and test) that you could provide? In general, you should be able to inject plain ol' Date and Time objects while testing.

Hi @craiglittle, I was trying to write a test for the following method:
  def is_due?(task)
    due_date = Biz.time(task.due_time, :hours).after(task.created_at)
    due_date <= Time.now
  end

Thanks to @Intrepidd now I'm using #travel_to (blog entry about it), available on rails >= 4.1 which solved the problem of setting a fixed date to be able to write test cases for that method.
Thanks for the quick response!

Ah, gotcha! timecop is another good option when you're not in a Rails environment.
Glad you got it worked out!


[explorer] - responsive footer update

Description

Add responsive styles for the updated Footer
Went back to using footerLinks to keep the markup a bit more tidy
Note: It was non-trivial to get the order of items to reverse between Desktop / Mobile, so I compromised for the sake of time

Test Plan
How did you test the new or updated feature?

If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.
Type of Change (Check all that apply)

[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

Current dependencies on/for this PR:

main

PR #11647   👈

This comment was auto-generated by Graphite.


Pie Graph/Chart

Any chance of having a pie chart as well? The syntax is very neat by the way, I love it!

What a pity!


Use nuget

This change is

............\ comes about because the nuget packages have been installed by default into my user directory. Where would you expect them to be installed?
No reference anywhere to 'Art of Reading' except in a control description, presumably for a tooltip. Where would you expect to find public constants declared?
the //old code removed
All Autofac references set to autofac6.1.0/lib/netstandard2.0

@madskinner I see. The problem is that the users directory may not be in the same relative directory for every setup. I think we'll need to reference them from the Output directory. Visual Studio will do the copy. Once you have everything using the Nuget packages the output folder should be safe.

Also, FYI you don't need to close a PR just because you have changes. You can make more commits to the branch and push it and the contents will update here.


Installs shared libraries without SONAME

Describe the bug
The FreeBSD ports framework complains:
Error: /usr/local/lib/python3.7/site-packages/ortools/.libs/libortools.so.8 is linked to /usr/local/lib/libabsl_flags.so.2103.0.0 which does not have a SONAME.  devel/abseil needs to be fixed.
Error: /usr/local/lib/python3.7/site-packages/ortools/.libs/libortools.so.8 is linked to /usr/local/lib/libabsl_flags_reflection.so.2103.0.0 which does not have a SONAME.  devel/abseil needs to be fixed.

Could you please add SONAME?
Thanks,
Yuri

Duplicate of #862


FE-24: Like

What's up?
We need to design and implement Like interface.
To Do

[x] Create the necessary interfaces for the like functionality.
[x] Connect it to backend.

Deadline
19.12.2022 @23.59
Additional Information
No response
Reviewers
No response

Items mentioned above are implemented.

Reviewed #374, everything is ok. Completed.


UTable #actions-data outdated when rows change

Environment

Operating System: Darwin
Node Version:     v20.15.1
Nuxt Version:     3.12.3
CLI Version:      3.12.0
Nitro Version:    2.9.7
Package Manager<EMAIL_ADDRESS>
Version
v2.17.0
Reproduction
https://stackblitz.com/edit/github-hpn25g
Description
We recognized that the row data which is passed to actions does not reflect the new state of the row data. This is relevant because row actions might lead to change of data in the backend.
Repro:

Load the page with the table (posts are fetched)
Rename a single post via Rename action (modal input and confirm)
Posts are rerendered, table properly reflects new name for one of the posts
Reopen the Rename modal --> see there is still the former name present
Expectation would be that the table component always forwards the current data to the actions

I am aware that I have to "fake" the refetch mechanism since the used dummy API does not allow mutations on the server. In our real world scenario we have a real backend but the issue is still reproducible.
As a workaround we use a constant field of a model (e.g. "id") and grab the item from the refetched lists of models before we pass it to the modal.
Is this a real bug or are we doing something wrong here? If you could give me some hints where/how to start I could try to provide a PR with a fix.
Additional context
No response
Logs
No response

Hello. I try and its work for me

<template>

  <UTable :columns="columns" :rows="posts">
    <template #actions-data="{ row }">
      <UButton title="Rename" color="primary" @click="rename(row)">Rename</UButton>
    </template>
  </UTable>

</template>

<template>
  <UModal>
    <UCard>

        New name for post with title  {{ oldTitle }}  is:

      <input v-model="newTitle" placeholder="New title" />
      <template #footer>

          <UButton variant="ghost" @click="emit('cancel')">Cancel</UButton>
          <UButton color="primary" @click="emit('confirmRename', newTitle)">Rename</UButton>

      </template>
    </UCard>
  </UModal>
</template>

True. I forked a Stackblitz with your code: https://stackblitz.com/edit/github-hpn25g-femxdi Yes it is working. Played around a bit and the reason/difference is, that you manipulate the data array (posts) in place instead of replacing the whole array. That is interesting, but not really a solution since in a real-world scenario we use the "refresh" function of AsyncData. So why does the table component have a problem with replacing the whole data array?

this works fine https://stackblitz.com/edit/github-hpn25g-oehbpx
i change this func
async function postsRefresh(updatedPost) {
  posts.value = posts.value.map((item) => {
    if (item.id === updatedPost.id) {
      return updatedPost;
    }
    return item;
  });
}

to this
function postsRefresh(updatedPost) {
  posts.value.forEach((post) => {
    if (updatedPost.id === post.id) {
      post.title = updatedPost.title
    }
  })
}

@ldiellyoungl I already got the difference when you answered the first time. It is not a fix to the problem since you are talking about code which we do not have in production scenario. The StackBlitz is only for demonstration purposes. In reality we do not have such a postsRefresh method. In reality we refetch the whole list again from backend. Then the list of models is replaced completely (so .map is actually more realistic than .foreach).
The real bug is that after replacing the data array the table does reflect the new model properties, so the rendering is correct. But what is not correct is the model data which is passed to actions ("#actions-data").

hmmm... maybe i dont understand, sorry. In my production i use websockets and change array with forEach method refresh  method not my chose :/

No worries. Using foreach and keeping the existing items list is also some kind of workaround for this problem described here (could be problematic if there are new or removed items from the list though?). We work around the issue like initially mentioned: we just ignore the model data which goes into #actions-data and find the model by id from the data array which is always uptodate.

I experement with your code
async function postsRefresh(updatedPost) {
  posts.value = posts.value.map((item) => {
    if (item.id === updatedPost.id) {
      return updatedPost;
    } else {
      return item;
    }
  });
  console.log(posts.value)
}

and got this:

first object is not proxy. maybe this is problem?
i try this code and its work! Can you check this? Most likely I did the same as above....
async function postsRefresh(updatedPost) {
  posts.value = posts.value.map((item) => {
    if (item.id === updatedPost.id) {
      return Object.assign(item, updatedPost);
    } else {
      return item;
    }
  });
  console.log(posts.value)
}

I have the same problem. #<column>-data slot's row property is not updated after refreshing table data. As a workaround similar to @robert-gruner 's I use the index property of the slot to lookup the actual row data.
As a side note index slot property is not documented here: https://ui.nuxt.com/components/table#column-data


Fix format in checkArgument/checkState calls

Fixes some bugs introduced in https://github.com/prestosql/presto/pull/214 (thanks @electrum for spotting this!) and some more.

squashed


[App Configuration] Add sorting to the different List views

Name of feature to enhance

We have a couple of list views on the app the main ones being the registers and tasks lists on the app.
The ability to sort them using a couple of resource attributes will be key for optimizing the UX.

Description of feature

[ ] Add the ability to sort the different resources in a list based on certain attributes of the resources in question.

[ ] We will support 2 sorting functionalities Ascending and Descending e.g [A-Z, 0-9] or [Z-A, 9-0]

Describe the enhancement

[ ]  Ordering the resources in a list in Ascending or Descending order based on certain resource attributes.

How would this tackle give a use case?

Ascending/Descending by Given name
Ascending/Descending by Family name
Ascending/Descending by Registration date
Ascending/Descending by Last Encounter

Would it be possible to

Allow the user to define a set of [name, query] pairs in an order key field in the config, where the query is defining the order, and have a convention for asc/desc, so we'd support asc/desc for every order query they define?
How would we execute the above? What would be the structure of that config?

Blocker:  Need to check on wether to do a Sort from Db or to do it programmatically from code. TBD

This will have to depend on how much data exists, but there'll definitely be a point when sort in code will be too slow, so I think regardless we have to do sort from DB.
Maybe later we could add an optimization where it sorts code if there are less than N elements, but if db queries are fast enough that won't be necessary.

I agree with @pld supporting sorting from the DB based on the indexed attributes per resource (which I think are all the attributes.
I might need to confirm this)
I think we can always review situations to filter or sort form code e.g We would need to sort the Groups by code if we wanted to sort or filter them by name since the name attribute is not an allowable search attribute for the Group.
We already have sorted by some of these attributes and we are using ASC as the default sort order. The user can however decide to use a different sort like in this example.
https://github.com/opensrp/fhir-resources/blob/273e5860d0ae710f4ea3ea45f324f68bbde69ebc/ecbis_liberia/app_configurations/ecbis_cha_preview/registers/household_register_config.json#L30-L36

So I would say this issue has the following tasks.
Sort & FIlter from the DB

[ ] Write a generic filter function that can filter all the resources from the DB using the available indexes.
[ ] Write a generic sort function that can sort all the resources from the DB using the available indexes.
[ ] All the above methods should be triggered from the configs.

Sort and filter using code - This can be a lower priority than the 2 above.

[ ] Write a function to filter and sort generic resources based on other attributes that are not indexed.

This method should also be triggered from the configs.

NB we can decide to limit the above method to just a list of resources or we can leave it open and advise against using it because of the performance implications.

@pld @f-odhiambo let me know what you think so I can extract the points above to individual issues that different people can pick up.

Can we not do filter and sort for attributes that are not indexed? Is there a way to manually add an index? If you can do that, then we shouldn't need this, we just need docs that say, first add the index, then you can do the filter/sort

@pld this is possible but I think we need to update the SDK so that it provides a way for implementors to easily define the extra attributes to cache.
We can also choose to add this to fhircore then make sure we maintain it.

Ah ok yea that seems like a generally useful API feature to have: asking to index specific kvs
Is that something we need in the first version, like do we have known use cases where we'll want to filter or sort on an attribute that isn't indexed?

We will need to sort Families (represented by the Group resource) using the name. The name attribute is not part of the searchable attributes of the Group resource.
This means it's not indexed.

OK, then this plan sounds good, we can add an issue for a create-index API to the Android FHIR SDK, and if we see queries on non-indexed fields going too slow maybe do that work in the SDK or in FHIRCore, depending on timing

On Feb 4, 2023, at 4:20 PM, Benjamin Mwalimu @.***> wrote:
We will need to sort Families (represented by the Group resource) using the name. The name attribute is not part of the searchable attributes of the Group resource.
This means it's not indexed.
—
Reply to this email directly, view it on GitHub https://github.com/opensrp/fhircore/issues/1789#issuecomment-1416852668, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAMMETUPWXVC52MJEQDLMDWV3BZ7ANCNFSM6AAAAAARR3VLVY.
You are receiving this because you were mentioned.

This seems like it will enable this functionality, https://github.com/google/android-fhir/pull/1778

This test case seems like a good guide, https://github.com/aditya-07/android-fhir/blob/master/engine/src/test/java/com/google/android/fhir/MoreSearchParametersTest.kt

We can close this issue. A separate GitHub issue can be created to track sorting _revinclude/forward included resources (the current workaround is to use RulesEngine sort functionality that leverages FHIRPath and Kotlin sort function). We partially support database sorting for base resources and code-level sorting that can be applied via Rules Engine.


doc: add fullscreen link to root readme

:tada: This PR is included in version 7.5.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[BUG] : 'Next' and 'Previous' are needed to be resized when the size of the screen changes to mobile screen.

'Right arrow button ' or ' Next button ' as well as 'Left arrow button ' or ' Previous button '  buttons are overlapping the content of carousel in the mobile screen.
Steps to reproduce the behavior:

Go to the Home page of the website.
Resize the size of the screen to the mobile screen size or view the page on mobile screen
Next and previous buttons are overlapping the content of the carousel.

The buttons are expected not to be overlapping the content. So that the user experience could be increased.

can you assign me this under hacktoberfest please ?


Fix test_version to resolve unused expression warning

resolves https://github.com/cnpryer/huak/issues/633

Thanks!


Fix flakiness in this test: ReloadSchema might be asynchronous.

@enisoc @sougou @guoliang100 whoever gets there first :)

lgtm


Links to frees-rpc docs

Fixes https://github.com/frees-io/freestyle-rpc/issues/51

Codecov Report

Merging #22 into master will not change coverage.
The diff coverage is n/a.

@@         Coverage Diff          @@
##           master   #22   +/-   ##
====================================
  Coverage       0%    0%
====================================
  Files           5     5
  Lines         126   126
  Branches        1     1
====================================
  Misses        126   126

Continue to review full report at Codecov.

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


Update io.github.lact-linux.desktop

Thanks, forgot to update that description. It probably still says AMDGPU in a bunch of places


Bug: a test must be modifying global state

Failure in CI running the example tests after the unit tests that doesn't occur if the example tests are run independent of the unit tests or before the unit tests:
 =================================== FAILURES ===================================
__________________ test_optional_dependency_in_openapi_schema __________________

    def test_optional_dependency_in_openapi_schema() -> None:
        with TestClient(app=dependency_default_value_no_dependency_fn.app) as client:
            r = client.get("/schema/openapi.json")
>       assert r.status_code == HTTP_200_OK
E       assert 404 == 200
E        +  where 404 = <Response [404 Not Found]>.status_code

examples/tests/dependency_injection/test_dependency_default_value_no_dependency_fn.py:10: AssertionError
________________ test_optional_dependency_not_in_openapi_schema ________________

    def test_optional_dependency_not_in_openapi_schema() -> None:
        with TestClient(app=dependency_default_value_with_dependency_fn.app) as client:
            r = client.get("/schema/openapi.json")
>       assert r.status_code == HTTP_200_OK
E       assert 404 == 200
E        +  where 404 = <Response [404 Not Found]>.status_code

examples/tests/dependency_injection/test_dependency_default_value_with_dependency_fn.py:10: AssertionError

---------- coverage: platform linux, python 3.10.7-final-0 -----------
Coverage XML written to file coverage.xml

The command for the tests on 3.10 in CI is:
poetry run pytest tests examples --cov=starlite --cov=examples --cov-report=xml

If I swap the order so examples run first, everything is fine:
poetry run pytest examples tests --cov=starlite --cov=examples --cov-report=xml

If I had to guess, I'd say some test is modifying the default openapi config in-place, but I haven't had the time to look into it.


Postgres SSL option in database config does not work as a Netlify environment variable

Bug Description
In database.config.json, an ssl option can be supplied for Postgres, which can be true, false, or an object. If you include an object, it works when set locally, but does not work when the options from the local database config are inputted as environment variables in Netlify.
Expected Behavior
We should be able to successfully use the same database config options locally as in Netlify.
Error Messages and Screenshots
Netlify build fails when ssl environment variable supplied
Workarounds
Use a connectionString instead of separate credentials (username, password, etc.) and append ?sslmode=no-verify to the end.
Connection string takes this form:
postgres://[user]:[password]@[host]:[port]/[database]?sslmode=no-verify
And is included in database.config.json in place of all other credentials, so the database config will look as follows:
{
     "connectionString": "postgres://[user]:[password]@[host]:[port]/[database]?sslmode=no-verify"
}

Workaround Example (using mock credentials):
Credentials:
host: ec2-34-948-34-100.compute-1.amazonaws.com
database: ijefij426jnk32k
port: 5555
user: fehuwifheiu
password: 1ab2cd3
Connection String:
postgres://fehuwifheiu:1ab2cd3@ec2-34-948-34-100.compute-1.amazonaws.com:5555/ijefij426jnk32k?sslmode=no-verify
Database Config File:
{
     "connectionString": "postgres://fehuwifheiu:1ab2cd3@ec2-34-948-34-100.compute-1.amazonaws.com:5555/ijefij426jnk32k?sslmode=no-verify"
}

Netlify Environment Variable:
Variable Name: connectionString
Variable Value: postgres://fehuwifheiu:1ab2cd3@ec2-34-948-34-100.compute-1.amazonaws.com:5555/ijefij426jnk32k?sslmode=no-verify
Database:

[x] Postgres
[ ] BigQuery
[ ] Snowflake
[ ] Redshift
[ ] MySQL
[ ] Other (please specify)

It might be this line in the postgres package causing the issue:
ssl: database ? database.ssl : process.env["ssl"] ?? false,
Locally, we can receive an object as the ssl option, but it appears that Netlify cannot accept the same input.
We've tried using double underscore notation to indicate a value stored within an object (e.g., ssl__rejectUnauthorized), but it is unclear if that works. If Netlify is able to process that, the line in the postgres package shown above may default to false since it is not receiving a specific ssl option.

Using the below should work:
ssl: no-verify

Closing since ssl: no-verify works for this case


Add flag for generating shell completions

This adds a flag (--gen-completions <shell>) that generates shell completions to stdout. It also changes CI so that all completions should be generated and provided up front (assuming I didn't subtly bork the release CI job)
I can confirm that it works on zsh for me

Everything looks good to me. Thank you for this 👍.


Everything setup correctly but not working

Hi everyone, is there a way to debug what maybe noticed are not being pushed to slask? it is all setup and integrated.... app enabled for the chatroom and the incoming webhook url but dont seem to sending to the channel

Generally if there's an error, it will be logged to /usr/local/cpanel/logs/error_log (as is normal with noticies in cPanel & WHM). If it is failing silently, debugging statements would likely need to be added.

Nothing i can see in the log file, what would i need to do to add debugging statements? I am a developer but not familiar with these scripts. Thanks in advance

Hmm, there should at least be a line in /usr/local/cpanel/logs/error_log at least saying something like:
'[2018-01-01 00:00:00 -0500] info [quotacheck] Quota::DiskWarning Notification => info@xxxxxxxxxx
http://some.incoming.webhook/url***** via SLACK [eventimportance => High (1)]`
if it is properly configured.
If not, you'd need to check in WHM >> Contact Manager or /etc/clevels.conf (and make sure it is configured to send notifications for the level you've set Slack notices to get sent).
Hopefully it is just something that simple, but if not, I'll work on adding a debugging test that enables a lot of prints, etc. similar to my XMPP and IRC providers (probably this weekend).

I've updated the readme with instructions on how to run a full test of the provider (it doesn't have to be on a box with cPanel installed if you just run make depend and then run scripts/generate_testing_configuration.pl btw).
I just received a message in the slack room I setup to receive message from my incoming webhook URL, so it "works for me" as a test. If you can't get it to send with that test, it should at least throw some sort of error in the test. IF it sends OK in that test (but not in cPanel), then I'd suspect a misconfiguration, likely related to how you've setup something in /etc/clevels.conf.
Let me know how all that goes, I tried to make all this "so simple a soldier could do it," but I'm sure my idea of what constitutes that is probably still too complex for most.

Closing since no response for a while... feel free to comment with more information on what system you were setting this up on. Things I'd want to know:

cPanel version
Notification type we were attempting to send on
Contents of /var/cpanel/clevels.conf
Contents of /var/cpanel/icontact_event_importance.json

If I knew all of the above, I could definitively rule out user error (at least).


Regression: range vector queries report samples multiple times

What did you do? / What did you expect to see?
Set up Prometheus with the SNMP exporter. Ran the following query through the web interface:
ifInUcastPkts{ifIndex="10648"}[2h]
Got the following results:
3548205210 @1508765401.244
3548946632 @1508765701.244
3549646577 @1508766001.266
3550396212 @1508766301.268
3551081575 @1508766601.367
3551730505 @1508766901.244
3552413802 @1508767201.244
3553181994 @1508767501.244
3554009421 @1508767801.254
3554675942 @1508768101.244
3555286317 @1508768401.244
3555922170 @1508768701.244
3556718950 @1508769001.244
3547556798 @1508765101.244
3548205210 @1508765401.244
3548946632 @1508765701.244
3549646577 @1508766001.266
3550396212 @1508766301.268
3551081575 @1508766601.367
3551730505 @1508766901.244
3552413802 @1508767201.244
3553181994 @1508767501.244
3554009421 @1508767801.254
3554675942 @1508768101.244
3555286317 @1508768401.244
3555922170 @1508768701.244
3556718950 @1508769001.244
3557449169 @1508769301.244

Notice that there are only 15 unique entries in the output; some of them are duplicated. I would have expected this to return
3547556798 @1508765101.244
3548205210 @1508765401.244
3548946632 @1508765701.244
3549646577 @1508766001.266
3550396212 @1508766301.268
3551081575 @1508766601.367
3551730505 @1508766901.244
3552413802 @1508767201.244
3553181994 @1508767501.244
3554009421 @1508767801.254
3554675942 @1508768101.244
3555286317 @1508768401.244
3555922170 @1508768701.244
3556718950 @1508769001.244
3557449169 @1508769301.244

At the same time, I notice that more elaborate queries like rate(ifInUcastPkts{ifIndex="10648"}[30h]) don't work as expected. Every couple of hours, we see a huge jump in the graph that shouldn't be there:

If I do the math by hand on the raw sample data, it should be a relatively flat graph with a value of ~2000; not peaks of millions.
Environment

System information:

Linux 3.16.0-4-amd64 x86_64

Prometheus version:

The official Prometheus 2.0.0-rc.1 Docker image.

Can you reproduce this across all metrics or just a few selected ones?

(Hi Fabian! Thanks for your awesome work on the new storage backend!)
It seems I can reproduce this across all metrics; even implicit ones like up.

Okay thanks, looks like I can reproduce it. By first intuition this may be the reason for the spike artefact you are seeing. But I cannot reproduce this equally right away.

So for me it seems the repeated samples when doing a simple range query only happens once, i.e. if you do it over longer and longer periods of time, there's always the fixed same amount of lines at the beginning being duplicated.
Can you confirm that behavior?
Also, do I see it correctly that your scrape interval is 5 minutes? This is kind of a threshold number for scraping. Just for debugging purposes, could you reduce it to at most 2 minutes and check if the spiking behavior in the rate() queries remain (for the newly collected data)?

So for me it seems the repeated samples when doing a simple range query only happens once, i.e. if you do it over longer and longer periods of time, there's always the fixed same amount of lines at the beginning being duplicated.
Can you confirm that behavior?

Yes, that seems to be the case exactly!

Also, do I see it correctly that your scrape interval is 5 minutes?

That's correct.

This is kind of a threshold number for scraping. Just for debugging purposes, could you reduce it to at most 2 minutes and check if the spiking behavior in the rate() queries remain (for the newly collected data)?

I'll give that a try first thing tomorrow morning, leave it running throughout the day and report back to you at the end of the day.

It looks like switching to a two minute scraping interval worked around the issue:

Upgraded to RC2 this morning and set the scraping interval back to five minutes. No anomalies so far. I think it's safe to close this issue. Fabian, thanks for solving this so quickly!


Added animated theme

Added animated theme with streamlit lottie component and brought title to centre for better visibility.

@Ashish-Abraham please resolve the conflicts

Conflicts are resolved now. Please check. Thanks😊


Contributing guidelines

First of all, congrats for this awesome repository :), I would love to contribute but I have no previous experience developing VSCode extensions.
I'm using a Mac, so I have modified the file at ' ~/.vscode/extensions/esri.arcgis-jsapi-snippets-1.1.2/snippets/javascript.json' but I don't know how I can test if it's properly working. How do I do it? Do I need to refresh/reload the extension?
Thanks!!

Note: I would be nice to include the readme / contributing.md simple instructions on how to do it.

I just found that adding a "prefix" and with Cmd+Shift+P -> "Developer: Reload Window" just works :)

One question, do you think it could make sense to add the following resources to the contributing.md:

Snippets in Visual Studio Code
Your First Extension | Visual Studio Code Extension API
JS IntelliSense in VSCode
IntelliSense in VSCode

I would be welcome to add a PR if you think so :).
// cc: @RalucaNicola, @andygup , @kellyhutchins, @ekenes

that sounds like a good idea to me!

Great, I'll add it then 😄 .
One more question, do you recommend me to link then snippets in the ~/.vscode/extensions with the snippets on this repo?, or do you just use a developer version of the snippet in local to avoid conflicts?

my workflow is usually to test them locally, just by adding them to the user snippets files (Cmd + P >Preferences: Configure user snippets, pick the language and add the snippet in there. If it's good, then I add it to this repo and I check it out once it's published. Then I usually remove the snippets from the local file.
I am not very proficient in this though, so there are for sure better ways to test the snippets before adding them to production.

Cool! I didn't know about the "Cmd + P >Preferences: Configure user snippets" either! 😅 , that another easy way to test snippets 👍 .
Anyway, I have also to install the VSIX from local (thinking that it might link the original folder):

But it makes a copy of the folder, then ask me to update to 1.1.2 and do another copy:

And of course, none of those folders are identical to the original, so... I will wait for @kellyhutchins 😄 .
Thanks one more time!!

I always test snippets locally using the same steps as Raluca outlines above. Installing from .vsix is just another way to install the existing published extension.
I will work on updating and publishing the updated extension to marketplace this week. Thanks for your contributions!

Thank you! I just did my PR so I'm going to close this 😉


Night Mode

Stratux version:1.4r2

Stratux config:
SDR

[ ] single
[x] dual

GPS

[x] yes
[ ] no
type: GPYes

AHRS

[x] yes
[ ] no

power source:
usb cable:

EFB app and version: ForeFlight
EFB platform: iOS 10.3
EFB hardware: iPad Mini 2

Description of your issue:  Enhancement request.  What are the chances of doing a night mode in the UI?  Just a button that would turn off all the lights in the Pi, SDRs, and GPYes?

If possible, enable "Replay Logs", reproduce the problem, and provide a copy of the logs in http://<IP_ADDRESS>/logs/stratux/ and http://<IP_ADDRESS>/logs/stratux.log.

Thanks for asking this. See below, the webui looks pretty decent with inverted colors.
Settings -> General -> Accessibility -> Accessibility Shortcut (all the way at the bottom) -> Invert Colors
Then triple-tap the home screen button to activate/deactivate.

Does that turn off the hardware lights?  That's really my problem right now.

On Aug 28, 2017, at 21:08, cyoung<EMAIL_ADDRESS>wrote:
Thanks for asking this. See below, the webui looks pretty decent with inverted colors.
Settings -> General -> Accessibility -> Accessibility Shortcut (all the way at the bottom) -> Invert Colors
Then triple-tap the home screen button to activate/deactivate.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

No, it doesn't.
Lights on the SDRs - which ones do you have? Recent ones don't have LEDs.
GPYes - can't turn those off in software.
We'd be looking at only turning off the LEDs on the Pi.

@cyoung are we sure the GPYes can't be turned off in software? When I looked into the u-blox 7 spec a few weeks ago it seems like a CFG-TP5 serial message (ID 0x06 0x31) could be formatted to deactivate the default time pulse output, which I believe is wired to the PPS LED on the GPYes.  I don't have anything to test on at the moment, but I would guess this should work?

@ijessen - you're right. Just tried this with u-center and that did it.

Okay, added NightMode. If someone wants to add something to the webui to toggle globalStatus.NightMode, that should round it out. Default state on led0 is "off". It will still flash if there's a critical system error.


Resetting user kind to null on login to trigger re-check for Github a…

After a lot of analysis, this is one of the few spots where we can put this code.

Makes sense to me...

Some working notes are available at the link here: https://github.com/ubccpsctech/classy/issues/90.

Looks good to me.
Would be good to get Reid's opinion...

Agreed. I will invite him to the ubccpsctech organization before this code finds its way further upstream.

@andrewstec this looks good. I'm sure we just cached this as a form of optimization that was probably premature anyways.


INFO: ===========
LINTING RESULTS
82 tests passed   2 tests had warnings   2 tests failed
WARNING: Test Warnings:
http://nf-co.re/errors#4: Config variable not found: params.reads
http://nf-co.re/errors#4: Config variable not found: params.singleEnd
ERROR: Test Failures:
http://nf-co.re/errors#8: Conda environment name is incorrect (nf-core-bcellmagic-1.0, should be nf-core-bcellmagic-1.0dev)
http://nf-co.re/errors#8: Conda dependency did not have pinned version number: bioconda::biopython
ERROR: Sorry, some tests failed - exiting with a non-zero error code...

As far as I got from http://nf-co.re/errors#4, we can ignore the warnings.
Conda environment name is incorrect -> I will fix this.
Conda dependency did not have pinned version number: bioconda::biopython -> As far as I got it from the linting discussion, the linting tool can not resolve this on its own. I will add a specific biopython version. I will also built the container again, to verify, that it is working.

solved now by PR #35


Added support for model : _TZE200_vzekyi4c

Smoke detection works. Though other readings like Battery do not show up.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5511

Relevant Lines:
6852

💛  - Coveralls

Could you edit the first comment and add that keyword reference, please?
Fixes: #1529

Thanks.

Sure, done
On Tue, 24 May 2022 at 18:45, javicalle @.***> wrote:

Could you edit the first comment and add that keyword reference, please?
Fixes: #1529
Thanks.
—
Reply to this email directly, view it on GitHub
https://github.com/zigpy/zha-device-handlers/pull/1581#issuecomment-1135756649,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGY3SM64L3LLUM772HH2C3VLSXM5ANCNFSM5WXWNZOQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Remove unsafe access to .data() and _data in Matrix

This PR removes access to the unsafe .data() and _data in Matrix.
Linked PRs which should be merged first:
https://github.com/PX4/Matrix/pull/90
https://github.com/PX4/ecl/pull/641

Nice!

Now depends also on https://github.com/PX4/Matrix/pull/94

We have to update the ecl again after it get merged.


rm ClimaCore deprecated functions

Purpose
update deprecated ClimaCore function calls
closes #490

Just one question - will we need to eventually pass in the comms_context from the driver? If that is the case, should we add it to the arguments of the domain constructors instead of making internally?

Good point - I'll add it as optional parameters to the domain constructors


add ability to retrieve current list of embedding trigger strings

This PR adds a new attributer to ldm.generate, embedding_trigger_strings:
gen = Generate(...)
strings = gen.embedding_trigger_strings
strings = gen.embedding_trigger_strings()

The trigger strings will change when the model is updated to show only those strings which are compatible with the current model. Dynamically-downloaded triggers from the HF Concepts Library will only show up after they are used for the first time. However, the full list of concepts available for download can be retrieved programatically like this:
from ldm.invoke.concepts_lib import HuggingFAceConceptsLibrary
concepts = HuggingFaceConceptsLibrary()
trigger_strings = concepts.list_concepts()

@blessedcoolant @keturn - Looks like you're up for review as code owner in order to get this merged.

Think this needs a review from @keturn .. I'm the codeowner for this for some reason the PR is still requesting it from a code owner.

…why is this so many lines?
Oh, I see.
In the future, if you're going to reformat a file and it changes more than a handful of lines, please do that in its own commit — if not its own PR. It'll make it much easier to see which changes are functional.


Create itivolta-edu-it

Request N. 52FB3643ED58E dated 2014, February 12th
Request for Accreditation to the
Directory of Public Administration
To the Manager of Directory of Public Administration
Pec<EMAIL_ADDRESS>This institution
ISTITUTO TECNICO TECNOLOGICO STATALE ALESSANDRO VOLTA
represented by
MARIA CRISTINA BERARDINI
requests the accreditation to the Directory of Public Administration and provides the required
information, as requested.
The undersigned declares that the mentioned institution,  for which registration to IPA is required,
belongs to the categories identified by Article 2 Comma 2  of Legislative Decree  82, 2005 March 7th
, named “Code for Digital Administration”, i.e. it is a public administration according to Article 1
Comma 2 of  Legislative Decree  165, 2001 March 30th.
Hence, being fully aware about the criminal liability for perjury, according to the current statutory
legislation, the undersigned takes on responsibility about the truthfulness of the stated declaration.

@ProfGallo lib/domains/it/itivolta-edu-it: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.


Lessons on automating with Openconfig

Hi,
I am thinking of the a 5-stage lesson for a Openconfig and custom Yang automation

Stage 1 install OC Yang on Junos box and provision OC config using CLI
Stage 2 provision OC config using netconf / ansible to show it works exactly as standard junos config
Stage 3 Custom Yang modules (service based e.g. l2service) and custom yang translator
Stage 4 Custom Yang config provisioning using CLI and netconf
Stage 5 Verification and Troubleshooting (show commands , logs, etc)

This is targeting beginner - immediate level users. Comments are welcomed.
tonychan@jnpr

@valjeanchan Looks good to me. Let me know when you have a PR for review, and if there's anything I can do to help.


Feature/add versions table in model group detail

Description
Add model details panel content
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #170 (95ec03e) into feature/model-registry (7c5cdf3) will increase coverage by 2.84%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@                    Coverage Diff                     @@
##           feature/model-registry     #170      +/-   ##
==========================================================
+ Coverage                   79.55%   82.40%   +2.84%
==========================================================
  Files                          84       92       +8
  Lines                        1521     1699     +178
  Branches                      274      318      +44
==========================================================
+ Hits                         1210     1400     +190
+ Misses                        248      229      -19
- Partials                       63       70       +7

see 31 files with indirect coverage changes


lang/org: Add evil-friendly calendar navigation

A few bindings for navigating the calendar in the org-read-date prompt. The default bindings are using Shift and Shift-Meta + the arrows, which is having us leave the comfort of the home row.

Looks good! Thank you for your help!


F/implement subgit import

Fixed #34
Implemented 'subgit import' with access to github and gitlab.
Fixed bugs related to subgit import which affected other commands. One of which being 'subgit pull' not being able to pull from config file where branch name is empty.
Updated README to include how to get started with 'subgit import' since it requires aditional dependencies which subgit does not help the user aquire.

I also tested using subgit import with the sub-group and it worked like a charm


paint cluster  in one location without making markers

hello, I am developing web site using google map,
I want to show  cluster like following image,
In one location,  having over 1000 markers markers and represent cluster by marker's count.
I edited prototype.calculator_ function , but I think there is no solution like this situation.
Is there any soltion ?

Yes, of course you can do it with MarkerClusterer Plus, this is exactly what it is designed to do. Of course, it may act a little slowly if you are trying to cluster thousands of markers.

Closing this unless there are specific issues encountered.


Implement redirection email

The redirection email has been forgotten to implement as part of the model and LDAP-Importer.
Besides that, the mail given in the „Email“ field of the user creation page is currently stored as the mail of the user. This is counter-intuitive, since the usual mail that is given during creation is the address for redirection.
Both mail attributes should be easily editable by the user manager.

…this is not the pycroft repository. Dang.


Bump bedrock for stream pump updates

must be merged alongside https://github.com/black-parrot/black-parrot/pull/1121 (when it merges to dev or master on BP)


Portal footer: Upraded PnP jsto 3.11.0

Category

[ ] Bug Fix
[x] New Feature
Related issues: fixes #X, partially #Y, mentioned in #Z

For the above list, an empty checkbox is [ ] as in [SPACE]. A checked checkbox is [x] with no space between the brackets. Use the PREVIEW tab at the top right to preview the rendering before submitting your issue.
(DELETE THIS PARAGRAPH AFTER READING)

What's in this Pull Request?
Upgraded PnP JS to 3.11.0. Removed usingCaching method since it's not available in PnP 3.11.0 version.

merged to v3 branch, thank you for the contribution.


Bundle not showing as a running applicication

Good afternoon
I just created my Burp install using your build scripts and everything seemed fine.  I launch that application from the Application folder and it runs but for some reason it is not showing as a running application on that mac.  I do not have any of the menu options so I can't fully utilize the tool.  Do you have any recommendation as to what could be the cause of this?  Thanks in advance for your help.

@Rharris11 Burp is still a java application, no matter what wrapper I placed around it. Unfortunately that means it will never be able to use the native macos gui, unless Apple decides to allow it, or Burp is ported to a native language( unlikely ). You should still see the burpsuite menus along the top of the application window though - if the case is that you aren't seeing those menus, there's something wrong.

Over a year old with no further information, closing for now.


Erro working with Flash Plugin in Windows

So I was trying to embed some videos in a electron app, but the following code works in linux and not in windows. In Windows it just says couldn't load plugin.
I am working on a Windows 8 64bits, with electron 0.29.2 and io.js 2.3.1
In Linux I am using a Ubuntu 14.04 LTS, electron 0.29.2 and io.js 2.3.1
Both the .dll and the .so files are in the same folder under application
var app = require('app');
var BrowserWindow = require('browser-window');
var dialog = require('dialog')

require('crash-reporter').start();

var mainWindow = null;

app.on(
  'window-all-closed', function() {
    if(process.platform != 'darwim'){
      app.quit();
    }
  });

if(process.platform  == 'win32'){
  app.commandLine.appendSwitch('ppapi-flash-path', __dirname + '/pepflashplayer.dll');
  app.commandLine.appendSwitch('ppapi-flash-version', '<IP_ADDRESS>');
} else if (process.platform == 'linux') {
  app.commandLine.appendSwitch('ppapi-flash-path', __dirname + '/libpepflashplayer.so');
  app.commandLine.appendSwitch('ppapi-flash-version', '<IP_ADDRESS>');
}

app.on('ready', function(){

	mainWindow = new BrowserWindow(
		{
			width:800,
			height:600,
			'web-preferences': {
				'plugins': true
			}
		});

	mainWindow.loadUrl('file://' + __dirname + '/index.html');
	mainWindow.openDevTools();

	mainWindow.on('closed', function(){
		mainWindow = null;
	});
});

My psychic debugger says that you've got a different architecture Electron vs. your pepflashplayer.dll (i.e. 64-bit Flash and 32-bit Electron.exe or vice-versa)

Thanks @paulcbetts:
I wiil lookup further into this.

Your psychic debugger was absolutely right, it was a problem with the version of electron that I had downloaded, replaced it with the 32 bits .exe and it worked.
Just a last question how can I get the .dll for a 64 bit electron.

Already found the dll thanks.

@rjusher where did you find dll for 64 bits

How were you able to get the 64 bit version of Electron?

@piecedigital check the link below
https://github.com/atom/electron/releases


Router appendRoutes() error

Fatal error: Call to a member function add() on a non-object in /var/www/webiny/src/Webiny/Component/Router/Router.php on line 226

This happens if you don't call Router::setConfig(). Setting router config should not be mandatory. I very often append routes to Router myself, and settings 1 config is not possible due to loading different modules each of which have its own config.
The problem is in this method inside Router.php:
public function appendRoutes(ConfigObject $routes)
    {
        foreach ($routes as $name => $routeConfig) {
            self::$_routeCollection->add($name, $this->_loader->processRoute($routeConfig));
        }

        return $this;
    }

self::$_routeCollection is not initialized if Router::setConfig() is not called, causing this error to be thrown.

Solved with the latest release, the Router has an initialize() method that can be called, when you are not defining your methods using a config file.


Deleted randutils module and moved its used functions to misc module

Why are we moving it to misc again? I thought #154 was just about removing the randutils functions that we don't need.
A module named misc is always a compromise for putting things that don't have a natural home. Removing modules with clearly defined name and purpose and shoving their functionality into misc should not be a goal...

Yes, I agree, I think I'm to blame on this.
Let's then rename this module to random, so it conforms to the naming pattern of other modules in easypy (everything is "utils").
clamp can be replaced by the easypy.misc.clamp, and interpolate can be discarded.


FileMode Error

Currently on the develop branch, couldn't figure it out, that's why posting here.
Whenever I open cpp files, in the bottom status area it shows error like "File Mode Specification Error (wrong argument type .. )" , and maybe because of this completion is also not working. Started facing this issue very recently. Sidenote: I barely made any modifications in the private module.

Figured it out, I tried this, and found that I had set line numbers to relative using (setq line-numbers 'relative), I removed that, now it's working fine.
One issue though, previously when you had all the keybindings for different modes set in their specific config.el files, you had keybindings for org mode for cycling the item sequence from TODO -> NEXT and so on, and many other bindings bound to local leader. Can you bring those bindings back to the main repo?


[master] FATA[0002] networks.yaml field paths.varRun error: path "." is not an absolute path

Description
In the master branch of Lima (05ed15108b952f5cc7a0664eaee57f385360cec9), limactl start template://vmnet fails with an existing  networks.yaml
$ limactl start template://vmnet
? Creating an instance "vmnet" Proceed with the current configuration
FATA[0002] networks.yaml field `paths.varRun` error: path "." is not an absolute path

~/.lima/_config/networks.yaml:
# Path to socket_vmnet executable. Because socket_vmnet is invoked via sudo it should be
# installed where only root can modify/replace it. This means also none of the
# parent directories should be writable by the user.
#
# The varRun directory also must not be writable by the user because it will
# include the socket_vmnet pid file. Those will be terminated via sudo, so replacing
# the pid file would allow killing of arbitrary privileged processes. varRun
# however MUST be writable by the daemon user.
#
# None of the paths segments may be symlinks, why it has to be /private/var
# instead of /var etc.
paths:
# socketVMNet requires Lima >= 0.12 .
# socketVMNet has precedence over vdeVMNet.
  socketVMNet: "/opt/socket_vmnet/bin/socket_vmnet"
# vdeSwitch and vdeVMNet are DEPRECATED.
  vdeSwitch: /opt/vde/bin/vde_switch
  vdeVMNet: /opt/vde/bin/vde_vmnet
  varRun: /private/var/run/lima
  sudoers: /private/etc/sudoers.d/lima

group: everyone

networks:
  user-v2:
    mode: user-v2
    gateway: <IP_ADDRESS>
    netmask: <IP_ADDRESS>
    # user-v2 network is experimental network mode which supports all functionalities of default usernet network and also allows vm -> vm communication.
    # Doesn't support configuration of custom gateway; hardcoded to <IP_ADDRESS>/24
  shared:
    mode: shared
    gateway: <IP_ADDRESS>
    dhcpEnd: <IP_ADDRESS>
    netmask: <IP_ADDRESS>
  bridged:
    mode: bridged
    interface: en0
    # bridged mode doesn't have a gateway; dhcp is managed by outside network
  host:
    mode: host
    gateway: <IP_ADDRESS>
    dhcpEnd: <IP_ADDRESS>
    netmask: <IP_ADDRESS>

A workaround is to remove the VDE lines from networks.yaml

@jandubois Could you take a look? 🙏
The validation code is hard to understand
https://github.com/lima-vm/lima/blob/103a9e40defa68b7eed252d4cad15f2ce8bff909/pkg/networks/validate.go#L18-L55

The problem is not the validation code, but the YAML parsing: https://github.com/lima-vm/lima/blob/103a9e40defa68b7eed252d4cad15f2ce8bff909/pkg/networks/config.go#L135
Since we pass the yaml.Strict() option, the Paths field will not be unmarshalled, so SocketVMNet, VarRun, and Sudoers will all be empty strings.
And filepath.Dir("") will return . (called by findBaseDirectory), explaining the error message.
I'm somewhat surprised that UnmarshalWithOptions does not return an error in that case, and does fill in the Group and Networks fields.
Anyways, 2 possible fixes are:

remove the yaml.Strict() option
keep the unused fields in the Paths struct

Given that the yaml.Strict() option doesn't seem to result in a proper error, I would tend towards the first option. If we can fix the error reporting, then I would favour the second one.


Code about simulating events on FlyingChairs2

Nice work!
Can you release the codes about simulating events on FlyingChairs2 or release the event datasets for easy use.

In order to avoid unnecessary misunderstanding of rights, we do not plan to provide simulated event data. If anyone needs the code of event simulation, please leave an email and I will send it to you via email.

Hi, my email is<EMAIL_ADDRESS>

Hi, my email is<EMAIL_ADDRESS>
I have already sent it. If there are no further questions, I will close this issue in a week.


fix: missing \ in dockerfile 8.3

In commit #715 a change was made that removed the \ at the end of the command 'echo "extension=swoole.so" > /etc/php/8.3/cli/conf.d/20-swoole.ini'. It was added back because it was breaking the image build!

Oh my bad! Miss typing thanks for fixing it.

Hey @driesvints can you please create a patch release for this PR? Thanks!
Closes #719 and #720


Wiki for our Horizontal Prototype (static, and link to high fidelity prototype)

I'll be doing that in XCode storyboards, trying to display pretty much all frontend features from https://github.com/mobileappdevhm19/apps4students-team-3/wiki/Design-Phase

first draft looks feature-complete, will put screencaps on the wiki for comparison

Screenshots are in the Wiki! so this issue is completed.