Nothing Works

The link to a running demo under "Try it out
Test angular2-grid in your browser. (https://tonicdev.com/npm/angular2-grid)" and clicking "run" just produces error "Cannot find module 'angular2/package.json'".
Pulling down the code, and running all the MPM commands, then trying to run it with http-server either in the root directory, or the demo directory fails - lot's of 404 errors.
Your sample code as written doesn't work, or at best has large omissions...

"import { NgGrid, NgGridItem } from 'angular2-grid';"  this will not work  - you need to specify the path to the angular2-grid ts file  ( for example if I were to copy the nggrid.ts file into the same directory as the hosting page's ts file I would have something like this "import {NgGrid, NgGridItem} from './nggrid';"  or if I try to import from where the "npm install angular2-grid" put's it code it would be something like "import {NgGrid, NgGridItem} from '../../../../node_modules/angular2-grid/src/nggrid';"

Finally your sample block of using the grid dosn't work at all:

{{box.title}}

{{box.text}}

Firstly the syntax of the *ngFor is outdated - it should be : *ngFor="let country of _countriesDS"
Then it uses an undefined array of undefined objects.  Evidently an array of <unknown Objects> in the undeclared array variable: `var boxes: = [new (),...]
`
I see some potentially good stuff in here, but noting is usable with the information provided.

OK, so I have no idea where the tonicdev link comes from. I think that's an npm thing? My demos are at https://bmorton.co.uk/angular/ and https://bmorton.co.uk/angular_test/, both of which work.
The root directory package is designed to be run from within another web server. I currently have it running behind apache. The built files end up in the dist/ folder and you can then serve them up from a http server of your choice.
The demo folder should be self-contained, however, as of the angular 2 rc there are quite a few fixes required (see #75). You'll need to make sure that the angular 2 version is correct. That should then be able to run behind http-server, though I don't personally use that as I have an apache server running.
The sample code, is just that - samples. It's not designed as a fully running/working component.

import { NgGrid, NgGridItem } from 'angular2-grid'; will work (unless something has drastically changed in TypeScript's module resolution). If you have TypeScript configured correctly, it will work (see this demo file)

Yes the sample block in the readme is out of date. I haven't updated it since the change. Thanks for pointing it out.

The sample block assumes that you have an array called boxes in your component. Though, as there is no actual component created in the sample you're gonna have to create that yourself.

The readme is intended as a "quick start", assuming that you have a component that already exists and that you are trying to add to it. Hence not being a 100% complete example of how to use the grid. For a full example of how to use it, see https://github.com/BTMorton/angular2-grid/blob/master/src/app.ts and the accompanying files.
For what it's worth, the readme could probably be improved. If you're willing to give that a go, I'll gladly accept a pull request.

I appreciate the reply Ben,
        I’ll have to see what direction my boss wants to go in. We are currently developing an Angular2 application which has page displaying a list of Counties obtains from a table in our M$ Sql Server database.  Our Angular2 application is built on the M$ MVC.Net framework.

We can display the data find in a simple HTML table from an array of data called “” like this:

 0)">

        DisplayName

        CountryId_Guid

        Abbreviation

        {{countryDS.DisplayName}}

        {{countryDS.CountryId}}

        {{countryDS.Abbreviation}}

But cannot get anything to display trying your grid like this:

{{country.DisplayName}}

{{country.CountryId_Guid}}

{{country.Abbreviation}}

</div

With my component file like this ( I copied the nggrid.ts file from  '../../../../node_modules/angular2-grid/src/nggrid' from the same file the component was defined in  from where the npm install put it --- I also tried “import {NgGrid, NgGridItem} from 'angular2-grid';” and “import {NgGrid, NgGridItem} from '../../../../node_modules/angular2-grid/src/nggrid':
…
import {NgGrid, NgGridItem} from './nggrid';
…
@Component({
selector: "angularUiGrid",

templateUrl: "app/components/sandbox/angular2_grid.html",

providers: [CountryService, NotificationService],

directives: [NgGrid, NgGridItem]

})
…
export class Angular2_grid implements OnInit {
private _countriesDS: Array<Country>; // for data obtained from DataService?

constructor(public countryService: CountryService         //We inject the CountryService in the constructor...

    , public notificationService: NotificationService)    //We inject the NotificationService in the constructor...

{

    this.getCountriesDS();

}

…
I don’t have a config file nor am I sure WHERE to set up the configuration, but figured it was just styling and SOMETHING should get displayed without it, but I get nothing ( and the array length is 247 ).
If you can see the issue and feel inclined to reply with suggestions I’d be happy to try again.
From: Ben Morton<EMAIL_ADDRESS>Sent: Wednesday, May 4, 2016 14:00
To: BTMorton/angular2-grid<EMAIL_ADDRESS>Cc: fubar<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [BTMorton/angular2-grid] Nothing Works (#76)
OK, so I have no idea where the tonicdev link comes from. I think that's an npm thing? My demos are at https://bmorton.co.uk/angular/ and https://bmorton.co.uk/angular_test/, both of which work.
The root directory package is designed to be run from within another web server. I currently have it running behind apache. The built files end up in the dist/ folder and you can then serve them up from a http server of your choice.
The demo folder should be self-contained, however, as of the angular 2 rc there are quite a few fixes required (see #75 https://github.com/BTMorton/angular2-grid/issues/75 ). You'll need to make sure that the angular 2 version is correct. That should then be able to run behind http-server, though I don't personally use that as I have an apache server running.
The sample code, is just that - samples. It's not designed as a fully running/working component.

 import { NgGrid, NgGridItem } from 'angular2-grid'; will work (unless something has drastically changed in TypeScript's module resolution). If you have TypeScript configured correctly, it will work (see this demo file <https://github.com/BTMorton/angular2-grid/blob/master/demo/app/app.component.ts> )

 Yes the sample block in the readme is out of date. I haven't updated it since the change. Thanks for pointing it out.

 The sample block assumes that you have an array called boxes in your component. Though, as there is no actual component created in the sample you're gonna have to create that yourself.

The readme is intended as a "quick start", assuming that you have a component that already exists and that you are trying to add to it. Hence not being a 100% complete example of how to use the grid. For a full example of how to use it, see https://github.com/BTMorton/angular2-grid/blob/master/src/app.ts and the accompanying files.
For what it's worth, the readme could probably be improved. If you're willing to give that a go, I'll gladly accept a pull request.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub https://github.com/BTMorton/angular2-grid/issues/76#issuecomment-216967440 https://github.com/notifications/beacon/ALLv9Sx3TvUetOFjpX-_pallGQVVRZIpks5p-OzLgaJpZM4IXbIk.gif

OK, first question: are you looking for a dynamic data table? If so, I don't think is the plugin for you.
If not, the first thing I see is that you have:

You don't have box defined in this config. I suggest you either add a new function which returns the item config (e.g. [NgGridItem]="getConfig(country.Id)") or you add a property to each country with the config in ([(NgGridItem)]="country.config"). Note: no two-way binding on the function version.
The config file I was referring to is your tsconfig.json. In there you should have a line moduleResolution: 'node', or similar. That should then take care of the include issues. Assuming of course that a) your typescript compilation root directory is the same as your node_modules directory location and b) your node_modules folder is accessible on your public root (or you have a System.js redirect in place for it). If that's all correct then I have no idea. I'd have to see a working example, I'm afraid.


processSignalingMessage executed if stream is local.

Possible fix for issue #616

Also refer to https://github.com/lynckia/licode/issues/616#issuecomment-278402223 for an explanation of the fix.

Thanks for the PR. However, I have concerns about this solution. Here my explanation. If you agree with the workaround I propose, could you implement it?

@aalonsog have you read my comment about this PR? Any news about it?

I updated this pull request according to my discussion with @aalonsog.
Anyway I guess we should continue our discussion about failed streams.
Please, let me know.

I'm going to close this PR since I created it from my master branch and it causes confusion every time I update my master branch.
I'll resend a new PR from a dedicated clean branch.
Sorry about this.

Ok thanks!

@aalonsog PR already sent!
Please have a look at it and send me some feedback.


Adding support for eth chain

Eth can't be run within the current setup because there is an inherent assumption of a chain being cosmos-based. To overcome this, I introduce a concept of a virtual chain with subtypes. It might not be the perfect abstraction but that's how we've been thinking about this at Polymer. Let me know what you think of that approach.
Here is the config file I used for polymer version:
chains:
  - name: ethereum
    type: virtual
    subtype: eth
    numValidators: 1
    prysmCtl:
      image: ghcr.io/polymerdao/prysm-prysmctl:v4.0.3-light-client-1
    geth:
      image: ethereum/client-go:v1.10.26
    beaconChain:
      image: ghcr.io/polymerdao/prysm-beacon-chain:v4.0.3-light-client-1
      args:
        - --datadir=/ethereum/consensus/beacon
        - --min-sync-peers=0
        - --no-discovery
        - --interop-genesis-state=/ethereum/consensus/genesis.ssz
        - --interop-eth1data-votes
        - --bootstrap-node=
        - --chain-config-file=/etc/config/config.yml
        - --chain-id=32382
        - --execution-endpoint=http://localhost:8551
        - --accept-terms-of-use
        - --rpc-host=<IP_ADDRESS>
        - --grpc-gateway-host=<IP_ADDRESS>
        - --suggested-fee-recipient=0x123463a4B065722E99115D6c222f267d9cABb524
        - --jwt-secret=/etc/secrets/jwt.hex
        - --enable-polymer-devnet-mode
    validator:
      image: ghcr.io/polymerdao/prysm-validator:v4.0.3-light-client-1
      args:
        - --beacon-rpc-provider=localhost:4000
        - --datadir=/ethereum/consensus/validator
        - --accept-terms-of-use
        - --interop-num-validators=36
        - --interop-start-index=0
        - --force-clear-db
        - --chain-config-file=/etc/config/config.yml
        - --config-file=/etc/config/config.yml
        - --suggested-fee-recipient=0x0C46c2cAFE097b4f7e1BB868B89e5697eE65f934
        - --enable-polymer-devnet-mode

What is a virtual chain? and what other kinds of chains can be part of virtual chain?
Can we just not call it eth directly?

What is a virtual chain? and what other kinds of chains can be part of virtual chain? Can we just not call it eth directly?

In polymer's terminology, a virtual chain is any non-ibc chain that can be represented as an ibc chain.
The reason I don't want to call it eth directly is because as more non-ibc chains added, the bigger if statements will become in the templates. Additionally, I was thinking of adding a cosmos type to  what already exists but it looks like a big structural changes and I wasn't sure if you want to go that way.

I think we can do some refactor for this, but this is a good start for getting eth working in Starship. Lets just merge this first and get you unblocked. I can do the refactor as part of backlog.
Only thing needed for this to work is to have a test case for this, so that we are atleast spining this up in as part of the tests. We would have to ignore the actual test cases to run against this. We might also have to change our testing paradigm for this to work, but we can actually make it part of the refactor itself.

How can I make a test case for this? When I look at the test cases, they are all tailored for a cosmos chain. So I can't really use the existing machinery and probably need to set up a new one even if it's just a stub.

Ok, lets just go ahead with this without tests, but I would need to understand your requirements to be able to refactor this properly. I think we need a way to be able to write tests for new types of chains as well. Redesign to be able to have different kind of chains (cosmos, eth, polkadot etc), would have a set of tests that are specific to them.
Usally we can just use something like helm tests to be able to just spin up the infra without errors. We might just have to comeup with a new testing system ourselves


"Video could not be played" all resolutions

Hey,
I can play videos in the official floatplane app without issues, but lately I get "Video could not be played" for every video and resolution on the LTT supporter plus channel using hydration 1.4.5.
Edit: signing out and in again doesn't help either

They've changed up some API responses, it seems. Working on a fix for both AndroidTV and Roku.

Problem is back here.


Experimental API #2 - tiny-atom/quarks

Ok, so this one's Alan's idea!
This might be a simpler and a more powerful abstraction compared to #32.
Check this yo:
const createAtom = require('tiny-atom/quarks')
const merge = require('tiny-atom/deep-merge')
const debug = require('tiny-atom/log')

// signature is now (options, initialState = {})
const atom = = createAtom({ merge, debug })

First of all, you can add top level state and actions:
atom({ a: 1 }, {
  plus: (get, split) => split({ a: get().a + 1 })
})

And extend the top level on demand:
atom({ b: 1 }, {
  minus: (get, split) => { split({ b: get().b - 1 })
})

But you can also create scoped slices of atom - quarks
const todo = atom('todo', { list: [], val: '' }, {
  add: (get, split, item, root) => {
    // gets are scoped
    const curr = get().list
    const next = curr.concat([item])

    // splits are scoped
    split({ list: next })

    // can call local actions
    split('reset')

    // can call global actions
    root.split('plus')
    root.split('minus')
    root.split('analytics.track')

    // and access global state
    root.get()
    root('analytics').get()
  },

  reset: (get, split, item, root) => {
    split({ val: '' })
  }
})

And another one
const analytics = atom('analytics', { events: [] }, {
  track: (get, split, event) {
    split({ events: get().events.concat([event] )})
  }
})

It's possible to nest these infinitely - might be good or bad, but doesn't matter, it seems fairly elegant in general:
const deep = analytics('deep', {})
deep.split({ value: 123 })

Observation only possible at the top, to emphasize this is still one tree and all:
atom.observe(render)

All actions go through the global evolver. And it's actually possible to override the global evolver at runtime! This means, however, that you have to fully manually manage what happens with all the actions and how you namespace your state, etc.
atom({}, customEvolve)
function customEvolve (get, split, action) {
  console.log('All actions are now intercepted', action)
}

Instead, it's also possible to only override evolve of the subatoms:
atom('analytics')('deep', {}, evolve)

And once you've constructed your atoms, you can pass them around or provide/connect:
e.g. in the root
<Provide atom={atom} />

e.g. in some subapp
<Provide atom={atom('feedback')} />
<Connect>
  {(state, split) => <div>{state.text}</div>}
</Connect>

Seems pretty elegant overall! Lots of little hidden features, but the core usage is quite pragmatic and straightforward and it's the same atom we know (and love, right, right?)! Typical usage would probably be:
const atom = createAtom({ merge, debug })

atom('onboarding', ...require('./actions/onboarding'))
atom('widget', ...require('./actions/widget'))
atom('feedback', ...require('./actions/feedback'))
atom('feeds', ...require('./actions/feeds'))
atom('entities', ...require('./actions/entities'))
atom('api', ...require('./actions/api'))

// and then pass this stuff around
atom('feedback').split('submit')
atom('feedback').get()

// or also
atom.split('feedback.submit')
atom.get().feedback

And a really cool thing is that these atoms and subatoms (mostly) conform to the atom interface. They preserve all of the original features and goals of the tiny-atom:

it's open and flexible
it's pragmatic
it's tiny (?)
works with all of the batteries - logger/devtools/merge/react/preact
it doesn't assume anything about your state - you can use Immutable.js for example
because it supports the top level stuff too it means you can drop it into any existing app

Tangent
The only thing that's removed here compared to #32 is splits as functions. String actions just keep proving to be simpler to implement and more powerful. It's still possible to bind all these splits as functions as demonstrated in #32, but you'd need to customize your evolver somehow.
Not sure how to elegantly incorporate it into the API and whether to do that at all. Feels like it would only make sense to use function actions if you completely remove splits as strings (?) to remove the duality. It's not cool that you can do both atom.split.onboarding.start() and atom.split('onboarding.start').

The full implementation is here examples/nested-example/atom-simpler.js.

FYI, the state looks like this, each quark is in the top level with the namespace as the key:

This keeps the implementation really simple, avoids naming collisions (quark shadowing parent's state key) and makes things a ~bit simpler when using alternative state data structure (e.g. immutable).

Love it!
Except one thing: Why restrict observe to the top level? Isn't it more elegant if the whole tree was a fractal, i.e., self-similar on every level? You can never predict what ingenious use case someone might come up with, and placing artificial limits like this (like axios did) is almost never a good idea.
Or am I missing something?

I like it. Nice that you no longer need to write an evolve function and you can attach actions directly.
The one thing that bothers me is this function signature
add: (get, split, item, root) =>

it's getting pretty long now, and it's strange that root comes after the payload. Could you return an actions creator function instead of an object?
Global
atom({ b: 1 }, (atom) => {
  const { get, split } = atom
  return { minus: (payload) => { split({ b: get().b - 1 }) }
})

Scoped
atom('todo', { b: 1 }, (atom) => {
  const { get, split, root } = atom
  return { minus: (payload) => { split({ b: get().b - 1 }) }
})

I also find the combination of global + scoped actions a little confusing... both seem useful separately but if a project ever used both together I think it would be easy to get lost.
Also, would like to see more info on merge and debug.

I've spent a lot of time thinking about this. I've also tried incorporating this into some of our existing projects. My conclusion thus far is that both this #33 and  #32 are unnecessarily complicated. While scoped and fractal atoms seam neat, for a typical app it's awkward to work with local/global state by having to use get/get.root or get/root.get or anything like that. You take something simple, which is - here's your overall app state grouped by some concern, and here are some functions that operate over that and transition that state from one phase to the next. And you add more layers - oh, this one's locally scoped, so I need to call this like that, and to get global state I need to do that, but if I'm in a component, I need to do something different, because of how it's connected.
All in all, I'm inclined to close both of these PRs without merging them into the core src of the library. I'll probably keep them as examples in the examples dir for future reference. The original tiny-atom api with it's flexible evolve remains unbeaten in my book, because it allows you to adapt tiny-atom to exactly what your application needs without making many assumptions or introducing many global library concepts.
In fact, if we add "2 cores" or add "actions as strings and functions" or "gets/splits can be local and global" or "atoms can be global or scoped" to the core library, maybe that defeats a lot of the simplicity of tiny-atom. It'd be fine if those were truly "the best way" to do things, or had a lot of conceptual synergy and elegance. But from just trying to use them in a larger app, I immediately got put off. Maybe we'll come up with some better strategy in the future, but for now I think the experimentation should remain in each of the applications and good patterns can only emerge from writing a real app, not from messing with small examples.
So yeah - this point still stands: https://qubitproducts.github.io/tiny-atom/best-practises#extend.
Finally, some closing thoughts. If your app say really wants to separate concerns strongly between 2 subapps - say we have some global actions, experiences substate and events substate. You can do something as simple as this:
module.exports = function createEvolve (actions) {
  const registry = { '': Object.assign({}, actions) }

  return { evolve, slice }

  function slice (namespace, actions) {
    registry[namespace] = actions
  }

  function evolve (get, split, action) {
    const namespace = action.type.split('.').slice(0, -1).join('.')
    if (namespace && registry[namespace]) {
      get = Object.assign(namespacedGet(get, namespace), { root: get })
      split = Object.assign(namespacedSplit(split, namespace), { root: split })
    }
    registry.actions[action.type](get, split, action.payload)
  }

  function namespacedGet (get, namespace) {
    return () => namespace.split('.').reduce((ref, segment) => ref[segment], get())
  }

  function namespacedSplit (split, namespace) {
    return (...args) => typeof args[0] === 'string'
      ? split(`${namespace}.${args[0]}`, ...args.slice(1))
      : split({ [namespace]: args[0] })
  }
}

The usage would be:
const { evolve, slice } = createEvolve(actions.global)
slice('experiences', actions.experiences)
slice('events', actions.events)

const atom = createAtom(initialState, evolve)

Now, your experiences actions, for example, are scoped by default and state lives within a namespaced slice. And those actions can still access the global state and global actions via get.root() and split.root(...). This might make sense for your app, but it won't make sense for all apps, so therefore is not enforced by the lib, but rather is an easy to write extension (that I'll write up  as a guide in docs).

Personally, I never cared about scoping at all. I'm just interested in decoupling. I want to do this:
// ----- feature1.js -----
import myAtom from 'myAtom'
myAtom(feature1State, feature1Actions)
// ----- feature2.js -----
import myAtom from 'myAtom'
myAtom(feature2State, feature2Actions)

instead of this:
// myAtom.js
import {feature1State, feature1Actions} from 'feature1'
import {feature2State, feature2Actions} from 'feature2'
myAtom({
  ...feature1State,
  ...feature2State
}, {
  ...feature1Actions,
  ...feature2Actions
})

Discussed in person with @mhelvens. I'm generally open to adding the "extend" state+actions feature. But the problem is that tiny-atom currently has no concept of actions. It's easy to write a wrapper that provides this functionality, but I'll keep this idea in mind for the future.
module.exports = function createAtom (options) {
  const registry = {}
  const atom = createAtom({}, evolve, options)
  return Object.assign(atom, { fuse })

  function fuse ({ state, actions }) {
    Object.assign(registry, actions)
    atom.split(state)
  }

  function evolve (get, split, action) {
    registry.actions[action.type](get, split, action.payload)
  }
}

Will merge via a new branch for history. Closing this one.


AbortionError(code=StatusCode.NOT_FOUND, details="FeedInputs: unable to find feed output ToFloat:0")

Hello
I am trying to host the ssd_mobilenet_v1_coco model from the Tensorflow Object Detection API model zoo with Tensorflow Serving.
I was able to successfully export the model with the exporter script in models/object_detection/exporter.py as a SavedModel. The issue arises when I tried to run the modified client
from __future__ import print_function
from grpc.beta import implementations
import tensorflow as tf
from tensorflow_serving.apis import predict_pb2
from tensorflow_serving.apis import prediction_service_pb2

tf.app.flags.DEFINE_string('server', 'localhost:9000',
                           'PredictionService host:port')
tf.app.flags.DEFINE_string('image', '', 'path to image in JPEG format')
FLAGS = tf.app.flags.FLAGS

def main(_):
  host, port = FLAGS.server.split(':')
  channel = implementations.insecure_channel(host, int(port))
  stub = prediction_service_pb2.beta_create_PredictionService_stub(channel)
  with open(FLAGS.image, 'rb') as f:
    data = f.read()
    request = predict_pb2.PredictRequest()
    request.model_spec.name = 'mobilenet_v1'
    request.model_spec.signature_name = 'serving_default'
    request.inputs['inputs'].CopyFrom(
        tf.contrib.util.make_tensor_proto(data, shape=[1]))
    result = stub.Predict(request, 10.0)  # 10 secs timeout
    print(result)

if __name__ == '__main__':
  tf.app.run()

Here's the traceback for the gRPC request
Traceback (most recent call last):
  File "/home/mcw-nn/Documents/SSP/serving/bazel-bin/tensorflow_serving/example/inception_client.runfiles/tf_serving/tensorflow_serving/example/inception_client.py", line 56, in <module>
    tf.app.run()
  File "/home/mcw-nn/Documents/SSP/serving/bazel-bin/tensorflow_serving/example/inception_client.runfiles/org_tensorflow/tensorflow/python/platform/app.py", line 48, in run
    _sys.exit(main(_sys.argv[:1] + flags_passthrough))
  File "/home/mcw-nn/Documents/SSP/serving/bazel-bin/tensorflow_serving/example/inception_client.runfiles/tf_serving/tensorflow_serving/example/inception_client.py", line 51, in main
    result = stub.Predict(request, 10.0)  # 10 secs timeout
  File "/usr/local/lib/python2.7/dist-packages/grpc/beta/_client_adaptations.py", line 324, in __call__
    self._request_serializer, self._response_deserializer)
  File "/usr/local/lib/python2.7/dist-packages/grpc/beta/_client_adaptations.py", line 210, in _blocking_unary_unary
    raise _abortion_error(rpc_error_call)
grpc.framework.interfaces.face.face.AbortionError: AbortionError(code=StatusCode.NOT_FOUND, details="FeedInputs: unable to find feed output ToFloat:0")

I am using this commit of tensorflow to build the model server which is the default submodule of tensorflow serving

I fixed it. I'm leaving the solution here if anyone faces this issue in the future.
Turns out I was trying to host with an older commit of tensorflow/models (which also happens to be the default submodule of tensorflow serving).
When I switched to the master branch of tensorflow/models, the error went away!


How to make a foreign key of a table be the primary key of another table that has a composite primary key ?

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the SQLModel documentation, with the integrated search.
[X] I already searched in Google "How to X in SQLModel" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to SQLModel but to Pydantic.
[X] I already checked if it is not related to SQLModel but to SQLAlchemy.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
from typing import Optional, List

from sqlmodel import SQLModel, Field, Relationship, create_engine, Session

engine = create_engine("postgresql://username:password@localhost:5432/db_dynamic")

class Model1(SQLModel, table=True):
    id: Optional[int] = Field(default=None, primary_key=True)

    model2_id: Optional[int] = Field(default=None, foreign_key="model2.id")
    model2: "Model2" = Relationship(back_populates="model1")

class Model2(SQLModel, table=True):
    id: Optional[int] = Field(default=None, primary_key=True)

    model3_id: Optional[int] = Field(default=None, foreign_key="model3.id", primary_key=True)
    model4_id: Optional[int] = Field(default=None, foreign_key="model4.id", primary_key=True)

    model1: List[Model1] = Relationship(back_populates="model2")

class Model3(SQLModel, table=True):
    id: Optional[int] = Field(default=None, primary_key=True)

    model4: List["Model4"] = Relationship(back_populates="model3", link_model=Model2)

class Model4(SQLModel, table=True):
    id: Optional[int] = Field(default=None, primary_key=True)

    model3: List[Model3] = Relationship(back_populates="model4", link_model=Model2)

if __name__ == "__main__":
    SQLModel.metadata.create_all(engine)

    with Session(engine) as session:
        model3 = Model3()
        model4 = Model4()

        session.add(model3)
        session.add(model4)
        session.commit()

        model2 = Model2(**{"model3_id": 1, "model4_id": 1})

        session.add(model2)
        session.commit()

        model1 = Model1(**{"model2_id": 1})
        session.add(model1)
        session.commit()

Description
I have link table represented by Model2, which establishes a many to many relation between Model3 and Model4.
Now I want to establish a many to one relation between Model1 and the link table. To do that, I created a third primary key id in Model2 and a foreign key in Model1 which points to the id of Model2.
I'm using a PostgreSQL database created using a docker-compose.yml as follows:
version: '3.7'

services:
  db:
    image: postgres:13.0-alpine
    volumes:
      - postgres_data:/var/lib/postgresql/data/
    environment:
      - POSTGRES_USER=username
      - POSTGRES_PASSWORD=password
      - POSTGRES_DB=db_dynamic
    ports:
      - "5432:5432"

volumes:
  postgres_data:

However, I'm getting the following error:
sqlalchemy.exc.ProgrammingError: (psycopg2.errors.InvalidForeignKey) there is no unique constraint matching given keys for referenced table "model2"

[SQL:
CREATE TABLE model1 (
	id SERIAL,
	model2_id INTEGER,
	PRIMARY KEY (id),
	FOREIGN KEY(model2_id) REFERENCES model2 (id)
)

However, it works well with a SQLite database. The problem seems to be in how SQLModel (or SQLAlchemy) generates the schema for the PostgreSQL database.
Someone knows how to solve it ?
Operating System
Linux
Operating System Details
No response
SQLModel Version
0.0.6
Python Version
3.9.5
Additional Context
No response

Solved by letting just the pair of ids being the primary key and the id being an unique index


LEGO® MARVEL Super Heroes: Universe in Peril [PCSE00250]

App summary

App name: LEGO® MARVEL Super Heroes: Universe in Peril
App serial: PCSE00250
App version: US

Vita3K summary

Version: v1.6.0-3
Build number: ?
Commit hash: https://github.com/vita3k/vita3k/commit/[commit]
CPU backend:
GPU backend:

Test environment summary

Tested by: ForgetSora
OS: Android
CPU: Qualcom Snapdragon 680
RAM: 6GB

Issues
The game runs well at start, but textures, voices and sounds are very bug. When the gameplay starts the game crash
Screenshots
![]

Log
Recommended labels

Grafics Bug
Audio Bug
Crash

Need test pc version and no Android, no using Android version for issue curently

Oh... Sorry, i didn't know that
El mié, 22 feb 2023 1:07, Zangetsu @.***> escribió:

Need test pc version and no Android, no using Android version for issue
curently
—
Reply to this email directly, view it on GitHub
https://github.com/Vita3K/compatibility/issues/1252#issuecomment-1439251283,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5776AH25HQZOOFQQOROQLLWYVKDZANCNFSM6AAAAAAVBEP4RY
.
You are receiving this because you authored the thread.Message ID:
@.***>

no worry, is just android version have LOT diference commpare pc lol

https://drive.google.com/file/d/1CltL6W7h-NM0eW-9PCiOSuAyo9fp3LHZ/view?usp=drivesdk
Use This Driver, After History Screen The Game Will Run.
I Used The POCO X3 PRO.

New Link Corrected, Please Check It To See If There Are Any Problems!

New Driver :
https://mega.nz/file/b6wiwIgB#7fhCsevvlqpLzd7hJqCKMHBZQKqoba590w-vMIIrv_8

should close this issue.

not accurate reporting

Где скачать
?


pull request for e2e test at 11/15/2016 00:45:11

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

Details

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


docs: tell how to deploy demo app in Hubble CLI guide

When following the guide on Inspecting the Network flow
with Hubble's CLI the Demo App is required to be deployed
but there is no reference to that anywhere following the
sequence of steps from the Getting Started guide.
Fixes: #15972
Signed-off-by: Lorenzo Fundaró<EMAIL_ADDRESS>

But I don't think that including the whole file for the instructions related to the SW demo is the best option

Yeah, I also feel like it will diverge people's attention out of this guide.
I'm proposing adding a note, before the guide start, check it out and let me know what you think. Cheers !

Looks good to me, thanks a lot! Just one more thing from me, could you please squash your commits into a single one?

sure, done 👍 . I also updated the commit message to better match the intent of this PR.
Cheers !


Added dice_coef

For segmantation dice_coef function so important.
If we want to segmantate image to classes, other functions will look all image area(with background so other functions look remaining background will behaving like higher accuracy.) but dice_coef look only segmantatin area matching, not background.
In this way, we can estimate the correct error rate.
dice_coef = (2*intersection) / ((first area) + (second area))
Example:
(White: Segamantation, Black: Background)
Label:

Predict:

Computer's say true segments area is big (Only little area(according to white areas) is wrong but not):

True segments areas must be like(So prediction is wrong):

This will be an estimation of the dice coefficient, since the true dice coefficient will require to round the input to 0 or 1. (But then it will not be continuous anymore.)
It is also a global dice value over the batch. More info here:
https://www.kaggle.com/c/ultrasound-nerve-segmentation/discussion/21358#131741

@Dref360 I use dice_coef first in my Cat-Segmentation repository.
Example:(This example include very base model because it is for testing)

Input
Output

Thanks for the PR. As a few general style notes:

Don't use the "loss" suffix in a loss
Use full words in function names, e.g "coefficient". If it needs to be shortened, you can create an alias
The metric name and loss name should be the same
A Keras loss is meant to return one scalar per sample (they're samplewise functions), not one scalar in total

In particular because of the last point, we won't merge this PR in core Keras.

@fchollet If you reopen this Pull Request, We will fix the problems you mentioned.

@ardamavi beyond the style/technical issues, which can be fixed, I don't think there is a strong case to include it. If people keep requesting it, we will revisit this decision in the future. That's how decisions are made: if enough people seem to need a feature, we add that feature, otherwise we don't. Including new features has a significant cost.

@fchollet For segmantation Dice Coefficient function so important. In the field of deep learning, Dice Coefficient has a very important place and is popular. Especially in medical community.
I think this append is very important for Keras. Like @Dref360  said. Am I wrong?


fix: sort failures appropriately

This makes sure failed items are not shown as fastest.

That's a good remark, thanks for noticing.
I think we can simplify this just by setting the mean of a failed items to 0 in https://github.com/ShogunPanda/cronometro/blob/master/src/index.ts#L23.
This way b.mean - a.mean holds a negative number for failed items.
What do you think?

Sure. Feel free to close this is you want to go the other route.


Make the Super Key of keyboard to open the menu in GameMode

Describe the bug
Without a controller, in game mode it's not possible to open the menu after opening a game or an app.
Is it possible to make the super key/ windows key as an steam input to open the menu, to go back to home or to juste shutdown ?
What did you expect to happen?
I expected the super key to open the menu like an steam input.
Output of rpm-ostree status
No response
Hardware
I'm using a home theater pc
Extra information or context
No response

This is not something we control, this is a steam issue


Implement property based tests

Description
Implement property based tests and shrinking to test shenlong generated llmv files.
Acceptance criteria

Property based testing is implemented
Shrinking is implemented

Fixed by #54


Arrays in C++

Description
An array is a collection of elements of the same type placed in contiguous memory locations that can be individually referenced by using an index to a unique identifier.
I would like to contribute to this project.
Hello @Lakhankumawat, I am a GSSOC'22 contributor. I am eager to contribute, please consider the issue and assign me to this project.
Thank you.
Domain
CPP
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of LearnCPP.

@anveshajain19 please be more specific, what would you like to contribute?
What are you planning to implement in array?
Or you want to write readme for it?

@n4i9kita I am planning for some topics which you can put in the read me section like:
Standard Problems
Factorial of large numbers
Subarrays
Median Sorted Arrays
Circular array problems and much more.

@anveshajain19 I think a better way to go about this would be to pick one topic (let's say Factorial of large numbers), add it's code with proper comment; and create an issue for each one separately. This way, others can also work on some other problem on arrays as well.
@everyone : Mention the problem(or whatever you wish to implement) in the issue properly, and not just the data structure's name that you'd be working with. Better be, mention why the algorithm that you are implementing is better than the other ones out there to solve the problem.

okay
I will make standard problems on Arrays so that others can also contribute in it.

Let's join the meet. I think @Lakhankumawat  will clear it properly.

okay

Either we can close this issue, and you can create a new issue with the particular problem/ algorithm you are going to work on. Mention the time complexity of your approach, and maybe compare it to other approaches that are already there to solve it. Like, you can create an issue on "Median on sorted array". Do go through the contributing guidelines as well.

Okay,  I will create a new issue with the particular problem/ algorithm I am going to work on.


CodeMirror: Incremental parse bug

Delete/replace the colon after the last if True in the while. Notice the structural highlighting is broken for the last if/else.
Edit the last if/else and it fixes itself.
Needs minimising and a CodeMirror bug reporting (once we've checked we're on the latest).
# Add your Python code here. E.g.
from microbit import *

while True:
    display.scroll('Hello, World!')
    display.show(Image.HEART)
    sleep(2000)
    if True:
        pass
    if True:
        pass

if True:
    pass

if True:
    pass
else:
    pass

CM bug is fixed (unclear by what change, but doesn't reproduce with the latest deps)..
Next step is to upgrade CM and test.

Closed by #232.

Reopening as we've had to revert the CodeMirror upgrade due to #243.

We re-upgraded. Closing.


WIP: feature: OpenAPI Specification

Initial PR for OpenAPI Spec. Can be opened in something like https://stoplight.io, we can also export it as a .json file and it would then be importable to something like Postman.
I've put in some APIs that would likely suit the mobile app, but can tweak them as needed of course.

Closing due to inactivity, since this PR was introduced newer solutions in the community exist for creating OpenAPI specs from your Vapor routes (and automatically generating a Stoplight documentation page). If desire still remains, I'd recommend looking in that direction to minimise the amount of code we need to maintain.


Fix packager issue that happens as a result of incorrect node-haste callsites.

Resolves issue @mkonicek mentioned here: https://github.com/facebook/react-native/issues/6036#issuecomment-186388671

cc @cpojer

We should also lock node-haste to revisions as there are breaking changes between minor versions at the moment (react-natives specifies v2.0.0 but there are breaking changes in 2.2.0) or respect semver in node-haste. This will be problematic especially when there will be release versions that use node-haste from npm.

@janicduplessis haste is fixed to ~2.0.0 in 89ea98554046356f934d7607270a22d0924b49c9.

I guess this is covered in b41ad9f5dc556f119f5512ec2e0b048aa16db642


Include styled-jsx in the learn-starter

Without this the project doesn't start

This could be related to your package manager. If styled-jsx is installed under node_modules/next/node_modules instead of installed at the top-level node_modules directory in your folder, there might be an issue.
https://github.com/vercel/styled-jsx/pull/788

This could be related to your package manager. If styled-jsx is installed under node_modules/next/node_modules instead of installed at the top-level node_modules directory in your folder, there might be an issue.
vercel/styled-jsx#788

I understand your point that is similar but is not the same. According to the issue that you link @leerob, this is related to react 18, not 17 which is used in this example.
I can easily reproduce it with npm from https://github.com/npm/cli/releases/tag/v7.20.3 to https://github.com/npm/cli/releases/tag/v8.1.0... It seems that it was fix on https://github.com/npm/cli/releases/tag/v8.1.1
You can easily reproduce it with the following dockerfile:
FROM node:16.14.0

# Install packages
RUN apt-get update -y && apt-get install -y git

RUN npm install -g<EMAIL_ADDRESS>
RUN npx create-next-app nextjs-blog --use-npm --example "https://github.com/vercel/next-learn/tree/master/basics/learn-starter" && \
    cd nextjs-blog && \
    npm run build

You can just run docker build on this dockerfile and check it for your self.
So saying this, maybe its a good idea to put some kind of warning on this tutorial:
https://nextjs.org/learn/basics/create-nextjs-app/setup about npm being not working for some versions and recomend one of outside the npm versions that I said above.


feat: Add a frameless param

I'm adding a param called frameless
So that different view of MASS can be used in cards (original suggestion)

Added 2 new params

frameless: Load just the content without the footer and navigation. It won't be saved in the store so that it goes away everytime we reload
targetPlayer: using that to select the activePlayer. I know we have a player param already. Put that param open the fullscreen player. And it might not be what we want.

Moved code that handled player to the defaultView so that everything is in the same place

https://github.com/user-attachments/assets/9091d7b5-3901-453c-a4c6-ac9c18896079

@marcelveldt I thought of that first. But I was not sure about adding a breaking change. If we are fine with that I'm happy to change

I think its fine as we are still early on in the cycle and this feature is not very known.
Let's just make it work great instead of half baken. We should just be aware to mention it in the docs and release notes once this gets released.

@marcelveldt I thought of that first. But I was not sure about adding a breaking change. If we are fine with that I'm happy to change

I think its fine as we are still early on in the cycle and this feature is not very known. Let's just make it work great instead of half baken. We should just be aware to mention it in the docs and release notes once this gets released.

@marcelveldt sounds good, let know what you think of it. Happy to rename anything if you want it

This will be a great update! Thanks @felipetoffolo1

wait - so https://myurl:xxxx/#/playlists?frameless=true&targetPlayer=MyBasementPlayer will give me the playlist window, without the media player and the menu bar?
THIS-IS-HUGE.
When will it be released into the beta?
If I could beg for one more addendum:
?OnlyShowFavorites=true&sort=playcount&view=list&toggleAlbumArtist=true
At this point, you have effectively made MA totally modular.
The 'views' will be your 'opinionated' views (which are responsive and great) -- but we will be able to do almost anything within HA without having dedicated MA-HA cards. Awesome.
s.


Game customization

Add a couple of basic options that the game host (the player who created the game) can set, implement a corresponding data container to keep them and their default values.

answer delay
word length (requires separate implementation)
word list (requires separate implementation)
multi-round mode (requires separate implementation)

Make sure the commands are properly documented. Consider adding a !help command.

The following are already implemented now:

word length
word list


[ButtonGroup] Split button requires an extra tab to get focus into the drop-down

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
The split button drop down (https://mui.com/components/button-group/#split-button ) can be opened from keyboard using space bar. However, up and down arrows do not allow user to select options. An extra tab is required to put focus into the drop down.
Composite components should only have one tab stop. the WAI-ARIA Authoring Practices 1.1 specification says:
A primary keyboard navigation convention common across all platforms is that the tab and shift+tab keys move focus from one UI component to another while other keys, primarily the arrow keys, move focus inside of components that include multiple focusable elements. The path that the focus follows when pressing the tab key is known as the tab sequence or tab ring.
Expected behavior 🤔
the tab and shift+tab keys move focus from one UI component to another while other keys, primarily the arrow keys, move focus inside of components that include multiple focusable elements.
Steps to reproduce 🕹
Steps:

Go to https://mui.com/components/button-group/#split-button

Tab to the down arrow in the control. Hit space to open the drop down.

Arrow keys don't allow selecting options.

Context 🔦
Keyboard-only user can't use the control as expected.
Your environment 🌎
No response

Here’s an example; click in the select to open the dropdown, immediately use the arrow keys to navigate through (no extra tab required) https://react-spectrum.adobe.com/react-aria/useSelect.html

Thanks for the report. Looks like we were missing an autoFocusItem prop on the MenuList. I am creating a PR for it. Here is an updated demo with the changes - https://codesandbox.io/s/splitbutton-material-demo-forked-fi9gf9?file=/demo.tsx


Add input file and os.Stdin support

now it can be used as $cat name card.vcf | qrcode -o name card.png

Thanks Tom, you are right.


Add options for fields

Add

nullable
unsigned
default

to options for fields

Hello again @phillipmadsen 👍  , This is not yet implemented ,but is a good suggestion . I will note this change  for the next version .Thanks


Sanddance Mutates Frozen Objects

Issue
If an object is frozen ( by default React Apollo does this ), the chart fails to render and it doesn't tell you the error unless you break on uncaught exceptions.
Exception: TypeError: Cannot add property GL_ORDINAL, object is not extensible at
http://localhost:3000/static/js/bundle.js:159322:61 at Array.forEach (<anonymous>) at assignOrdinals
(http://localhost:3000/static/js/bundle.js:159321:10) at Viewer.<anonymous>
(http://localhost:3000/static/js/bundle.js:160146:83) at Generator.next (<anonymous>) at
http://localhost:3000/static/js/bundle.js:159693:67 at new Promise (<anonymous>) at __awaiter
(http://localhost:3000/static/js/bundle.js:159672:10) at Viewer._render (http://localhost:3000/static/js/bundle.js:160133:12)
at Viewer.<anonymous> (http://localhost:3000/static/js/bundle.js:160090:29)

Can you iterate over your data and add a wrapper over each item? {...item}

Yes - thats how I resolved it.
Ideally Sanddance would probably either make its own copy or use a lookup table for this metadata ( < personally I would lean towards that approach ).

We avoid making a copy to minimize memory usage for large datasets.

Related: https://github.com/vega/vega/issues/2453


Improve function documentation

All exported functions need to have their inputs and outputs documented.

Done in https://github.com/rgiordan/zaminfluence/pull/11

Done in https://github.com/rgiordan/zaminfluence/pull/11


Unable to inspect global/script-wide variables in IronPython, works in Python

PTVS: Dev Build - 2015-07-09
IronPython: 2.7
I have a very simple test application that has a base and child class, where the child inherits the base class:
import sys

class BaseClass(object):
    def __init__(self):
        self.bValue = 2
    def Ping(self):
        print "Pong"

class Child(BaseClass):
    def Wacky(self):
        print "Wacky"

c = Child()
c.Ping()

I've noticed that when using PTVS and standard Python, I am able to debug the instance of the Child script-wide variable c in the debugger.  However, when using IronPython I am not able to inspect the variable.

If I step in to c.Ping(), self is visible by the debugger on both ends.
Additionally, if I create a method called test and perform the same kind of instructions d = Child(); d.Ping(); and inspect d it is visible in IronPython with PTVS.

I think IronPython has some subtleties about the global namespace (or possibly just globals()) that we aren't accounting for. @DinoV probably knows better than I do.

This occurs for me with the IronPython (.NET) launcher. Try switching to the Standard Python launcher and it seems to work properly.


test: rework Playwright cleanup using fixtures

Hi Kent!
This is a PR regarding your recently filed issue in https://github.com/microsoft/playwright/issues/27058. We at Playwright recommend fixtures for the cleanup of resources, which make it super easy to cleanup things after the use. In this PR I moved some of your resources over to fixtures, so its a bit move obvious to you to understand what we described in the issue, let me know if you have any questions!
I saw your UniqueEnforcer instance, this might be problematic, since it does not sync across multiple Playwright workers. Also it might make sense to move loginPage over to a fixture, so it always has access to the page instance. Not sure about the design choices about the User type, I just created it for now, might be changed to type inference etc. later.
(Marked as a draft for now, since this is more a FYI / how we would do things PR)

Thanks for this @mxschmitt! This helped me try fixtures out again and I'm pretty happy with where I landed: https://github.com/epicweb-dev/epic-stack/commit/85465dbf83cfc70486043666c08270636b6dafb7
I really appreciate you taking the time to demonstrate this!


required input field not validating on pagination

Required attribute is not being applied to the input field when checked in the CMS. It gets ignored by user forms validation.
Context
When creating a multi-page form with user define forms, all other required fields validated upon pagination except for the addressfinder field.
Possible solution
In AddressFinderField.php, in _constructor line 64.
    public function __construct($name, $title = null, $value = null)
    {
        $this->addressField = new TextField("{$name}[Address]", $title);
        $this->addressField->setAttribute('autocomplete', 'off');
        $this->manualToggle = new HiddenField("{$name}[ManualAddress]");
        $this->manualFields = new FieldList();

add conditional statement if required is checked in CMS -  $this->addressField->setAttribute('required', 'required');

This feels like this should be apart of the expected behaviour of this module, rather then an extension.
This is a handy module, thanks for putting together.
Regards ,
Janson

@jansonMaharaj Thanks but perhaps try when you do your validator use RequiredFields::create(['AddressFieldName[Address]']) rather than 'AddressFieldName' as the required field and see if that works out of the box

Hi,
Tried implementing this with a simple form, but the validator doesn't seem to call the Required() function as other FormFields do. Can you confirm a snippet that works for you? Here's mine:
use FullscreenInteractive\SilverStripe\AddressFinderField;
use SilverStripe\Forms\FieldList;
use SilverStripe\Forms\TextField;
use SilverStripe\Forms\RequiredFields;
use SilverStripe\Forms\Form;
use SilverStripe\Forms\FormAction;

class PageController extends ContentController {
	private static $allowed_actions = [
        'HelloForm',
        'search'
    ];

  public function HelloForm() {
    $fields = new FieldList(
        // TextField::create('Name', 'Your Name'),
        $test = new AddressFinderField('AddressField')
    );

    $test->setMaxLength(2);

    $actions = new FieldList(
        FormAction::create('doSayHello')->setTitle('Say hello')
    );

    $required = RequiredFields::create(['AddressField[Address]']);

    $form = new Form($this, 'HelloForm', $fields, $actions, $required);
    $form->disableSecurityToken();

    return $form;
  }

  public function doSayHello($data, Form $form)
  {
    $form->sessionMessage('Hello ' . $data['Name'], 'success');

    return $this->redirectBack();
  }

Hi @wilr ,
Just a bit more info:
The validation on 'AddressField[Address]' is not kicking in because this field doesn't belong to the form. If we set the form on that child field, then things start working:
$fields = new FieldList($address = new AddressFinderField('AddressField'));
$actions = new FieldList(FormAction::create('submitAddress')->setTitle('Address'));
$required = RequiredFields::create(['AddressField', 'AddressField[Address]']);
$form = new Form($this, 'AddressForm', $fields, $actions, $required);
$address->getAddressField()->setForm($form);
return $form;

However, the validate function in AddressFinderField should then call
parent::validate($validator);
or execute the current logic depending on manualToggle
(The javascript should also be updated to remove the required attribute on 'AddressField[Address]' when entering the address manually)
What do yo uthink?

HI @wilr, Any thoughts on the suggestions above?
Cheers,
Uili

@Uili-Simteau I've tested this on a simple form...
$fields = new FieldList(
            new TextField('FirstName', 'First name'),
            new TextField('Surname', 'Surname'),
            new EmailField('Email', 'Email'),
            new HiddenField('TakeUserToo', '', $nextStep),
            new AddressFinderField('Address'),
            $password = new PasswordField('Password', 'New Password'),
            new CheckboxField(
                'Terms',
                DBField::create_field(
                    'HTMLText',
                    sprintf(
                        'I agree to the Privacy Policy and Terms of Conditions for using our website, app and related services',
                        $privacy,
                        $terms
                    )
                )
            )
        );

        $password->setDescription('Make sure your password is secure. Want a random password? Try '. singleton(Member::class)->generatePassword());

        $actions = new FieldList(
            new FormAction('doRegister', "Register")
        );

        $validator = new RequiredFields(
            'FirstName',
            'Surname',
            'Email',
            'Password',
            'Terms',
            'DateOfBirth',
            'Address'
        );

Address is set as a required field, and I correctly get a warning.

It doesn't set the required attribute via HTML5 since that would prevent the manual fields working but backend functionality appears to work fine.

This is on Silverstripe 4.5 with latest AddressFinder release (d6367b72189cae861f0c5b28d203595c54d96902).


Question: How do I use the plugin with an external rule set?

I have created a custom rule set and would like the plugin to also highlight for these rules - is that possible?
Thank you!

Update: I had to use a search in the IntelliJ settings menu to actually find the settings for the plugin - which include the ability to point to a local jar.


Need Arbitrum tokenlist

Required:

[ ] arbitrum.ts created and lists exported
[ ] arbitrum tokenlist follows patterns
[ ] should include empty Index Coop tokenlist, appended to main list, just like the other lists in src/lists
[ ] exported appropriately in src/index.ts
[ ] should roughly mirror the tokenlist here

Guess I'll do this too while I'm already adding some others


[SourceKit] Use deep stack when parsing in the XPC service

Othwerise we were performing the syntactic parsing on a background queue that had a reduced stack size which could result in stack overflows.
rdar://84474387

@swift-ci Please smoke test

Could we only wrap the parsing part in a deep-stack WorkQueue? Kicking off a new thread for every request is a bit heavyweight since we don't implement it as a pool (it was intended for heavyweight operations like typechecking, so optimizing out the overhead of starting a new thread for that didn't seem very important).

@swift-ci Please smoke test

Changed it to only switch to a thread with a deep stack when actually performing the parsing and added a test case because we are no longer hitting an assertion failure now that https://github.com/apple/swift/pull/40065 is merged.

@swift-ci Please smoke test


Installation Issues

I followed the instructions on the documentation site - https://raysect.github.io/documentation/installation.html and then running the cornell_box demo failed, with import errors
  File "cornell_box.py", line 4, in <module>
    from raysect.primitive import Sphere, Box
  File "/home/adavis/.local/lib/python2.7/site-packages/raysect-0.5.4-py2.7.egg/raysect/primitive/__init__.py", line 30, in <module>
    from .box import Box```

I added the raysect egg folder into my PYTHONPATH. Box looks to be in primitive, so maybe this is out of date?

I expected to see some CI status from Travis/Circle but I didnt see any, I presume you do have CI but maybe its private? I figured it may hint what im doing wrong? I guess insideof there you run all the demos and all the nosetests, I guess it would be useful to expose that to users? I'll keep pluggin along in case its just an out of date file.

Hi Andy,
Looks like you are using python 2.7? Raysect only works for Python 3.5+.
Matt

Oh i see, I figured that one would need pip3, but the docs indicated pip, is that a convention thing?

It depends on how your environment is set up. For me pip maps to python 3.5.

Also, same issues im afraid
Traceback (most recent call last):
  File "cornell_box.py", line 4, in <module>
    from raysect.primitive import Sphere, Box
  File "/home/adavis/.local/lib/python3.6/site-packages/raysect-0.5.4-py3.6.egg/raysect/primitive/__init__.py", line 30, in <module>
    from .box import Box
ModuleNotFoundError: No module named 'raysect.primitive.box'``

The second problem was more to do with the documentation. The latest master doesn't automatically build with cython. Needs to be installed with python setup.py install --use-cython. The documentation needs to be updated to reflect this.

Yeah, personally I think 'out of the box' it should just build, the users first impression is really critical here. I think there are a few issues

There was no indication from the build system that anything was wrong, that should obviously be rectified
I think its really a situation that shouldnt arise, it should use cython by default, an advanced option (i suspect for a more advanced user) should be to pull it from some precompiled location. I guess there are several problems with that too, is there guaranteed ABI compatibility for cython compiled on linux to osx?
As @mattngc says, the documentation should really reflect this.

Raysect is a python extension and as such, strictly, only the .c files are necessary for the package to build. the cython stage just rebuilds the .c files. I'm a little stumped why it didn't compile! As it happens we've changed the default behaviour back to rebuilding the .c files in the next (development) release.
Annoyingly this does force the user to have cython installed, something I was trying to avoid.

Could it be that I only had the Intel compiler suite available at the time? Could it expect GCC?

Hmm, possibly. The c extension compliation is handled by a package in the standard library (setuptools). We don't define it's behaviour beyond providing it c files - see the setup.py file. It is possible it had an issue with the compiler.

If you install it again at any point could you capture the log file? We can't recreate the error.

Alex thinks this problem was caused by the c files not being included in the git repository. So the testing worked locally but failed when installed through pip or a clone. We have reverted to the original behaviour where cython compilation happens by default unless it is explicitly turned off.
Fixed in commit 6ef264b.


Add new synomtv user to plugin-defensics developers.

Description
Add new synomtv user to plugin-defensics developers. Related JIRA issue INFRA-2647, repository https://github.com/jenkinsci/defensics-plugin/ and users @synhal @synomtv .

Submitter checklist for adding or changing permissions

Always

[x] Add link to plugin/component Git repository in description above

For a newly hosted plugin only

[ ] Add link to resolved HOSTING issue in description above

For a new permissions file only

[ ] Make sure the file is created in permissions/ directory
[ ] artifactId (pom.xml) is used for name (permissions YAML file).
[ ] groupId / artifactId (pom.xml) are correctly represented in path (permissions YAML file)
[ ] Check that the file is named plugin-${artifactId}.yml for plugins

When adding new uploaders (this includes newly created permissions files)

[x] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[x] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[x] Make sure to @mention the users being added so their GitHub account names are known if they require GitHub merge access (see below).
[x] All newly added users have logged in to Artifactory at least once

Reviewer checklist (not for requesters!)

[ ] Check this if newly added person also needs to be given merge permission to the GitHub repo (please @ the people/person with their GitHub username in this issue as well). If needed, it can be done using an IRC Bot command
[ ] Check that the $pluginId Developers team has Admin permissions while granting the access.
[ ] In the case of plugin adoption, ensure that the Jenkins Jira default assignee is either removed or changed to the new maintainer.
[ ] If security contacts are changed (this includes add/remove), ping the security officer (currently @daniel-beck) in this pull request. If an email contact is changed, wait for approval from the security officer.

There are IRC Bot commands for it

User name not known to Artifactory: synomtv

Hmm, I have logged in few times to https://repo.jenkins-ci.org/ with 'synomtv'. Let's see if there's some delay in syncing user information.

User name not known to Artifactory: synomtv

Hmm, I have logged in few times to https://repo.jenkins-ci.org/ with 'synomtv'. Let's see if there's some delay in syncing user information.

Now the JSON list used in CI seems to contain that user. Removing WIP. Was there some Github comment to try re-build this PR?

you can close, pause, re-open PR to trigger a rebuild.
or a select few people can re-trigger it directly (I've done that)

@synhal can you approve please

Easiest way to get a new developer access in future is to create a PR to here, jira isn’t as closely monitored

Easiest way to get a new developer access in future is to create a PR to here, jira isn’t as closely monitored

Thanks for information and merging!


Feat: Add curried heterogeneous function types

See #10389

@eric-wieser It is possible to put ↿f as the body of uncurry₃, and I'll change that if needed, but it's not possible to change the type signature of  uncurry_3_eq_uncurry₃ to uncurry f = (↿ f) ∘ (finThreeArrowEquiv α); lean throws the error failed to synthesize instance HasUncurry (OfArity α β 3) ?m.12793 ?m.12795. Since my intent with uncurry₃ is to provide specifications for cases where the types may be pretty complex (like these heterogeneous functions or the ternary functor uncurrying from the other PR) it needs to have a concrete type for unification

bors merge


[BUG] Window shadows and window preview bugs

I have found two bugs:

When mica is enabled and set to dark mode, the window shadow is twice as thick as it would be if it was disabled or set to light mode.
The window preview (hovering the mouse over the app icon) is transparent where the mica effect should be shown, this is especially noticeable in light mode:

^ Actual window

^ Window preview

Same as #18, cannot be fixed. I just made sure my app has a dark mode to fit on the dark mode task bar.

@alexmercerind I see, what about the first issue? (window shadow)

I am also able to replicate number 1 in my apps too @alexmercerind, both with a custom frame and the default one:

I am also able to replicate number 1 in my apps too @alexmercerind, both with a custom frame and the default one:

@alesimula @aguilaair,
Same is noticeable in Windows's File Explorer. Intensity of the shadow seems to be higher in dark mode, but there is nothing I can do again. Best I can say is to inform Microsoft through Feedback Hub or support mail.


GH-676: KafkaEmbeddedHolder - creating and adding topics.

Fixes https://github.com/spring-projects/spring-kafka/issues/676. we have also migrated topics from array to set.

No concerns; LGTM; @nklmish thanks for the contribution.

Sure thing, @nklmish !
Thank you very much for the contribution.
Looking forward for more!

@garyrussell @artembilan Thank you very much

... and cherry-picked to 2.1.x.


Resolve runtime error RedisArray

If I'm trying to use cluster = true  i got a runtime exception.
use \RedisArray;
resolves the issue

@vetruvet Can you create a new tag with this?

done.


[Feature request] URL parameter API with JSON output for the web app

I would love to see a discussion on using the web-app URL-interface, as a way into using the various AI models and getting some form of semantically structured JSON back. This would make integrating the library a lot easier I think.
In my mind I see the tasks as a function-name name and the model context data as arguments on that function. There can also be global arguments, which are applied to all tasks, eg. the format type of the output (and perhaps the language).
I'm currently using the demo app with some URL parameters: https://conze.pt/app/ai/?task=summarization&arg1=Lightning%20is%20a%20naturally%20occurring&l=en
Maybe a good first step would be to have a list of JS objects describing the:

Available tasks,
Supported models for each task,
The model arguments (defaults, optional. required). Later these could be more formalized using some schema.org schema (or similar), eg: AchieveAction

js const tasks = {

  'summarization':

    'desciption' : 'text summarization',
    'type' : 'text-generation',

    'models': {

      't5-small' : {
        'options' : {
          'output_formats' : [ 'text, 'json' ],
          'output_name' : 'summary',
          'max_new_tokens': 50,
          'num_beams': 1,
          'temperature': 1,
          'top_k': 20,
          'do_sample': true,
          'summary_text' : 'text',
         }
       },

      't5-base' : ...,
      't5-v1_1-small' : ...,
      't5-v1_1-base' : ...,
      'facebook/bart-large-cnn' : ,
      'sshleifer/distilbart-cnn-6-6' : ...,
      'sshleifer/distilbart-cnn-12-6', ...,
    }

  // ...another task

}

TODO: Maybe add some "required / optional" and data-type declarations on the model options too.
Once we have this we can create a URL parameter structure to call a task with its options and get some plaintext or JSON back.
And perhaps (later) look into creating an OpenAPI schema (never did that myself): https://editor.swagger.io
I'm not yet sure what the best way would be to use larger amounts of text for models, I'm currently using HTTP GET, but a POST might be better suited for this. The drawback is the linking needs to become indirect then.

I like this idea! Will make the demo much more "share-able" too.


Find way to hide NUnit.Compatability.Path from intellisense

The .NET Core version of nunit.framework contains a public class called Path. This shows up in intellisense at the top when you try to auto-complete a reference to Path. This can be annoying because Path is a commonly used class.

@jcansdale I am pretty sure it is gone in the latest versions on master. It was annoying me to, so I removed it from the compatibility classes in the portable driver because it wasn't really needed. It was left over from when the driver was a PCL.
I just searched the code and can't find it, but I'll leave this open until the next release just in case I'm blind :smile:

@rprouse,
This was with 'nunit.framework' v3.4.1 (lousy bug report missing that).
Good to hear you're on top of it. :smile:

Ah, that is where it was coming from. Most people won't be including the NUnit.Compatibility namespace when using the PCL, but it should probably be internal anyway. We might also be able to drop it when we switch the PCL to .NET Standard.
I am going to move this to the main NUnit repo. I am pretty sure it belongs there.

Issue moved to nunit/nunit #1663 via ZenHub

Most people won't be including the NUnit.Compatibility namespace when using the PCL, but it should probably be internal anyway.

The problem is with intellisense, if you don't already have a using System.IO. This is compounded by the fact that System.IO.Path is in a separate assembly if you're using .NET Core (therefore appears further down the intellisense suggestions).


Both Editors Building Editor Events

Both text and visual editors are building the event nodes, text editor doesn't need them and its extra resources that are going to be taking up memory, which lead to tons of orphaned resources on close, and seem to at least be in part stability issues with alpha versions of Godot 4
Should probably actually be implemented as a flag for whether you want them built or not, which then could also be used for #1101 to allow building a timeline editor at runtime as requested there.

build_event_editor() is called because we (for whatever reasong) decided to override _get_property_list() instead of creating our own method. I think that is called.
I do not think that actual nodes are created on the text editor tho, but oc I could be wrong.

We can create our own method. We were using the _get_property_list one to show in the inspector, but we don't use the inspector for it any more.

I do not think that actual nodes are created on the text editor tho, but oc I could be wrong.

i found in debugging on #1207 that they were, and also on runtime as well (which I put in a check to disable). there not adding to anything, but they are being created by the event creation itself

huh? but how? only the EventNode.tscn actually adds any input field nodes. And that scene is only created by the VisualTimeline editor.


Migration to Swift 3

Create a new swift3 branch and update the APIs to be conform to the Swift API Guideline.

V2.0.0 is out now!


Prevent removal of inlined mutable vars from optimized AST

This prevents inlined mutable vars to be removed from optimized AST.

Can you give more context on this? What is this solving?

Can you give more context on this? What is this solving?

I do get why this is needed.
The compiler inserts various compiler-generated bindings during type checking, but we make an effort to remove these when revealing the TAST via quotations and FCS expression trees, to make the public form of the expression trees less sensitive to compiler changes and less revealing of compiler internals.  We do this by simply substituting let compgen = e1 in e2 away replacing all compgen in e2 with copies of e1.
However, this may be incorrect when a compiler-generated intermediate value is mutable.  I'd imagine the mutability is relevant in some situation related to Fable
@ncave two things

Can you measure the impact of making the corresponding change in quotations.fs please?
Can you add some tests please?  I'd like to understand the situations where this matters, particularly w.r.t. quotations.

Can you give more context on this? What is this solving?

@TIHan
This prevents incorrect removal of mutable variables after inlining functions that use them (in optimized AST), e.g. List.average, etc.

@TIHan This prevents incorrect removal of mutable variables after inlining functions that use them (in optimized AST), e.g. List.average, etc.

That is, in the context of clients who look at expression trees.
It sounds like this doesn't affect quotations as yet, since they don't reveal expression trees after inlining.

@dsyme That's right, as stated, in optimized AST only. And it's perfectly fine to keep it downstream (Fable-only), I just wanted to bring it up to see if anything can be done here.
Basically, is that optimization (extra variable removal) worth applying unconditionally, if there are (not so edge) cases when it impact (breaks) simple code that just uses FSharp.Core functions (e.g. List.average etc.). Again, just in the optimized AST, not the generated IL, but still.

@dsyme That's right, as stated, in optimized AST only. And it's perfectly fine to keep it downstream (Fable-only), I just wanted to bring it up to see if anything can be done here.

Your fix is good.  Can you add a test case please?
thanks

Thank you both. It makes sense to me.

@dsyme Added some tests.


Fix after uninstalling Merbridge, the recvmsg and sendmsg programs remain

Codecov Report

Merging #230 (748d508) into main (3b7188d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #230   +/-   ##
=======================================
  Coverage   40.33%   40.33%
=======================================
  Files           7        7
  Lines         419      419
=======================================
  Hits          169      169
  Misses        236      236
  Partials       14       14

Flag
Coverage Δ

unittests
40.33% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Combined timeline keeps opening

A lot of actions lead to displaying "combined" feed, when not explicitly asked. Two actions I know are easily reproducible:

Leaving the app for a second (pressing the Back button one time too much, and holding the Home button to go back to it again)
Changing screen orientation: this is absurd but I tilted my phone and the screen has gone blank and Combined feed was displayed

But there's more, I was completely disoriented by this when I first started using &s, then I learned to pay close attention to the feed selection widget.

The combined option could be put as a selectable option in settings. I would presume the "combined" feature only benefits a few? It's a nuisance to have it sit in the menu, its being displayed as the default setting when opening AndStatus which assumes users constantly have the option to switch "combined" off or on. Personally I am confused & annoyed by this feature and I wouldn't mind seeing it hidden when "combined"is not used on the app.

@bulletbites "Combined" is for everybody having more than one account in AndStatus. It allows to switch less between account. Even if a User needs to turn it on or off seldom, this cannot be put into "Settings".
Retarding unwanted switching to " Combined" timeline - this will be fixed, of course.
How many accounts do you have?

As of v.27.03 there are some improvements on this: "Default" timeline opens in many cases where some "Combined" timeline opened earlier.
You can set any timeline as your Default Timeline in the "Manage timelines" view.

Thank you. It's still confusing, though: I was viewing Home feed for account X, which is not my default account. I've pressed Home and clicked on &s icon. I was looking at Friends Combined feed of account X now.
On July 15, 2016 7:33:54 AM CEST, Yuri Volkov<EMAIL_ADDRESS>wrote:

As of v.27.03 there are some improvements on this: "Default" timeline
opens in many cases where some "Combined" timeline opened earlier.
You can set any timeline as your Default Timeline in the "Manage
timelines" view.

You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/andstatus/andstatus/issues/389#issuecomment-232863673

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

@yvolk I have only one active GNU account in AndStatus. When starting AndStatus the default setting for the timeline is "Home combined". Now I tried unticking all the stuff in "Manage timelines" but still it shows  "Home combined". I vaguely get what the feature adds to AndStatus but again I would think the "combined" feature is too prominent for its own good.

@bulletbites You have to set your default timeline explicitly.  I was furiously trying to still fix my issues with "Home combined". I finally figured it out!
In "Manage Timelines look if there is a "D" somewhere showing in one of the timelines listed. I totally overlooked this "D". It was showing up in between the column "Displ." and "Title". The "default" will choose that one for your default TL and if it happens to be for instance the timeline "Home combined" this will show up as the default when you start the app up, when you "Go to the Top", refresh or renew the TL.
Once you figure out that that pesky little "D" is the culprit (read the default) you can figure out that you can now set ANY timeline as your default. The issue is resolved and I hope others experiencing similar problems can understand that the solution lies in "setting the little d" in manage timelines. I'll provide screenshots to clarify if this post still confuses people. Anyway, thanks @yvolk for your patience.

@bulletbites Maybe I should change default value of the ”Defaul Timeline” to ”Home Timeline of the first account added to AndStatus” ?!

@yvolk in my own case I had trouble understanding how to properly SET my TL stuff. What you are suggesting should already help so that single account users aren't pestered with a TL setting they rather see changed.
I also wonder how many people actually use the "combined feature". I know what it does but I think most people use AndStatus as THE microblogging app (together with twitter). So 1 app for a Twitter-alternative and one separate app for Twitter (the default twitter app). To be honest I don't understand why people are combining twitter and gnusocial in AndStatus: the posting functionality of twitter gets gimped thru AndStatus.

@bulletbites I saw several posts, where people were satisfied exactly with having both Twitter and GNU Social Timelines in one application. I'm with them on this :-)
BTW What AndStatus lacks for you to be used as a Twitter client also?

On a whole I think AndStatus could be cleaner and leaner.

Posting: replying/posting should be just like the twitter app does. A new clean posting interface not embedded in the TL but in its own window.
Vids/Stats: both are not yet supported by gnu, but still.
Instant working buttons below the message. It's flaky at the moment. Often when I press the heart button it does not seem to register it or seems to do it somewhat delayed.
Timeline interface: in it's current state I would prefer having NOT to micromanage how and what my TL does, it makes things more confusing and its not always clear how and if it actually does what you expect it to do.
search-icon should NOT be a globe but a looking glass
buggyness or is it: A lot of the tickets get opened on github and I think its a combination of people not understanding something and having it set the wrong way or it actually IS a bug and it often becomes a challenge for you to understand, reproduce and hopefully fix the issue.

The confusion problem is also a little bit the main problem with AndStatus. Its heavy on options, heavy on settings and often I don't trust the setting to actually do what I think it would do.
For a dedicated gnusocial app that even comes close to the twitter-app AndStatus is the BEST there is at the moment but if the twitter app would allow for combined/multiple microblogging systems (adding gnusocial to it) I would dump AndStatus in a second, no hesitation. I would prefer it if AndStatus would mimic the twitter app as best as possible without all the quirkiness.
I often wondered how AndStatus could be improved in respect to ease of use in general. I think a online instructions manual would benefit it greatly. Confused users could look up stuff if they want to set AndStatus up or customize it (like with the manage timelines thingy) and it would make it easier for you too because you could just point them to the manual to resolve the issue at hand. If you consider making a manual I'm sure the community (me included) would be willing to help you with that.

Actually combined timeline is needed to combine information gathered from different GNU social instances. As you know, different GNU social instances have different (although partially overlapping) information. This is why I, for example, use several accounts on different GNU Social hosts.
In order to fight with duplicated posts found on different networks I'm implementing ”Collapse duplicated posts” feature, see https://github.com/andstatus/andstatus/issues/337

@bulletbites Regarding own window for an Editor please read/look and discuss here: https://github.com/andstatus/andstatus/issues/290

Instant working buttons below the message. It's flaky at the moment. Often when I press the heart button it does not seem to register it or seems to do it somewhat delayed.
We will discuss this in #410

search-icon should NOT be a globe but a looking glass

@bulletbites As you can see, we have two types of Search:

Search in local cache only. Quicker and doesn't require Internet connection. This doeas have a common "looking glass" icon.
Global Search. The same as above plus querying remote servers via Internet.It reuses another available material icon: Globe. In the old versions of AndStatus I made special icon for this case: combined general Search icon with a Globe. But as we moved to Android team-provided icons I decided that it is too hard for me to draw my custom icon for the "Global search"...

The confusion problem is also a little bit the main problem with AndStatus. Its heavy on options, heavy on settings and often I don't trust the setting to actually do what I think it would do.

I think that we need to use all ways to minimize such a confusion. Currently I see these:

Extending User Manual collaboratively: https://github.com/andstatus/andstatus/wiki/AndStatus-User-Manual
Choosing better default values so that most of users don't need to change any settings.
Breaking settings into Simple and Advanced, as discussed here: https://github.com/andstatus/andstatus/issues/237

I'm not really sure what the posters above are discussing, but the original issue persists as of 27.06: i clicked a link on my home timeline, it opened in firefox, then clicked back, found myself in &s and home timeline has been replaced with my default timeline.
So &s still loses its previous view on resume.

@gjedeer I think that the issue title confuses people.
So the problem now is not exactly about ”combined" timeline, but about the Timeline view state persistence?!

Yes, other people hijacked the thread.
On August 19, 2016 6:08:35 PM CEST, Yuri Volkov<EMAIL_ADDRESS>wrote:

@gjedeer I think that the issue title confuses people.
So the problem now is not exactly about ”combined" timeline, but about
the Timeline view state persistence?!
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/andstatus/andstatus/issues/389#issuecomment-241060923

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

@gjedeer I just tested both cases:
rotation and clicking on a link and then returning back to AndStatus;
on two devices: Android 6 2GB RAM and Android 4.4 512 MB RAM (Nexus One)
In both cases AndStatus restored a Timeline (not a default one!) and a position on it.
So I cannot reproduce the problem now... Maybe you have some settings, which force Android System shut down completely applications, which lost focus?! - In that case yes, default timeline will be opened.

Thank you for your answer.
I don't think I have such settings. K-9 Mail, which also reacts poorly to killing (saves edited message to drafts and closes editor), never had any related issues on this phone. It's a 4.4 with 2GB RAM. No task killers too.
So no plans to restore state in onResume?

@gjedeer Please understand me correctly: AndStatus does restore its Timeline view state on resume, but not on it creation. I would like to find out, why it doesn't resume on your device, but I cannot reproduce it.

Got it.
I've just spent some time trying to reproduce it, happened to me once during a screen flip but i was too slow to hit pause in catlog. My next post will be with logcat, any app create events should be there.
On August 20, 2016 5:51:50 AM CEST, Yuri Volkov<EMAIL_ADDRESS>wrote:

@gjedeer Please understand me correctly: AndStatus does restore its
Timeline view state on resume, but not on it creation. I would like
to find out, why it doesn't resume on your device, but I cannot
reproduce it.
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/andstatus/andstatus/issues/389#issuecomment-241176907

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

@gjedeer On creating logs in AndStatus see https://github.com/andstatus/andstatus/issues/225


Flaky-test: BasicEndToEndTest.testPatternMultiTopicsConsumerPubSub

Search before asking

[X] I searched in the issues and found nothing similar.

Example failure
https://github.com/apache/pulsar/runs/7739162912?check_suite_focus=true#step:11:185
Exception stacktrace
/pulsar/pulsar-client-cpp/tests/BasicEndToEndTest.cc:2184: Failure
Expected equality of these values:
  ResultTimeout
    Which is: TimeOut
  consumer.receive(m, 1000)
    Which is: Ok
[  FAILED  ] BasicEndToEndTest.testPatternMultiTopicsConsumerPubSub (1173 ms)
[----------] 1 test from BasicEndToEndTest (1173 ms total)

[----------] Global test environment tear-down
[==========] 1 test from 1 test suite ran. (1173 ms total)
[  PASSED  ] 0 tests.
[  FAILED  ] 1 test, listed below:
[  FAILED  ] BasicEndToEndTest.testPatternMultiTopicsConsumerPubSub

This can be relevant to https://github.com/apache/pulsar/issues/16802.
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

It seems in that attempt multiple tasks failed due to timeout. Perhaps it's an issue with GitHub CI resource unstable.
Keep it open for a few days to see if there're other instances. If no further reported, anyone can close it as invalid.

I think this problem may caused by #16968 and fixed by #17018.
Currently, the fix PR already merged, maybe we can close this issue until we got it again.

@mattisonchao OK.


Representing QR codes in Julia terminal?

I am so sorry if I am bothering someone but I am opening this issue due to the discussion on Twitter:
https://twitter.com/mhsatman/status/1563975585948483585
The main question is:
Is it possible to represent a QR  code in Julia terminal using its ASCII equivalents? It may be useful if it was possible.
Thank you in advance.

Here you go!

I've created an example on how to create a custom AbstractRenderable subtype for a QR code visualization starting from a BitMatrix.
It takes advantage of the half block character (U+02580) ▄ (see https://docs.julialang.org/en/v1/manual/unicode-input/) and different foreground/background colors to create "squared pixels" which would be otherwise hard to get in the terminal (the aspect ratio of terminal line's is not square).
This could be improved (besides using a proper QR code generator) to "scale" the visualization such that one pixel in the BitMatrix corresponds to an nxn square in the renderable, but its a good start.
By changing the way qr is created you can also do bar codes:
qr = fill(0, (10, 100))
idxs = rand(1:100, 50)
qr[:, idxs] .= 1
qr = BitMatrix(qr)

Hope this helps!

Great! Thank you.


Kamel kit create to use mount/camel traits

The command still use properties/configmap/secret parameters https://github.com/apache/camel-k/blob/main/pkg/cmd/kit_create.go#L70-L72 and they could instead use the logic encapsulated in camel trait (for properties) and mount trait (for configmap/secret).

Hi, could this be assigned to me?

I fixed this issue in pr #3160

I can see @Gre8t was working on it

hi @squakez this issue is resolved in pr #3160


Issue with linting only range and not the whole file.

Hi, and thanks again for this great project!
Is your feature request related to a problem? Please describe.
We're using the no-unused-vars rule.
With this diff
const foo = "..."
const bar = "..."

- console.log(foo, bar)
+ console.log(foo)

Eslint with the diff plugin does not see any issue here.
However when running eslint standard, it detects properly the issue: 'bar' is assigned a value but never used.
It's coming from https://github.com/paleite/eslint-plugin-diff/blob/53fd185e1efec3fdf99fc1848ae67ecb7d318bb7/src/processors.ts#L103-L107.
It's the same for unused imports rules, and all the ones that are based on part that are not touched.
Describe the solution you'd like
I don't think we actually can safely lint part of files with ESLint. I would remove this part of code. The impact should not be very important.
Describe alternatives you've considered
We could use a flag, to allow diffing only on files, and not lines.
Happy to make the PR of the chosen solution.
👋

Hi, thank you for your report and your patience in waiting for my response. Your input has sparked an idea in me that could potentially address this issue in an upcoming release. However, currently, this limitation is inherent to the design of the plugin itself. Nonetheless, there are alternative approaches that can still help you identify the mentioned issues.
You can employ different settings for ESLint depending on the context in which you are using it. For instance, you can utilize diff/ci to disable the eslint-plugin-diff on your local machine. This will ensure that you receive all ESLint errors, including the ones you mentioned. In CI environments, however, eslint-plugin-diff will be enabled, resulting in the exclusion of those specific errors.
Another option is to utilize other tools that can compensate for the limitation. TypeScript, for example, can be configured to throw an error for unused variables, thereby aiding in the identification of such issues.
Please let me know if you need any further assistance or clarification.


Bug: RPI OS is not autodetected

Describe the bug
RPI OS is not autodetected because the /etc/os-release ID is "debian".
Solution
Check for "/boot/issue.txt" and if it exists, set distroid to "raspbian".

It should probably also partially check that file issue.txt for the text Raspberry to avoid false detection.


Update training to Plone Conference 2024

📚 Documentation preview 📚: https://plone-training--886.org.readthedocs.build/

@ericof do you have the request/response diagram available as SVG or the source files you used to generate it? This would be a good Mermaid or Graphviz diagram for documentation.

@stevepiercy Do we support Mermaid in our docs? I did not have time to check it, so I generated the png

@ericof yes, the recent release of sphinxcontrib-mermaid 1.0.0 now makes it work with Plone Sphinx Theme 1.0.0.
You can see some examples at https://6.docs.plone.org/conceptual-guides/package-dependencies.html. @jensens used https://www.mermaidchart.com/play with a live preview, then copy-pasted it into the docs, and I did a little fine tuning so that lines show up in dark mode. See also the docs for usage at https://6.docs.plone.org/contributing/documentation/myst-reference.html#diagrams-and-graphs-with-mermaid.
I think this diagram would be good on a new page under https://6.docs.plone.org/conceptual-guides/index.html, as it would describe the concept of the request-reponse cycle in Plone.


Debug go in devcontainer doesn't hit the breakpoint

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.94
OS Version: mac os sequoia 15.0.1

Steps to Reproduce:

.devcontainer/devcontainer.json:

{
    "name": "Pdf Service Dev",
    "dockerComposeFile": "./docker-compose.yml",
    "service": "profile-pdf-ws",
    "shutdownAction": "stopCompose",
    "workspaceFolder": "/workspace",
    "forwardPorts": [
        4204
    ],
    "containerEnv": {
        "GOARCH": "amd64",
        "CGO_ENABLED": "0",
        "GOOS": "linux"
    },
    "customizations": {
        "vscode": {
            "forwardSSH": true,
            "debuggerType": "chrome",
            "settings": {
                "terminal.integrated.shell.linux": "/bin/bash",
                "go.inferGopath": true,
                "go.useLanguageServer": true
            },
            "extensions": [
                "redhat.vscode-yaml",
                "GitHub.copilot-chat",
                "GitHub.copilot",
                "streetsidesoftware.code-spell-checker",
                "golang.Go"
            ]
        }
    }
}

// .devcontainer/docker-compose.yml
services:
  profile-pdf-ws:
    container_name: profile-pdf-ws
    build:
      dockerfile: ./Dockerfile
      context: ./
    volumes:
      - ../:/workspace:cached
    security_opt:
      - seccomp:unconfined
    command: /bin/sh -c "while sleep 1000; do :; done"
    env_file:
      - ../.env
    ports:
      - "4204:4204"
    networks:
      - profile-network

networks:
  profile-network:
    external: true

///.devcontainer/Dockerfile
FROM golang:1.23.1-alpine3.20
RUN go install -v<EMAIL_ADDRESS>RUN go install -v golang.org/x/tools/gopls@latest
RUN go install -v honnef.co/go/tools/cmd/staticcheck@latest

// .vscode/launch.json
{
    "configurations": [
        {
            "name": "Launch Package",
            "type": "go",
            "request": "launch",
            "mode": "debug",
            "program": "${workspaceFolder}",
            "apiVersion": 2,
            "showLog": true,
        }
    ]
}

In devcontainer: Pressing F5 was able to launch the go project, it just doesn't hit the breakpoints, and the breakpoint greyed out

In Host environment: everything works fine

The issue is I set the dev container env:
GOOS=darwin
GOARCH=arm64

remove those env variables make it works again in the dev container


[SPARK-48706][PYTHON] Python UDF in higher order functions should not throw internal error

What changes were proposed in this pull request?
This PR fixes the error messages and classes when Python UDFs are used in higher order functions.
Why are the changes needed?
To show the proper user-facing exceptions with error classes.
Does this PR introduce any user-facing change?
Yes, previously it threw internal error such as:
from pyspark.sql.functions import transform, udf, col, array
spark.range(1).select(transform(array("id"), lambda x: udf(lambda y: y)(x))).collect()

Before:
py4j.protocol.Py4JJavaError: An error occurred while calling o74.collectToPython.
: org.apache.spark.SparkException: Job aborted due to stage failure: Task 15 in stage 0.0 failed 1 times, most recent failure: Lost task 15.0 in stage 0.0 (TID 15) (ip-192-168-123-103.ap-northeast-2.compute.internal executor driver): org.apache.spark.SparkException: [INTERNAL_ERROR] Cannot evaluate expression: <lambda>(lambda x_0#3L)#2 SQLSTATE: XX000
	at org.apache.spark.SparkException$.internalError(SparkException.scala:92)
	at org.apache.spark.SparkException$.internalError(SparkException.scala:96)

After:
pyspark.errors.exceptions.captured.AnalysisException: [INVALID_LAMBDA_FUNCTION_CALL.UNEVALUABLE] Invalid lambda function call. Python UDFs should be used in a lambda function at a higher order function. However, "<lambda>(lambda x_0#3L)" was a Python UDF. SQLSTATE: 42K0D;
Project [transform(array(id#0L), lambdafunction(<lambda>(lambda x_0#3L)#2, lambda x_0#3L, false)) AS transform(array(id), lambdafunction(<lambda>(lambda x_0#3L), namedlambdavariable()))#4]
+- Range (0, 1, step=1, splits=Some(16))

How was this patch tested?
Unittest was added
Was this patch authored or co-authored using generative AI tooling?
No.

cc @cloud-fan and @ueshin

This is more like an unsupported feature, that we don't support Python UDF as the lambda function within a higher order function. How about the UNSUPPORTED_FEATURE error?
BTW, will we ever support it in the future?

I don't think we will ever support this :-). Let me fix up the error

Merged to master, thank you @HyukjinKwon @cloud-fan