[BUG] In BIDS, data-type may be at subject or session level

Currently NeuroBlueprint mandates the placement of data-type folders (behav, ephys, funcimg) in the session level, and histology in the subject level. We discussed a little on slack about moving histology into the session level for simplicity. However of course BIDS does not actually mandate the session level, and data-type folders can be at the subject or session level.
I think for BIDS compliance it makes sense to also support having this flexibility. However it does make data-discovery in things like spikewrap more difficult, and datashuttle will require some massaging + a lot more tests and checks (e.g. error if trying to put a data-type folder at the subject level that is already somewhere in the session level,  and vice-versa). But I think in principle it makes sense?

I think ideally users could do both, I can see microscopy only projects that could be either:

Just one "session", i.e. a single confocal image, for which adding creating ses-01 seems onerous
A more complex project involving tissue clearing, imaging session 1, restaining, imaging session 2, sectioning, imaging session 3. Here sessions makes much more sense.

If this can be implemented within DataShuttle and NIU tools that ingest these data it would be great, but that could be a lot of overhead. If it's too much work, I'd be happy to prescribe what users should do.

A more complex project involving tissue clearing, imaging session 1, restaining, imaging session 2, sectioning, imaging session 3. Here sessions makes much more sense.

Actually, that makes sense. I was not familiar with such histology workflows.
How about this:

make session level optional (up to the user)
if no session level, put all datatypes under subject (obviously)
if session level is used, put datatypes under a session (incl. histology)
do not mix subject-level and session-level datatypes.

I think this is most clear and would avoid lots of problems with parsing. Technically BIDS does allow this however, but it is under SHOULD
In datasets where at least one subject has more than one session, this additional subdirectory later SHOULD be added for all subjects in the dataset.

I guess this becomes a more general question, as how strictly it is feasible to align with BIDS / if we should enforce some BIDS SHOULD as MUST to make software development easiest. Some possible options:

Follow BIDS exactly in NeuroBlueprint but in specific tools enforce stricter behaviour (probably not good because we cannot then claim our tools work with NeuroBlueprint
Have NeuroBlueprint  be more restrictive than BIDS in places (not good as then there is confusion between the specifications, and essentially they have diverged).
Allow full BIDS (possibly lead to hellish file / folder discovery problems).

I think all of these are bad, personally I'd be up for trying to maintain full alignment with BIDS unless there is a specific instance where it is just not possible to robustly perform file / folder search operations. I could try and handle this case in datashuttle and see how hard it is.

personally I'd be up for trying to maintain full alignment with BIDS unless there is a specific instance where it is just not possible to robustly perform file / folder search operations.

I think this should be the general aim. We should align with BIDS as long as it allows us to achieve our other goals. If we need to diverge, then we should. There's a reason why we created NeuroBlueprint in the first place. Turning "real" BIDS into something that supports all of our data types is currently too much work to be feasible in the time frame we have.

Ofc I hope that NeuroBlueprint and BIDS will converge eventually

Actually, that makes sense. I was not familiar with such histology workflows.

@niksirbi to be clear, these are rare, but it's something we would like to make possible at SWC with the mesospim etc.

I think this should be the general aim. We should align with BIDS as long as it allows us to achieve our other goals. If we need to > diverge, then we should. There's a reason why we created NeuroBlueprint in the first place. Turning "real" BIDS into something > that supports all of our data types is currently too much work to be feasible in the time frame we have.

This makes sense, if we are getting bogged down and spending a lot of time trying to maintain BIDS compatability, and making some more stringent criteria will make life much easier and the software more robust we should do it. I suggest we start with full BIDS compliance and then reduce it if issues arise with implementation.

updated with decision in #45


Example use: Complex Function Grapher

Not an issue but an example  putting the custom shader material to use (I think this is where you said to post it still?).  Starting with PlaneGeometry, rewrite the vertex shader to map points to a parametric surface (here, a projection to R3 of the graph of some complex polynomials in R4).
The result is then textured using a fragment shader  whose output is used as the map: on a MeshPhysicalMaterial.

As a comparison with a similar program I wrote doing this with THREE.ParametricGeometry (re-creating the geometry when the function changes), CustomShaderMaterial let me subdivide the domain 100x finer, at which point the result was still running 4x faster (60fps) than my original version.

That’s so cool! Do you have the code for this?
Can you fork this repo and add the code to a separate directory in the examples folder?
Update me here once it’s done, I’ll integrate it into the demo site then we can merge it into this repo!

Done!

Can you make a PR into the dev branch?

Closed by #8. If you have any more examples, would love to see them! Create a PR and they will be merged. Also feel free to edit your examples to include a little description for the uninitiated.
Thank you very much! 😄

Sounds good! Later this week when I have some time I'll add a description and throw send over the next example when its done.  Here's the start to it!

Looking forward to it!


Calculate relative size based on exclusive zone

it is able to set exclusive zone for ourself but we can not find a way to get the available area size.
one way to do this is through wayland-compositor, get available size though ipc:

set 2 small widgets on 2 opposite corner
get the coordinate of these widgets(through compositor ipc)
calculate size

This works but it feels a bit weird.
The hyprland feature hasn't have update for this for a long time, it's unusable right now.
maybe we'll find a better way, or just stick with this idea.

i give up, i don't really use it
reopen this if anyone need it


Random username

I need to make a command that can pull a random username out of chat
how would you go about doing this?

please get back to me. I need this im trying to help a friend

This Github repo is not a general Twitch/programming help forum, such as this official Twitch third-party help/announcement/bug report/discussion forum. However, what you want is to use this undocumented endpoint to get a list of chatters in a channel, concat all of the arrays, filter out the broadcaster and/or others, then pick a random index from the array.
Here's a Node example using request-promise:
const rp = require('request-promise');

function getChatters(channelName, _attemptCount = 0) {
    return rp({
        uri: `https://tmi.twitch.tv/group/user/${channelName}/chatters`,
        json: true
    })
    .then(data => {
        return Object.entries(data.chatters)
            .reduce((p, [ type, list ]) => p.concat(list.map(name => {
                if(name === channelName) type = 'broadcaster';
                return { name, type };
            })), []);
    })
    .catch(err => {
        if(_attemptCount < 3) {
            return getChatters(channelName, _attemptCount + 1);
        }
        throw err;
    })
}

function getRandomChatter(channelName, opts = {}) {
    let {
        onlyViewers = false,
        noBroadcaster = false,
        skipList = []
    } = opts;
    return getChatters(channelName)
    .then(data => {
        let chatters = data
            .filter(({ name, type }) =>
                !(
                    (onlyViewers && type !== 'viewers') ||
                    (noBroadcaster && type === 'broadcaster') ||
                    skipList.includes(name)
                )
            );
        return chatters.length === 0 ? null : chatters[Math.floor(Math.random() * chatters.length)];
    });
}

client.on('message', (channel, userstate, message, fromSelf) => {
    if(fromSelf || message[0] !== '!') return false;
    let chan = channel.slice(1);
    let params = message.split(' ');
    let command = params.shift().slice(1).toLowerCase();
    if(command === 'randomuser') {
        // Get a random user but skip the user requesting a random user
        getRandomChatter(chan, { skipList: [ userstate.username ] })
        .then(user => {
            if(user === null) {
                client.send(chan, `${userstate.username}, there was no one to choose.`);
            }
            else {
                let { name, type } = user;
                client.send(chan, `${userstate.username}, I chose "${name}" with type ${type}!`);
            }
        })
        .catch(err => console.log('[ERR]', err));
    }
});


Add an option to toggle rendering of padded rows.

Hi, this is a feature suggestion to toggle the rendering of padded rows.
This PR adds a new prop showPaddedRows. It's useful if you want the height of the table to be determined by the number of visible, populated rows.
We found it a bit more performant than setting style:{display:none} on empty rows via getTrProps (particularly when iterating through small data sets with large defaultPageSizes).
Included is a story which demonstrates the behaviour of the prop.

I feel like this is unnecessary with the minRows prop, which you can set to 0.  Does this achieve something different than that?

Ha, too true! Looks like I missed that prop before venturing down this rabbit hole.
Suggest adding minRows to the props section of readme.md. It's currently not listed as one of the available props.

Submit a pr? ;)
On Fri, May 26, 2017 at 7:35 PM ldstein<EMAIL_ADDRESS>wrote:

Ha, too true! Looks like I missed that prop before venturing down this
rabbit hole.
Suggest adding minRows to the props section of readme.md. It's currently
not listed as one of the available props.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/tannerlinsley/react-table/pull/295#issuecomment-304417965,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFUmCXVRJRgi-TfudpxgVuzRVkZDOZtEks5r933hgaJpZM4NnUnc
.

Updated this PR.

Putting this as a default prop makes things kinda wonky for me. I think the docs should be enough in their own


add trajectory calculation

This PR:
Cleans up the cache and adds more data, such as trajectory:

The good morning message will now include the trajectory calculation.

Also you'll be able to run this in REPL.

cleaned up the cache a little bit more:

Awesome, thanks for the contribution!


🔄 Synced file(s) with BetaHuhn/github-files

Synced local file(s) with BetaHuhn/github-files.

Changed files

Synced local  .github/FUNDING.yml with remote  configs/FUNDING.yml

This PR was created automatically by the repo-file-sync-action workflow run #768123274

:tada: This PR is included in version 1.1.3 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


electron-json-storage integration

I've successfully integrated electron-json-storage. I've used promises, because electron-json-storage is async. Promises are better than callbacks, because you can avoid a "callback hell" (llike this: https://www.reddit.com/r/ProgrammerHumor/comments/5un65s/it_took_some_time_to_fix_this/) and it has a nice error handling system. If you need more information about promises, you can ask me, it's very easy to use.
It does save config files even when using a compiled asar archive, but it can't toggle mods, because you haven't finished the react client rendering yet (don't worry, I didn't touch the front-end code).

I'm helping you with the front-end react code, I'm doing it slowly and carefully and I trying to learn how it works. I understand the actions system and I made the start factorio button work again.

I'm having some trouble with the toggle mod system. I updated some things in ProfileManager.toggleMod, but I don't know what's the proper way to do it.

Seems to be a problem with 'true' vs true and 'false' vs false

It works now, but appears that if you don't wait 5 seconds before reopening the app after closing it, the config files get corrupted. I'm still investigating the problem.

So about pulling in other branches, I think it's best to keep a focus on the branch you're working on.
In this case, think of develop as the primary branch, where all branches come from and eventually go to. Unless it's a primary branch, generally a branch is worked on by a single developer and merged separately onto a primary branch. It's easier to handle things that way.
With the json-storage thing, the only files needing changes would be in the lib folder (appManager and profileManager I think), and the branch for that feature should only include those changes. Then when the react/redux branch is finished, that will merge into develop as well. In the event problems do arise between the two, that's the appropriate place to handle changes.
You've pulled in my half-finished branch, which will make things a bit weird when I finish it and merge it in to develop.
In short, can you take out everything unrelated to implementing electron-json-storage out of this branch before making another pull request?
And I'm happy for you to help with things but to be just a bit selfish, this little pet project is still my baby, I'd appreciate finishing up my own branch.

I fixed it!!!! I forgot to wait until electron-json-storage was finished saving the config files before quitting the app

Ok, I'll do that

Sorry, this is taking very long, I feel like I'm screwing everything up

Sorry for the weird things I did, but now I understand what you mean with the primary branch concept

No worries about any issues, I've been inconsistent with the project myself so I feel weird telling you off, but with two developers now's the time to do things right! And don't sweat how long you take to do something, it's personal time so there's no rush. I'll work on a contribution guide to point out some ways I'd like to do things if I'm going to be picky about it.


pulling in upstream versioning

versioning module needed to be pulled in from: https://github.com/sigstore/rekor/blob/main/cmd/rekor-cli/app/version.go#L19
This should allow us to modify it to include downstream build information. We will then pull it in in cosign and gistign.
/cc @lance

moved to: https://github.com/securesign/go-version-package/pull/1/files


fix: Video source and old versions update in samples

Updated video source and library version

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


implement authentication plugin

The plugin uses the existing authentication plugins and allows for rules such as 'at least one must be successful' or 'both this and this one need to be successful', for 2 factor schemes.

https://gist.github.com/fkooman/9220885cc4f5ed1905f6

Rudimentary support is implemented now, it allows for multiple authentication plugins to be implemented at the same time, for example both Basic and Bearer support where one of them has to succeed.
For 2 factor more consideration is needed. For example a method ::require(array('Basic', 'Yubi')) may be required to specify the methods.

we move this to a new project, no need to have authentication in main project


use originalname instead of .name

when generating results for the test right now, it creates it in this structure of
test_results/[test_name][param]
with this change it will generate the folder structure of
test_results[test_name]
this drops off things at the end like [py0] when using parameters

I included the py_with_config as I'm aware that it won't hit the issue as likely without such reasons to pass in a parameter.  But the PR is really just for using "request.node.originalname"

Thanks for submitting a PR, @theqa-ninja! There are a few things I want to mention.
First, you should follow the guidelines that are described in the CONTRIBUTING.md documentation. You are missing a lot of things. For example, creating an Issue first so we can discuss it before submitting the PR.
Second, this should be broken into two different issues since they are two different requests.
Lastly, kind of tied to the first, you need to explain the reasoning behind your requested change. Without this context and any examples, I'm not sure what the value is of originalname change is.
Most of this is solved by following the CONTRIBUTING docs, so please take some time to look through it, update your PR, and then I'll look at it again. Thank you!

Sorry, will clean this up.  I should've read the docs first (I really was just digging through stuff and went ah-ha and threw it in quickly).

No movement for a long time. Closing the PR


Test D-Voting with New Mino P2P

[ ] modify c4dt/dvoting to use mino P2P
[ ] run dvoting local tests to verify it works

c4dt/d-voting > run_local.sh > echo "  Create a new chain with the nodes" > ordering export`:

prints address{peer id} after each mino instance is started (on a diff process, or host if running not on local)
admin collects printed address{peer ids} and export to root node
root  node then uses these marshalled/unmarshalled addresses{peer ids} to call/stream

clone d-voting/cli/dvoting
mod.go > runWithCfg() > mino.NewController() use minows instead
run_local.sh > init_nodes() "starting nodes" change start up command dvoting ...> change grpc to minows


feature proposal: keyUp output event

Hi, i need to output an event on key up,
would you accept this feature if I make the PR ?

Hi, can you tell a bit more about the use case?
It might be better to make a custom component for it

I need to catch keyUp on ng completer input to set the form dirty
2017-04-03 20:36 GMT+02:00 Ofer Herman<EMAIL_ADDRESS>
Hi, can you tell a bit more about the use case?
It might be better to make a custom component for it
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/oferh/ng2-completer/issues/186#issuecomment-291233331,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAU396hX4vNjKue6SBX-TwrKLr8RmwlOks5rsTwZgaJpZM4Mxq6Y
.

--
Thomas Desvenain
Téléphone : 09 51 37 35 18


Sik-Ho Tang | Review: Deep Image -- A Big Data Solution for Image Recognition in ILSVRC 2015.

Sik-Ho Tang. Review: Deep Image — A Big Data Solution for Image Recognition in ILSVRC 2015.

Overview
In this story, Deep Image [1] is reviewed. Deep Image achieves 4.58% error rate which surpasses the human-level performance in ILSVRC 2015.
However, Baidu violated the rule of ILSVRC at that moment. They created 30 accounts such that they had at least 200 submissions, and more than 40 submissions over 5 days from 15 March, 2015 to 19 March, 2015. Due to such frequent submissions, they violated the rule of ILSVRC that only allows 2 submissions in 1 week.

Supercomputer Setup
As mentioned, Minwa, a custom-built supercomputer, with 36 server nodes, is used.

Each node has 2 six-core Intel Xeon E5–2620 CPU processors, 4 Nvidia Tesla K40m GPUs, each with 12GB of memory, and 1 FDR InfiniBand which can provide 56Gb/s interconnection data speed.

In total, it has 6.9TB host memory and 1.7TB device memory.
As we can see, it is actually scaling out rather than scaling up!
With more GPUs, larger batch size can be used. It is crucial to have a larger batch size in deep learning network, e.g. AlexNet used 2 GPUs. It is a difficult task for one single computer to increase the number of GPUs due to limited PCI-E slots, power supply, and high computer temperature problem. The best solution is to scale out!!
With 1024 batch size, and 64 GPUs, the speed up can be up to 47 (Green) as shown below:

And the time to converge is much faster, it only takes 8.6 hours by using 32 GPUs for training to get up to 80% accuracy while 212 hours are needed with only 1 GPU.
If the training time of your model takes weeks, right now, it can be reduced by scaling out. Then it is possible to collect all the current day new data, train the model at midnight, and deploy the trained model before the next business day starts, or in a weekly-basis!

Data Parallelism
Each GPU is responsible for $\frac{1}{N}$ mini-batch. During backpropagation, all GPUs compute the gradient based on the local training data, then exchange gradients and update the local copy of weights, as below:

Experimental Results
VGGNet [2] is used. Multi-scale training is also used. The model is pre-trained from ILSVRC and then fine-tuned based on the new dataset.

Model Architecture.
CUB-200–2011
200 bird species recognition, 11,788 images.

Oxford 102 Flowers
102 different categories of flowers, 8,189 images.

Oxford-IIIT Pets
37 classes, 7,349 images.

FGVC-Aircraft
100 aircraft variants, 10,000 images.

MIT-67 Indoor Scenes
67 indoor scenes, 15,620 images.

ILSVRC
1000 images in each of 1000 categories. Indeed, the best error rate is of 4.58%. But finally they are withdrawn from ILSVRC due to violation of rule.

Multi-Model.

Single Model.

References

[2015 arXiv] [Deep Image] Deep Image: Scaling up Image Recognition.
[2015 ICLR] [VGGNet] Very Deep Convolutional Networks for Large-Scale Image Recognition.


Frozen problems are running in the wrong environment

That's what 14c40c0362ae3d8b7b212e0239f882e75c41912f is fixing in an ad-hoc way
You can also see this as a failed test on 47b121d7ae6ee2aafbc13aed58c6c320649a9992
cabal test --test-options "-p CaseTest.pi"
Elaborating module "CaseTest"
warning: pi/test/CaseTest.pi:8:3:
  supposed to reduce b for now returning it without reduction
  In the expression
    case b of
      True -> False
      False -> True
FAIL
      Exception: Prelude.head: empty list

which on 14c40c0 becomes

long test output
  Files typechecked
    test/CaseTest.pi: Parsing File "pi/test/CaseTest.pi"
Elaborating module "CaseTest"
warning: pi/test/CaseTest.pi:8:3:
  supposed to reduce b for now returning it without reduction
  In the expression
    case b of
      True -> False
      False -> True
warning: After checking an entry there are unsolved constraints
  [  constraint 4: (?_2 ~ True) : Bool raised at pi/test/CaseTest.pi:4:6:, constraint 14: (Bool -> Bool ~ ?_11 -> ?_13) : Type raised at pi/test/CaseTest.pi:7:7:, constraint 19: ?_13  ]
warning: pi/test/CaseTest.pi:13:6:
  supposed to reduce
    Bool -> Bool
  for now returning it without reduction
  In the expression
    not b1
warning: After checking an entry there are unsolved constraints
  [  constraint 4: (?_2 ~ True) : Bool raised at pi/test/CaseTest.pi:4:6:, constraint 14: (Bool -> Bool ~ ?_11 -> ?_13) : Type raised at pi/test/CaseTest.pi:7:7:, constraint 19: ?_13, constraint 20: (?_2 ~ True) : Bool raised at pi/test/CaseTest.pi:4:6:, constraint 26: (Bool -> Bool ~ ?_23 -> ?_25) : Type raised at pi/test/CaseTest.pi:13:6:, constraint 27: (Bool ~ ?_23) : Type raised at pi/test/CaseTest.pi:13:6:, constraint 28: (?_25 ~ Bool) : Type raised at pi/test/CaseTest.pi:13:6:  ]
warning: After checking an entry there are unsolved constraints
  [  constraint 4: (?_2 ~ True) : Bool raised at pi/test/CaseTest.pi:4:6:, constraint 14: (Bool -> Bool ~ ?_11 -> ?_13) : Type raised at pi/test/CaseTest.pi:7:7:, constraint 19: ?_13, constraint 20: (?_2 ~ True) : Bool raised at pi/test/CaseTest.pi:4:6:, constraint 26: (Bool -> Bool ~ ?_23 -> ?_25) : Type raised at pi/test/CaseTest.pi:13:6:, constraint 27: (Bool ~ ?_23) : Type raised at pi/test/CaseTest.pi:13:6:, constraint 28: (?_25 ~ Bool) : Type raised at pi/test/CaseTest.pi:13:6:, constraint 29: (?_2 ~ True) : Bool raised at pi/test/CaseTest.pi:4:6:  ]
warning: pi/test/CaseTest.pi:24:3:
  supposed to reduce c for now returning it without reduction
  In the expression
    case c of
      FirstC -> SecondC
      SecondC -> FirstC
warning: After checking an entry there are unsolved constraints
  [  constraint 4: (?_2 ~ True) : Bool raised at pi/test/CaseTest.pi:4:6:, constraint 14: (Bool -> Bool ~ ?_11 -> ?_13) : Type raised at pi/test/CaseTest.pi:7:7:, constraint 19: ?_13, constraint 20: (?_2 ~ True) : Bool raised at pi/test/CaseTest.pi:4:6:, constraint 26: (Bool -> Bool ~ ?_23 -> ?_25) : Type raised at pi/test/CaseTest.pi:13:6:, constraint 27: (Bool ~ ?_23) : Type raised at pi/test/CaseTest.pi:13:6:, constraint 28: (?_25 ~ Bool) : Type raised at pi/test/CaseTest.pi:13:6:, constraint 29: (?_2 ~ True) : Bool raised at pi/test/CaseTest.pi:4:6:, constraint 34: (?_2 ~ True) : Bool raised at pi/test/CaseTest.pi:4:6:, constraint 35: (?_32 ~ FirstC) : TwoConstructors raised at pi/test/CaseTest.pi:20:7:, constraint 45: (TwoConstructors -> TwoConstructors ~ ?_42 -> ?_44) : Type raised at pi/test/CaseTest.pi:23:10:, constraint 50: ?_44  ]
warning: pi/test/CaseTest.pi:29:7:
  supposed to reduce
    TwoConstructors -> TwoConstructors
  for now returning it without reduction
  In the expression
    nother nb1
warning: After checking an entry there are unsolved constraints
  [  constraint 4: (?_2 ~ True) : Bool raised at pi/test/CaseTest.pi:4:6:, constraint 14: (Bool -> Bool ~ ?_11 -> ?_13) : Type raised at pi/test/CaseTest.pi:7:7:, constraint 19: ?_13, constraint 20: (?_2 ~ True) : Bool raised at pi/test/CaseTest.pi:4:6:, constraint 26: (Bool -> Bool ~ ?_23 -> ?_25) : Type raised at pi/test/CaseTest.pi:13:6:, constraint 27: (Bool ~ ?_23) : Type raised at pi/test/CaseTest.pi:13:6:, constraint 28: (?_25 ~ Bool) : Type raised at pi/test/CaseTest.pi:13:6:, constraint 29: (?_2 ~ True) : Bool raised at pi/test/CaseTest.pi:4:6:, constraint 34: (?_2 ~ True) : Bool raised at pi/test/CaseTest.pi:4:6:, constraint 35: (?_32 ~ FirstC) : TwoConstructors raised at pi/test/CaseTest.pi:20:7:, constraint 45: (TwoConstructors -> TwoConstructors ~ ?_42 -> ?_44) : Type raised at pi/test/CaseTest.pi:23:10:, constraint 50: ?_44, constraint 51: (?_2 ~ True) : Bool raised at pi/test/CaseTest.pi:4:6:, constraint 52: (?_32 ~ FirstC) : TwoConstructors raised at pi/test/CaseTest.pi:20:7:, constraint 58: (TwoConstructors -> TwoConstructors ~ ?_55 -> ?_57) : Type raised at pi/test/CaseTest.pi:29:7:, constraint 59: (TwoConstructors ~ ?_55) : Type raised at pi/test/CaseTest.pi:29:7:, constraint 60: (?_57 ~ TwoConstructors) : Type raised at pi/test/CaseTest.pi:29:7:  ]
Checking module "CaseTest"
FAIL
      test/Tests/Full.hs:30:
      Type error:Internal error: can't have metavariables in the core ?_2
      When checking the term  ?_2 against the signature [b1] : Bool

confirmed by 3596942c781f46fb03aa9a3759352fe748dccd5f

fixed by 1be09da


pip install -r requirements.txt Fails

When I try to do the pop install I get this error
ImportError: cannot import name 'Feature' from 'setuptools' (c:\users\kkashani\python-docs-hello-world\env\lib\site-packages\setuptools_init_.py)

pip install feature
Or
pip3 install feature

I had the same issue, but this didn't work, however I uninstalled setuptools (had version 47) and then installed version 45.0.0 because the problem is for versions higher than 46

@kashanico Can you git pull the sample and try again? We had an old version of MarkupSafe in requirements.txt, which I've cleaned up to reference only Flask, which should pull in the correct versions of its own dependencies and fix this error.

Confirmed that this fixed the issue after testing locally. I'm closing this for now, but if you have any other problems, please raise another issue.


test(scanv4): New test: upgrading using bundles.

Description
Adds a new test scenario for upgrading from pre-Scanner V4 bundles.
Checklist

[ ] Investigated and inspected CI test results
[x] Unit test and regression tests added
[x] Evaluated and added CHANGELOG entry if required
[x] Determined and documented upgrade steps
[x] Documented user facing changes

If any of these don't apply, please comment below.
Testing Performed
Manual + CI.
Reminder for reviewers
Took this PR over from @kurlov.

/retest


log rotation at OS level

The docs say that there are better ways to do log rotation at OS level. I'm new at this and any references would help.

Search for logrotate.

Thanks @Marsup

Does good-file reopen the log file on SIGHUP?


feat: fix variable replacements in payloads and support pruning duplicates and storing duplicate_responses

Codecov Report
Attention: Patch coverage is 41.59292% with 66 lines in your changes missing coverage. Please review.

Project coverage is 51.84%. Comparing base (bb65fd8) to head (605ad76).
Report is 5 commits behind head on main.

Files with missing lines
Patch %
Lines

py/atlas_init/cli_tf/mock_tf_log.py
51.06%
23 Missing :warning:

py/atlas_init/cli_tf/debug_logs_test_data.py
24.13%
22 Missing :warning:

py/atlas_init/cli_tf/debug_logs.py
26.31%
14 Missing :warning:

py/atlas_init/repos/go_sdk.py
41.66%
7 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #26      +/-   ##
==========================================
- Coverage   51.97%   51.84%   -0.13%
==========================================
  Files          49       50       +1
  Lines        4949     5046      +97
  Branches      660      672      +12
==========================================
+ Hits         2572     2616      +44
- Misses       2297     2350      +53
  Partials       80       80

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


Add tpc (Cancel teleport)

still didnt test lmao

Hey, thanks for your contribution.
However, I will not invest more time in maintaining the main branch.
If you want this command in the next release, can you port this as a class CancelCmd extends TpCommand to the dev branch?

Also, I am not really convinced this would be so useful that it is worth implementing. What is the use case for a cancel command?


Settings_Mudae.json not found

I installed discum but whenever I launch "MudaeAutoBot.py" I get a prompt saying "there is no such file or directory "Settings_Mudae.json"" any clue why?

Personally hate windows store version and even when you attempt to install the standalone version windows just loves to force you to use their windows store version

If you're sure that the settings file is in that same folder and is named correctly, try opening a terminal/command prompt in that folder (shift right click in the folder, click "open in terminal" or "open in command window" depending on your OS) and run the script with the command "python MudaeAutoBot.py"
I use the windows store version of python on windows 11 and it works fine for me this way.

I opened the file in a command window and whenever I run the script the whole thing just freezes. I press enter after entering "python MudaeAutoBot.py" and nothing happens afterwards and the prompt takes no inputs.

Also yes I did have a standalone version of python but windows forced the windows store one.

Make sure you have Discum installed (I recommend you install git and then install Discum using the command on their GitHub page readme).  If you have Discum installed, make sure you entered your login token in the top of the settings file.


NETOBSERV-1734: change subnets order priority

Description
Make sure auto-detected openshift labels are appended at the end of the subnets list, so that they have lower priority. It makes it possible to override in-cluster subnets.

Dependencies

n/a
Checklist
If you are not familiar with our processes or don't know what to answer in the list below, let us know in a comment: the maintainers will take care of that.

[x] Is this PR backed with a JIRA ticket? If so, make sure it is written as a title prefix (in general, PRs affecting the NetObserv/Network Observability product should be backed with a JIRA ticket - especially if they bring user facing changes).
[ ] Does this PR require product documentation?

[ ] If so, make sure the JIRA epic is labelled with "documentation" and provides a description relevant for doc writers, such as use cases or scenarios. Any required step to activate or configure the feature should be documented there, such as new CRD knobs.

[ ] Does this PR require a product release notes entry?

[ ] If so, fill in "Release Note Text" in the JIRA.

[ ] Is there anything else the QE team should know before testing? E.g: configuration changes, environment setup, etc.

[ ] If so, make sure it is described in the JIRA ticket.

QE requirements (check 1 from the list):

[x] Standard QE validation, with pre-merge tests unless stated otherwise.
[ ] Regression tests only (e.g. refactoring with no user-facing change).
[ ] No QE (e.g. trivial change with high reviewer's confidence, or per agreement with the QE team).

/ok-to-test

/label qe-approved

Verified that this fixes the OpenShift auto-detect issue described in NETOBSERV-134.  If you use an internal CIDR and openShiftAutoDetect is set to true, it is now able to show your custom name and the predefined names (e.g. Pods, Services).
/lgtm

/approve


Fix computation of argument index in the presence of indirect error results

Fixes rdar://124108894
With typed throws, we can have indirect error results, this changes how we compute argument index. Update 2 incorrect instances exposed by  rdar://124108894.

There are other instances where only getNumIndirectSILResults() seem to be used, I have not surveyed any of them. Fixing just 2 incorrect instances exposed by the bug.

@swift-ci test

@swift-ci smoke test

@swift-ci smoke test


[HELP] Post-processing example for OpenCV and/or TensorFlow Lit

Describe the issue
Cannot find any code snippet related to post-processing with OpenCV and/or TensorFlow Lite at https://www.raspberrypi.com/documentation/computers/camera_software.html#getting-help
Help request

I have followed all the steps successfully to prepare for post-processing with OpenCV and TFL.
I have built libcamera as outlined in the documentation
I have built (from source v4.5.3) OpenCV
I have confirmed simple image processing works with the OpenCV deployment including primitive object detection
I have built TFL apps for object detection and classification for Android devices (that use OpenCV for image capture and Google online models)

Is there a very simple code snippet that illustrates post-processing of libcamera output using OpenCV and/or TFL? I understand that it is impractical to accommodate diverse examples but something very simple to ensure a practical pipeline would be very helpful for my continuing self-paced exercises.
My predicament is that several JSON files are mentioned as being used during the libcamera image acquisition process. Unfortunately, I have no idea how to leverage these files when running OpenCV. If the libcamera team has done some work on this topic would it be possible to share a few lines of skeleton code snippet to get a newbie started? Thanks.
Kind regards.

I presume you have looked at the relevant documentation sections, particularly this and this?
All our example postprocessing stages that use either TFlite or OpenCV can be found in the post_processing_stages subdirectory of this repo.
Depending on what you are doing, it might also be worth looking at picamera2.  There are some examples of OpenCV postprocessing that are available in the repo as well.  You can find a comprehensive manual on picamera2 here.

Thanks @naushir for the heads up. I do have the links you provided. I simply cannot tie the loose ends (my issue). I'll step through the documentation again step by step.
I understand the advantages of integrating libcamera for OpenCV and TFL use. My earlier use of pure OpenCV for object detection was laggy. I will definitely keep trying. Thanks again for the excellent work on libcamera-apps.
Regards.


Update README to have more detailed information

Trying to get this more friendly, ideally including some how to guides with the current UI.

Yes, this could really use some documentation on how to use it.  Where is the run script?  I'm guessing from some of the file extensions that it might be a .NET app.  Do I need Windows to run it?


Add version nubmers?

Thank you very much for creating these symbols, they are very useful. Would you mind adding tags for versions so we can package them more easily to repositories like Nixpkgs or AUR?

Thanks for the suggestion, I just pushed up v1.0.0

Seems that there are no tags yet. Could you please try git push --tags again?

Sorry about that, should be pushed now.


Revert "Re-Re-land Implement flutter build bundle with assemble"

Reverts flutter/flutter#41230
TBR @zanderso

Didn't catch the latest failure because I wasn't running from a clean-slate locally :(

lgtm


Submissions: test server reruns the same commit multiple times

Repository URL: git@github.com:Vlad-Shcherbina/icfpc2020-tbd.git
The day before yesterday I submitted, among others, the commit bd58c6. Test server successfully ran it.
However, yesterday it ran it 7 more times without any action on my side.
Just letting you know.

Yesterday around 17:30 UTC we had a glitch in our submission system (still being actively developed :-)) which triggered multiple builds for existing submissions. Shortly after problem was solved. Sorry for inconvenience.


lock on adding to list to avoid dupes

this is @sperand-io on wesley's computer. upon further testing with wes, we were unable to avoid creating dupes by passing properties.$id in our add_to_list calls.
However, when we reversed the order and waited to identify (which is async) until after hearing back from the sync add_to_list endpoint, no more dupes were created.
One other change involved here is the stripping of redundant traits. Should we include that here?
The remaining question is how to handle falling back to anonymousId (and whether we should at all).
Options:

remove fallback outright
make it an option
cut 2 versions and put new users on version with no anonId fallback.

@sperand-io I think we should probably save the stripping of redundant traits on a separate PR since despite that is the right way to go moving forward, probably best to just isolate this PR since it is technically "breaking" assuming Klaviyo creates custom traits for the redundant traits we are currently sending.
I though they told us to not send anonymousId? for the sake of getting this out and for least version complication, we should just make it an option and have everyone by default not fallback  but for existing users they should since that's the current behavior

👍 to everything han said.
We should have separate PRs for the different behavior changes.
A setting is better than multiple versions, because we don't have to maintain two branches that way.

Sounds great, thanks for the review! Here's the plan then to be executed in order of priority

[ ] this PR will only change ordering of requests to remove susceptibility race condition— can then go out immediately
[ ] separate PR for optionally falling back to anonymousId (set true for all existing users, off by default for new)
[ ] separate PR for stripping redundant traits

Did some testing and it looks to me like the addToList endpoint actually does everything the identify one does, so it doesn't make sense to worry about the two racing at all. We can just use identify when addToList isn't the appropriate measure. And since it's synchronous and Klaviyo locks on the primary identifier we can get away without locking on our end as well.


Ignore - Unknown tag

Is it possible to ignore "Unknown tag" messages?

I would also find that very useful, since that'd allow me to focus on one tag at the time when fixing stuff


Inspection/Evasion tasks roadmap

These are a very popular class of RL problems for spacecraft tasking. What would we need to do to enable this type of research?

[ ] Per #201, there should be an option to add non-agent satellites to the environment. They should just execute some default FSW mode and not be included in reward/observations/actions/etc.
[ ] Thruster dynamics model.
[ ] FSW for controlling thrusters via absolute thrusts, relative OE control, etc.
[ ] Action for setting thrusters + duration
[ ] Observations that are relative to passive satellite (how to link sats?)
[ ] Reward model for agent observation amount, penalty model for use of thrust
- Requires #105 to use both


Integrate Sniffs with WP-CLI

We could introduce a new WP-CLI subcommand called lint or phpcs which would apply the sniffs in this project to the WordPress install in the current scope.
I'm not sure of the best route to go as for integrating the sniffs into WP-CLI. Perhaps it would mean forking WP-CLI https://github.com/wp-cli/wp-cli and then adding this repo as a submodule dependency.

What's the status on this from the WP-CLI side? (I'm not familiar with WP-CLI)

I am a little "why bother" on this. What does running it with wpcli does which is better than... just running it?

I am a little "why bother" on this. What does running it with wpcli does which is better than... just running it?

That's a good point. If one has command-line access to run WP-CLI, then one could also use the command line to do phpcs too.
Might be some advantage to being able to have WP-CLI loop through each plugin and theme and trigger phpcs for each with any custom phpcs.xml? It's not really how many folks work though.

What does running it with wpcli does which is better than... just running it?

I don't see much utility in a WP-CLI command for running phpcs.

@westonruter do you have any additional thoughts on what merit of such command could be?

No. I don't remember why I thought this was a good idea 😄


WebPartTitle styles while in edit mode override the styles set by className.

Category
[ ] Enhancement
[X] Bug
[ ] Question
Version
Please specify what version of the library you are using: [  1.1.1       ]
Expected / Desired Behavior / Question
When switching between 'live/preview' to 'edit' mode my className styles should still prevail.
Observed Behavior
When switching between 'live/preview' to 'edit' mode the styles set by the className are overridden by the controls 'edit' style. The problem is that I may be using a background color that is close or matching the style used in edit mode making the text unreadable. Additionally, it doesn't allow you to see the true styles of the title while in edit mode.
Steps to Reproduce
Set the className for the WebPartTitle component. Start the workbench and switch between Preview and Edit mode.

Closing this issue as it is fixed in the current version.


[BUG] Play game code block returns exceptions, "No module named 'regex._regex'; "

🐛 Bug Report
Describe the bug
Exception thrown when running #Play code after running installation and model download.
How to Reproduce

Go to https://colab.research.google.com/github/nickwalton/AIDungeon/blob/master/AIDungeon_2.ipynb
Run the install block, wait for completion message
Run the download model block, wait for completion message
Run play game block
See error:

  File "play.py", line 6, in <module>
    from generator.gpt2.gpt2_generator import *
  File "/content/AIDungeon/generator/gpt2/gpt2_generator.py", line 7, in <module>
    from generator.gpt2.src import encoder, model, sample
  File "/content/AIDungeon/generator/gpt2/src/encoder.py", line 7, in <module>
    import regex as re
  File "/content/AIDungeon/venv/lib/python3.6/site-packages/regex.py", line 402, in <module>
    import _regex_core
  File "/content/AIDungeon/venv/lib/python3.6/site-packages/_regex_core.py", line 21, in <module>
    import regex._regex as _regex
ModuleNotFoundError: No module named 'regex._regex'; 'regex' is not a package```

**Additional context**
- Colab or local machine? - Colab

It's working again for me

Yikes!

I'm not getting this when I play on master. Maybe it's fixed?

Appears fixed.


problems getting dashboard addon working

inspection-report-20200427_172851.tar.gz
I am just getting started with microk8s on an Ubuntu 18.04 workstation that I'd like to do some cluster testing on.  I've done the getting started, and have moved on to the dashboard addon.   The port-forward command runs, but gives errors and nothing is serving on localhost.
What I notice is coredns is running, but not ready.  Then, at some point, hostpath-provisioner and kubernetes-dashboard start erroring or crashlooping.
I've gone through several issues at this point, doing the suggested ufw commands, and installing/re-installing,   But still the same result.
Is there any clue in the inspection report?
Example hostpath-provisioner log
F0428 00:32:15.028585       1 hostpath-provisioner.go:162] Error getting server version: Get https://<IP_ADDRESS>:443/version: dial tcp <IP_ADDRESS>:443: i/o timeout

Example kubernetes-dashboard log
panic: Get https://<IP_ADDRESS>:443/api/v1/namespaces/kube-system/secrets/kubernetes-dashboard-csrf: dial tcp <IP_ADDRESS>:443: i/o timeout

goroutine 1 [running]:
github.com/kubernetes/dashboard/src/app/backend/client/csrf.(*csrfTokenManager).init(0xc0005388c0)
	/home/travis/build/kubernetes/dashboard/src/app/backend/client/csrf/manager.go:40 +0x3b4
github.com/kubernetes/dashboard/src/app/backend/client/csrf.NewCsrfTokenManager(...)
	/home/travis/build/kubernetes/dashboard/src/app/backend/client/csrf/manager.go:65
github.com/kubernetes/dashboard/src/app/backend/client.(*clientManager).initCSRFKey(0xc0001b6080)
	/home/travis/build/kubernetes/dashboard/src/app/backend/client/manager.go:494 +0xc7
github.com/kubernetes/dashboard/src/app/backend/client.(*clientManager).init(0xc0001b6080)
	/home/travis/build/kubernetes/dashboard/src/app/backend/client/manager.go:462 +0x47
github.com/kubernetes/dashboard/src/app/backend/client.NewClientManager(...)
	/home/travis/build/kubernetes/dashboard/src/app/backend/client/manager.go:543
main.main()
	/home/travis/build/kubernetes/dashboard/src/app/backend/dashboard.go:105 +0x212

I ended up just shutting off ufw, thought that's not a great long-term solution.  When it was active, I did have cni0 open:
$ sudo ufw status
Status: active

To                         Action      From
--                         ------      ----
Anywhere on docker0        ALLOW       Anywhere
Anywhere on cni0           ALLOW       Anywhere
Anywhere on cbr0           ALLOW       Anywhere
Anywhere (v6) on docker0   ALLOW       Anywhere (v6)
Anywhere (v6) on cni0      ALLOW       Anywhere (v6)
Anywhere (v6) on cbr0      ALLOW       Anywhere (v6)

Anywhere                   ALLOW OUT   Anywhere on cni0
Anywhere                   ALLOW OUT   Anywhere on cbr0
Anywhere (v6)              ALLOW OUT   Anywhere (v6) on cni0
Anywhere (v6)              ALLOW OUT   Anywhere (v6) on cbr0


[add] swift sekelton for cli

Add a template for making cli swift apps. Follows Swift(2.1) project convention.

At the moment can't validate whether this file works with progeny, am having odd issue. @shaggytwodope said he'll look into it later.

File an issue report on the problem you were experiencing (on the progeny repo). This works in my local installation.


Migrate existing PR

The PR at https://github.com/0xTim/Vapor-PasskeyDemo/pull/1 should be migrated over to this library as is was missed (I believe) in the original port

Already done


BUG: scipy.optimize.linprog exits the entire program without raising an assertion that can be catched

Describe your issue.
There are inputs (like the one shown in the example below) that make scipy.optimize.linprog exit on a failed c++ assertion.
This exception cannot be catched in python. The program just exits.
Reproducing Code Example
from scipy.optimize import linprog
import numpy as np

objective = np.array([1, 1,  0, -1,  0,  0])
a_opt = np.array(
[[-1, -1,  1,  0,  0,  0],
 [-1, -1,  0,  1,  0,  0],
 [ 0,  0,  0, -1,  4,  1],
 [ 0,  1,  0, -1,  4,  0],
 [ 1,  0,  0,  0,  0,  0],
 [ 0,  0,  0,  0,  5,  0]])
b_opt = np.array(
[0, 1, 0, 0, 2, 1])

res = linprog(c=objective, A_ub=a_opt, b_ub=b_opt, bounds=(None, None))

Error message
Assertion failed: model_status_ == HighsModelStatus::kInfeasible || model_status_ == HighsModelStatus::kUnbounded, file ../../scipy/_lib/highs/src/lp_data/Highs.cpp, line 1125

SciPy/NumPy/Python version information
1.9.3 1.23.5 sys.version_info(major=3, minor=10, micro=0, releaselevel='final', serial=0)

Thanks for reporting the issue, @iincer.  I have confirmed the crash, and also confirmed that the problem has been fixed by https://github.com/scipy/scipy/pull/17430, so I'm closing the issue.  The fix will be in SciPy 1.10.0.

Perfect. Thank you, @WarrenWeckesser.


Fixed Tutorial steps missing at first launch

Fixes  #1089 issue
@llaske pls review it

I don't understand your fix.
The issue is related to the home view, not to the first screen.
Have a look on the video included with https://github.com/llaske/sugarizer/issues/1089

I don't understand your fix. The issue is related to the home view, not to the first screen. Have a look on the video included with #1089

I'm still exploring codebase my fix might be wrong, but does it fix the issue pls check it on your side.

@llaske Here is the explanation why my fix works :

For some reason the firstscreen gets rendered twice on the screen
1st time on FirstLaunch
2nd just before the homeview
Hence, launching 2nd tutorial when the view is actually homeview  (since, we're calling tutorial once view gets rendered)
now, all elements related to the view will get added in tutorial steps.
then the tutorial steps will  filter out steps if the element is not present on screen.

What about the tutorial launch once homeview being created

It'll get cancelled bcoz of this line :
https://github.com/llaske/sugarizer/blob/f766b334a6f59af0bb04319b1171e4c89195e887/lib/tutorial.js#L508

Solution : Start the tutorial at the end while creating firstscreen (just like it is in homeview)

Thanks 👍


slack: convert names with spaces

The original Slack IRC bridge converted user names with spaces in them to use underscores, to comply with the IRC protocol. mm-go-irckit instead sends messages where usernames and hostmasks can contain spaces in violation of the IRC protocol's formal grammar. It should convert them transparently the same way Slack's IRC bridge did.

This was fixed by 666f9185fb9118b724813f9d334423dc6a388922.


Question regarding resource pack handling (1.18.2 if relevant)

Hi, I am using this mod in a modpack on 1.18.2 Forge.
My modpack utilizes resource packs and replaces entities, too.
Is there an incompatibility with those, or does it recognize resource packs and replace the entity models accordingly?
Would love to get some insight. Thanks in advance.

There shouldn't be any issues with resource packs that just replace entity textures. Since custom entity models (like those from OptiFine) aren't a vanilla feature, I haven't done any testing with them, but I believe that they should still work fine in most cases.
The only exception that I can think of would be stuff that uses GeckoLib, as entities that use GeckoLib usually have issues with their animations when placed on trophies. That's why the default resouce pack currently doesn't have any trophies for mods that use it.

Thanks, I see. I am utilizing EMF which is basically a port of CEM but for Oculus. Some mods I use have GeckoLib, so I will see what happens, in some time. I'll return if I find anything noteworthy to report, albeit it will take a lot of time until I could properly test everything. Let's hope for the best I'd say.


Tensorflow_gpu crash on windows server 2012(Nightly build Feb 15, 2017 1:25:00 AM)

Environment info
Operating System:
Windows Server 2012 R2
GPU GeForce GTX 750 and 1080 in two windows server workstation.
Installed version of CUDA and cuDNN:
CUDA 8.0 installed
CuDNN download, and set path to system environment variable %PATH%
If installed from binary pip package, provide:
I install the fallow whl: "tensorflow_gpu-1.0.0rc2-cp35-cp35m-win_amd64.whl"
http://ci.tensorflow.org/view/Nightly/job/nightly-win/85/DEVICE=gpu,OS=windows/artifact/cmake_build/tf_python/dist/tensorflow_gpu-1.0.0rc2-cp35-cp35m-win_amd64.whl
I only import tensorflow, and it's crash.
(venv64) C:\work\keras01>python
Python 3.5.2 (v3.5.2:4def2a2901a5, Jun 25 2016, 22:18:55) [MSC v.1900 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import tensorflow as tf
Traceback (most recent call last):
  File "C:\work\keras01\venv64\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 18, i
n swig_import_helper
    return importlib.import_module(mname)
  File "C:\work\keras01\venv64\lib\importlib\__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 986, in _gcd_import
  File "<frozen importlib._bootstrap>", line 969, in _find_and_load
  File "<frozen importlib._bootstrap>", line 958, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 666, in _load_unlocked
  File "<frozen importlib._bootstrap>", line 577, in module_from_spec
  File "<frozen importlib._bootstrap_external>", line 906, in create_module
  File "<frozen importlib._bootstrap>", line 222, in _call_with_frames_removed
ImportError: DLL load failed: The specified module could not be found.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "C:\work\keras01\venv64\lib\site-packages\tensorflow\python\__init__.py", line 66, in <module
>
    from tensorflow.python import pywrap_tensorflow
  File "C:\work\keras01\venv64\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 21, i
n <module>
    _pywrap_tensorflow = swig_import_helper()
  File "C:\work\keras01\venv64\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 20, i
n swig_import_helper
    return importlib.import_module('_pywrap_tensorflow')
  File "C:\work\keras01\venv64\lib\importlib\__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
ImportError: No module named '_pywrap_tensorflow'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\work\keras01\venv64\lib\site-packages\tensorflow\__init__.py", line 24, in <module>
    from tensorflow.python import *
  File "C:\work\keras01\venv64\lib\site-packages\tensorflow\python\__init__.py", line 72, in <module
>
    raise ImportError(msg)
ImportError: Traceback (most recent call last):
  File "C:\work\keras01\venv64\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 18, i
n swig_import_helper
    return importlib.import_module(mname)
  File "C:\work\keras01\venv64\lib\importlib\__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 986, in _gcd_import
  File "<frozen importlib._bootstrap>", line 969, in _find_and_load
  File "<frozen importlib._bootstrap>", line 958, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 666, in _load_unlocked
  File "<frozen importlib._bootstrap>", line 577, in module_from_spec
  File "<frozen importlib._bootstrap_external>", line 906, in create_module
  File "<frozen importlib._bootstrap>", line 222, in _call_with_frames_removed
ImportError: DLL load failed: The specified module could not be found.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "C:\work\keras01\venv64\lib\site-packages\tensorflow\python\__init__.py", line 66, in <module
>
    from tensorflow.python import pywrap_tensorflow
  File "C:\work\keras01\venv64\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 21, i
n <module>
    _pywrap_tensorflow = swig_import_helper()
  File "C:\work\keras01\venv64\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 20, i
n swig_import_helper
    return importlib.import_module('_pywrap_tensorflow')
  File "C:\work\keras01\venv64\lib\importlib\__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
ImportError: No module named '_pywrap_tensorflow'

Failed to load the native TensorFlow runtime.

See https://github.com/tensorflow/tensorflow/blob/master/tensorflow/g3doc/get_started/os_setup.md#im
port_error

for some common reasons and solutions.  Include the entire stack trace
above this error message when asking for help.

Have you followed the advice given regarding the _pywrap_tensorflow problem. Check out: https://stackoverflow.com/q/42011070
It looks like you are missing a DLL. However, I had this problem not long ago even though I did have the DLL. Just ensure that your environment variables are set correctly. If playing around with that doesn't work i'll try and remember the other troubleshooting steps I did to fix it. (As it was a bit tricky to solve!)

Thanks your reply, my computer has already install visual studio 2015 and also install 2015 Redistributable
in previous version of tensorflow_gpu 0.12.x don't have this issue.

Can you try installing via pip install tensorflow-gpu
I seem to recall that the nightly build I used threw this error even though the DLL was there and set up correctly. Installing straight from pip solved this issue. Uninstall the current version first.

Sometimes pip may misbehave on windows.
Could you try this command out with an administrator terminal:
pip install --ignore-installed --upgrade tensorflow-gpu

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!

Though I am able to import tensorflow, I am not able to create a session and test it.
It gives below error.
(tensorflow-gpu) C:\Users\ssheshap>python
Python 3.5.2 |Continuum Analytics, Inc.| (default, Jul  5 2016, 11:41:13) [MSC v
.1900 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.

import tensorflow as tf
tf.Session()
2017-11-16 19:52:51.979025: W c:\l\work\tensorflow-1.1.0\tensorflow\core\platfor
m\cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use SSE ins
tructions, but these are available on your machine and could speed up CPU comput
ations.
2017-11-16 19:52:51.979894: W c:\l\work\tensorflow-1.1.0\tensorflow\core\platfor
m\cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use SSE2 in
structions, but these are available on your machine and could speed up CPU compu
tations.
2017-11-16 19:52:51.980766: W c:\l\work\tensorflow-1.1.0\tensorflow\core\platfor
m\cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use SSE3 in
structions, but these are available on your machine and could speed up CPU compu
tations.
2017-11-16 19:52:51.981665: W c:\l\work\tensorflow-1.1.0\tensorflow\core\platfor
m\cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use SSE4.1
instructions, but these are available on your machine and could speed up CPU com
putations.
2017-11-16 19:52:51.982829: W c:\l\work\tensorflow-1.1.0\tensorflow\core\platfor
m\cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use SSE4.2
instructions, but these are available on your machine and could speed up CPU com
putations.
2017-11-16 19:52:51.984912: W c:\l\work\tensorflow-1.1.0\tensorflow\core\platfor
m\cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use AVX ins
tructions, but these are available on your machine and could speed up CPU comput
ations.
2017-11-16 19:52:52.026790: E c:\l\work\tensorflow-1.1.0\tensorflow\stream_execu
tor\cuda\cuda_driver.cc:405] failed call to cuInit: CUDA_ERROR_UNKNOWN
2017-11-16 19:52:52.031638: I c:\l\work\tensorflow-1.1.0\tensorflow\stream_execu
tor\cuda\cuda_diagnostics.cc:158] retrieving CUDA diagnostic information for hos
t: WIN-0LVFD08FG54
2017-11-16 19:52:52.033113: I c:\l\work\tensorflow-1.1.0\tensorflow\stream_execu
tor\cuda\cuda_diagnostics.cc:165] hostname: WIN-0LVFD08FG54
<tensorflow.python.client.session.Session object at 0x0000000C22A2B358>
exit()

Please help.
Thanks


[e2e]: delegator-lowperms release main default slsa3

Repo: https://github.com/slsa-framework/example-package/tree/v43.0.360
Run: https://github.com/slsa-framework/example-package/actions/runs/9753907233
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.delegator-lowperms.release.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.delegator-lowperms.release.main.default.slsa3.yml
Trigger: release
Branch: v43.0.360
Date: Tue Jul  2 02:51:39 UTC 2024

Repo: https://github.com/slsa-framework/example-package/tree/v43.0.361
Run: https://github.com/slsa-framework/example-package/actions/runs/9770781902
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.delegator-lowperms.release.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.delegator-lowperms.release.main.default.slsa3.yml
Trigger: release
Branch: v43.0.361
Date: Wed Jul  3 02:47:15 UTC 2024
Tests are passing now. Closing this issue.


Enh raster fillin callback

attn @klauer I know this is replicating code you have, but I was not sure where that was.

There is a slight amount of overlap, but as mine includes the ugliness of grabbing preview data from an in-progress HXN fly-scan, it's something that doesn't belong anywhere near the bluesky repo.

demo still not working quite right
from bluesky.examples import Mover, SynGauss
from bluesky.spec_api import mesh
from bluesky.callbacks import LiveRaster
from bluesky.standard_config import gs

Mover = bluesky.examples.Mover
SynGauss = bluesky.examples.SynGauss
Syn2DGauss = bluesky.examples.Syn2DGauss

theta = Mover('theta', ['th'])
gamma = Mover('gamma', ['ga'])
delta = Mover('delta', ['dt'])
theta_det = SynGauss('theta_det', theta, 'th', center=0, Imax=1, sigma=1)
gamma_det = SynGauss('gamma_det', gamma, 'ga', center=0, Imax=1, sigma=1)
delta_det = SynGauss('delta_det', delta, 'dt', center=0, Imax=1, sigma=1)

tgd = Syn2DGauss('theta_gamma_det', theta, 'th', gamma, 'ga',
                 center=(0, 0), Imax=1)

cb = bluesky.callbacks.LiveRaster((25, 20),  'theta_gamma_det', clim=[0, 1])
gs.DETS = [theta_det, gamma_det, tgd]
mesh(theta, -2, 2, 25, gamma, -2, 2, 20, False, subs=cb)

@danielballan this is ready to go


Added a menu-item to choose current renderer

I propose to add an Edit -> Choose Renderer menu item that triggers a pure Fiji plugin (so that necessary GUI params harvesting can take place). The Fiji plugin needs to get SciView service injected and than it works on its own. Specifically:

when started, it considers given SciView context (actually, the current scenery.Renderer) and populates the choice box with available options

avail options when OpenGL backend is the current one are:

OpenGL to force the scenery to restart this backend
if scenery.Renderer.Device is available, the second menu item contains it,
else the second menu item says "Vulkan: first available renderer"

avail options when Vulkan backend is the current one are:

list of discovered vulkan devices, one by one, allowing user to switch among available vulkan devices (which is what Vlado needs now)
OpenGL to switch back to OpenGL backend

Essentially the idea is that user touches this menu item only when she is not satisfied with the choice of renderer that scenery has made. If you end up in OpenGL, you can switch to Vulkan. If after that your not happy with the currently chosen Vulkan, you can alter it. And vice versa.
Since the choice is a standard scijava plugin, the last choice is memorized (which becomes useful if switching again from OpenGL back to vulkan, it can go to the last chosen vulkan device directly).

The PR however assumes that world is perfect and that switching between renders work well....
on my current machine works OpenGL backend, and 2nd from two avail vulkan devices.
Scenery, with no startup params, choses the first vulkan -- which freezes it. So I don't get a chance to actually swtich to anything else.

yep, builds must be failing... it requires the complementary feature in scenery

I've extended  Edit -> Choose Renderer (and underlying RenderingDeviceChooser class) dialog to provide:

a static method RenderingDeviceChooser.setupSystemProperties() that can setup the system properties (e.g. scenery.Renderer) based on the choice a user has made previously via the dialog (provided no system properties overriding would take place)
two entry points into SciView now check what was the last choice of the Choose Renderer dialog and trigger the .setupSystemProperties() method to make sure sciview starts with the previously-chosen renderer again
a menu item "SciView decides itself at start up" was added into the dialog; this way, one can unset whatever her choice was before (because this option renders .setupSystemProperties() into a do-nothing call)

I don't intend to add stuff to this anymore -- I'm happy with it now, unless you request anything
Yes, this PR still requires the PR in the scenery to be processed first

sorry, was cleaning up branches in my fork... accidentally removed this one...restored and reopened

closing because OpenGLRenderer is about to be deprecated


Remove webpack requirement, use tsc to build

This removes webpack and uses tsc to build.
The only functional change is that the .d.ts files are generated on all builds, not just production builds. This shouldn't be an issue though, they're ignored from source control.
The start command has also been removed - there isn't a watch function in the raw TypeScript compiler.

:tada: This PR is included in version 2.2.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add async mode

The new 1.2.3 release calls proc.wait(), which makes this synchronous. An async option should be provided.

We will need a change from the syntax proposed in #7. Specifically, attributes cannot be in square brackets, since we now allow custom formatters. Specifically, consider this:
a = 1
custom_formatters[a]`cat {file}`

should not treat a as an attribute. Instead, the new regex is
(?:([a-zA-Z0-9_.]+)(?:\.(foreach)(\([a-zA-Z0-9_]+(?:,\s*?[a-zA-Z0-9_]+)*\))(?::([^`]*))?)?)?({[a-z]+})?`

which moves the attributes to the very end of the regex, and uses curly braces, which are not valid Python. As such, the following are valid:
{a}`long_process`
list{b}`cat {file}`

where b might be a future attribute. Note that if we use async, we need to ensure that no formatter is used.


Changed star to "like" while maintaining circular style

Fix for https://github.com/Swati4star/NSIT-Connect/issues/41

@HEATH3N Can you rename the icons and rather change in xml file?
I dont want a file name star to contain a like icon.

Thank you @HEATH3N
Merged :+1:


Fix many=False Issue #12

pass the "many" variable on javascript file. when many=False the results drop the previous selected in $chips else append the new option.

Hi, thanks for your contribution! It seems like the tests are failing due to unexpected count of items, can you please check it?
Also, please remove the commented code lines.
Thanks!


Import Cost Question

Hi,

Was just wondering if the newer version have optimized the import cost for all components? Usual sizes would go above 400k and larger than usual ? Thank you~
The pictures are Typography comparison from mui/material versus our material-tailwind/react

import { Typography } from '@material-tailwind/react/components/Typography' // 226k
import like this. 👍🏻


Add Contribution Doc & GitHub Templates

Changes
Summary of changes included in this PR:

[X] Create Contributing markdown file with info on building/contributing
[X] Create GitHub PR Template
[X] Create GitHub Issue Templates
[X] Updated Readme with Contribution info

Issues Completed

Closes #15

CC - @traviskosarek @calebcartwright

Coverage remained the same at 100.0% when pulling b0771370311f88d5decd869bb5f038486ffea1b8 on docs into c4e0d2874c689ebbb4e94abc111155d8787e920c on master.


add apt repo in enable-repo playbook

What type of PR is this?
/kind feature
What this PR does / why we need it:
add apt repo in enable-repo playbook, fixed Ubuntu offline installation issue.
Which issue(s) this PR fixes:

Fixes https://github.com/kubean-io/kubean/issues/388
Special notes for your reviewer:

After importing the os-package and iso to minio, verify the enable-repo playbook with the following command:
# Set the apt repo configuration
$ ansible-playbook -i hosts-ubuntu.yml -b --become-user root enable-repo.yml \
    -e"{repo_list: ['deb [trusted=yes] http://<IP_ADDRESS>:9000/kubean/ubuntu/amd64 focal/', 'deb [trusted=yes] http://<IP_ADDRESS>:9000/kubean/ubuntu-iso focal main restricted']}"

# Restore the apt repo configuration
$ ansible-playbook -i hosts-ubuntu.yml -b --become-user root -e undo=true enable-repo.yml

https://github.com/ErikJiang/kubean/actions/runs/3853299162/jobs/6566160992

/lgtm

/lgtm


Passing an instance of Pathlib.Path to CliRunner.invoke() gets a TypeError

Consider the following example:
from pathlib import Path

import click
from click.testing import CliRunner

@click.command()
@click.argument("DIRNAME")
def print_directory(dirname):
    print(dirname)

runner = CliRunner()
print(runner.invoke(print_directory, ["example"]))
print(runner.invoke(print_directory, [Path("example")]))

With Python 3.6.4 and click 7.0, this is the output you get:
<Result okay>
<Result TypeError("object of type 'PosixPath' has no len()",)>

This can be easily fixed by wrapping in str(), i.e.,
print(runner.invoke(print_directory, [str(Path("example"))]))

But in general I've got used to the implicit contract (at least in the standard library) that I can use a Path object anywhere I can use a string representation of a path. Since paths are often the arguments to command-line tools, it'd be nice to be able to use a Path object here as well.
Alternatively, raise a more helpful TypeError if you pass a non-string object into invoke() – I spent a while poking around in my code to see where I was calling len() on the wrong object before realising it was actually coming from click.

This code change would solve your problem in core.BaseCommand.main:
        if args is None:
            args = get_os_args()
        else:
            args = [arg.__fspath__() if hasattr(arg, "__fspath__") else arg
                    for arg in args]

However, that would still not make support for other types work, such as integers. In general, the parser assumes all inputs are strings. To properly handle this, a pre-processing step would have to be done to determine which arguments on the command line are associated with which types. However, this adds a lot of complexity, since command line arguments like dashes need to be handled. This would be a huge overhaul to the parser and is probably not a good idea.
I suggest closing this issue and just using the noted workaround.

I think it’s reasonable to say “we can’t coerce all arguments to strings”, but the confusing error message remains.
Would it be possible to add a check something like:
if not all(isinstance(arg, string_types) for arg in args):
    raise TypeError(“args must all be strings; got %r”)

to make it more obvious when somebody has made this mistake?

If the documentation is unclear that the invoke-related APIs expect strings, that's something I'm ok with changing, but otherwise I think this is just expected behavior. In general, and I know this is somewhat contradictory given that Click does a bunch of validation, Python shouldn't be checking the types of arguments.


Custom Monsters that share models

Hi,
I have an issue with custom monsters not being identified with a symbol when there are multiple.
For example I have a custom monster that uses the "MonsterBase" catagory, and a "Zombie" monster, however when they both spawn at the same time, neither has an identifier to tell them apart:

Have I missed something here, or is it a bug?

This is a bug, need to think about this one.

Hi @scrubbless,
can you please provide a small scenario to reproduce that problem ?

Example Scenario Attached.
ExampleCustomM.zip

Thank you for the example, it is now fixed.

Will be available in release 2.1


djvulibre: doesn't build

The new release of djvulibre doesn't appear to come with a configure file, so autogen.sh needs to be run in the build.
@ectlunya
@git-bruh

Changed on official community pull request

Changed on official community pull request

@ectlunya djvulibre has a wrong checksum pushed.

@ectlunya djvulibre has a wrong checksum pushed.


Android: Adjustable Text Size for Lists

The range of DPI values we're seeing in the wild across devices is getting wider and wider, making it hard to pick a correct font size for feed lists and story lists and story metadata.  We've now got compelling examples where our chosen size is both too small to read and too large to fit much on screen.  We should consider adding a text size chooser for the rest of the app, not just story reading.

Yes I agree. I have two sizers for web and iOS: feed and story list, and story detail.

done in #930


No more messages recieved on firebase google console with 9.4.2 upgrade, fixed rolling back 9.4.1

[REQUIRED] Step 2: Describe your environment

Operating System version: ubutu
Firebase SDK version: 9.4.2
Library version: sorry dunno
Firebase Product: FCM

[REQUIRED] Step 3: Describe the problem
No more messages recieved on FCM console
Steps to reproduce:
Don't know, upgrade to 9.4.2 ?

And no exception or no logs of any kind found about that

Hey @pierre-gautier, I am not able to reproduce this on my side. Since you are not seeing any errors it suggests that this may not be related to #1045 either. Could you check your dashboard again to confirm that it was not a temporary issue. Also could you provide us with a bit more information on the type of messages you are sending to help us narrow down the cause.

Sorry for my late answer.
After downgrading to 9.4.1 the notifications came back instantly.
We didn't find any trace in our datadog but since we rolled back pretty quick without deep investigation I can't swear it isn't related to #1045 which actually seems pretty likely.
Thanks for the investigation


Enable choice between icon first or last in fabs

The extended fab in MDC for web let you decide whether the label or the icon comes first based on ordering of elements (https://github.com/material-components/material-components-web/tree/master/packages/mdc-fab). The React version isn't quite so flexible.
In this version, I have introduced a new property called iconFirst that defaults to true, which is currently the default behaviour.
When iconFirst is false, the text label span is written first, and so appears in front of the icon.

I signed it!

Codecov Report

Merging #904 into master will decrease coverage by 0.05%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master     #904      +/-   ##
==========================================
- Coverage   93.99%   93.94%   -0.06%
==========================================
  Files          85       85
  Lines        3599     3602       +3
  Branches      568      571       +3
==========================================
+ Hits         3383     3384       +1
  Misses         90       90
- Partials      126      128       +2

Impacted Files
Coverage Δ

packages/fab/index.tsx
86.66% <33.33%> (-5.93%)
:arrow_down:

Continue to review full report at Codecov.

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

How do we get a reviewer?

taking a look

:ping_pong: @ndtreviv

Yep - sorry! I made those changes but the tests don't pass:
Chrome 74.0.3729 (Mac OS X 10.14.5) Fab i tag is rendered with mdc-fab__icon class FAILED
Chrome 74.0.3729 (Mac OS X 10.14.5) Fab span tag is rendered with mdc-fab__icon class FAILED
Chrome 74.0.3729 (Mac OS X 10.14.5) Fab a tag is rendered with mdc-fab__icon class FAILED
	Error: Method “hasClass” is meant to be run on 1 node. 2 found instead.

Too wiped to resolve it today. I'll get on it tomorrow.

sounds good thanks! Please ping me again when you're ready to go


[14.0.x] ISPN-14902 Exception is thrown while rebuilding an index

https://issues.redhat.com/browse/ISPN-14902
https://issues.redhat.com/browse/JDG-6256
Backport of https://github.com/infinispan/infinispan/pull/10992

Looks like at least 2 related test failures :( https://ci.infinispan.org/job/Infinispan/job/PR-10991/4/

Looks like at least 2 related test failures :( https://ci.infinispan.org/job/Infinispan/job/PR-10991/4/

I converted it to draft. I think that we need much time to solve the issue properly.

Ci looks good.

Thanks @wburns

Thanks @ryanemerson for merging it


lets make it easy to switch between http/https and generate signed certs when on kubernetes

switch to latest ingress controller
add kube-lego app
exposecontroller config to toggle protocols

[merge]


Relationships not displaying in Prisma Studio when using direct SQL insertion with SQLx

Bug description
I'm encountering an issue with many-to-many relationships not appearing in Prisma Studio when inserting data directly into the database using SQLx in Rust.
I have a database set up with prisma for many-to-many relationships (models: Track, Artist, TrackToArtist), but when I insert data into the database using sqlx in rust:

insert track
insert artist
insert tracktoartist
it does get inserted, but the relationship isn't showing in the prisma studio.

how can I fix this?
How to reproduce

Setup postgresql database with prisma for many-to-many relationships
Setup a rust project with sqlx
insert the data using sqlx

Expected behavior
The relationship should be shown in prisma studio
Prisma information

model Track {
  album             Album           @relation(fields: [album_id], references: [id])
  artists           Artist[]
  available_markets String[]
  disc_number       Int
  duration_ms       Int
  explicit          Boolean
  external_ids      Json
  external_urls     Json
  href              String
  id                String          @id
  name              String
  popularity        Int
  preview_url       String?
  track_number      Int
  type              String
  uri               String
  album_id          String
  History           History[]
  Scrobble          Scrobble[]
  TrackToArtist     TrackToArtist[]
}

model TrackToArtist {
  Track     Track  @relation(fields: [track_id], references: [id])
  Artist    Artist @relation(fields: [artist_id], references: [id])
  track_id  String
  artist_id String

  @@id([track_id, artist_id])
}

model Artist {
  external_urls Json
  href          String
  id            String          @id
  name          String
  type          String
  uri           String
  tracks        Track[]
  albums        Album[]
  TrackToArtist TrackToArtist[]
  AlbumToArtist AlbumToArtist[]
}

Environment & setup

OS: Windows 11
Database: PostgreSQL
Node.js version: v22.11.0

Prisma Version
prisma                  : 6.0.1
@prisma/client          : 6.0.1
Computed binaryTarget   : windows
Operating System        : win32
Architecture            : x64
Node.js                 : v22.11.0
Query Engine (Node-API) : libquery-engine 5dbef10bdbfb579e07d35cc85fb1518d357cb99e (at<EMAIL_ADDRESS>Schema Engine           : schema-engine-cli 5dbef10bdbfb579e07d35cc85fb1518d357cb99e (at<EMAIL_ADDRESS>Schema Wasm             : @prisma/prisma-schema-wasm 5.23.0-27.5dbef10bdbfb579e07d35cc85fb1518d357cb99e
Default Engines Hash    : 5dbef10bdbfb579e07d35cc85fb1518d357cb99e
Studio                  : 0.503.0

const tracks = await prisma.track.findMany({
	include: {
		artists: true,
	},
});

Also returns an empty artists array.


Refactor tests using Theory

While doing major changes (eg changing from Sprache to Superpower) I had some trouble with testing, because the current Fact-Methods do fail at the first failing case. That way it is hard to check whether you fixed something and just broke a different case, or if it does not fix the intended case at all.
Additionally you always have to care about whether the testcases within a method are really "the same", or if for example some check "AtEnd" and some do not.
That's where Theories come into action.
They reduce many of the test-methods to one-liners, which prevents you from merging different aspects of testing.
Instead it enforces to have a precise test-statements which should be tested with different inputs.
Additionally it reports each single failing input instead of just failing at the first of them.
Is that something you would support?
It is not finished, and of course some scenarios have to be handled differently, we should not parse an entire file for each single test for example.
It's not really possible to check that request via git-compare at the moment, too many changes which are not recognized in the correct order.
If there is an interest in having this, I'd split this commit into single method-change-commits to enable a proper review.
And it is of course still a draft, it is not even fully finished for ParserTests.

You don't need to switch to Theory to do what you are doing. It is very doable with XUnit to pass arguments to functions for various tests.
The problem with this change is about setup and teardown (ctor + dispose). If you separate the tests into multiple 1 liners, you run setup and teardown for each and every one of those, because that's the whole point, and that can add significant slowdowns to test running. In this set of tests, perhaps it won't be too noticeable, but there is a real chance that these changes could 3x-10x test runtime.
So, with these changes I would be concerned about showing things down annoyingly, and also unnecessarily dripping XUnit.
If you can address/justify those concerns, I would be open to this continuing.

You don't need to switch to Theory to do what you are doing. It is very doable with XUnit to pass arguments to functions for various tests.

But with calling methods you won't get individual errors, since the test still fails at the first error.

The problem with this change is about setup and teardown (ctor + dispose). If you separate the tests into multiple 1 liners, you run setup and teardown for each and every one of those, because that's the whole point, and that can add significant slowdowns to test running. In this set of tests, perhaps it won't be too noticeable, but there is a real chance that these changes could 3x-10x test runtime.

Sure, as I already stated we should not parse entire files for each individual test for example.
There such expensive setup-actions are required, I need to find a solution addressing this concern, or the tests just stay as they were.
The huge advantage of individual errors is mainly about the direct (small) Parser tests, where you can directly see which cases are problematic and which are still green.

So, with these changes I would be concerned about showing things down annoyingly, and also unnecessarily dropping XUnit.
If you can address/justify those concerns, I would be open to this continuing.

Does this address the concerns so far?
If your concern is only about performance, I guess there is no reason to stop until we see performance issues.
And if so, we can just leave these tests untouched.

Small addition:
I found some explanation about SharedContext in X-Unit, which could solve performance issues: https://xunit.net/docs/shared-context
I need to read through, but my first impression ist, that this can address all performance issues.

Actually it turns out what you are doing is the XUnit way of doing this, I think I was thinking of nunit's approach, or some other framework.
So I support the path you are on. For sure this makes things easier to isolate for failures. And we can check performance at the end.
Thanks!

Splitted to single commits for each changed method now, that way a review should be possible.
Updating the other methods will follow soon.

Ready with ParserTests, but searching for a better solution for ParseDotenvFile()-refactoring.
I think it has the advantage of separate failing, but it is not a really nice solution so far...
But at least the refactoring of this class can be reviewed, then I can use that feedback for refactoring the next classes.

Found the solution for ParseDotenvFileShouldParseContents:
Using MemberData combined with TheoryData removes all these dictionaryRefs.
The solution is typesafe and you cannot miss any tests, compared with the loosely coupled DictionaryIndexing with InlineData.

While refactoring LoadOptionsTests I realized one real trade-off:
There is no good solution for showing which InlineData or TheoryData is under tests (no link back to the code-line).
While that is mostly acceptable for most tests, there are cases (mainly with TheoryData I think) where this is quite inconvenient.
With LoadOptions it was quite horrible, because it ist nearly impossible to see from the shown data (checking the compiled code is not a good way I guess :D ).
That's why I added a workaround, adding a new Type IndexedTheoryData.
There are some tradeoffs, the compiler cannot create exceptions for wrong parameters on the test method for example, but still the whole thing is typesafe etc. so that should not be a real problem (you get well explaining exceptions if you have wrong parameters).

I just had a look through the other TestClasses now:
HelperTests and EnvReaderTests seem hard to rewrite, and 'm not sure how valuable this is.

I think this functionality is quite stable, there won't happen much
the focus should be on IConfiguration, which is the preferred way of ConfigurationData-Usage imho (IConfiguration allows to deserialize data into classes directly, no "manual" conversions)
all these methods do not work if you do not use SetEnvVars, which is a huge restriction (and an additional reason to use IConfiguration instead)

I would postpone EnvTests and EnvConfigurationTests, because there are potentially huge simplifications when working on #105 (decoupling Parsers and EnvVars) first.
What do you think so far?
I'm not completely happy with some of the code-constructs now, but in general I think it's  a good direction.

Hey sorry for the delay, I did not realize you were waiting on me for next steps. Honestly, this is hard for me to confirm that you retained all existing tests correctly, but if this all passes, it looks broadly correct, and I do like the improvements. I would in fact encourage not doing more than this at this time and instead stop here and call it done, if indeed you are done with these changes.
So, I am fine with merging as-is if you are complete on this set of changes.

Not much time at the moment, and I'm still thinking about better solutions regarding some of the drawbacks...
Checking the PR is only possible commit by commit.
In general I just took the tests as-is and have them only re-written, but there are some special cases with changes or even completely gone.
I will comment those things as soon as I find some time again.


Display form types other than text or checkbox

Do you have any advice on how to modify the tree to display/edit (for example) a date picker instead of just text. I would like to fork this and use to display a dynamic form (whose attributes exist in the node to determine a form component to display such as a date picker or dropdown, radio button ...).  I think the NodeContent.vue is where I want to make the changes to support this  ?

There is an example...
https://jsfiddle.net/amsik/6jm2b1dq/?utm_source=website&utm_medium=embed&utm_campaign=6jm2b1dq
You don't change NodeContent... Define the data as described in the doc an add Your individual data in the data property.  In Your the template which is embedded in a scoped slot You can do what You want with the node data even pass them to other components.
Here the snippet from the fiddle above
`

          <template v-else>
            <i :class="[node.expanded() ? 'ion-android-folder-open' : 'ion-android-folder']"></i>
            {{ node.text }}
          </template>
        </span>
      </tree>`


[GLUTEN-4889][VL] Support approx_percentile

What changes were proposed in this pull request?
Support approx_percentile for VL backend
(Fixes: #4889)
How was this patch tested?
UT

CC: @zhztheplayer

I believe the change on Gluten side is ready now. But the ApproxPercentileAggregate implements in velox has to make some minor modification, becasue when add intermediate data, it checks the row vector encoding in addIntermediateImpl
https://github.com/facebookincubator/velox/blob/main/velox/functions/prestosql/aggregates/ApproxPercentileAggregate.cpp#L652 ,
I believe the encoding is guaranteed by extractAccumulators https://github.com/facebookincubator/velox/blob/main/velox/functions/prestosql/aggregates/ApproxPercentileAggregate.cpp#L262.
But when it comes in Gluten, the output of PartialAgg are processed by Shuffle, which will flatten all vectors, so that the encoding is not guaranteed when added to FinalAgg.
https://github.com/apache/incubator-gluten/blob/main/cpp/velox/shuffle/VeloxShuffleWriter.cc#L343
cc @zhztheplayer @liujiayi771 @ulysses-you

Thank you in advance!

I believe the change on Gluten side is ready now.

Did that mean this PR can be merged prior to merging Velox changes?

@WangGuangxin Perhaps we could create an issue in the upstream Velox community to remove this verification.

Hi @WangGuangxin so the current plan is to merge https://github.com/WangGuangxin/gluten/commit/97db869a56e8d42c0782e4e8218e1937e40682ce to upstream Velox before this ? Am I understanding correctly?

this

yes, we should merge to upsteam first before merge this PR

@WangGuangxin Any progress of this PR?

@WangGuangxin Thanks for supporting this! Any new process abt this pr?

@WangGuangxin Thanks for supporting this! Any new process abt this pr?

@boneanxs Hi, I'll do rebase today since main branch changed a lot

@WangGuangxin, could you let us know this work's state and the blocking issues? If you have no bandwidth, I can help cover it. Thanks!

@WangGuangxin, could you let us know this work's state and the blocking issues? If you have no bandwidth, I can help cover it. Thanks!

@PHILO-HE Sorry for this. Before this I have a PR to velox and I'll rebase it and send out today.


A bunch of unexpected behaviour

Sorry for the not very specific title, but having a few issues!
Firstly, the exhaustive setting does not appear to be working as expected. Despite having this enabled for each category (features, fixes, tests) on a category label plus a product label, features are appearing in the fixes list, etc.
Secondly, date ranges between tags don't seem to be resolving properly. Despite finding the right tag range (@neo4j/graphql@2.4.0...@neo4j/graphql@2.5.0), the date range that it infers from that (2021-11-01T10:03:42.000Z to 2021-12-01T13:39:00.000Z), as the previous tag was made 22 days ago, so just over a week over what it's some up with as the start date.
Any ideas on the above? Thanks for any help!
Full log below:
Run mikepenz/release-changelog-builder-action@v1
  with:
    configuration: config/release-changelog-builder-action/configuration-graphql.json
    toTag<EMAIL_ADDRESS>    ignorePreReleases: false
    failOnError: false
    commitMode: false
  env:
    GITHUB_TOKEN: ***

📘 Reading input values

🔖 Resolve previous tag
  ℹ Found 57 (fetching max: 200) tags from the GitHub API for neo4j/graphql
🚀 Load pull requests
  ℹ Comparing neo4j/graphql - '@neo4j/graphql@2.4.0<EMAIL_ADDRESS>  ℹ Found 271 commits from the GitHub API for neo4j/graphql
  ℹ Fetching PRs between dates 2021-11-01T10:03:42.000Z to 2021-12-01T13:39:00.000Z for neo4j/graphql
  ℹ Retrieved 173 merged PRs for neo4j/graphql
  ℹ Retrieved 271 release commits for neo4j/graphql
📦 Build changelog
  ℹ Sorted all pull requests ascending: ASC
  ℹ Used 1 transformers to adjust message
  ✒ Wrote messages for 24 pull requests
  ℹ Ordered all pull requests into 3 categories
  ✒ Wrote 18 categorized pull requests down
  ✒ Wrote 6 non categorized pull requests down
  ✒ Wrote 0 ignored pull requests down
  ℹ Filled template

Configuration below:
{
    "categories": [
        {
            "title": "## 🚀 Features",
            "labels": ["feature", "graphql"],
            "exhaustive": true
        },
        {
            "title": "## 🐛 Fixes",
            "labels": ["fix", "graphql"],
            "exhaustive": true
        },
        {
            "title": "## 🧪 Tests",
            "labels": ["test", "graphql"],
            "exhaustive": true
        }
    ],
    "tag_resolver": {
        "method": "sort"
    }
}

Hi there @darrellwarde
Thank you for the report. I'll have a look at this.
Was this the current specification in your github actions? https://github.com/neo4j/graphql/commit/5020318277535ce805f848ff32c08b624fadc7fb#diff-d44dd879a02a07f5781424a643498042e49b0b3d49d2682d176d7322e89f69a6R384-R391

Hi there @darrellwarde
Thank you for the report. I'll have a look at this.
Was this the current specification in your github actions? neo4j/graphql@5020318#diff-d44dd879a02a07f5781424a643498042e49b0b3d49d2682d176d7322e89f69a6R384-R391

Apologies, I should have given you that as well to start with!
That's from a different workflow file, but identical YAML, the actual block is https://github.com/neo4j/graphql/blob/master/.github/workflows/release.yml#L518-L525
Thanks for the speedy response!

Oh one thing I saw based on your mono repo setup. The action offers the ability to pre-filter tags or transform tags if they are prefixed like in your case. So you can still use real semver ordering.
This was showcased for example:
https://github.com/mikepenz/release-changelog-builder-action/commit/d5c72c27f04d4eef7dd41c8423433dfa07da8e05#diff-5dd6a20ac6fda7c3a6519cec8213222616372409d92d79bc83c9e8942638d32cR52-R60
And maybe also helpful. It's possible to checkout this repo and run parts of the action locally. So you don't have to test it within the repo. This even comes with debugging support:
https://github.com/mikepenz/release-changelog-builder-action/issues/576#issuecomment-963406744
I am currently using this branch: https://github.com/mikepenz/release-changelog-builder-action/tree/feature/605_test
And run the test via:
npm test -- neo4j.test.ts

Do you possibly have an example of which PR ended up in the wrong category?
Based on the below output it seems to at least make sense
    ## 🚀 Features

    - Feat/fulltext
       - PR: #563
    - Sorting by relationship properties for union and interface fields
       - PR: #598
    - Feat/Aggregate Sum
       - PR: #597
    - Feature/connectOrCreate
       - PR: #575

    ## 🐛 Fixes

    - Fixed edge updates on one to one relationships
       - PR: #579
    - fix/559: add subscription type fields to resolve tree field resolver
       - PR: #576
    - Querying DateTime in the edge of a connection throws an error
       - PR: #587
    - Generate scalar typedefs before extra definitions
       - PR: #599
    - Fix/564: Async error when reading from JWKS endpoint
       - PR: #578
    - Fix for missing auth parameter
       - PR: #603
    - Fix: Project All Interface Fields For Types Implementing `@relationship` Interface
       - PR: #590
    - Fix: Escape all @node directive labels
       - PR: #617

Strange, our real output was very different! Luckily we haven't fixed the changelog yet, but it ended up like https://github.com/neo4j/graphql/releases/tag/%40neo4j%2Fgraphql%402.5.0
You can see that almost every, if not every, PR is in each category.

@darrellwarde oh, I should have looked at that earlier. It looks you are using the v1 of the action:
mikepenz/release-changelog-builder-action@v1

https://github.com/neo4j/graphql/blob/master/.github/workflows/release.yml#L520
If you want to use v1, this was the latest v1 README and state for the action:
https://github.com/mikepenz/release-changelog-builder-action/tree/v1.8.3
Exhaustive did not exist back then, as graphql will match for every category.

Well now I feel like an absolute numpty! Thank you for spotting that, you've probably saved me hours of scratching my head over this.

No worries :D I didn't look at it directly either.
Just edited my message above with some additional details.

But yeah if you want to try and modify the action and configurations, it's much faster and easier if you run it locally :). So you can pre-test things without needing to tag on the repo, and wait until things run through.

This was really great support, thank you! Pulled down that branch you pointed me to and used it to generate the correct changelogs to copy and paste in, saved me a load of time!
Now bumped the version to v2 and doing a release of a different product to validate. Many thanks again! 😄

Thanks for the kind words, you have a great day too!


Upgrade async-http-client to 2.0.2

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

Merging [#616][cc-pull] into [master][cc-base-branch] will increase coverage by -2.13%

@@           master       #616   diff @@
========================================
  Files         196        392    +196
  Lines        4312       4513    +201
  Methods         0       4083   +4083
  Branches      607        240    -367
========================================
+ Hits         2561       2584     +23
- Misses       1751       1929    +178
  Partials        0          0

3 files (not in diff) in ...eware/authentication were modified. more
3 files (not in diff) in ...4s/server/middleware were modified. more
2 files (not in diff) in ...la/org/http4s/server were modified. more
3 files (not in diff) in ...cala/org/http4s/util were modified. more
2 files (not in diff) in ...a/org/http4s/headers were modified. more
22 files (not in diff) in ...ain/scala/org/http4s were modified. more

Misses -22
Hits +22

2 files (not in diff) in ...la/org/http4s/client were modified. more

Hits +1

3 files (not in diff) in .../http4s/server/blaze were modified. more
3 files (not in diff) in ...rg/http4s/blaze/util were modified. more
5 files (not in diff) in .../http4s/client/blaze were modified. more

Powered by Codecov. Last updated by 51a5b71
[cc-base-branch]: https://codecov.io/gh/http4s/http4s/branch/master?src=pr
[cc-pull]: https://codecov.io/gh/http4s/http4s/pull/616?src=pr

This looks good to me. I don't know why codecov is unhappy but I didn't look very closely.

Something changed in codecov behavior recently.  I'll have to dive into that later.