Create a good README

Please read all of https://docs.techstartucalgary.com/projects/readme/index.html and create a good readme in the root directory.
Information on markdown can be found here https://www.markdownguide.org/cheat-sheet/

Closed by #225


editableInput event is not trigger

editableInput event is not triggered when some medium-button takes effect. this causes inconsistent state.

At least 1.1.2 does seem to fix the issue. AFAICS it was fixed with https://github.com/arcs-/medium-button/issues/14.


supply configurable initial reducers in SSR

so that we can control what preloaded state we want in our initial client app. #1914

This is the only place where they should be filtered: https://github.com/plone/volto/blob/147b3cd1d6b3b326cd7225b132f214c9bb3670a3/src/helpers/Html/Html.jsx#L151
The big concern is about the data that's transfered from the server to the browser. See this place for a starting point:
https://github.com/eea/volto-bise/blob/master/src/customizations/volto/helpers/Html/Html.jsx#L14
Though we want to have these reducers configurable, so that part is ok (though those reducer names are not part of serverConfig). Right now that optimization is disabled in BISE, see this commit:
https://github.com/eea/volto-bise/commit/001728a2e4a39114017a7c3d570d050ef882da8a#diff-eab43a4915429976633277419de252a8790fe08898718804e0c3c62b95ebcf37

This is the only place where they should be filtered: https://github.com/plone/volto/blob/147b3cd1d6b3b326cd7225b132f214c9bb3670a3/src/helpers/Html/Html.jsx#L151
The big concern is about the data that's transfered from the server to the browser. See this place for a starting point:
https://github.com/eea/volto-bise/blob/master/src/customizations/volto/helpers/Html/Html.jsx#L14
Though we want to have these reducers configurable, so that part is ok (though those reducer names are not part of serverConfig). Right now that optimization is disabled in BISE, see this commit:
https://github.com/eea/volto-bise/commit/001728a2e4a39114017a7c3d570d050ef882da8a#diff-eab43a4915429976633277419de252a8790fe08898718804e0c3c62b95ebcf37


how to update by using extend

app.factory("Message", function($firebaseArray) {
function Message(snap) {
// store the user ID so AngularFire can identify the records
// in this case, we store it in a custom location, so we'll need
// to override $$getKey
this.message_id = snap.key();
this.message = snap.val();
}
Message.prototype = {
update: function(snap) {
// store a string into this.message (instead of the default $value)
if( snap.val() !== this.message ) {
this.message = snap.val();
return true;
}
return false;
},
toJSON: function() {
// tell AngularFire what data to save, in this case a string
return this.message;
}
};
return Message;
});
how can I update Message object from a controller? what parameter should be placed onto snap for update function? Can you give me a example?

Not at all sure what this means. There is no use of $extend or AngularFire in your example, although it does inject the unused $firebaseArray as a dependency. Could you clean this up, add formatting, and provide a complete use case of what you're trying to accomplish here?

The above code is an example from the official AngularFire documentation. What I understand from the code is "Message" Factory create a class and the "MessageFactory" extends firebaseArray object with a "Message" Class. Once I get a $firebaseArray via "MessageList factory". How can I update a Message class object itself directly.
Once, MessageList factory instantiates with a Firebase reference. It returns an array with Message class objects. The Message class object has update member method from the document. But it keeps giving me a error because of snap parameter's "undefined val() function".
If I get an array like this way,
...
controller('ctrl', function($scope, MessageList){
var array = new MessageList(ref);
// array[0] is Message class
array[0].update (??) // what parameter do I need to put in?

....
})

The MessageList is used just like any other $firebaseArray instance. Reading and setting data is covered in the guide here. Check out the Full Example near the bottom for a solid overview.


Compile errors due to missing meta file

After installing the package with open upm I get this error:

Which seems to be causing compilation errors.

https://github.com/MirrorNG/MirrorNG/commit/de24dc4dee7279b299642732286d7cc4273692e5
Will be fixed on next release

fixed


Replace goerli with sepolia as the testnet

Replaces goerli with sepolia throughout the app. Kept everything in mev-boost and did as little changes as possible to keep this PR small.
This PR:
-> removes relay-goerli.ts and implements a mechanism to detect the chain id in relay.ts
-> Removes unused chains from networks.ts
-> Implements a new function getNetworkConfig in networks.ts that returns relevant flashbots-related urls
-> Uses JsonRpcProvider instead of AlchemyProvider in useRecoveryProcess as sepolia is not supported for AlchemyProvider in the version of ethers we are using
-> Changes how the asset detection logic entirely: Uses getTokenBalances, getTokenMetadata, and getNftsForOwner instead of getAssetTransfers, as that endpoint does not support sepolia testnet.
-> Updates the readme and the relevant pieces of code to say "sepolia" instead of "goerli"
Next steps

[ ] Think about testing? (maybe playwright?)
[ ] Might want to start using MEV-Share (It works on a hwr clone i built, so the code is ready)

Love to see all this refactoring. I will test extensively and leave a review when I get a chance. Thanks, @portdeveloper!


Fix community sidebar issues

Closes #235
In addition, makes links for any zero-length 3rd party service handles hidden
Preview
Mainnet

@ailisp I'm looking into it

It mostly looks good. One issue is team admins are not displayed, only global moderators, which I think it's a bug (it also shows in current production). @ori-near can you confirm whether this is expected?

Hey @ailisp – you are right, this is a bug. We need to only show Community Admins.
@carina-akaia Please let me now if I should create a new bug for this or if you want to fix this as part of this.

@ori-near I'd prefer to merge this PR and create a separate ticket in order to delegate it to someone else, as I don't have the capacity to keep up with the sidebar

@ori-near please check out this preview and let me know whether any additional changes are required or not

Code looks updated but preview didn't update and still showing moderators, @carina-akaia can you update the preview?

@ailisp apologies, I redeployed another branch to that account by accident. Fixed!

@carina-akaia the preview is still not fixed:
https://near.social/root.akaia.near/widget/gigs-board.pages.community.activity?handle=contract-standards&nearDevGovGigsContractAccountId=devgovgigs.near
I suspect your deployment didn't succeed, because changes below doesn't reflect in the preview

@ailisp check https://near.social/carina.akaia.near/widget/gigs-board.pages.community.activity?handle=near-discovery&nearDevGovGigsContractAccountId=devgovgigs.near
( also updated the link in the description )

Hi @carina-akaia thanks for posting this. Right now the GitHub link is incorrect ... but it's because the user entered "https://github.com/near/near-discovery" instead of just handle. I'll fix this on the community settings page, but we will address this potential user error in: https://github.com/near/neardevhub-widgets/issues/173.


Code formatting stopped working

Code formatting stopped working after I click ALT+SHIFT+F...
It work:
class Test  {
    constructor() {
        this.value = true;
    }
}

Test    .run = function () {
    return true;
};

It does't work:
class Test {
    constructor() {
        this.value = true;
    }
}

Test.prototype    .run = function () {
    return true;
};

PS: I use "Beautify" code formatter.

If you have Beautify configured as formatter for JS/TS and you see this error - you should report this issue in Beautify extension repo.
/needsMoreInfo
Please provide all info required by issue template (OS, versions, etc,).
Is it reproducible with all extensions disabled?
You can try this with F1 and >Developer: Reload Window With Extensions Disabled
Just tried with built-in formatter (but on VS Code insiders and TS nightly) everything works as expected.

@IllusionMH Built-in formatter works fine! Thanks for the advice!


⚠ Bofrost.lu has degraded performance

In fbdd81f, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1180 ms

Resolved: Bofrost.lu performance has improved in 3bd3b44 after 25 minutes.


Refs #3998 Make mutationTries configurable [5018]

Related with #324.

Build status:

Linux
Mac
Windows

Build status:

Linux
Mac
Windows


ConstEval API

I'm trying to write a plugin that requires evaluating combinatorial circuits.  From what I can gather ConstEval is the tool which does this.  However, the API is not so clear to me.  Is there somewhere a rundown of the members of ConstEval and what they do?

I have now posted an answer to that question to stackoverflow: http://stackoverflow.com/questions/32093541/how-does-the-yosys-consteval-api-work
Feel free to ask followup questions either here or there.


Use DeepEqual to replace DeepDerivative

fixed: https://github.com/open-cluster-management/backlog/issues/13507
Signed-off-by: clyang82<EMAIL_ADDRESS>

/test test-e2e

/retest


Dimmers turn on to previous brightness by default

This reverts the code in #125 so that dimmers can turn on to their previous brightness.

Thanks @jshridha


Use batch optimizations, load params in groth16::Verifier, verify Spend & Output descriptions

Motivation
Tie together the existing async groth16::Verifier to the bellman batch optimizations and the required loading in of the different VerifyingKeys and Params for each statement type.
Solution
Supports the bellman batch math optimizations, and creates 3 static instances,
one for Spend, Output, and JoinSplit, per their different VerifyingKeys.
Depends on #1704, and https://github.com/zkcrypto/bellman/pull/59
The code in this pull request has:

[ ] Documentation Comments
[ ] Unit Tests and Property Tests

Review

Related Issues
#1645, #406
Follow Up Work
tbd

Rebased onto https://github.com/ZcashFoundation/zebra/pull/1725 while in progress

@yaahc I'm running into conflicts between bellman's rand/rand_core traits, and redjubjub's rand/rand_core traits, which I thought I had resolved to the same, but rustc still mad at me, maybe you can take a look?

There are still some things to add (tests beyond syncing&verifying the chain, cleanup) but it's mostly ready for eyes.

Nice

Woot

https://github.com/ZcashFoundation/zebra/runs/2001346234?check_suite_focus=true

Resolves #323, #324

I've just started an ephemeral mainnet and testnet sync for this PR, with some local peers for speed.

Commit a143434 (merge 0a0d11bd) synced within 3 hours on testnet, and 5 hours on mainnet. That's pretty typical for me, I'm network-limited, and my local peers are a much larger proportion of testnet. (Also testnet data is much smaller.)
Didn't notice much CPU load, but I do have a lot of cores.
I've restarted with the latest version of this PR 525a6f6 (merge 611b335f), and I'll let it run overnight. Feel free to ping me if I forget to update this ticket.

I've restarted with the latest version of this PR 525a6f6 (merge 611b335), and I'll let it run overnight. Feel free to ping me if I forget to update this ticket.

Seems fine 👍


The Restart button makes Testbed go back to Tiles rather than restart the current test

What steps will reproduce the problem?
1. Compile 2.3.2 and run Testbed.
2. Choose any test other than Tiles, then click Restart.
3. Rather than restarting the current test, the program goes back to Tiles.

What is the expected output? What do you see instead?
Expected: when I click Restart I expect Testbed to restart the current test,
like it did on 2.3.0. What I see instead: rather than restarting the current
test (which is not Tiles), the program goes back to Tiles.

What version of the product are you using? On what operating system?
2.3.2 on Windows 8.1 Pro x64, compiled with MinGW-builds gcc 4.9.1 32-bit (the
one bundled with Qt 5.4/Qt Creator 3.3.0).

Please provide any additional information below.
I've attached a patch that maked the Restart button behave the same way it did
in 2.3.0; if the change in behavior was on purpose, please close the issue.
By the way, Testbed wouldn't compile as-is without #include <crtdbg.h> on my
platform. The patch addresses this as well.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 10 Feb 2015 at 1:52
Attachments:

fix-restart_and_mingw-crtdbg.patch

Works for me. Does this still happen for you?


⚠ gorzdrav.org has degraded performance

In 61b03c0, gorzdrav.org (https://gorzdrav.org) experienced degraded performance:

HTTP code: 200
Response time: 1632 ms

Resolved: gorzdrav.org performance has improved in 731344c.


Include export extension in install

The export extension is a python package, while the other frontend extensions are javascript. Either figure out a way to incluide this python package in the install for the extensions or rewrite it as JS

This is implemented as part of #122.


MediaStreamTrack.getSources() Deprecated

According to Chrome MediaStreamTrack.getSources() is deprecated and will be removed in Chrome 56 (currently in dev channel, so coming around December or January).
Since the only current use of that function is to find any environment facing cameras and set a preference for it, this code could be replaced entirely by using constraints. The newer format would simply be {facingMode: 'environment'} to express optional preference for environment facing cameras.
While making this change it would also be useful to plumb down additional constraints through ContinuousScanner. I've implemented this as looking for a videoConstraints key in the options for ContinuousScanner and passing it to LocalVideoCapturer which now takes an optional constraints object.
Plumbing through additional constraints is very useful for multi-camera environments. I have a setup with two cameras and allow the user to set a preference (saved to LocalStorage). I can enforce this preference by providing {deviceId: {exact: "foobar"}} as an additional constraint and it works great.

Sounds great, PR welcome :)

Now that Chrome 55 has landed the deprecation notice has been updated to specify Chrome 56 will remove the feature likely in January 2017.
Unfortunately switching to just the {facingMode: 'environment'} constraint doesn't work well with Android as facingMode is currently broken there.
The code could be switched to use MediaDevices.enumerateDevices() which is the spiritual successor of MediaStreamTrack.getSources(), but compatibility is unclear. Testing on desktop, it appears to be usable for Chrome, Firefox, and Opera. I can only test Chrome Android and it appears to be accessible there as well.
I'll work on a pull request with that functionality.


[AAP-9251] Fixed exclusion of disabled rules

A rule can optionally be disabled by setting enabled=false. This is handy when testing and certain rules need to be disabled temporarily. Fixed the json schema, added an example.
https://issues.redhat.com/browse/AAP-9251

eda-controller must be updated as well: https://issues.redhat.com/browse/AAP-9257

@Alex-Izquierdo Fixed typo


Fixing a couple of minor bugs for the Steel Defender

I noticed a few minor bugs in my playthrough, so I think these commits should fix them.

looks good to me. New release soon. Thanks!


Revert "Webinstall improved"

Reverts bagisto/bagisto#2727 .
This pull request is not working well.

@rahulcs0082 more info please?


[WIP] MMI graph compilation inside dataloader

This code works only with DataLoader num_workers=0, and when we comment out the assertion assert num.requires_grad == is_training.
As to num_workers, when we set it to 1 or greater, it will spawn subprocesses and require the objects passed between to be pickle-able. I think that Fsa/FsaVec has no pickling support at the moment which is why the subprocess crashes (it's actually frustrating because you can't easily attach to it with GDB).
As to num.requires_grad, I am not sure if it needs to be True - the training seems to work fine when it's False? It's not like we're updating the numerator FSA parameters, since it gets deleted soon after the batch is processed anyway. In any case, the line where I set it to True actually does nothing - I'll get rid of it before we merge this (eventually).

I think the issue with requires_grad is that we train the bigram language model by means of the numerator graph, so a grad does get propagated there.  That may actually make this approach non-workable, because it means that there is a connection between the model's P matrix and the num graph, and if the data-prep process is independent of the process that contains the model there may be a problem.  (Might not exist for certain other training mechanisms though.)

I might be getting it wrong, but:

requires_grad=False only means that you won't be storing the gradient information for a given tensor/object, but it can still be backprop'd through so that the preceding parameters get their gradient
the bigram LM in P.scores is a separate entity from the numerator graph, that is attached to the model and gets normally updated (per https://github.com/k2-fsa/snowfall/pull/113/files#diff-f7ad18b0ff25da80362b606e8e147895f23b943ad66aea4e54aa8b17bc924d6bR527)

So I think if we're able to work around the pickling issues, it should work OK.

No, you can't backprop through it if it was forward prop'd with requires_grad=False.
The problem is that the connection between P.scores and the numerator is part of the graph that PyTorch creates, it's not really "movable", it wouldn't survive pickling.

How about the following snippet? I think it shows the reverse.
>>> import torch
>>> from torch import nn
>>> x = torch.randn(32, 100)
>>> l1 = nn.Linear(100, 200)
>>> l2 = nn.Linear(200, 10)
>>> l2.requires_grad_(False)
Linear(in_features=200, out_features=10, bias=True)
>>> l1.weight.requires_grad
True
>>> l2.weight.requires_grad
False
>>>
>>> z = l2(l1(x))
>>> z.sum().backward()
>>> l1.weight.grad
tensor([[-3.9523e-01,  1.0352e+00,  1.0160e-01,  ...,  3.4121e-02,
         -4.9239e-01, -3.2230e-01],
        [-7.5320e-02,  1.9729e-01,  1.9362e-02,  ...,  6.5026e-03,
         -9.3837e-02, -6.1422e-02],
        [ 5.9474e-03, -1.5578e-02, -1.5289e-03,  ..., -5.1346e-04,
          7.4096e-03,  4.8500e-03],
        ...,
        [ 3.4034e-01, -8.9147e-01, -8.7489e-02,  ..., -2.9383e-02,
          4.2401e-01,  2.7754e-01],
        [ 8.4615e-02, -2.2163e-01, -2.1751e-02,  ..., -7.3051e-03,
          1.0542e-01,  6.9002e-02],
        [ 1.7086e-01, -4.4754e-01, -4.3921e-02,  ..., -1.4751e-02,
          2.1286e-01,  1.3933e-01]])
>>> l2.weight.grad
>>>

think of it in terms of the computation tree.
the matrix product thats.computed.will have.requires grad.true.
On Wednesday, March 3, 2021, Piotr Żelasko<EMAIL_ADDRESS>wrote:

How about the following snippet? I think it shows the reverse.

import torch>>> from torch import nn>>> x = torch.randn(32, 100)>>> l1 = nn.Linear(100, 200)>>> l2 = nn.Linear(200, 10)>>> l2.requires_grad_(False)Linear(in_features=200, out_features=10, bias=True)>>> l1.weight.requires_gradTrue>>> l2.weight.requires_gradFalse>>>>>> z = l2(l1(x))>>> z.sum().backward()>>> l1.weight.gradtensor([[-3.9523e-01,  1.0352e+00,  1.0160e-01,  ...,  3.4121e-02,
-4.9239e-01, -3.2230e-01],
[-7.5320e-02,  1.9729e-01,  1.9362e-02,  ...,  6.5026e-03,
-9.3837e-02, -6.1422e-02],
[ 5.9474e-03, -1.5578e-02, -1.5289e-03,  ..., -5.1346e-04,
7.4096e-03,  4.8500e-03],
...,
[ 3.4034e-01, -8.9147e-01, -8.7489e-02,  ..., -2.9383e-02,
4.2401e-01,  2.7754e-01],
[ 8.4615e-02, -2.2163e-01, -2.1751e-02,  ..., -7.3051e-03,
1.0542e-01,  6.9002e-02],
[ 1.7086e-01, -4.4754e-01, -4.3921e-02,  ..., -1.4751e-02,
2.1286e-01,  1.3933e-01]])>>> l2.weight.grad>>>

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/k2-fsa/snowfall/pull/113#issuecomment-789018825, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZFLOZGFNZ3ZPIE4FICZD3TBUEITANCNFSM4YNZITCA
.

OK, I went back through the MmiTrainingGraphCompiler code and I think I understand now. I previously missed the point that P LM is used not only for the dens, but it also affects the nums score. In that case yeah, we can't do the composition in a subprocess, as it would have kept using the initial, un-updated version of P. (I'm spelling it out to make sure I got you right)

Yes.
On Wed, Mar 3, 2021 at 10:12 PM Piotr Żelasko<EMAIL_ADDRESS>wrote:

OK, I went back through the MmiTrainingGraphCompiler code and I think I
understand now. I previously missed the point that P LM is used not only
for the dens, but it also affects the nums score. In that case yeah, we
can't do the composition in a subprocess, as it would have kept using the
initial, un-updated version of P. (I'm spelling it out to make sure I got
you right)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/k2-fsa/snowfall/pull/113#issuecomment-789742370, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZFLO5LJDQPDHQHCGDWE5LTBY7WVANCNFSM4YNZITCA
.

I am closing it since it's a dead-end at this time.

ok.
On Saturday, March 6, 2021, Piotr Żelasko<EMAIL_ADDRESS>wrote:

Closed #113 https://github.com/k2-fsa/snowfall/pull/113.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/k2-fsa/snowfall/pull/113#event-4415240665, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZFLO7UGLCLK6PYHJQVGTTTCEZS5ANCNFSM4YNZITCA
.


feat: dock beta banner on top

Closes #x
Objectif(s)

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


Handle android activity life cycle and event loop

Handle android activity life cycle and event loop
Related PR: https://github.com/tomaka/winit/pull/153

I did a rebase and adapted the code to the new EventLoop API

@tomaka I see that the API has changed and EventLoop has been moved to winit. Now I can't create the EventLoop override to handle EGL life cycle events on Android:

If I create a EventsLoop override in Glutin (like this PR did), it can't be used for the glutin::GlWindow::new(window, gl_context, &events_loop).unwrap()creation because the argument uses winit::EventLoop type
If I create a EventsLoop override in Winit, I don't have access to the gl_context defined in winit.

We need the android life cycle merged to unfork glutin in Servo. What's the best way to do this now?

Err I knew there would be a quirk with exposing winit.

I thought a bit about this, and considered the following solutions:

Letting the user handle the situation explicitly when they receive a Suspend event.
Wrapping around the EventsLoop again.
Try automatically detecting this in make_current and is_current.

But in the end I think the best thing to do is make a tweak to the winit's EventsLoop to make it possible for glutin to register a callback when a suspend event happens.
This is a bit of a hack, but I think that it doesn't have any downside.
The solution would be specific to Android (with a trait hidden with #[cfg(target_os = "android")].
Do you think that's ok?

Yes, I though doing a similar thing too but with all events. Doing with suspend event only it's ok. I don't think that we'll need to handle other events directly in Glutin.

@tomaka Is what you describe here https://github.com/tomaka/glutin/pull/868#issuecomment-319442402 still the approach you recommend?

@paulrouget The discussion has more or less moved there: https://github.com/tomaka/winit/issues/231
There is still no good solution for this right now.

@MortimerGoro winit 0.11.1 has been published.

@paulrouget thanks for the notfication. PR updated!
@tomaka We need to update winit dependency from 10.x to 11.x here. Do you want a separate PR for that change?

@MortimerGoro Yes please.

@tomaka r? PR ready

Thanks, looks good except for a few style issues.


Bump container versions to planned v0.19.0 for release

The Problem/Issue/Bug:
Our container code is now part of the main code tree, so during development we used temporary pushed container.
Now, though, we're about to do a release with the code in our containers. But we want to be able to test and reference the new containers (all tagged v0.19.0 on dockerhub) at the time of the tag.
To do that we have to temporarily push v0.19.0 containers, so I've just done that with the current code.
Manual Testing Instructions:
Use it, look at the tests.
Release/Deployment notes:
The 4 containers should be re-pushed at tag time.

Thanks for taking the extra effort for the manual review @andrewfrench !

I'm having a bit of trouble with the testbots, but it doesn't seem to have anything to do with this PR. So as soon as both are up again, I'll pull this and we'll rely on the master test.


Homebrew install

Review the installation on mac OS X
Indicate that this walkthrough is aimed at developers
Related PRs:

https://github.com/ome/homebrew-alt/pull/120
https://github.com/ome/omero-install/pull/143

Looking at the developers section, I have not found a suitable place to insert that page into.
cc @hflynn

job is now green https://ci.openmicroscopy.org/job/OMERO-DEV-merge-homebrew/ICE=3.6,OSX=snapper/411/

Changes look good and render correctly. No reference to omero52 left. Good to merge.

Do you want to edit the entry about this page in the see also box at the top of https://www.openmicroscopy.org/site/support/omero5.3-staging/sysadmins/unix/server-installation.html as well?
You could also add a link to https://www.openmicroscopy.org/site/support/omero5.3-staging/developers/installation.html#building-omero if you liked, although that does already link to the installation overview page so it isn't really necessary.

For the second one, pointing at the overview page is better for now until we only have centos7 and ubuntu

@hflynn: updated server-installation

Looks good thanks.


For a predicate, operand is permuted with value when using variables

This has been checked on at least less-than and more-than.
Reproduction step:
<variable name="ThreeHoursAgo">
  <script language="c-sharp">
    DateTime.Now.AddHours(-3)
  </script>
</variable>
<test name="Now is more-than three hours ago">
  <system-under-test>
    <resultSet>
      <query>
        <![CDATA[ select getdate() as Now, @ThreeHoursAgo ]]>
        <parameter name="@ThreeHoursAgo">@ThreeHoursAgo</parameter>
      </query>
    </resultSet>
  </system-under-test>
  <assert>
    <all-rows>
      <predicate operand="Now" type="dateTime">
        <more-than>@ThreeHoursAgo</more-than>
      </predicate>
    </all-rows>
  </assert>
</test>

Reproduction developed in #000081 (QueryAllNoRows.nbits) without success. Should investigate further on the original environment with the last version of NBI.
PS: improved error handling when the variable has incorrectly case.

No bug, the issue was in the test-suite.


Encountering error using options_for_select

Or at least it seems like it's related to options_for_select. The error is reproducible with a fairly simple component:
class TestComponent < ApplicationComponent
  include Phlex::Rails::Helpers::FormWith
  include Phlex::Rails::Helpers::Label
  include Phlex::Rails::Helpers::OptionsForSelect
  include Phlex::Rails::Helpers::Select

  def template
    form_with url: "#" do |f|
      f.label :some_attribute
      f.select :some_attribute, options_for_select(my_options)
    end
  end

  private

  def my_options
    []
  end
end

and here's the error from the rails server with a full stacktrace:

I can only guess but it seems like something is going awry in the handoff between Phlex and the rails helper method. Any help is much appreciated! 🙏

Thanks for opening the issue @phillipspc. I checked the API docs, and it looks like options_for_select is meant to be used to output <option> tags. I think you could use it like this:
f.select :some_attribute do
  options_for_select(my_options)
end

Thankyou for your response @joeldrapper, that helped move things forward for me.
Since I was wanting to add more attributes to the f.select (classes, data attributes), I found it difficult to make this work with options_for_select. But ultimately I found that I didn't actually need to use that helper at all (not sure why I was thinking I did).
The final working solution looks like this:
      f.select :category,
               categories_for_select,
               { selected: selected },
               class: "...",
               data: {
                 action: "...",
               }

I don't know if that would also be possible using options_for_select, but it doesn't really matter since this way works. Appreciate the response! I'll close this since my issue has been addressed.


Execute tests in parallel for CI

Summary
This commit switches the test runner we use to use stestr [1] for
running tests in parallel instead of the built in unittest test runner
which just runs things serially. Right now a test job takes ~3 minutes
to run all the tests serially. This can be sped up by running tests in
parallel. In testing locally it's consistently taking between 70-80
secs with parallel execution.
Details and comments
[1] https://stestr.readthedocs.io/en/latest/MANUAL.html

The python 3.5 windows CI failures will be fixed by #264

It looks like the speedup in CI was not as much as when running locally. When I ran things locally with 2 process (to try and match the CI environment as opposed to 16 which is the default on my desktop to match the core count) it took ~70 secs but in CI it looks like it took ~170secs which is still faster but not as much of a speedup when running on a faster machine. I'm going to try grouping tests by class to see if that speeds things up or not.

I've optimized things as much as I can in the travis environment with the current way that tests are structured. By adding caching for the previous results timing the limiting factor becomes how quickly the rb tests can be run: https://travis-ci.com/Qiskit/qiskit-ignis/jobs/230643853#L467 You can see there the stestr scheduler puts all the other tests on one worker and it executes those in less time than it takes travis to execute the rb tests. We probably can speed things up more by breaking that test up into smaller tests using ddt so they can be scheduled independently.


Fixing up opengv doc

I made a small correction to the opengv docs so that they can simply be copy pasted into the shell.

thanks @FreakTheMighty !

No problem, thanks for the sweet library!


[6.0] @_extern is an experimental feature.

Explanation: @_extern was accidentally enabled as a mainline feature even though it's experimental. This restores its experimental status.
Scope: Bug fix.
Issue: rdar://132013078
Original PR: https://github.com/swiftlang/swift/pull/75334
Risk: Low. Projects shouldn't be using @_extern, but if they are, they have to -enable-experimental-feature Extern.
Testing: Swift CI
Reviewer: @tshortli

@swift-ci Please test

@_extern is a new experimental feature in the 6.0 compiler, right? As in, no projects can be using this feature in production because there's no released compiler that supports it?

Yeah, it was added in 6.0, so it shouldn't have had an opportunity to be used in production yet. It looks like it was originally implemented as an EXPERIMENTAL_FEATURE and changed to a baseline feature by accident.

@swift-ci Please test

@swift-ci Please test

@swift-ci Please test


Change flowrouter to no ssr version so app works again

meteor 1.3-modules-beta.8 breaks the app when it uses kadira:flow-router-ssr.  This just changes the app to use kadira:flow-router

Yeah. I think this is good.


Pretty print

Addressing issue #51
To add indentation/make more specialized formatting for different AST nodes, I added another layer of indirection between the IR and the pretty printer output:
src/IR.hs:         indentPretty

                        |
                        |
                        v

src/IR/IR.hs:           ws       // Whitespace type class

                        |
                        |
                        v

src/IR/IR.hs:         pretty     // Pretty type class

The middle layer sort of acts as a more adjustable wrapper for the pretty function. I'm not convinced a typeclass is definitely the right tool here, but I am using that for now and can adjust the code structure as I get a better idea of what is needed.
Currently:

hello1.ssl can be exported and fed back into the compiler successfully, and looks much more readable.
Input hello1.ssl program:

main ( cout : & Int ) -> () =
  after 10 , (cout : & Int) <- 72
  wait (cout : & Int)
  after 10 , (cout : & Int) <- 101
  wait (cout : & Int)
  after 10 , (cout : & Int) <- 108
  wait (cout : & Int)
  after 10 , (cout : & Int) <- 108
  wait (cout : & Int)
  after 10 , (cout : & Int) <- 111
  wait (cout : & Int)
  after 10 , (cout : & Int) <- 10
  wait (cout : & Int)

output pretty print using stack exec -- sslc --dump-ir-final tests/hello1.ssl

main cout : (&Int) -> () =
 after 10, cout <- 72
 wait cout
 after 10, cout <- 101
 wait cout
 after 10, cout <- 108
 wait cout
 after 10, cout <- 108
 wait cout
 after 10, cout <- 111
 wait cout
 after 10, cout <- 10
 wait cout

Next steps:

prettify match statements with indentation
make sure nested match, lambdas, and lets are indented properly
add support for pretty printing type definitions
solve issue where pretty printer spits out constructor functions starting with "__" which cannot be parsed
double check annotation on/off switch works as expected and can be fed back in

Update:

match and nested match statements are pretty
ADT type definitions are pretty
__functions can be removed from pretty printing via flag
map.ssl and adt-recursive.ssl are pretty and can be successfully fed back in to compiler ( __funcs OFF)

Next Steps:

fix issue with nested lambdas and __funcs ON
make sure type annotated output can be fed back in
make sure rest of examples look pretty and can be fed back in
double check nested lambdas look pretty and can be fed back in
refactor code/use better function naming

Example input/output of pretty printer:
Original:
type List
  Cons Int List
  Nil

print_list2 (out: &Int) (l: List) -> () =
  match l
    Cons i l0 =
      after 1, out <- i
      wait out
      after 1, out <- 32
      wait out
      print_list2 out l0
    Nil = ()

print_list (out: &Int) (l: List) -> () =
    after 1, out <- 91
    wait out
    after 1, out <- 32
    wait out

    print_list2 out l

    after 1, out <- 93
    wait out
    after 1, out <- 10
    wait out

map (f: Int -> Int) (l: List) -> List =
  match l
    Cons i l0 = Cons (f i) (map f l0)
    Nil = Nil

add (a: Int) (b: Int) -> Int =
  a + b

main (out: &Int) -> () =
  let l1 = (Cons 104 (Cons 101 (Cons 108 (Cons 108 (Cons 111 Nil)))))
  print_list out l1
  let l2 = map (add 1) l1
  let l3 = map (add 4) l1
  print_list out l1
  print_list out l2
  print_list out l3

Output without __funcs:
type List
 Cons Int (List)
 Nil

print_list2 (out : &Int) (l : (List)) -> () =
 match l
       Cons i l0 = after 1, out <- i
                   wait out
                   after 1, out <- 32
                   wait out
                   print_list2 out l0
       Nil  = ()

print_list (out : &Int) (l : (List)) -> () =
 after 1, out <- 91
 wait out
 after 1, out <- 32
 wait out
 print_list2 out l
 after 1, out <- 93
 wait out
 after 1, out <- 10
 wait out

map (f : Int -> Int) (l : (List)) -> (List) =
 match l
       Cons i l0 = Cons (f i) (map f l0)
       Nil  = Nil

add (a : Int) (b : Int) -> Int =
 a + b

main (out : &Int) -> () =
 let l1 = Cons 104 (Cons 101 (Cons 108 (Cons 108 (Cons 111 (Nil)))))
 print_list out l1
 let l2 = map (add 1) l1
 let l3 = map (add 4) l1
 print_list out l1
 print_list out l2
 print_list out l3

Output with __funcs:
type List
 Cons Int (List)
 Nil

print_list2 = fun out fun l match l
      Cons i l0 = after 1, out <- i
                  wait out
                  after 1, out <- 32
                  wait out
                  print_list2 out l0
      Nil  = ()

print_list = fun out fun l after 1, out <- 91
wait out
after 1, out <- 32
wait out
print_list2 out l
after 1, out <- 93
wait out
after 1, out <- 10
wait out

map = fun f fun l match l
      Cons i l0 = __Cons (f i) (map f l0)
      Nil  = Nil

add = fun a fun b a + b

main = fun out let l1 = __Cons 104 (__Cons 101 (__Cons 108 (__Cons 108 (__Cons 111 (Nil)))))
print_list out l1
let l2 = map (add 1) l1
let l3 = map (add 4) l1
print_list out l1
print_list out l2
print_list out l3

__Cons = fun __arg0 fun __arg1 Cons __arg0 __arg1

Update: Incorporated PR feedback

removed FMT typeclass and moved all pretty printing changes to the pretty typeclass
preserved old pretty printer with type checking in the Lengthy typeclass
updated spaghetti to call lengthy instead of pretty
tightened up code in DConToFunc.hs/added more documenation
replaced magic numbers with indentNo constant defined in Pretty.hs
updated/improved comments

Next Steps:

I think this PR is ready to merge into main now
Implementing minimum parentheses algo or selective type annotations could be good work for future PRs
I think the original pretty printer may have only ever passed unit tests? Maybe we can talk more about this at the meeting.

Hi John,
- I believe I have completely separated Dumpy and Pretty per your request
- I'm not sure about the change you requested on ast.hs; could you respond to my question on what to do for this part?
- while updating dumpy to work with the regression tests, I found a bug in the dConToFunc.hs pass which I fixed
- without pattern desugaring (Leo's pass), all the regression tests work with dumpy, however when I test dumpy with pattern desugaring, map.ssl,map-inc.ssl and adt-partially-applied.ssl all fail. I was trying to resolve this issue this week, but I was getting stuck and thought I better check in with you.
I realize the semester is over now so maybe I shouldn't be working on this, but I remember you saying you wanted to merge in the pretty-print branch soon and I didn't want to leave this PR in an unfinished state for you. If you'd prefer I stop working on this and you can take over from here, let me know. Otherwise I will keep working on getting dumpy to pass all regression tests. Let me know your thoughts/ what you would like me to do!


Keep Java version of exemplar in sync with Kotlin version

We've added a bunch of actions to the Kotlin exemplar (for websockets, for form actions) but haven't kept the Java version up to date. Need to add these to the Java variant to ensure they function properly in Java (right now Java exemplar doesn't even start).

Closing - we'll only support Kotlin.

Closing - we'll only support Kotlin.


Voting: Heatmaps

If you want to this feature then post 👍  to below.
Native Google Maps Android API has heat-map in utility library.

https://developers.google.com/maps/documentation/android-api/utility/heatmap

Google Maps SDK for iOS doesn't have it.
But, I found library named "LFHeatMap" what implement heat-map for iOS Maps.

https://github.com/rmarinho/LFHeatMap

This is great works,
I think we can implement heat-map for X.F.GoogleMaps using these.

Yes please! This would be great!

This is the feature I wish the most! I'm currently trying to use https://github.com/alexsorokoletov/Xamarin.Android.Maps.Utils on Android and LFHeatMap on iOS, but having it unified into X.F.GoogleMaps would be great.

@ricardoglc  Any idea on how to implement heat map in xamarin forms?

Actually I have added this to my own project using Platform Effects (just for iOS at least).
I'd welcome this feature right into the library.

Hi @amay077 , Is this planned in the near future?

@amay077  maybe now?


Add --depth option to --feature-powerset

--depth <NUM>
    Specify a max number of simultaneous feature flags of --feature-powerset.

    If NUM is set to 1, --feature-powerset is equivalent to --each-feature.

    This flag can only be used with --feature-powerset flag.

Closes #57
Closes #58

bors r+


RenderPeople asset links and how to rig them

Thanks for sharing your awesome work.
In the paper, you state that PSP uses RenderPeople human models and added custom rigs to animate them. Could you provide the links to those assets? Would the models be simulation ready after applying animations with mixamo?
Section 3.1 in the paper:

PEOPLESANSPEOPLE has a set of 28 scanned 3D human models from RenderPeople [36]. These
models are ethnically- and age-diverse, fully re-topologized, rigged, and skinned with high-quality
textures (Fig 2a). We added a character control rig to pose or animate them with motion capture data

Thank you for your interest in this work.
In section 6 of the PeopleSansPeople paper we mention:

We also provide a Unity template environment with example assets and full functionality of PeopleSansPeople. However, due to RenderPeople redistribution and licensing policies, we do not provide direct access to the 3D human assets; instead, we provide detailed instructions and examples for sourcing and making the human assets simulation-ready.

Following the instructions on the paper, and the example assets we provided in the Unity template environment, you should be able to bring in any human assets into Unity and make them simulation ready.

The unity env of this repo only includes the human assets you own, not all the models used to generate the actual dataset. I meant the URLs to the specific human assets you downloaded from RenderPeople to reproduce your work.

As I mentioned in my comment above, for RenderPeople we do not own redistribution and licensing policies to allow us to send them to you. This includes redistributing links. You can look at the figures on the paper including this one to help you choose those assets from the RenderPeople library.
You must follow the same prefab setup as in the human prefab we have provided in the template Unity environment. We cannot know beforehand whether you need to modify the rigs or not, because the assets may have been sourced from anywhere. What's important is that you make them look like the one we have provided; i.e., set up the rig, randomizers, labelers, etc. such that your asset matches ours. Hope that is a good guideline for you.

Thanks for your clarification. I see, you mean the "redistribution" includes giving out the URLs of assets. I'll manually try to look for similar assets in that case.
My second question is more of whether you applied special modifications to the RenderPeople models or their rigs with other tools than Unity. Did you only need to adjust settings within Unity / Unity Perception package to make animations work correctly? As RenderPeople models are not free, I was hoping to confirm this.

You will need to adapt those assets to look exactly like the sample asset we have provided. Look under each provided asset, and also in the randomizers where those assets and their components are referenced.
If you want to do any clothing shader graph randomization, your materials, textures, albedos, and normals need to be separated for clothing.
You may or may not have to modify the rig too to ensure that the animation library you are going to be using can retarget well to your human assets.


What should I modify to make the model generate video under both pose  video control and image guidance?

At present, the model seems cannot input both image and control video at the same time?

I believe that is correct.  I was trying to do the same thing.

philpilkington

Hi, do you have any ideas or results? I thought of several ways to inject image conditions like concat or adaln, but haven't experimented yet.


Browser search doesn't scroll to position of result

When there are multiple tables with floatThead, and searching for text in thead, the browser search feature is unable to scroll to the position of thead of the matched result.

Steps to reproduce
Go to https://jsfiddle.net/6fuLmjyb/
Use the browser search feature to search for "whisper".

The default for window scrolling is fixed uses position. This takes the element out of the document flow and so the browser doesnt know how to scroll to them. If you want search to work you must use absolute positioning like this:
https://jsfiddle.net/hqLyxn3e/
var $table = $(".table_demo");
$table.floatThead({
    position: 'absolute', // <--- You need this!
    responsiveContainer: function ($table) {
        return $table.closest(".table-responsive");
    },
});

Works, Thank you :pray:


Feature Request: re-centering on fullscreen switch

it would be nice if the center in the embedded view was used as the center when going to full screen; what i'm seeing is that the embedded view ends up in the upper left corner. likewise, using the same center when switching out of full screen would be nice.

interesting. it does appear to recenter and refresh properly in Firefox, but not in safari. i'd tried safari 11 days ago which is when i opened this request.


Allow using alternate cache manager for ImageManager::cache()

I want to implement a custom folder hierarchy for caching images which is not possible to achieve using the intervention/imagecache package. So it would be nice if it was possible to configure our own cache class. Checking ImageManager:cache()'s as long as the cacher has a get() method which returns back the imagemanager instance or boolean there really shouldn't be a problem.
I have made a proof of concept patch which hopefully should allow setting a custom cacher. Or perhaps you could add an interface to this package which any class can implement to be able to be used as an image cacher.

What exactly do you mean with "custom folder hierarchy"?

For e.g if checksum of operations (like the one generated by ImageCache::checksum()) is 123abc, i want the image to be saved under 123/abc folder hierarchy in the configured cache directory. Common practice to avoid storing large number of files in a single folder.
If the class used by ImageManager::cache() was configurable I could just extend Intervention\Image\ImageCache and override the relevant parts.

Would be nice if intervention/imagecache had a base class with all the goodies to maintain the operations chain and generating checksum without dependency on other Illuminate packages and have get()method as abstract to allow one to implement it as they wish.

I see. Maybe a good solution would be, to just use the Interfaces of Illuminate/Contracts as abstract dependencies and allow the user to use other cache drivers. I think storing files is entirely the task of the actual cache driver and not the imagecache class.

I think storing files is entirely the task of the actual cache driver and not the imagecache class.

I agree.


SP metadata is lacking the AttributeConsumingService key

should be fixed by https://github.com/simevo/spid-php2/commit/a836d92a301d8999046974a6070833d49581a009

we can use this schema for validation:
https://github.com/italia/spid-metadata-builder/blob/master/saml-schema-metadata-SPID-SP.xsd

add these two keys to the $settings array:
// assertion consuming services
// in this array, the order is important: the 0-base index will be used
// as assertion consuming service index in the API calls
'assCs' => array(
    $base . "/acs.php", // full url
    $base . "/acs1.php", // full url
),
// attribute consuming services
// in this array, the order is important: the 0-base index will be used
// as attribute consuming service index in the API calls
'attrCs' => array(
    array('name', 'familyName', 'fiscalNumber', 'email'),
    array('fiscalNumber', 'email')
),


Update "Use your own data" tip dialog

Update the "User your own data" tip dialog. @saleiva , please assign issue accordingly.

The grammar can be improved
The numbers (A,B,C) are not intuitive (we already removed these from the "Welcome" tip dialog, so we should be consistent

https://github.com/CartoDB/cartodb/blob/master/lib/assets/javascripts/cartodb/common/views/create/create_onboarding.jst.ejs. Update text as follows:
Use your own data,

A dataset can be used in several maps and updated via an API. This enables any changed data to be reflected in your maps.

Connect datasets using one of our external service connectors, upload your own data files, or use a public dataset from the Data Library.

Thinking of starting from scratch? You can also create an empty dataset.

Related to Docs issue#326

Does not apply with the Builder. Closing this, and the related Docs issue!


Update the pulsar admin cli java doc to support set/get/remove deduplication on topic level

Describe the bug
In PR (https://github.com/apache/pulsar/pull/7821), set/get/remove deduplication policy is on topic level. But the pulsar admin cli java doc is not supported accordingly.
https://github.com/apache/pulsar/blob/master/pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdTopics.java
Expected behavior
Update the CmdTopics.java doc.
@315157973 , cloud you please help update the CmdTopics.java.

@315157973 Could you please help take a look at this issue?

please assign it to me


[CNETFranceBridge] Initial commit

CNET France News but with filters on title or url.
Default filters are provided. This Bridge can produce false-positive, but I think it better to miss a news than receiving sometime more "ads" news than real ones

Thanks for the PR and welcome here !


Tradesim can force other players mons to evolve

Example:
/tradesim player whomst isnt me 4 5
They have a Seadra with a dragon scale in slot 4, I can force it to become a Kingdra despite the fact I'm using the command on another player instead of myself.

This bug is now fixed, thank you for the report :)


undefined is not an object (evaluating RNSound.isAndroid)

I get this error message on iphone simulator only by using this
const Sound = require('react-native-sound');

...any ideas?

I am resolved this issue by manually link library in X code, actually i did react-native link that is not giving any error so assume it's linked but when i have checked xcode then see this library is not linked so when i have linked manually and create new build then it's working perfectly.

Can you explain more how did you link it ?

same issue

same issue
I solved this problem by this way.

https://github.com/zmxv/react-native-sound/wiki/Installation
But don't forget to change compile word to just like others at android/settings.gradle. I think it was "implements" or  "implement" something like that. compile is the old word RN doesn't use it anymore.
And also I didn't use react-native link because when I used it generated a huge problem.

Thanks @naklow12 , great solution! Works for me.
Wonder why react-native link breaks things though ...

Hi! If you're still having this problem, could you please post a link to a project that reproduces the problem, give information about your environment, and say what steps you have already tried?
This is the most common issue faced by react-native-sound users and it would be good to add documentation listing the possible resolutions. A pull request would be welcome; I've opened #592 to track that.


View Main Activity Board

As a user, I should be able to see the main activity board that contains four tabs, namely:

Todo
In-progress
Upcoming Deadline
Expired Deadline

Acceptance

[ ] I have four tabs that have sections named, Todo, In-progress, Upcoming deadline, and Expired deadline.

I will be working on this issue.


haxr 3000.11 :: connection refused

New haxr version 3000.11 provide a critical bug on node start stage:
 Prelude B N A T L> runNode "n" $ advertise "a" (T.repeatM (return (Byte 1)))
*** Exception: connect: does not exist (Connection refused)

WireShark says that app access to 443 port, but master port is 11311.

This problem exist only on 3000.11 version of haxr, around this https://github.com/byorgey/haxr/commit/acd31f70f7da0fd9a57cf1fce04bc06eaa51b0e6 seems to me.

Temporary solution
haxr == 3000.10.* in cabal file.
About haxr
Currently author have migration to http-streams. This package doesn't support plain-text HTTP requests(no gzip encoding) by design: http://hackage.haskell.org/package/http-streams-<IP_ADDRESS>/docs/src/Network-Http-RequestBuilder.html#http, BUT Python xmlrpc based master process doesn't support gzip encoded requests and gives 500 error.

Ok, we should get that quick fix in quickly. I've given you push access to the repository, too, btw. The only guideline is that we try to make changes via pull requests so people can comment. Fixing breakage like this can go straight to master. I rebase a lot to avoid incidental merges in the commit history, but that's a best effort thing.
Back to this issue, did you come across evidence that http-streams deliberately does not support plain-text? If it's just something they don't support yet, we can pull in other haxr and http-streams developers to the conversation, and/or add support for that ourselves.

Apparently haxr 3000.11 has a several problems:

bugs with port/path capturing from URI;
hardcoded https only request;
chunked encoding of request data by default.

Items 1 and 2 fixed in https://github.com/akru/haxr/commit/c95025249ee339c4c06c8e74c194f334fcc22682, pull request to author repository created.
Item 3 currently isn't solve.

Is the issue of chunked encoding just a matter of calling setContentLength somewhere here?

Really, content-length header is a perfect solution! Patched haxr already works with rosmaster.
Waiting for the PR is accepted.

PR accepted, haxr-3000.11.1 works good.

Great that this is resolved. Thanks for taking care of it!


Ignore undefined fields in Collection.find()

I noticed when calling the find(selector, fields) method on a collection with fields set to undefined, I get a TypeError: Object.keys called on non-object. When checking out the problem, I found that the collection.js module checks if argument.length >= 2 which evaluates to true even if argument[1] is not explicitly set. I.e. the code is trying to use the fields argument as if it's set, throwing the error eventually.
Fixed it by comparing fields !== undefined instead of using the argument count.
In our application, we basically do something like this:
// fields is optional
function getRecords(arg1, arg2, fields) {
    // validate arg1, arg2 etc...

     // build the query with arg1, arg2
    var query = {};

    return collection.find(query, fields);
}

I changed fields to fields || {} as a quick work-around for now, but I'm pretty sure undefined should be just ignored in the module.

this is legacy behavior you should use. I support it as best as possible but the code is brittle so any changes like these breaks behavior that unfortunately people still relay on.
db.collection('data').find({}).project({a:1}).toArray()

I understand your point of view, although in this particular situation I have my doubts that developers could rely on this behaviour. You just wouldn't expect a function to try and parse a argument that is not really passed to it.
To be honest I'm not completely sure why JS also counts undefined values in arguments.length, maybe there is actually a good reason for this. But I think that for most use cases you shouldn't check if an optional parameter is passed or not in this way. Just as a future consideration then :)
Anyway, I wasn't aware that this is a legacy way of using the module. Will update my code accordingly. Thanks for the quick response!


[microNPU] Add support for depthwise conv2d on U65

Add support for depthwise conv2d running on two cores in U65,
similarly how it is done for conv2d.

cc: @manupa-arm @lhutton1 @NicolaLancellotti @dchauhan-arm


Separate PooledTransactionElement into variants

Describe the feature
With the introduction of EIP-4844, signed transactions must have a blob sidecar when obtained from the network. Decoding EIP-4844 transactions should then require that a blob sidecar is present. Currently, PooledTransactionElement may include a 4844 variant (part of its Transaction/TransactionSigned variant) that does not have a sidecar.
pub enum PooledTransactionElement {
    /// Legacy transaction variant
    Legacy {
        /// The inner transaction data
        transaction: TxLegacy,
        /// The hash
        hash: TxHash,
        /// The signature
        signature: Signature
    },
    /// EIP-2930 transaction variant
    Eip2930 {
        /// The inner transaction data
        transaction: TxEip2930,
        /// The hash
        hash: TxHash,
        /// The signature
        signature: Signature,
    },
    /// EIP-1559 transaction variant
    Eip1559 {
        /// The inner transaction data
        transaction: TxEip1559,
        /// The hash
        hash: TxHash,
        /// The signature
        signature: Signature,
    },
    /// EIP-4844 transaction variant
    Eip4844 {
        /// The inner transaction data
        transaction: TxEip4844,
        /// The hash
        hash: TxHash,
        /// The signature
        signature: Signature,
        /// The blob sidecar
        sidecar: BlobSidecar,
    }
}

Additional context
No response

just updated - instead of proposing a refactor to TransactionSignedNoHash / TransactionSigned this is now just proposing to change PooledTransactionElement.


feature: adapter packages can be downloaded and configured

Description
This merges the Adapters feature branch into main, bringing with it the new section of bot project settings (under the "new creation experience") feature flag.
Task Item
closes #5907
closes #5584
Screenshots

Coverage decreased (-0.4%) to 53.431% when pulling 56e9f1027b6a228cfa49457bb532fd33ea3d71e3 on feature/adapters into b05aa30023d120b0ec0f637531c805de35bc43c8 on main.

@carlosscastro let's go ahead and take this.. but can you address conflicts?

To reviewers: a bit more code was added around package manager but it was reviewed and tested separately here: https://github.com/microsoft/BotFramework-Composer/pull/6035


Add support for space-separated RGB

This is almost exactly the same as @htunnicliff's excellent PR #55, but just for the RGB case instead of HSL. It adds support for values like:
rgb(255 255 255)
rgb(255 255 255 / 1)

The motivation for this was that Chrome's developer tools now uses that form of notation for RGB colors, and I got bitten by blindly copying and pasting one into my CSS 😅

Thank you @JJC1138 for the awesome work!
@Qix- please merge and release, we need this!

Does the RE support alpha percentage like rgb(0 255 38 / 7%)?

No, I'm afraid not. I thought that was disallowed by the spec, but I see was wrong 😣 Sorry about that. I'm not using the library in my project anymore so I don't have an immediate need to work on that myself. You're very welcome to fork my branch to add support, of course.

@JJC1138 I just fix it to work with alpha percentage and added more tests. Everything passes and it works.
Please check this PR https://github.com/JJC1138/color-string/pull/1 and merge.

Done. Thanks very much!

Thanks :)

Published as 1.7.0.

You're very welcome @Qix- , thank you for making this!
Now that I'm looking at it again, I think I should have used parseFloat for the percentage instead of parseInt. The spec says that it's a number, not an integer, thus it may have floating point numbers. What do you think?

PR welcome. :)


move code from custom into their respective files

head_custom.html injects its contents into the head. Putting the navbar code there results in the browser automatically moving it to the body while complaining about  tags in . Additionally I've modified the styling a little to put a margin on top of the article-meta to eliminate the need for the  after the navbar.
I've also moved code from foot_custom.html to footer.html.

Groovy - thanks qqwui!!


Support callback

Codecov Report

Merging #11 (f2c0199) into main (2462cbf) will decrease coverage by 3.04%.
The diff coverage is 38.46%.

@@            Coverage Diff             @@
##             main      #11      +/-   ##
==========================================
- Coverage   52.88%   49.84%   -3.04%
==========================================
  Files           4        5       +1
  Lines         295      321      +26
  Branches        3        3
==========================================
+ Hits          156      160       +4
- Misses        139      161      +22

Files
Coverage Δ

mod.ts
52.70% <100.00%> (-0.74%)
:arrow_down:

kachaka/value_handler.ts
37.25% <37.25%> (ø)

:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!


Managed Instance preview status

This page states that Managed Instance is in Preview. This page states it is GA.
https://techcommunity.microsoft.com/t5/azure-sql-blog/data-virtualization-now-generally-available-in-azure-sql-managed/ba-p/3624292#:~:text=We are excited to announce the general availability,new supported option for authenticating to storage accounts.

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

ID: b7ebefef-f8c5-e429-269e-1ef7a96264fd
Version Independent ID: a8009495-9bb9-87f4-85af-0ac0074749e7
Content: Introducing data virtualization with PolyBase - SQL Server
Content Source: docs/relational-databases/polybase/polybase-guide.md
Service: sql
Sub-service: polybase
GitHub Login: @WilliamDAssafMSFT
Microsoft Alias: wiassaf

@mshelt -- Mike, thank you for catching this.  I am updating the article.


It seems that getting current price of storage in BZZ is missing in the API

Summary
It seems that getting current price of storage in BZZ is missing in the API.
Expected behavior
https://docs.ethswarm.org/api/ should contain an endpoint to obtain current storage price.
The documentation should explain whether it is per-byte-second, per-chunk-second, or whatever.

It is in undocumented /chainstate. I am closing this bug and opening a bug in bee-docs.

It is in undocumented /chainstate. I am closing this bug and opening a bug in bee-docs.

But apparently this issue is still open?  @vporton If the bee-docs issue has been opened, please close this one.

It should be in main API rather than in Debug API.

@agazso this issue has been closed with https://app.zenhub.com/workspace/o/ethersphere/bee/issues/3112


Fix prmopt prod skipping if has no tty

Changes

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

main

PR #390

PR #392   👈

This stack of pull requests is managed by Graphite.


How to limit the number of options for setting parameters to avoid Ddos-attacks?

For example, attacker can use something like that:
https://example.com/image.jpg?w=401
https://example.com/image.jpg?w=402
https://example.com/image.jpg?w=403
...
https://example.com/image.jpg?w=9999

I don't fully understand the request. Is your need to apply throttling on the number of incoming requests, for example, limiting each IP to 30 requests per minute?
In that case, there are plugins like caddy-ratelimit that would do the job.
Or are you imagining a limit (like a range) for certain parameters?
For example, specifying that the width can only be between 100px and 1000px.
The same could apply for other parameters.
This would help limit CPU time, as a request for 10_000px could easily consume a lot of CPU time and overload memory as well.
I also imagine putting a time limit on generation, to stop the process if it takes more than 5 seconds, for example. I need to look into this in more depth to see if it's possible, though I'm not sure.
If you have other ideas, I'm listening!

specifying that the width can only be between 100px and 1000px.

Yes, I meant it. But also with discrete step. For example, only this values may be applied - 100, 200, ..., 1000.

specifying that the width can only be between 100px and 1000px.

Yes, I meant it. But also with discrete step. For example, only this values may be applied - 100, 200, ..., 1000.

Okay, I will think about a solution for this.
It will take some time, and I think I will try to add several layers of control.
For now, I will work on implementing these two restrictions:

Disabling soecific parameters.
Limiting width and height within certain bounds. I will consider discrete values later.

We need to decide whether we prefer to return a 400 error if one of these conditions is not met, simply ignore it, or serve the original image by bypassing the processing.
I have other ideas for improving security, but they will take much longer to implement.

@monochromer
Implemented, see README and use 2.3.1 release, here is an example:

localhost:80 {
	import common
	root test-dataset
	file_server
	image_processor {
	    on_fail bypass
            security {
                on_security_fail bypass
                constraints {
                    h range 60 480
                    w {
                        values 60 130 240 480 637
                    }
                }
           }
       }
}

Thanks!


[FLINK-3722] [runtime] Don't / and % when sorting

Replace division and modulus with addition and subtraction.
The timing chart below has three columns for two Gelly algorithms with increasing scale. The first is timings for the master branch. The second column replaces division and modulus with shift-and-bitmask by storing each element in a power-of-2 chunk of memory. The third column is for this PR which modifies the sort algorithm to track the buffer number and offset for each index. This code has the advantage that there is no wasted memory.
In this comparison, for both int and long, we look to be already operating on power-of-2 size elements. I would expect this PR to perform relatively better with the FixedLengthRecordSorter instrumented from FLINK-4705 where shift-and-bitmask would be padding elements.
This initial PR does not modify HeapSort. I wanted to first get acceptance for the current set of modifications. There are further optimizations which can be benchmarked in follow-on tickets.
HITS, scale=16, INT   : runtime=   1.618 (8)      1.524 (8) ( -5.82%)   1.546 (8) ( -4.47%)
HITS, scale=16, LONG  : runtime=   1.579 (8)      1.552 (8) ( -1.76%)   1.541 (8) ( -2.43%)
HITS, scale=16, STRING: runtime=   1.774 (8)      1.739 (8) ( -1.99%)   1.701 (8) ( -4.11%)

HITS, scale=17, INT   : runtime=   2.032 (8)      1.950 (8) ( -4.06%)   1.937 (8) ( -4.71%)
HITS, scale=17, LONG  : runtime=   1.986 (8)      1.923 (8) ( -3.15%)   1.926 (8) ( -3.01%)
HITS, scale=17, STRING: runtime=   2.377 (8)      2.275 (8) ( -4.30%)   2.289 (8) ( -3.68%)

HITS, scale=18, INT   : runtime=   2.894 (8)      2.761 (8) ( -4.59%)   2.762 (8) ( -4.54%)
HITS, scale=18, LONG  : runtime=   2.863 (8)      2.754 (8) ( -3.81%)   2.735 (8) ( -4.48%)
HITS, scale=18, STRING: runtime=   3.683 (8)      3.455 (8) ( -6.18%)   3.461 (8) ( -6.02%)

HITS, scale=19, INT   : runtime=   4.914 (8)      4.631 (8) ( -5.76%)   4.595 (8) ( -6.49%)
HITS, scale=19, LONG  : runtime=   4.792 (8)      4.578 (8) ( -4.45%)   4.538 (8) ( -5.28%)
HITS, scale=19, STRING: runtime=   6.484 (8)      6.149 (8) ( -5.18%)   6.118 (8) ( -5.64%)

HITS, scale=20, INT   : runtime=   8.662 (8)      8.086 (8) ( -6.64%)   8.091 (8) ( -6.59%)
HITS, scale=20, LONG  : runtime=   8.407 (8)      7.993 (8) ( -4.93%)   7.918 (8) ( -5.82%)
HITS, scale=20, STRING: runtime=  11.946 (8)     11.327 (8) ( -5.18%)  11.272 (8) ( -5.64%)

HITS, scale=21, INT   : runtime=  16.248 (8)     15.137 (8) ( -6.84%)  15.207 (8) ( -6.41%)
HITS, scale=21, LONG  : runtime=  15.907 (8)     14.750 (8) ( -7.28%)  14.724 (8) ( -7.44%)
HITS, scale=21, STRING: runtime=  23.634 (8)     22.596 (8) ( -4.39%)  22.546 (8) ( -4.61%)

HITS, scale=22, INT   : runtime=  31.964 (8)     29.954 (8) ( -6.29%)  29.808 (8) ( -6.74%)
HITS, scale=22, LONG  : runtime=  31.244 (8)     29.241 (8) ( -6.41%)  28.829 (8) ( -7.73%)
HITS, scale=22, STRING: runtime=  47.870 (6)     45.942 (6) ( -4.03%)  45.987 (8) ( -3.93%)

HITS, scale=23, INT   : runtime=  65.146 (4)     60.245 (4) ( -7.52%)  60.008 (4) ( -7.89%)
HITS, scale=23, LONG  : runtime=  64.770 (4)     59.828 (4) ( -7.63%)  59.334 (4) ( -8.39%)
HITS, scale=23, STRING: runtime= 103.690 (2)     99.335 (2) ( -4.20%)  98.919 (3) ( -4.60%)

HITS, scale=24, INT   : runtime= 156.567 (2)    138.923 (2) (-11.27%) 141.185 (2) ( -9.82%)
HITS, scale=24, LONG  : runtime= 141.904 (2)    132.292 (2) ( -6.77%) 129.677 (2) ( -8.62%)
HITS, scale=24, STRING: runtime= 221.081 (1)    217.128 (1) ( -1.79%) 213.822 (1) ( -3.28%)

HITS, scale=25, INT   : runtime=                                      280.720 (1)
HITS, scale=25, LONG  : runtime= 321.418 (1)    297.764 (1) ( -7.36%) 293.982 (1) ( -8.54%)

JaccardIndex, scale=12, INT   : runtime=   0.413 (8)      0.424 (8) (  2.57%)   0.408 (8) ( -1.30%)
JaccardIndex, scale=12, LONG  : runtime=   0.440 (8)      0.414 (8) ( -5.85%)   0.424 (8) ( -3.64%)
JaccardIndex, scale=12, STRING: runtime=   0.667 (8)      0.647 (8) ( -2.91%)   0.644 (8) ( -3.38%)

JaccardIndex, scale=13, INT   : runtime=   0.811 (8)      0.743 (8) ( -8.45%)   0.764 (8) ( -5.75%)
JaccardIndex, scale=13, LONG  : runtime=   0.885 (8)      0.822 (8) ( -7.12%)   0.805 (8) ( -9.01%)
JaccardIndex, scale=13, STRING: runtime=   1.599 (8)      1.489 (8) ( -6.86%)   1.486 (8) ( -7.04%)

JaccardIndex, scale=14, INT   : runtime=   1.809 (8)      1.621 (8) (-10.43%)   1.649 (8) ( -8.87%)
JaccardIndex, scale=14, LONG  : runtime=   2.003 (8)      1.834 (8) ( -8.43%)   1.800 (8) (-10.13%)
JaccardIndex, scale=14, STRING: runtime=   3.855 (8)      3.666 (8) ( -4.91%)   3.667 (8) ( -4.86%)

JaccardIndex, scale=15, INT   : runtime=   4.693 (8)      4.158 (8) (-11.40%)   4.148 (8) (-11.62%)
JaccardIndex, scale=15, LONG  : runtime=   5.205 (8)      4.936 (8) ( -5.17%)   4.712 (8) ( -9.48%)
JaccardIndex, scale=15, STRING: runtime=  10.882 (8)     10.429 (8) ( -4.16%)  10.529 (8) ( -3.24%)

JaccardIndex, scale=16, INT   : runtime=  13.504 (8)     11.852 (8) (-12.24%)  12.124 (8) (-10.22%)
JaccardIndex, scale=16, LONG  : runtime=  14.933 (8)     13.482 (8) ( -9.72%)  13.279 (8) (-11.08%)
JaccardIndex, scale=16, STRING: runtime=  31.802 (8)     30.549 (8) ( -3.94%)  30.910 (8) ( -2.80%)

JaccardIndex, scale=17, INT   : runtime=  36.742 (8)     32.542 (8) (-11.43%)  33.193 (8) ( -9.66%)
JaccardIndex, scale=17, LONG  : runtime=  40.648 (6)     37.355 (6) ( -8.10%)  36.786 (8) ( -9.50%)
JaccardIndex, scale=17, STRING: runtime=  89.418 (4)     84.632 (4) ( -5.35%)  86.596 (4) ( -3.16%)

JaccardIndex, scale=18, INT   : runtime= 102.126 (4)     91.521 (4) (-10.38%)  90.900 (4) (-10.99%)
JaccardIndex, scale=18, LONG  : runtime= 121.389 (3)    113.324 (3) ( -6.64%) 112.323 (4) ( -7.47%)
JaccardIndex, scale=18, STRING: runtime= 251.373 (2)    240.014 (2) ( -4.52%) 243.849 (2) ( -2.99%)

JaccardIndex, scale=19, INT   : runtime= 286.927 (1)    254.767 (1) (-11.21%) 260.911 (2) ( -9.07%)
JaccardIndex, scale=19, LONG  : runtime= 331.742 (1)    303.246 (1) ( -8.59%) 301.764 (2) ( -9.04%)

Thanks for these cool experiments. Looks like a nice improvement.
I'll try to dig into your code in the next days...

Thanks @ggevay for reviewing. I added a commit with additional comments.
I transcribed Quicksort so as to remove considerations of Java performance and inlining. It was not clear to me that if we encapsulated the index, page number, and page offset into an object that Java would inline the various increment and decrement functions. Also, I don't think this looks too bad. I'm happy to reformat if that is preferred.
I think this is the best time to investigate alternative methods. I'm not seeing how one would sort on top of InMemorySorter without deserializing records.

I transcribed Quicksort so as to remove considerations of Java performance and inlining. It was not clear to me that if we encapsulated the index, page number, and page offset into an object that Java would inline the various increment and decrement functions. Also, I don't think this looks too bad. I'm happy to reformat if that is preferred.

OK, I would say that it is OK like this, but let's see what the others will say.


Any hopes for a GDScript implementation?

i'm curious whether it is possible to implement this in the base godot 4.3 version without the .net stuff?

@mrSuave00 I'm not interested in re-implementing it in GDscript, but it's totally possible to do if someone decides to.
After all, original plugin for Godot 3 was done in two versions, GD and C#. You could try migrate it to Godot 4 with auto-script updater (not guaranteed success) or manually, but it obviously would not include all the changes that I have introduced here. Here's the link: Verlet Rope GD, if you are interested.
Overall: the .NET is more suitable for computations; the two versions of the plugin is just harder to support; and you can anyway add this plugin to any project (at the cost of including .NET runtime). So, I don't really see any reason to spend time migrating it (ignoring the fact that I just don't like GDScript, so it would not happen by me anyway).
Hope it clarifies things for you :)


Autoscroll not working in chrome browser greater than version 61

I forked this repo and solved scrolling issue
https://github.com/darron1217/enjoyhint.js
check it out :)

**please add previous button on tour...
its important for this product.....**


Question about Lane IoU implementation

i see that in your paper u said "For CULane dataset, we crop the input image below y = 270 and resize it to (800, 320) pixels"
but why in Lane IoU calculation, i see that u set default img_h=320, img_w=1640. And did you initialize prior exacly like CLRNet ?

We wanted to define the virtual lanes to calculated LaneIoU so that the width of the lanes are the same as the evaluation metric. The segmentation-based IoU calculation in the metric is done in (1640, 590) resolution.
https://github.com/hirotomusiker/CLRerNet/blob/main/libs/datasets/metrics/culane_metric.py#L22
As for prior initialization, we employ CLRNet's initial anchors.
https://github.com/hirotomusiker/CLRerNet/blob/feature/train/libs/core/anchor/anchor_generator.py#L31
Thank you.

thanks for fast reply <3. i want to ask 1 more question is that i see you set self.max_dx = 1e4. i want to ask what is this value and how do you find this?

The taget (GT) lanes have various lengths, and the rows where there are no lane labels (e.g. above the road) are set to -1e5.
https://github.com/hirotomusiker/CLRerNet/blob/ec392b95b8a076733bb879fc6b098e345a4d93f5/libs/datasets/pipelines/lane_formatting.py#L36-L38
So the following calculation should be very large value at the edge of the lane.
target_dx = (target[:, 2:] - target[:, :-2]) * self.img_w

target[0] :
 9.6844e+00,  9.6848e+00, -2.0619e+05,  -2.0620e+05,  0.0000e+00,  0.0000e+00,

We suppress the large value like -2.0620e+05 to zero to avoid erroneous target_width values.
max_dx can be set to a different value as long as it can filter the above large values.
We will add a comment on the max_dx part. Thank you.


Available to handle exception

I have problem using SeqUtils.
In case of passing exception as argument,  the argument is evaluated immediately and throw exception.
Seq(1).emptyOrElse(throw new Exception)
// => throw Exception

So, I think to use by-name parameter for argument type.
Thank you.

Thank you!
That's Great!
👍

Not a valid command: checkScalariform...


Mention exp apply --no-force option in "Persisting Experiments"

The option is only mentioned in the command reference but we don't have examples or motivations behind the usage.
I think it would be nice to add a mention to the option in the "Persisting Experiments" user guide.

@pmrowla I hadn't noticed the --no-force argument. On reflection, switching between experiments would necessitate use of a --force flag if it existed, and I guess that use-case is considered the most common.
I will monitor how I use exp apply for now, and whether I can easily encorporate --no-force into my workflow.
Originally posted by @mattlbeck in https://github.com/iterative/dvc/issues/6930#issuecomment-970712018

I'm surprised it's not in https://dvc.org/doc/user-guide/experiment-management/persisting-experiments. I guess we skipped some options @iesahin ?
This has previously been discussed in https://github.com/iterative/dvc.org/issues/2397 so I'm going to merge the issues... ⌛

Hmmm actually that other issue should be closed as it referred to the cmd ref. This one can be about the user guide.

So, I think the default behavior should be definitely noted (that it destroys current changes i.e. the Git working tree) and it is (in the ref), but we could rephrase to make it clearer there.
We should add this explanation to the guide as well. I'm surprised it was left out of https://dvc.org/doc/user-guide/experiment-management/persisting-experiments actually. Did we leave out many other options from guides @iesahin ? Thanks

As far as I can tell, there is none.


Remove non-node support

It's 2016. Everyone is using a build process. Let's remove bower and component support, and use proper npm dependencies instead of homegrown methods.

I intend to merge this tonight if there's no objections - it will be a semver-major.

LGTM as a semver-major.  Onwards to the future!

Onward indeed.


Line numbers don't line up in gutter with ProggyTiny font

I'm trying to use the ProggyTiny font with Atom.
It's a bitmap font, i.e. designed to be viewed at a certain size. When I set the size to 16 (the only size where it looks right - I usually need to use 12 with other applications), the line spacing is much too big:

When setting the line spacing to 1.0 instead of the default 1.5, the line numbers start to act funny:

It's even worse with 0.8:

This is with Atom 1.0.19 on Debian Jessie, and I see this with the default themes (tried "Atom Dark" and "One Dark").

Hi, can you try this:

Open **File **> Settings from the menu
Click **Open Config Folder **
Open styles.less from the left pane
Modify the atom-text-editor block so it shows:

atom-text-editor {
   font-size: 12pt;
}

And let me know if that looks any better.

I'm unfortunately not using Atom anymore, so I'd be glad if someone else could try to reproduce it. Otherwise I'll try myself, but it might take me some days.

I just discovered I still had Atom installed at work, so I could give this a quick try - unfortunately that doesn't help, it looks the same.
Also I'm not sure why this has the "windows" label seeing that I have this on Debian Jessie :wink:

A similar bug appears when I set the font-size of line number to smaller number, for example 12px. Pls look at the screenshot of this bug.

Why is this issue closed if this exact problem persists? ... closing the issue does not resolve the issue .. .. i mean like .. really? ..amazing.. it's like a fault is reported in a nuclear reactor; to resolve the issue - the issue is trashed .. bravo to the automatic genius - get rid of the issue - no issue! :dancer:

For ProggyTiny fonts, the following [SOLVED] the issue:
.editor
{
    font-family: 'ProggyTinyTT';
    font-size: 16px;
    line-height: 17px;
}

In English:  because ProggyTiny fonts look best @ 16px, setting the line-height to font-size + 1px solves the issue (kind-of) .. if you're not too fussed about exact line-height -it's fine. .. must be a percentage (em?) scaling thing in the gutter, not sure, assumption is.. dangerous.. anyway, I hope it helps someone.

@CharlSteynberg, if you read the message from the bot that closed the issue, it tells you exactly why it was closed. Since it is still reproducable, I would expect it to get re-opened.
As for work-around. I use line-number padding to correct for fonts that don't line up.
atom-text-editor {
  .line-number{
    padding-top: 0.14em;
    padding-bottom: 0.14em;
  }
}

@damieng Hmm... I was going to argue with the claim that it was only on one font, because I deal with a similar issue.
But with a little testing before posting, I remembered that my misaligned line-numbers were caused by changing the font-size of the scroll-view (so that Ctrl-0 would zoom to my preferred font-size instead of default 14px).
// causes line-number mis-alignment for any font
`atom-text-editor .scroll-view {
  font-size: 1.21em;
}

So at the moment, I am unable to reproduce OP's case with another font.

@skylize - yup -but i like shaming bots, help's 'em learn I guess. .. can't wait to meet Sophia :)


Feature added

Now, the google clone can be in any language.

Hey Kushal,
Just wanted to update you that I have made the requested changes to the
pull request. Could you please take another look at it?
Thanks,
Payal
On Wed, Oct 11, 2023 at 9:24 AM Kushal Das @.***> wrote:

@.**** requested changes on this pull request.
@payallenka https://github.com/payallenka Kindly raise the issue first
then raise a PR , make sure describe what you have changes there .
—
Reply to this email directly, view it on GitHub
https://github.com/Kushal997-das/Project-Guidance/pull/929#pullrequestreview-1669791033,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AT6EWI6RBUH73JP2YPFKJALX6YJ5XAVCNFSM6AAAAAA5ZNK6COVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTMNRZG44TCMBTGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hello @Kushal997-das I don't have an idea on deployment but I'll add the link in the Readme file. Will that be okay?

Just add a the link in readme. md file
On Thu, 12 Oct, 2023, 9:32 pm payallenka, @.***> wrote:

Hello @Kushal997-das https://github.com/Kushal997-das I don't have an
idea on deployment but I'll add the link in the Readme file. Will that be
okay?
—
Reply to this email directly, view it on GitHub
https://github.com/Kushal997-das/Project-Guidance/pull/929#issuecomment-1759920514,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOUDPZPIBAXFAVHHEHSVKXDX7AICNANCNFSM6AAAAAA5ZNK6CM
.
You are receiving this because you were mentioned.Message ID:
@.***>

The link is already added in the Readme.md file. The changes will be visible once the PR is merged in the main project.
Thank you.

The link is already added in the Readme.md file. The changes will be visible once the PR is merged in the main project. Thank you.

The updated link is not present, here you need to deploy this project and then put the link in the readme.md file in order to see others.

Yeah, and i actually don’t know how to deploy it. Yet to learn it. Is there
anything else I can do?
On Sat, 14 Oct 2023 at 9:51 AM, Kushal Das @.***> wrote:

The link is already added in the Readme.md file. The changes will be
visible once the PR is merged in the main project. Thank you.
The updated link is not present, here you need to deploy this project and
then put the link in the readme.md file in order to see others.
—
Reply to this email directly, view it on GitHub
https://github.com/Kushal997-das/Project-Guidance/pull/929#issuecomment-1762590959,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AT6EWIZC4QEDDVACBSBEM6TX7IHLZANCNFSM6AAAAAA5ZNK6CM
.
 Yet to learn it. Is there anything else I can do?
…
On Sat, 14 Oct 2023 at 9:51 AM, Kushal Das @.> wrote: The link is already added in the Readme.md file. The changes will be visible once the PR is merged in the main project. Thank you. The updated link is not present, here you need to deploy this project and then put the link in the readme.md file in order to see others. — Reply to this email directly, view it on GitHub <#929 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AT6EWIZC4QEDDVACBSBEM6TX7IHLZANCNFSM6AAAAAA5ZNK6CM . You are receiving this because you were mentioned.Message ID: @.>

@payallenka Hello Look into this

@Kushal997-das

a little guidance here please

Hello kindly message me on LinkedIn
On Tue, 17 Oct, 2023, 8:49 pm payallenka, @.***> wrote:

@Kushal997-das https://github.com/Kushal997-das
[image: image]
https://user-images.githubusercontent.com/83643171/275885065-c91ff96b-757d-48e4-ac23-3030b873e025.png
a little guidance here please
—
Reply to this email directly, view it on GitHub
https://github.com/Kushal997-das/Project-Guidance/pull/929#issuecomment-1766637653,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOUDPZOGLOLQXF43SQBDYHLX72OZDAVCNFSM6AAAAAA5ZNK6COVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRWGYZTONRVGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

@payallenka Update on this else your PR gonna be close.

Please do, I am unable to message you over LinkedIn. Apologies for this
inconvenience.
On Fri, 20 Oct 2023 at 9:21 AM, Kushal Das @.***> wrote:

@payallenka https://github.com/payallenka Update on this else your PR
gonna be close.
—
Reply to this email directly, view it on GitHub
https://github.com/Kushal997-das/Project-Guidance/pull/929#issuecomment-1772035274,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AT6EWI6WNF7G2AAK3UKAM23YAHYMDAVCNFSM6AAAAAA5ZNK6COVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZSGAZTKMRXGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Allow to open either side of a diff editor as editor (fix #153786)

This is an attempt to implement https://github.com/microsoft/vscode/issues/153786 by offering a way of going back to a single window from a diff. It should also be possible to display either side of the diff in a single window (other editors allow this).
When doing code reviews on GitHub you can open a file fully for context, then click back into the diff when done viewing. It should be just as easy to do this in VSCode.
This change would greatly improve productivity as it helps with navigating around changes and not needing to learn different keybindings/commands from other extensions.
Things that are needed:

~To know which side of the diff has focus so I can open the right file, currently this doesn't seem possible as neither textDiffEditor nor diffEditorInput support showing which side has focus.~
~Back currently doesn't work, once you open up an editor you can't go back to the diff view~ Addressed in https://github.com/microsoft/vscode/issues/166805
~Eventually an opportunity for a keybinding to open a window for each split you’re focused on. This increases productivity quite a bit as you can view the full file for context then go back to the diff.~

Thanks for the PR, I can not guarantee a speedy reply on this, please be patient.

I've made some good progress on this, thanks to @eamodio's help.
The only issue remaining is you can't go back to a diff editor (I'm not sure why history doesn't support it).
This also sets up the unused ctrl/cmd + k, shift + O to open either side.
Here'

s a gif:

As this bug seems to have always been there and is separate to this, ive raised a new issue
Awaiting @bpasero to advise.

@bpasero you're right that looks like a mistake, I think I was experimenting there and didn't change it back before committing.
I've both fixed it and rebased.

Hey @bpasero let me know what you think of the changes, I've hopefully addressed both of your points above.

Thanks. I cleaned it up a little.

Thanks for reviewing @bpasero
Are you able to offer any guidance to https://github.com/microsoft/vscode/issues/166805? It feels a bit broken that you can navigate to a diff side but can’t go back.
Is there a reason diffEditors were never added to the history stack? It would be useful to fix this so navigation properly works (for reviewing changes between diff and full file).

I think we only keep editors in the history that can be addressed via plain old JS objects, not fully heavy typed editor inputs. We cannot know for sure that a closed editor can be restored from its EditorInput because it is disposed after being closed.


Error on "split" call

Hi, can you check this out?
┌────────────────────────────────────────────────────────────────────┐
│ Tinder Ruby AutoLiker v2.0                                         │
├────────────────────────────────────────────────────────────────────┤
│ Copyright © 2014-2016 Maxime Alay-Eddine @maximeae                 │
├────────────────────────────────────────────────────────────────────┤
│ Licensed under the MIT license.                                    │
└────────────────────────────────────────────────────────────────────┘
==== FACEBOOK ====
* Fetching Facebook data...
  - Fetching your Facebook Tinder token...
TinderAutoLike.rb:66:in `<main>': undefined method `split' for nil:NilClass (NoMethodError)

This project motivates me to learn Ruby :)

Hi
Have you filled the following part:
# -------------
# CONFIGURATION
# -------------
# Please insert your Facebook credentials below. They won't be sent to anybody except Facebook servers.
myLogin = 'YOUR_FACEBOOK_EMAIL_ADDRESS'
myPassword = 'YOUR_FACEBOOK_PASSWORD'
# We use them to connect you and to fetch your Facebook Tinder token, in order to authenticate with the Tinder server.

Best,

Hi,
Can't reproduce & closing. Likely either of the following has occurred during my experiments:

login credentials weren't filled properly (not at all or with typos)
some gems were disabled temporarily (I was playing with them)

Thanks!
cukubik

@cukubik Hi got the same error, my creditals are good so what is wrong ?


response instance of http.IncomingMessage

This make the response object returned on request responses be an instance of http.IncomingMessage
https://github.com/joyent/node/blob/master/lib/http.js#L247
https://github.com/joyent/node/blob/v0.4.12/lib/http.js#L209
The request object is already an instance of http.ClientRequest, why shouldn't it be the same for response?
Also, some programs check if a given stream like object is actually an instance of a stream. With this patch that is true, and there is no need to keep a version of the stream pipe method in its own file.

@fent Awesome again, thanks!


Add video recording preparation for Android

This feature is not included in DeepAR Android SDK (3.4.2).


Create a man page for gl

man gl should work

A tldr page would also be nice :)

https://github.com/gitless-vcs/gitless/pull/233
I created a man page, but installation / package management integration would require work from the side of the package maintainers.


Remove references to FireCloud from Terra.

We are trying to teach users that "it’s best to now think of “FireCloud” as synonymous with the NCI-sponsored project page that features TCGA and TARGET data". We therefore wish to remove any written references to FireCloud in Terra.
These references occur in the following places:
Dataset library.  Rename "FireCloud Dataset Library" to "Broad Data Workspaces"
Code&Tools library. Rename "FireCloud Methods Repository" to "Broad Methods Repository"
Workspace tool cards. Rename "Source: FireCloud" to "Source: Methods Repository"
Scan the code for other instances where the name "FireCloud" is presented to the user. If additional locations are found, suggest an appropriate renaming that omits the word "FireCloud".

Some discussion ongoing about whether we want to make this change at this time.

Fixed by #1347


Components -> Cliques in Dedupe class

If we don't learn blocking rules that lead to the comparison of every true dupes, we can get comparisons that look like this
record 10 - record 11 : 90% match
record 11 - record 12:  95% match
but no comparison for record 10 - record 11
Right now, in the clustering stage, we act as we did compare record 10 - record 12, and that the classifier said there was a 0% chance of matching. This often leads the cluster to put these three records into two clusters instead of one.
This is certainly not the optimal strategy.
Alternatives are to

Assume transitivity and calculate the transitive score for non-compared records within a connected component. So, we would assume that record 10 - record 12 would have a comparison score of .90 * .95 = 0.8550. This would be pretty easy.
After doing all the blocking and comparing all the records, find the connected components. Identify all the missing comparisons within a component and the compare those. This probably the most correct thing to do, but would require a significant changes to the code.

This PR has a hack to identify all the missing comparisons in a connected component and compare those. Right now the API for this is unacceptable, but this can either evolved into better code that uses this approach or it can provide the baseline against which we can compare the transitivity approach.

attn @mbauman, the approach currently in the code does seem to really improve recall with very modest recall loss.

Thanks for tackling this so quickly. I'll try this branch on my synthetic dataset next week.

I just tried this out, and I'm not seeing a big difference on Python 2. With 20k true identities and 410k messy rows, I'm still seeing < 0.5% false positive rate and 15-25% false negative rates.
On Python 3 I get an IndexError:
---------------------------------------------------------------------------
IndexError                                Traceback (most recent call last)
/Users/mbauman/dssg/superdeduper/.env/lib/python3.5/site-packages/dedupe-1.6.7-py3.5-macosx-10.10-x86_64.egg/dedupe/api.py in matchBlocks(self, blocks, threshold, *args, **kwargs)
    117         logger.debug("matching done, begin clustering")
    118
--> 119         clusters = list(self._cluster(matches, threshold, *args, **kwargs))
    120
    121         try:

/Users/mbauman/dssg/superdeduper/.env/lib/python3.5/site-packages/dedupe-1.6.7-py3.5-macosx-10.10-x86_64.egg/dedupe/clustering.py in cluster(dupe_components, threshold)
    134         if len(sub_graph) > 1:
    135
--> 136             i_to_id, condensed_distances, N = condensedDistance(sub_graph)
    137
    138             linkage = fastcluster.linkage(condensed_distances,

/Users/mbauman/dssg/superdeduper/.env/lib/python3.5/site-packages/dedupe-1.6.7-py3.5-macosx-10.10-x86_64.egg/dedupe/clustering.py in condensedDistance(dupes)
    103
    104     ids = candidate_set.searchsorted(dupes['pairs'])
--> 105     row = ids[:, 0]
    106     col = ids[:, 1]
    107

IndexError: too many indices for array
> /Users/mbauman/dssg/superdeduper/.env/lib/python3.5/site-packages/dedupe-1.6.7-py3.5-macosx-10.10-x86_64.egg/dedupe/clustering.py(105)condensedDistance()
    103
    104     ids = candidate_set.searchsorted(dupes['pairs'])
--> 105     row = ids[:, 0]
    106     col = ids[:, 1]
    107

ipdb> p ids
array([0, 1])

it looks like you aren't running the code that's in this branch under
python3
On Mon, Apr 17, 2017 at 5:46 PM, Matt Bauman<EMAIL_ADDRESS>wrote:

I just tried this out, and I'm not seeing a big difference on Python 2.
With 20k true identities and 410k messy rows, I'm still seeing < 0.5% false
positive rate and 15-25% false negative rates.
On Python 3 I get an IndexError:

IndexError                                Traceback (most recent call last)
/Users/mbauman/dssg/superdeduper/.env/lib/python3.5/site-packages/dedupe-1.6.7-py3.5-macosx-10.10-x86_64.egg/dedupe/api.py in matchBlocks(self, blocks, threshold, *args, **kwargs)
117         logger.debug("matching done, begin clustering")
118
--> 119         clusters = list(self._cluster(matches, threshold, *args, **kwargs))
120
121         try:
/Users/mbauman/dssg/superdeduper/.env/lib/python3.5/site-packages/dedupe-1.6.7-py3.5-macosx-10.10-x86_64.egg/dedupe/clustering.py in cluster(dupe_components, threshold)
134         if len(sub_graph) > 1:
135
--> 136             i_to_id, condensed_distances, N = condensedDistance(sub_graph)
137
138             linkage = fastcluster.linkage(condensed_distances,
/Users/mbauman/dssg/superdeduper/.env/lib/python3.5/site-packages/dedupe-1.6.7-py3.5-macosx-10.10-x86_64.egg/dedupe/clustering.py in condensedDistance(dupes)
103
104     ids = candidate_set.searchsorted(dupes['pairs'])
--> 105     row = ids[:, 0]
106     col = ids[:, 1]
107
IndexError: too many indices for array

/Users/mbauman/dssg/superdeduper/.env/lib/python3.5/site-packages/dedupe-1.6.7-py3.5-macosx-10.10-x86_64.egg/dedupe/clustering.py(105)condensedDistance()
103
104     ids = candidate_set.searchsorted(dupes['pairs'])
--> 105     row = ids[:, 0]
106     col = ids[:, 1]
107

ipdb> p ids
array([0, 1])
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dedupeio/dedupe/pull/544#issuecomment-294617638, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAgxbR-kBP8gMVTwou9Nr5skdPTUBl_mks5rw-vWgaJpZM4M9HuK
.

--
773.888.2718

@mbauman could you check to see if you are still running into the indexerror problem?

Yes, I'm still seeing the same error with your latest update.  And you were right — I had it installed incorrectly on Python 2.
IndexError: too many indices for array
> /Users/mbauman/dssg/dedupe/dedupe/clustering.py(105)condensedDistance()
    103
    104     ids = candidate_set.searchsorted(dupes['pairs'])
--> 105     row = ids[:, 0]
    106     col = ids[:, 1]
    107

ipdb> p ids
array([0, 1])
ipdb> p dupes['pairs']
array([3627, 3628])

okay, @mbauman I think the issue is that you are directly calling matchBlocks, and this hacked up version breaks matchBlocks. only the match method of Dedupe class is working (and densifying the connected components into cliques)

What are the next steps here?  Are you planning on doing any further work or should I try to block out some time to work on incorporating this into the Postgres dedupe example/pgdedupe?

I think I'm going to pursue #552 instead of the approach here.

Makes sense!  Are you planning on doing any postgres/database work there?

no. I think it's very unlikely there will ever be any database stuff in the
core library.
On Tue, May 16, 2017 at 12:10 PM, Matt Bauman<EMAIL_ADDRESS>wrote:

Makes sense! Are you planning on doing any postgres/database work there?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dedupeio/dedupe/pull/544#issuecomment-301850298, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAgxbUUHNDDyha_Wph6644oJXmuhXTjTks5r6dh4gaJpZM4M9HuK
.

--
773.888.2718


As a budget admin, I need to be able to pull a shopping cart report that will display service, quantity invoiced, total invoiced, date invoiced, which PI (account owner) and chart string.

Please add date filters (from: to:) based on transaction dates.

MIght also be useful to have an all rows all columns export of the orders table for a given cluster.