Getting couple of Xcode warnings with Xcode version 9.1

Hi,
We are using the latest version of CDTDatastore 1.2.2 however are getting a few warnings related to CDTDatastore while building the app.
We are using Swift 4 and have use_frameworks! in place. I have attached the screenshots below:

@navaneet tried to reproduce in XCode 9.1 and didn't get a warning - shown below are the signatures for all of the delegate methods
class MyDel : NSObject, CDTReplicatorDelegate
{
    func replicatorDidComplete(_ replicator: CDTReplicator) {
    }

    func replicatorDidError(_ replicator: CDTReplicator?, info: Error) {
    }

    func replicatorDidChangeState(_ replicator: CDTReplicator) {
    }

    func replicatorDidChangeProgress(_ replicator: CDTReplicator) {
    }
}

K let me try something, will provide you with more details tomorrow as to why it might be showing up. I did see somewhere that there might be an xcode bug related to the above mentioned warning.

@tomblench I guess the issue is similar to what I saw in stackoverflow https://stackoverflow.com/questions/39495773/xcode-8-warning-instance-method-nearly-matches-optional-requirement

OK, closing


Added sorting to the stats' table.

Commit to fulfill feature #329 .

+1
Merged this into my build and works fine 👍


Don't Clear mSecureSessionType on PairingSession::Clear()

Problem
#13711
Change overview
There is no need to clear the session type because it doesn't change.
I believe the more appropriate fix will be implemented with #9059. The Clear() method is not needed and a new session instance should be allocated and the old one released when required.
Testing
existing tests

fast track: small delta, several checkmars, PR has been up for sufficient time for cross timezone review.


Consider whether we should have an add row button in every row of slice and table views

Also, we need to ensure that the top level add button is added to the AppBar when it needs to be, and that the add and remove button headers of the table are not there when they shouldn't be. Furthermore, we might want to get rid of the delete button in each row of the slice/table and instead have it behind a context menu.

We will remove the add and delete buttons in each row and put them in the context menu instead. Also, we need to fix the slice view context menu not working for editable slice and table views, which is somewhat related to #672 and #681.


EN-35641 Parameterize timeout for deleteByQuery calls

This parameterizes the timeout for deleteByQuery requests, which are another source of the timeouts
that we've been observing in the new ES 7 secondary watcher.

[x] styleCheck passes
[x] unit tests pass
[x] integration tests pass
[ ] manual tests pass
- [ ] API docs updated

This build looks good.


feat: added more properties in pex2 schema

included mso_mdoc as valid format identifier (https://openid.net/specs/openid-4-verifiable-presentations-1_0.html#name-iso-mobile-driving-license-)
added format as input_descriptor's property (https://identity.foundation/presentation-exchange/spec/v2.0.0/#presentation-definition-in-an-envelope)
added intent_to_retain as field property (https://identity.foundation/presentation-exchange/spec/v2.0.0/#retention-feature)

great! If you are updating to latest V2, please feel free to ignore/delete this PR

No this needs to be in anyway. Thanks


Issue with _docs/reference/config/traffic-rules/routing-rules.md

Currently this states:

Note 1: The keys uri, scheme, method, and authority correspond to URI, protocol scheme (e.g., HTTP, HTTPS), HTTP method (e.g., GET, POST), and the HTTP Host header respectively.

The data attribute bag actually passes these prefixed with a colon (I have no idea why), so if you're writing a RouteRule like this:
# omitted for brevity
spec:
  destination:
    name: workspacer
  match:
    request:
      headers:
        authority:
          prefix: bobbytables
  precedence: 100
  route:
  - destination:
      name: dev-workspacer
    weight: 100

You'll likely be annoyed when you discover this is your issue:
      headers:
-        authority:
+        :authority:
          prefix: bobbytables

cc @ijsnellf / @GregHanson


compile flag to disable 'use of internal package not allowed'

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
1.7.1
What operating system and processor architecture are you using (go env)?
linux
amd64
What did you do?
I have a code generating tool which creates a main that imports (almost) all pkgs it inspects.
This generated code is deliberately ignoring all "internal" boundaries otherwise respected in normal code, running into imports XXX/internal/XXX: use of internal package not allowed error when compiling.
I would like to add a flag to the go compile command to disable the "internal" check.
Disabling the use of disallowInternal is enough to stop the issue.
I currently run a modified go build with this disabled but I imagine others would benefit from having this option - even if normally discouraged.
What did you expect to see?
What did you see instead?

Sorry, we're not keen on adding flags to begin with, much less to support behavior that we don't want to encourage.
You clearly need something, but I don't understand your problem enough to suggest what it might be. This feature request sounds like it might be an XY problem (http://xyproblem.info/) Let's understand the problem before we jump to solutions.
The mailing list or slack (see https://golang.org/wiki/Questions) would be a better forum.

Thank you - presented my issue in further detail in the Go Forum


Targeting netcoreapp3.1

Does netcoreapp3.1 target really required for the package? Looks like netstandard2.1 will be sufficient.

.NET Standard doesn't include System.Text.Json, and therefore, cannot be used as a minimal required version. That library was started only since .NET Core 3.0, but because of it's deprecation the minimal version of .NET is Core 3.1.

System.Text.Json is a separate package, which supports even .NET Framework 4.6.1 https://www.nuget.org/packages/System.Text.Json/4.6.0#supportedframeworks-body-tab

Then MSDN has wrong information about the minimal required version. In that case I can target it to a lower version.

Just add PackageReference to the System.Text.Json and target netstandard2.1

Not so easy actually because I used range and index types, so need to adjust code a bit to be .NET Standard compatible. It will be ready today with some bug fixes.

range and index types are compatible with netstandard2.1

Yeah, but I used .NET Standard 2.0 as System.Text.Json since the same code is proposed to be in .NET itself (no idea when it happen) and it would be easier to maintain if that repo and a PR moving it to .NET will contain the same code.
In case you're interested in the pull request on moving it to .NET, here is it: dotnet/runtime#69613.

Fixed via 82974163fb3a48fcffb9ef3fc60c4a3b91549640, thank you for bringing it up! A new version will be ready as soon as I address changes requested by @NinoFloris in the .NET repo here and there.


Simulated index abduction range of motion is wrong

While making tests with the ergoCub hand in simulation, we realized that the range of motion of the index abduction has been reported in the URDF in the wrong way. In the CAD, this joint can move from -3 degrees to + 12 degrees. Hence, the overall RoM is 15 degrees that - I guess - we hope to command YARP-side going from 0 to 15 degrees.
At the moment we see the following. In the video we are comparing the URDF model with a non-actuated mesh of the whole hand with the thumb abduction fully opened up to the mechanical limit:
https://github.com/icub-tech-iit/ergocub-design-lowerarm/assets/6014499/91512c2a-b468-4759-b4d4-e63b50f61c4c
As can be seen, the extra +3 degrees of range of motion are there, but in the wrong place. Hence, I think the "0" configuration within the URDF should be moved back by 3 degrees.
Hope the description above was clear, available for further clarifications.
@salvi-mattia @Lawproto @Nicogene @mfussi66 @traversaro

cc @fiorisi


Make it possible to remove timeout altogether

Some cases in SMS app requests are resolve from the network, this means that overall round trip time is unknown and could take several seconds. In this case it's best to never timeout.
var client = threads.client({
  service: 'my-service'
  endpoint: myEndpoint,
  timeout: false
});

timeout: 0 would work for me too

@steveck-chung volunteered to work on this :)

WIP in https://github.com/gaia-components/threads/pull/75

Not totally related to this issue: but I'm wondering if we should have separate timeouts for the connection and for the method call like some other web-services-in-the-wild have?
Currently I'm suffering from connection timeout when I refresh page with ctrl-f5 with our 2 hops communication channel, sometimes 1000ms is not enough for shared worker to start and initialize service. But at the same time method calls should be very fast and big global timeout can mask some potential performance issues for method calls.
Not really sure, but still feel a bit concerned.
What do you guys think?

I think it makes sense.
but tbh I'm not sure we need a timeout at all, ever... what exactly is the purpose of a timeout ?

but tbh I'm not sure we need a timeout at all, ever... what exactly is the purpose of a timeout ?

Mmm, good question :)
Here is where I-am-as-a-developer see timeout is useful - If client can't connect to the service it timeouts and we see that "connection" can't be established due to timeout, if connection is established, but method is not responding we see that "method" is not responding - easier to understand where the problem is. Connection is done under the hood when we call method - so with never-resolved-promise we can't say what's going on exactly.
Also Potentially clean-up/re-try/re-connect logic can be applied inside bridge automatically if service connection is time-outed - not really an argument since I'm not sure if it's practically needed and don't have real use cases (e.g. one of the ideas to handle SharedWorker->multiple iframes (one per app instance) was to automatically re-connect to the first available service-in-iframe that responds to connection request via BroadcastChannel in case previous time-outed, but we decided to not do it as it's impractical and damn complex :)).


camelCase directive in html referenced as kebab-case, The kebab-case shall be camel-case

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
What is the current behavior? (You can also link to an open issue here)
What is the new behavior (if this is a feature change)?
Does this PR introduce a breaking change?
Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines: https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#commit-message-format
[ ] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

Other information:
In line no 71 and 72 camelCase is directive name and in HTML the it is said that it should be kebab-case where as it should be 'camel-case'.
I felt that beginners who is trying to learn from tutorial can get confused.

Hm, I see the problem. We should use an example name for the component then, instead of camelCase, kebab-case, because the new example can also be confusing.

Hi Narretz,
travis failed what could be the reason, I just edited the doc. Is there is any problem from my side?

Thx for the PR, but this change is invalid. The used case is indeed called kebab-case.

I think it's best to add links to definitions of camel/kebab case.

I am not getting if I create a component with name 'camelCase' can I call that component in html by using 'kebab-case'?

The names are not the real component names.  And when using it in the template do: my-custom-component

cb31067 hopefully improves the explanation of casing.

Thanks for accepting the suggestion :+1: :)


Relocate init-preload-local

Thanks for your effort. Your Emacs dots facilitate my journey.
I was using straight to manage some packages, and recently decided to give it a try to elpaca.
On their installer code they mention it has to be loaded early into the init.el.
I've found init-preload-local.el and the issue #199, but it's being loaded after 'init-elpa' which makes elpaca to throw a warning.
I found that moving:
https://github.com/purcell/emacs.d/blob/adf0e27c95cae4e19e2fee174170bac2ff7e83d3/init.el#L56-L57
In between of:
https://github.com/purcell/emacs.d/blob/adf0e27c95cae4e19e2fee174170bac2ff7e83d3/init.el#L19-L21
Will reduce the warnings and keep the dots working as usual, at least with the default configs.
Would this change create any trouble?

I think that would break configurations for people who expect the package system to have been initialised when init-preload-local is loaded. But you could always modify init.el directly in your fork — I don't intend these init*-local.el extension points to cover all potential use cases anyway.


compress/zlib: compressed data is different from zlib-flate

What version of Go are you using (go version)?

$ go version
go version go1.18.2 linux/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE="on"
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/aagu/.cache/go-build"
GOENV="/home/aagu/.config/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/aagu/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/aagu/go"
GOPRIVATE=""
GOPROXY="https://goproxy.cn,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.18.2"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/dev/null"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build718718165=/tmp/go-build -gno-record-gcc-switches"

What did you do?

I am learning about the git object storage recently. I tried to use the compress/zlib to implment the git hash-object command, the snippet of exmple code could be found at https://go.dev/play/p/_a_CGxRHTcy. The compress data will be write to the .git/objects/[ref-header]/[ref-body].
What did you expect to see?
The content of git object can be displayed as: git cat-file -p [some digest].
What did you see instead?
The content could not be read by git:

git cat-file -p e69de29bb2d1d6434b8b29ae775ad8c2e48c5391
fatal: unable to stream e69de29bb2d1d6434b8b29ae775ad8c2e48c5391 to stdout

The content compress by compress/zlib was:

cat .git/objects/e6/9de29bb2d1d6434b8b29ae775ad8c2e48c5391 | xxd
00000000: 785e 4aca c94f 5230 6000 0000 00ff ff    x^J..OR0`......

I tried to decompress and recompress the content like this:

cat .git/objects/e6/9de29bb2d1d6434b8b29ae775ad8c2e48c5391 | zlib-flate -uncompress | zlib-flate -compress=5 | xxd
00000000: 785e 4bca c94f 5230 6000 0009 b001 f0    x^K..OR0`......

The output is now be the same with git hash-object does.
I have tried with compress level from 1 to 9 of the compress/zlib lib, but none of them can produce 785e 4bca c94f 5230 6000 0009 b001 f0
I am not familiar with the zlib algo, can any one be kind to help me out? Thanks in advance!

The underlying compression format is RFC 1951. The Go implementation only guarantees that it produces valid output that is compatible with RFC 1951, but does not guarantee that it outputs any specific sequence of bits.

@dsnet Thanks for the reply, it turns out that I just called Flush instead of Close for the zlib writer, so the output write to file is not correct.


Add support for Group lifecycle policies, classification list and blocked words etc.

Would love if we can add additional tenant settings as below in upcoming schemas, happy to contribute PRs:

Add support for Group lifecycle policies - ability set group expiration policy / expire groups after specified number of days (probably default to 180 days)  - groupLifecyclePolicy resource type
.

Would be great if we can setup expiry for groups(all, selected, none) and specify expiration for specific groups

Set the classification list for groups, add blocked words, modify usage guidelines url as in below image. Would like if we can implement this entirely:

Hi @gautamdsheth,
I just added support for Group Lifecycle Policies at schema level in schema 201909. About the AAD settings, I will keep it on hold and I will let you know what we can do. I'm not sure we can include it in the upcoming schema 201909, but in the worst case I will do my best to have it in the next one.

I just included a dictionary to collect the directory settings, too. It will show up in 201909 schema.


你一更新害死人了

能不能考虑下 这种不能升级node 版本的小公司
你一更新 我就发布了版
能不能考虑考虑

https://nodejs.org/en/blog/announcements/nodejs16-eol/
16 been EOL for 10 months. 14 has been EOL for over a year. Pin your deps and only upgrade carefully if you are using EOL platform versions.


Update to cargo 0.29 and fix build.

Please note that I have no previous experience with rust, but I'm happy to improve this PR.

Rebased to resolve the conflicts and merged it in.


BAD for psychophysical function estimation

Then implement a new psychometric function estimation package.

[x] Resolve #63 first.

Change of plan. It makes sense for this to go into a separate repo. This one is about DARC only.


GitHub releases for new versions

I noticed, that there is a relaese 0.3.2 at NPM, but it is not mentioned here as a release.
Would you mind tagging the release in GitHub also? :)

Right. I haven't been keeping the tags up. Fixed it. Now I have to remember to maintain it going forward.


Add freebie tag and tool tip for forward solve, backsolve, and freebie

Most of the changes to the html file were caused by me running
prettier hunts/templates/stats.html --write --config .prettierrc


v1.3.1 branch release

Codecov Report

Merging #122 (d535aab) into master (412e9f8) will not change coverage.
The diff coverage is 50.00%.

@@            Coverage Diff            @@
##             master     #122   +/-   ##
=========================================
  Coverage     64.55%   64.55%
  Complexity      664      664
=========================================
  Files           108      108
  Lines          2491     2491
  Branches        274      274
=========================================
  Hits           1608     1608
  Misses          782      782
  Partials        101      101

Impacted Files
Coverage Δ

src/main/java/seedu/address/MainApp.java
0.00% <0.00%> (ø)

...ava/seedu/address/model/person/AttendanceList.java
51.51% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Handle the case if a server has more than 25 plans

(according to documentation there is no way to request them at once)

Thanks for the improvement!


[Website] set up conversion tracking

As a stakeholder in the success of Mineral UI, I would like to know that my AdWords money is being well-spent.
AC:
I have a report that tells me how effective ad clicks are for driving users to the goals we set up in #486.

add conversion tracking code for important actions on mineral-ui.com

we don't have a clear conversion


Incorrect TOC in the website sidebar for tutorials

"process data", "loading pretrained elmo data", "putting everything together" should not appear in the sidebar.

This is already fixed in http://gluon-nlp.mxnet.io/master/index.html

Back to normal when I checked the website today again. Not sure why it happened. Let's reopen if it appears


Covert artifacts and parameters arguments in the CommandRunner constructor to strings

Is your feature request related to a problem? Please describe.
Corresponds to the TODO here: https://github.com/cmmorrow/build-magic/blob/main/build_magic/runner.py#L153
Describe the solution you'd like
Try to cast the values in the arguments to strings and raise an error if they cannot be cast. This should only be an issue when using the low-level API directly.

Maybe a dumb question, but what's an example of something that could not be cast to a string?

 I can't think of something that can't be case to a string since objects have a string representation. I think my original concern with this ticket was if an artifact or parameter value isn't already a string, it might cause an error when trying to copy files or interpret parameter values. It would be easier to check for this in the constructor rather than add try/except blocks everywhere the artifacts list and parameters map attributes are used.
I updated the description to reflect this change. Casting wouldn't really address the problem and could make an error more confusing.

After working on this, I think the more pythonic thing to do is just let build-magic error at the point of failure if a bad argument is provided. Values from the CLI will always be strings, so there's no worry there. Only when using the Python API can non-string arguments potentially be provided. In this case, the traceback should let the user know what and where the problem is, and docstrings should communicate the correct type to use.


Pokémon tv not working

Whenever I try to watch certain episodes,movies, and specials on poke-tv.vercel.app, it says “the media couldn’t be loaded, either because the server or network failed or because the format is not supported” can you please fix this?

https://github.com/seiya-dev/pokemon-tv/issues/77


added support Ping Cmd for the cluster

This is my First PR in OS , apologies if done anything wrong.
During integration i didn't find Ping command for health-check for all nodes in the cluster. Tried building it by myself.
Theory:  it will  run ping command on individual node and accumulate the response, if any node throws error, it will throw the same err.
example :

reply,err := cluster.Ping()

closing it..


Training the model using multiple onboard GPUs?

First, thank you for publishing the great work.
In order to enhance the speed of training, do you plan to optimize the training loop such that it can make use of multiple GPUs in the system?
I have 3 GPUs(Nvidia 1080ti) in my system. I am trying run the training on multiple GPUs using torch.nn.dataParallel() but am ending up with batch_size mismatch error. Can you please guide me through that?
Thank you
Best Regards

you can refer to this issue https://github.com/xinge008/Cylinder3D/issues/22

Thanks a lot for the lead.


shpool hides the motd when immediately executed

Right now, if you follow the advice in the README and configure shpool to immediately connect when you ssh into a remote machine, any MOTD (message of the day) will get hidden. To fix this, shpool should be able to read the MOTD and display it to users when first launching a shell or reconnecting.
I think this should be controlled with some new config options
mod_display_mode - { pager = "<cmd>" } | dump, controls how the MOTD is shown to the user. default { pager = "less" }
motd - never | attach | new_session, controls when the MOTD is shown (never, whenever a user connects, or only for new shells). Initially will default to never, but will default to attach after a while.
This is a migration of the internal bug b/303261756
I've already started work on a tool to dump the motd over at github.com/shell-pool/motd

https://github.com/shell-pool/shpool/pull/6 and https://github.com/shell-pool/shpool/pull/8 together resolve this issue.


Megan sandbox

I started to add support for fused unfold. It currently only supports convolution stride=1, dilation=1, padding=1, and square kernels. I added a test comparing to PyTorch version and a benchmark script for CNNs. For forward pass, (128, 128, 3, 3) filter and input (128, 128, 16, 16) (i.e. batch size 128), this fusing saves memory 3.8x(not sure why memory savings are this big) and runtime 1.9x. It's a work-in-progress---for the forward pass, I still need to:

add support for decreasing strides

return folded values? Currently returns unfolded values (as all intermediates are unfolded). I think returning folded values would be nicer interface (input and output dimensions match, but doesn't look like it would greatly affect runtime or memory usage)

need to test and debug when in_channels != out_channels

dies for large feature dimension (>16), need to debug why

general code clean up and add checks in code

add backward pass

Addressed all of the above except for (2) where I return unfolded values from the cuda kernel since the post-processing folding overhead seems small. I was having issues with rebasing on my side so there are a ton of commits---if this gets merged, we'll want to squash all these commits then. Currently the stable kernels are the older forward and backward pass without the recent additional optimizations so they can be faster.

Should support the same kernels currently used for untied in the master branch with fusing the unfolding, which mostly speeds up the forward pass (~2x).


IsoSurface: Opacity isn't honored

Describe the bug
Changing IsoSurface opacity doesn't seem to have any effect.
To Reproduce:

Open VAPOR, load CF compliant data: /home/shaomeng/little_glade/cm1.nc.
Create an IsoSurface renderer, and enable it
Go to Appearance Tab, enable 2 IsoSurfaces.
Try to adjust the opacity value from the TF widget. Notice that any adjustment wouldn't change the rendering.

Hmm, there's a use constant color box. The behavior is as expected once that box is clicked. This report is rejected.


How to make status bar color & Appbar color is different?

My English isn't good. Hope everyone understand.
If Appbar appear, color of status bar & background app bar to same. How to I make status bar color & Appbar color is different?
Ex: Appbar background is blue and status bar color is red (In this case, present both is blue)

You can use this package for android and ios : https://pub.dev/packages/flutter_statusbarcolor

@pqtrong17
Your question is better suited for StackOver flow. Please see https://flutter.dev/community for resources and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you


Add blocks support to Slacker

This commit will add the ability to use blocks to Chat.post_message, Chat.update, and Chat.post_ephemeral.
Blocks are an optional ability to add visual effects using json-like structure. Slack has been pushing its use since early 2019.
This is an attempt to squash and re-PR this. I am the newbest, so if this doesn't work, I'm just gonna let someone else make the like, 25 character change to have this work.

It looks good, thanks, @genghis!


Feat/improve api tokens

This pull requests contains two improvements to API TOKEN handling:

The first improvement (commit 957284f) allows reading the api token from an external command
The second improvement allows using PAT tokens with an on-premise (local) jira server

Closing due to inactivity. The branch has diverged a bit and some part of this PR is done in #327. Please feel free to comment or open a new PR for further discussion.


Add exit diagnostic session

If we have connected to an ecu by double clicking on Initiate Contact we still need a way to go back to default session.
Could you bind the ExitDiagnosticSession to an handler to double click again or somthing else?

I've changed a minor typo (copy, paste :p), otherwise it looks good to me! I'll look into automatically switching the session on disconnection or when exiting. Thanks for your time and effort, N0cynym.

I've changed a minor typo (copy, paste :p), otherwise it looks good to me! I'll look into automatically switching the session on disconnection or when exiting. Thanks for your time and effort, N0cynym.


[BUG] Input from v1 with inconsistent height

🐛 Describe the bug
The input in Nimbus V1 is 2 pixels bigger than in V2, this makes the input bigger than the filter button on new admin.

🧐 To Reproduce
Steps to reproduce the behavior:

Go to Storybook V1
Inspect the input height
Compare with Storybook V2

✅ Expected behavior
The V1 input should only have 32px of height.

Resolvido!


Removing DataRequestEventArgs as it is not required

@dotnet-bot test Windows_NT / Debug

@dotnet-bot test this please


Add navigation top and side bars

Mockups have been produced so would be good to get testing on live site.
When creating also need to take into consideration single component display mode (or perhaps link to router layout) - this won't affect the nav themselves but will likely need to be brought into the router component
child of #106

merge into #108

splitting this issue, the creation of navbars is now part of #106 , this can serve as a reminder that we will want to hide navigation entirely when we have chosen just to display a single component (like how the documentation component is displayed at documentation.preciousplastic.org)


Fix pyusb dependency

Installation of pyOCD fails at the time of this patch on platforms that
use pyusb, like Linux.  This is because the only released versions
of pyusb are alpha or beta, which pip does not install by default.
This patch sets the minimum version to the latest beta so pyusb
can be installed.

This should fix #177

@c1728p9 LGTM. How did you test this?

I was testing on an Ubuntu virtual machine.  I tested with the following sequence:

Uninstalled pyusb "pip uninstall pyusb"
Uninstalled pyOCD "pip uninstall pyocd"
Verified that pyOCD fails when trying to install current master "pip install -e ."
Apply changes verified install works using "pip install -e ."


[image_picker_for_web] Migrate image_picker to package:cross_file

Changed all usages of PickedFile to XFile, to follow the changes made to the platform interface (#4072)
This PR will remain in draft state until the platform interface changes have been merged and published (#4072), so that the pubspec can be updated properly.
Relevant issue:

flutter/flutter#70886

Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Note that unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Let's go! The platform_interface has been published here:
https://pub.dev/packages/image_picker_platform_interface/changelog#220

@googlebot I consent.

@BeMacized, I've brought the branch up to date with the latest master, removed the "deprecated" methods (let's add those only in the core plugin package), and used this opportunity to migrate the tests to run with flutter drive, instead of flutter test --platform=chrome.
There's documentation linked in the new README files, but TL;DR, this is how tests can be run now:

Ensure chromedriver --port=4444 is running
From the root of the repo, run: dart ./script/tool/lib/src/main.dart drive-examples --packages=image_picker/image_picker_for_web --web.

dit@dit:~/github/plugins$ dart ./script/tool/lib/src/main.dart drive-examples --packages=image_picker/image_picker_for_web --web
script/tool/lib/src/main.dart: Warning: Interpreting this as package URI, 'package:flutter_plugin_tools/src/main.dart'.

============================================================
|| Running for image_picker_for_web
============================================================

Running command: "flutter drive -d web-server --web-port=7357 --browser-name=chrome --driver test_driver/integration_test.dart --target integration_test/image_picker_for_web_test.dart" in /usr/local/google/home/dit/github/plugins/packages/image_picker/image_picker_for_web/example
Running "flutter pub get" in example...                            614ms
Launching integration_test/image_picker_for_web_test.dart on Web Server in debug mode...
Waiting for connection from debug service on Web Server...         16.6s
integration_test/image_picker_for_web_test.dart is being served at http://localhost:7357
The web-server device requires the Dart Debug Chrome extension for debugging. Consider using the Chrome or Edge devices for an improved development workflow.
Application finished.
[WARNING]: http://localhost:7357/dart_sdk.js 48732:16 "registerExtension() from dart:developer is only supported in build/run/test environments where the developer event method hooks have been set."
[WARNING]: http://localhost:7357/dart_sdk.js 48714:16 "postEvent() from dart:developer is only supported in build/run/test environments where the developer event method hooks have been set."
[INFO]: http://localhost:7357/dart_sdk.js 27220:14 "00:00 +0: Can select a file (Deprecated)"
[WARNING]: http://localhost:7357/packages/image_picker_for_web/image_picker_for_web.dart.js 204 File chooser dialog can only be shown with a user activation.
[INFO]: http://localhost:7357/dart_sdk.js 27220:14 "00:00 +1: Can select a file"
[WARNING]: http://localhost:7357/packages/image_picker_for_web/image_picker_for_web.dart.js 204 File chooser dialog can only be shown with a user activation.
[INFO]: http://localhost:7357/dart_sdk.js 27220:14 "00:00 +2: computeCaptureAttribute"
[INFO]: http://localhost:7357/dart_sdk.js 27220:14 "00:00 +3: createInputElement accept: any, capture: null"
[INFO]: http://localhost:7357/dart_sdk.js 27220:14 "00:00 +4: createInputElement accept: any, capture: something"
[INFO]: http://localhost:7357/dart_sdk.js 27220:14 "00:00 +5: (tearDownAll)"
[INFO]: http://localhost:7357/dart_sdk.js 27220:14 "00:00 +6: All tests passed!"
All tests passed.

------------------------------------------------------------
Run overview:
image_picker_for_web - ran

Ran for 1 package(s)

No issues found!

Just verified that tests are running in CI. See this run, for example.

@ditman Alright! I can confirm the tests succeed locally as well, so I've marked this PR as ready for review.

Published https://pub.dev/packages/image_picker_for_web/changelog#210


Fix bad property in attachment detection

The "dparameters" is undefined for me (why ? I don't know). The good value is "parameters".
My patch checks if "dparameters" is set, if not, tests on "parameters".

Nice! This looks good to me.


Zenodo?

See Sam's comments in MS doc.

high priority. This is relatively easy and doesn't require IPDS record because you can use provisional software release guidance from FSP. Just need to create a tag of relevant repos and either download the zip / tarball or integrate into zenodo. I can help out with this. I'd estimate this would take less than 1 hour to do and would allow you to point to it for processing descriptions

Determined to be not a viable option for us at this time. Closing and commenting more details in the reconciliation document.


c7n_mailer based on latest source code is broken

Following the installation guide, it looks like c7n_mailer is broken due to changes to mu.py.
(custodian)$ c7n-mailer -c mailer.yml
Traceback (most recent call last):
  File "/home/testusercloud-custodian/tools/c7n_mailer/c7n_mailer/cli.py", line 72, in main
    deploy.provision(config, functools.partial(session_factory, config))
  File "/home/testuser/cloud-custodian/tools/c7n_mailer/c7n_mailer/deploy.py", line 74, in provision
    archive = get_archive(config)
  File "/home/testuser/cloud-custodian/tools/c7n_mailer/c7n_mailer/deploy.py", line 38, in get_archive
    'memcache')
TypeError: __init__() takes at most 6 arguments (8 given)
> /home/testuser/cloud-custodian/tools/c7n_mailer/c7n_mailer/deploy.py(38)get_archive()
-> 'memcache')

By looking at the code, it looks like the older version of c7n installed from pip install compatible with the source code from top of the tree.   https://github.com/capitalone/cloud-custodian/commit/1be171dbdf48d80c1b640c499d9fe6b2194255dc
What should I do to get around this? just checkout a specific commit before the one that breaks c7n-mailer? Is there a planned new version of pip?

i'm also interested in a workaround to this. it appears it's fixed in <IP_ADDRESS>, but, i've yet to successfully install it.

@kennyg so I had to do a git checkout 34d74e0dce6a2eb8599311947a4b2fea3036d466 to rollback c7n_mailer that works with c7n <IP_ADDRESS>

that did the trick! thanks

any update on when the next version of c7n will be released through pip?

I'll try and get this sorted monday, currently traveling.

good to know, thanks @kapilt

so current trunk has a regression which is preventing release per #1111

0.8.24 released


REQUEST_NOT_SUCCESSFUL

File downloading 50% after it's not getting downloaded but file is working fine.
{
"autoRetryAttempts": 5,
"autoRetryMaxAttempts": 5,
"created":<PHONE_NUMBER>848,
"downloadOnEnqueue": true,
"downloaded": 17017742,
"downloadedBytesPerSecond": -1,
"enqueueAction": "UPDATE_ACCORDINGLY",
"error": "REQUEST_NOT_SUCCESSFUL",
"etaInMilliSeconds": -1,
"extras": {
"data": {
  }
},
"file": "/data/user/0/com.sdll/files/fetch/nNlcHuJfcFKzrg2Txe0yayOkQdvkg8GbKZL8njx8.mp4",
"group": 4,
"headers": {

},
"id": 211440922,
"identifier": 8,
"namespace": "DownloadListActivity",
"networkType": "ALL",
"priority": "NORMAL",
"status": "FAILED",
"tag": "6",
"total": 27391464,
"url": "https://green-jakobsen.s3-accelerate.amazonaws.com/sitemap/6/nNlcHuJfcFKzrg2Txe0yayOkQdvkg8GbKZL8njx8/nNlcHuJfcFKzrg2Txe0yayOkQdvkg8GbKZL8njx8.mp4"

}

@sanjaysingh1990 I was able to successfully download the file using Fetch.
"url": "https://green-jakobsen.s3-accelerate.amazonaws.com/sitemap/6/nNlcHuJfcFKzrg2Txe0yayOkQdvkg8GbKZL8njx8/nNlcHuJfcFKzrg2Txe0yayOkQdvkg8GbKZL8njx8.mp4

Here is the MD5 hash of the file: 93148314a92e6912490deefca14940d1 and its 16.2MB
The error REQUEST_NOT_SUCCESSFUL signals that there was an issue with the server REQUEST_NOT_SUCCESSFUL. Could be a very slow connection. Please try on a different connection and let us know how it worked for you.


[STYLE CHANGE] Improving the Navbar of Home Page

Description:
Improving the Navbar of the home page by removing the underline effect we get, when we hover on the options present in the Navbar.
Screenshots or Design Mockups

Priority Level

[ ] Low
[x] Medium
[ ] High

please assign me this project.

Please assign me this issue under GSSoC-2023 i love to contribute to this issue.

I would like to work on this project . Please assign it to me .

I am interested to work on this issue under GSSoC 2023 as a contributor

It's done. Do a fresh fetch to get the changes.


Cannot include yaml fragments in junction elements

See original issue on GitLab
In GitLab by [Gitlab user @tristanvb] on Jul 21, 2020, 11:25
Currently we have an undocumented limitation, which is that we fail to expand (@) files when these are declared inside a junction element.
This is not mentioned in the include documentation or in the junction documentation.
While it is possible to end up in unresolvable situations by allowing includes in junctions, it is perfectly possible to allow, we just need to have the correct early stage LoadErrors in place if ever an include leads to a circular reference into the same project that the given junction is loading. This error is already probably accounted for by LoadErrorReason.CIRCULAR_REFERENCE which is checked for and raised in the Loader.get_loader() API.
In the worst case, we need to document this as unsupported, but I think we should support this.

This needs to be verified again, this quite possibly works after some refactors made last year.
We should create a test case to validate whether this works or not and then document this accordingly.

Including now works both with BuildStream 1 and 2 but there may be some corner-cases with expansion of variables in some cases.


Feat fix #9

Added documented test

The old ImageUtilsTest should remain when merging, ie completely discard all changes made by this pr

Resolve the conflicts pls


Create metaclasse for joint (SOLID)

Except from the direct use of the module, many users may want to adapt objects defined here. Allowing them to follow the SOLID principles would be nice. Hence, metaclasses.

Class Joint was added in v0.6.0 :+1:


Support RSV on Results page

FRONTEND PULL REQUEST
Related Issue
resolves #6987
Changes Proposed

add RSV to the result page disease filter
add an RSV column in our CSV for downloading results
dynamically populate the "View details" modal with only the results that exist for the test event
add RSV copy so it shows up in both the print result and pxp (email/text results)

Additional Information

copy for the header for print and pxp and the guidance (more info section) will be updated in #7000

Testing
will deploy to dev3

oops I think the sorting on the view details modal is indeterminate 👀

do we want to do a general audit/change of the site (including the static site) to make it more clear/accurate that it's not just COVID-19 tests we are handling?

Thanks @emyl3! Created #7084


Change project name

Project : ros2_realtime_statistics
Packages :
[ ] realtime_statistics_json
[ ] realtime_statistics_plot
[ ] ?

OR

Project : ros2_realtime_statistics
Packages : realtime_statistics
Nodes :
[x] write_json
[x] plot_json

I think the second option is better because we can introduce both RT and non-RT system characteristics in the same package. Which would be better in the long run

what do you mean ?

I mean IF we want to develop this further, the main package stack would be ros2_realtime_statistics and inside it we can include support for both RT and Non RT stuff


any kind of error in callJs leads to JSON parse error, original error disappears

The code in neel.js (see neel.nim, line 254) for responding to a callJs request will swallow any exception caused by the function being called and make it look like the problem is a JSON parse error. Offending code section:
254 ws.onmessage = (data) => {
255     try {
256         let x = JSON.parse(data.data)
257         let v = Object.values(x)
258         neel.callJs(v[0], v[1])
259     } catch (err) {
260         let x = JSON.parse(data)
261         let v = Object.values(x)
262         neel.callJs(v[0], v[1])
263     }
264 }

If line 256 succeeds and the function called in line 258 then throws an exception, then the catch block attempts to re-parse the whole message as JSON, which fails with Uncaught SyntaxError: JSON.parse: unexpected character at line 1 column 2 of the JSON data and the original exception which was thrown in the try block is lost.
I think this would do the trick:
254 ws.onmessage = (data) => {
255    let x;
256     try {
257         x = JSON.parse(data.data)
258     } catch (err) {
259         x = JSON.parse(data)
260     }
261         let v = Object.values(x)
262         neel.callJs(v[0], v[1])
263 }

@LemongrabThree Apologies for the delay brother.
@NecroticOoze Take a look please

@LemongrabThree
We've just released 0.3.1 which includes your changes. Thanks for the contribution! Happy coding


Preserve custom script tags

I need to preserve custom script tags in the HTML. Right now they are removed during the build. Any ideas about how to stop this from happening?

Can you provide an example of the syntax used?
There are two ways to do it depending on the syntax

It is a JSON-LD for Gmail.


Facing some issues on install jit.realsense

Required Info

Camera Model
{ D415 }

Firmware Version

Operating System & Version
( MacOS )

Platform
Max 8

Issue Description
hello, i just got Realsense D415 and I'm facing some issues on install with Max 8.
I got this message "CMake Error: The source directory "/Users/Fai/build/path/to/jit.realsense" does not exist."
anyone can help me?

Hi @faiahmed   I believe that you are referring to the jit.realsense program that provides RealSense compatibility for Max.  Is that corrrect, please?
https://github.com/blueyeti/jit.realsense
https://cycling74.com/products/max-features
The instructions give me the impression that the appropriate course of action is to download or clone the jit.realsense GitHub using the Code button and then go to the parent folder of the downloaded repository, create the build directory and use the cmake path/to/jit.realsense -DCMAKE_BUILD_TYPE=Release command to point CMake to the CMakeLists.txt file in the parent folder.  That is how I personally interpret the process (I have not encounted Max before).

thank you your reply but i did everything you mentioned & still got the same issues.

It is very hard to work out the correct instructions from the official documentation available.  The best I can do I think is link you to another case related to MacOS installation problems with jit.realsense and see if it makes sense to you.
https://github.com/blueyeti/jit.realsense/issues/2
There does not seem to be anywhere else to go if that link does not prove useful unfortunately.  I do apologize.

It is very hard to work out the correct instructions from the official documentation available.  The best I can do I think is link you to another case related to MacOS installation problems with jit.realsense and see if it makes sense to you.
https://github.com/blueyeti/jit.realsense/issues/2
Unfortunately there does not seem to be anywhere else to research this particular question if that link does not lead to the solution.  I do apologize.

Case closed due to no further comments received and no other solutions available.


[SIP] fix error running data fusion queries - Physical input schema should be the same as the one converted from logical input schema

fixes: #655

@a-agmon this's a draft PR, Does it works for you?

Hey @FANNG1 , yesss, this works.
note that I had to pin the latest arrow version to resolve some conflict cargo found
arrow = "53.1.0"

@a-agmon @liurenjie1024 , sorry for the delay, it's ready to review now, please help to review when you have time, thanks

LGTM


Rules with numeric keys do not validate properly

Laravel Version
10.48.10
PHP Version
8.3
Database Driver & Version
No response
Description
Numeric rules (Eg: [4 => 'required', 8 => 'required']) when added to a validator are not saved properly, they are transformed into[0 => 'required', 1 => 'required'] in the constructor
I tracked it down to this line
https://github.com/laravel/framework/blob/bf7046f37a03ac006f60a811148f91393f305283/src/Illuminate/Validation/Validator.php#L1208-L1210
It seems the array merge recursive treats any numeric key differently and just appends it to the array instead of preserving the keys, this is explained in the php doc, but undesirable behavior in this case https://www.php.net/manual/en/function.array-merge-recursive.php
array_merge_recursive([], ['foo' => 'required', 4 => 'required', 8 => 'required']);
= [
    "foo" => "required",
    0 => "required",
    1 => "required",
  ]

Steps To Reproduce
Running the following in a tinker session is enough to reproduce
Validator::validate(['foo' => 'baz', 4 => 'foo', 8 => 'baz'], ['foo' => 'required', 4 => 'required', 8 => 'required']);

Results in
   Illuminate\Validation\ValidationException  The 0 field is required. (and 1 more error).

A possible solution is a different implementation of the array_merge_recursive as seen here https://www.php.net/manual/en/function.array-merge-recursive.php#106985
Prior attempt at a fix https://github.com/laravel/framework/pull/39368

Since this behavior has been present ever since the beginning I don't think we can change this. Should you want to propose this change I suggest you to attempt a PR to the master branch. Thanks

What is the actual, valid, use-case for this?
This seems very unlikely to anyhow useful in a real use-case.

@crynobone thank you for this very insightful copy-paste
But this "behaviour" is a bug and it existing forever is not a valid reason to not want to improve the tool/framework we are working on for future use

@Tofandel feel free to send a PR to master branch as suggested earlier.

As a workaround, you can wrap your fields' IDs into the same name key.
Code will explain better:
// routes/web.php
<?php

use Illuminate\Http\Request;
use Illuminate\Support\Facades\Route;

// from DB
$fields = [
    4 => ['label' => 'name', 'rule' => 'required'],
    8 => ['label' => 'surname', 'rule' => 'required'],
];

Route::get('/', fn () => view('form', ['fields' => $fields]));

Route::post('/', function (Request $request) use ($fields) {
    $rules = collect($fields)->mapWithKeys(fn ($field, $id) => [
        'field.' . $id => $field['rule'],
    ]);

    $attributes = collect($fields)->mapWithKeys(fn ($field, $id) => [
        'field.' . $id => $field['label'],
    ]);

    $request->validate($rules->all(), attributes: $attributes->all());

    return back();
});

{{-- resources/views/form.blade.php --}}

@json($errors->all(), JSON_PRETTY_PRINT)

<form action="/" method="POST">
    @csrf

    @foreach($fields as $id => $field)
        {{ $field['label'] }}
        <input type="text" id="field-{{ $id }}" name="field[{{ $id }}]">

    @endforeach

    <button type="submit">send</button>
</form>

@rodrigopedra Thanks, that's what I did but then the problem is that the path of the field in the error bag doesn't match with the output I want which is yet another workaround needed

Thanks. This has come up before quite a few times. We decided we won't be changing this behaviour because this could have quite an impact on existing apps, sorry for that. We could maybe make the docs more clear about this so feel free to attempt a PR to the docs.

@driesvints

quite an impact on existing app

I would like to see what exactly what that could be, because I don't see it at all. A validator works only on keyed arrays


error with loading plugins

so when im trying to start up my server it gives me this exact error Failed to load ZaupShop, unloading now... :System.IO.FileNotFoundException: Could not load file or assembly 'I18N.West, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=0738eb9f132ed756' or one of its dependencies.
File name: 'I18N.West, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=0738eb9f132ed756'
at ZaupShop.ZaupShop.Load () [0x00000] in :0
at Rocket.Core.Plugins.RocketPlugin.LoadPlugin () [0x00000] in :0
and then it starts unloading the plugin how do I fix this

Not a RocketMod bug, you installed the plugin wrong.
Copy libraries folder next time.


A way to get current timezone name?

Is there some way to get the name of the current timezone or is this functionality current missing? i.e. tzInfo("LOCAL").tzName == "Europe/London" or similar.

Very late answer, but no. This is not trivial to implement even on unix systems


Update example remove dba db master key

Type of Change

[ ] Bug fix (non-breaking change, fixes #)
[ ] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] Nunit test is included
[X] Documentation
[ ] Build system

Purpose

Update the example to reflect current parameters. Typo in database parameter name.

fabulocity of 10!


exclude_unset=True removes tokens field when streaming logprobs.

https://github.com/vllm-project/vllm/blob/54d3544784ff20e7038abf72793eaf734e727269/vllm/entrypoints/openai/serving_completion.py#L99
with exclude_unset=True.
data: {"id":"cmpl-6a6c465165c94d83bede2e6c8d33300c","created":12380229,"model":"google/gemma-7b-it","choices":[{"index":0,"text":"Hello","logprobs":{"top_logprobs":[{"Hello":-0.006023823
749274015}]},"finish_reason":null}]}

data: {"id":"cmpl-6a6c465165c94d83bede2e6c8d33300c","created":12380229,"model":"google/gemma-7b-it","choices":[{"index":0,"text":"!","logprobs":{"top_logprobs":[{"!":-1.317667841911316,"
,":-0.3176678419113159}]},"finish_reason":null}]}

data: {"id":"cmpl-6a6c465165c94d83bede2e6c8d33300c","created":12380229,"model":"google/gemma-7b-it","choices":[{"index":0,"text":" 👋","logprobs":{"top_logprobs":[{"▁👋":-0.0000207422017
4108632}]},"finish_reason":null}]}

data: {"id":"cmpl-6a6c465165c94d83bede2e6c8d33300c","created":12380229,"model":"google/gemma-7b-it","choices":[{"index":0,"text":"\n\n","logprobs":{"top_logprobs":[{"\n\n":-0.10709359496
831894}]},"finish_reason":"length"}]}

data: {"id":"cmpl-6a6c465165c94d83bede2e6c8d33300c","created":12380229,"model":"google/gemma-7b-it","choices":[{"index":0,"text":"","logprobs":{},"finish_reason":"length"}],"usage":{"pro
mpt_tokens":10,"total_tokens":14,"completion_tokens":4}}

data: [DONE]

with exclude_unset=False.
curl http://localhost:8000/v1/completions -H "Content-Type: application/json" -d '{"model": "google/gemma-7b-it", "prompt": "<start_of_turn>user\nHello<end_of_turn>\n<start_of_turn>model\n", "max_tokens": 4, "stream": true, "logprobs": 1}'
data: {"id":"cmpl-806feb2e4e6e4f26af94190e317c88ae","object":"text_completion","created":12380363,"model":"google/gemma-7b-it","choices":[{"index":0,"text":"Hello","logprobs":{"text_offset":[0],"token_logprobs":[-0.006023823749274015],"tokens":["Hello"],"top_logprobs":[{"Hello":-0.006023823749274015}]},"finish_reason":null}],"usage":null}

data: {"id":"cmpl-806feb2e4e6e4f26af94190e317c88ae","object":"text_completion","created":12380363,"model":"google/gemma-7b-it","choices":[{"index":0,"text":"!","logprobs":{"text_offset":[5],"token_logprobs":[-1.317667841911316],"tokens":["!"],"top_logprobs":[{"!":-1.317667841911316,",":-0.3176678419113159}]},"finish_reason":null}],"usage":null}

data: {"id":"cmpl-806feb2e4e6e4f26af94190e317c88ae","object":"text_completion","created":12380363,"model":"google/gemma-7b-it","choices":[{"index":0,"text":" 👋","logprobs":{"text_offset":[6],"token_logprobs":[-0.00002074220174108632],"tokens":["▁👋"],"top_logprobs":[{"▁👋":-0.00002074220174108632}]},"finish_reason":null}],"usage":null}

data: {"id":"cmpl-806feb2e4e6e4f26af94190e317c88ae","object":"text_completion","created":12380363,"model":"google/gemma-7b-it","choices":[{"index":0,"text":"\n\n","logprobs":{"text_offset":[8],"token_logprobs":[-0.10709359496831894],"tokens":["\n\n"],"top_logprobs":[{"\n\n":-0.10709359496831894}]},"finish_reason":"length"}],"usage":null}

data: {"id":"cmpl-806feb2e4e6e4f26af94190e317c88ae","created":12380363,"model":"google/gemma-7b-it","choices":[{"index":0,"text":"","logprobs":{},"finish_reason":"length"}],"usage":{"prompt_tokens":10,"total_tokens":14,"completion_tokens":4}}

data: [DONE]

Some initialized field has been removed while serialization. What is the intention of exclude_unset=True? If there is no reason, exclude_unset=False is much more reasonable response for the request.

I suspect that token_offset, token_logprobs, and tokens appends new data instead of set their field. Pydantic regards this field as unset and excludes from the dump output.


Updated wording

Updated wording

@JamesTran-MSFT : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Closing since this PR resolves the issue - https://github.com/MicrosoftDocs/azure-docs-pr/pull/188499


Custom template modules

Do you have an example for verb --set includes="my-include-templates" ...?
Tried it both, with a local module and a path following the example of readme-includes, but didn't succeed, always get:
async-helper-base: {%= include("extension-install") %} error: %j TypeError: Cannot read property 'render' of null
But settings in the app seems to be correct:
 store:
   { name: 'verb',
     path: 'C:\\Users\\me\\data-store\\verb.json',
     data: { includes: 'me-readme-includes' } } }

Hi,
I assume the right place to fix this would be `.\lib\transforms\init\load.js', right?
I don't see any code there which would load the setting from the data-store ...
Am I missing something?
Regards
Stefan

@stefanwalther it would it with this.get, imo.

@tunnckoCore, thx, will give it a try ...

sorry for taking so long on this. @doowb and I have virtually refactored the entire ecosystem underlying verb and other related projects. I'm just migrating now, so any day...

@jonschlinkert Any news on this?

Ping ;-)

Hi, getting this to work would be soooooooo great ;-)
@jonschlinkert & @doowb , any ideas?

@stefanwalther soon :) If you not noticed they work on underlying tons of packages (see dev branch).
You should definitely try the dev branch + verb-readme-generator.
I opened #87 and https://github.com/verbose/verb-readme-generator/issues/2, currently there are resolving problems.

yes we're very close. fwiw everything is working perfectly locally for me, but we need to add support for npm custom prefixes before we release - I think that's what is breaking @tunnckoCore's build, and probably will cause issues for others.
but I have to say, @doowb and I are really amazed at how awesome this is, and how well everything is coming together as we had planned. We're very close to getting everything "done"! thx for hanging in there and staying interested!

I think that's what is breaking

hm, that's remind me that i didn't try after I fixed the prefix problem. :D I'll try now.

after I fixed the prefix problem.

what did you do to solve it btw?

the solutions you pointed to seem like overkill for this, but maybe I'm missing something

what did you do to solve it btw?

just moved node executable to /usr/local/bin, lol

seem like overkill for this

why? ;d it didn't break anything, which was surprising for me too.

Btw, problem occurred with trying latest dev. Errors comes from templates, I'm checking and trying to realize it, then I'll open issue there.

@stefanwalther have you tried using 0.9? this was implemented

Hi @jonschlinkert, what is 0.9, the dev branch? I'll give it a try immediately, really awaiting that feature ;-)

Installing 0.9 (the dev branch), gives me the following errors:
npm install verbose/verb#dev

verb

{ [Error: base-runner#runner Cannot find module 'minimist'] code: 'MODULE_NOT_FOUND' }

After installing minimist, I get some other missing dependencies ... (omit-empty, etc.)

@doowb and I aren't able to reproduce. Since you installed locally, it's possible that verb is trying to use outdated modules (since npm doesn't force everything to reinstall the way it used to).
did you uninstall everything (delete node_modules and do cache clean) and reinstall? globally and locally?


Repeated "Namespace/tst configured" event

About every 5 minutes, I get a notification in Slack saying:

kustomization/flux-system.flux-system:
Namespace/tst configured

I see no reason why this is happening. This message started to show when I added another, differently named namespace. It seems a state bug. Any way I can reset this?

It may be that 2 Flux Kustomizations fight over the same resources. You can use flux tree ks and see who manages them. I recommend you follow the repo structure showcased here: https://github.com/fluxcd/flux2-kustomize-helm-example

This is about the structure we use.

I'm getting the following CLI error (should I create a new issue?): failed to get API group resources: unable to retrieve the complete list of server APIs: kustomize.toolkit.fluxcd.io/v1: the server could not find the requested resource

I downgraded flux CLI to RC.4 but I get the same error:
❯ flux tree kustomization environment-tst
✗ failed to get API group resources: unable to retrieve the complete list of server APIs: kustomize.toolkit.fluxcd.io/v1: the server could not find the requested resource

❯ flux --version
flux version 2.0.0-rc.4

You need to upgrade your cluster to the same version as the CLI

Maybe I'm missing something but they seem the same:

❯ flux version --client
flux: v2.0.0-rc.4

❯ flux version
flux: v2.0.0-rc.4
helm-controller: v0.28.1
image-automation-controller: v0.28.0
image-reflector-controller: v0.23.1
kustomize-controller: v0.32.0
notification-controller: v0.30.2
source-controller: v0.33.0

I will try to upgrade the cluster, as I am running an old version.

I see now, using flux tree kustomization flux-system I see that the namespace is included in two kustomizations. Thanks!

It's still unexpected, because the namespace is included in a kustomization that should not include it (I think):

environment.yml:
apiVersion: kustomize.toolkit.fluxcd.io/v1beta2
kind: Kustomization
metadata:
  name: environment-tst
  namespace: flux-system
spec:
  interval: 10m0s
  dependsOn:
    - name: repositories
    - name: infrastructure
  sourceRef:
    kind: GitRepository
    name: flux-system
  path: ./kustomize/environments/tst
  prune: true
  wait: true
  timeout: 5m0s

path points to a directory that does not include Namespace/tst, but it is listed under both Kustomization/flux-system/environment-tst and Kustomization/flux-system/flux-system. I would expect only the latter, as the namespace is not present in the environment-tst kustomization.

Move environments out of the clusters dir, in there you should have only Flux Kustomizations, see https://github.com/fluxcd/flux2-kustomize-helm-example as an example on how to structure the repo.

That seems to have resolved it 🎉


Ensure that binaryninjacore.h is valid C.

Fixes #4422.

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

Alright, I've poked this into the CI. Assuming it passes I'll merge this.

Looks like it passed. Thanks for the contribution!


Beta 6 fixes

Worked some more on overlay positioning. I now assign an id for the editorsession aka editorId. This I can later use to hide inactive overlays.
@oliver---- can you review and if good turn this into a new patch release? Then I could base the forms release on a substance release version.

I am not too happy about the editorId caching. It does not look like the right approach. Maybe ScrollPane is not the right one to aggregate all the hints necessary for positioning.
So it seems, that for positioning there are different hints needed, one being the selection rect. Isn't the editorId and the selectionRect very much related?

I am against rolling out this experiment. By then we need to make forms work with a sha, or maybe via a tar.gz.

We should put this into a branch with different name. It is not about fixing beta-6.
More like adding new features.

OK let's wait with this then and further discuss. There's another thing I find a bit weird: When I create the Overlay instances, I need to assign the editor as the parent in order to make the context available to the overlay component. Since the overlays are injected in the body scrollpane it's weird to have its parent set to the editor component. Possibly all needs more thinking. At least everything works smoothly now for forms (from a user point of view).

@michael is there a way that you base off beta-6.4 and add code which monkey-patches the core classes (i.e. overwriting the prototype impls with custom versions)?

I think it's fine to base it off a sha.


Feature Request: Add / Remove Groups

Awesome work! Will you be incorporating the add / remove from group feature from node-sonos-http-api
into echo-sono for voice control?

Sure, I can take a look at this

This would be a great add - I did notice that there is a group.js in the lib/actions folder - are there specific commands?

I just wanted to check in on this. It would be beyond awesome to have this feature. Your work is greatly appreciated.

Sorry, I'm very busy at work this month, but I'll try to take a crack at this sometime in the next couple of weeks

No worries. If there's any way I can help, please let me know.

Would it be possible to have a variation of join which does not requires PlayingRoom?  That is, if I'm the only one home, and only one room is playing, it would be nice to be able to say something like "add Living Room" or "add Kitchen" without having to also specify the playing room.
This would be a nice enabler to combine with some sort of (Bluetooth or other) presence detection that might automatically add a room as you walk around the house.
I don't know if the code would have to query Sonos to see what room(s) are playing, but presumably if only one room (or group) is playing, then PlayingRoom should be optional as the default should be to add to the already playing room.

exactly what i would be interested in as well, being the only person at
home while watching TV/Movies/Video games it would be nice to add in
speakers to the playing group or remove as needed.
For example:
Alexa, tell sonos to play rock party mode
On Thu, May 5, 2016 at 5:55 PM, Steve Lemke<EMAIL_ADDRESS>wrote:

Would it be possible to have a variation of join which does not requires
PlayingRoom? That is, if I'm the only one home, and only one room is
playing, it would be nice to be able to say something like "add Living
Room" or "add Kitchen" without having to also specify the playing room.
This would be a nice enabler to combine with some sort of (Bluetooth or
other) presence detection that might automatically add a room as you walk
around the house.
I don't know if the code would have to query Sonos to see what room(s) are
playing, but presumably if only one room (or group) is playing, then
PlayingRoom should be optional as the default should be to add to the
already playing room.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/rgraciano/echo-sonos/issues/16#issuecomment-217313065

--
Research is the key to unlock knowledge -
  OSCE - Offensive-Security Certified Expert
  OSCP - Offensive-Security Certified Professional
  OSWP - Offensive-Security Wireless Professional
  GPEN - SANS560 General Pentester

@lemketron, @subinacls - there's a general need for coming up with a way to default act on the only playing system without a PlayingRoom parameter.  As you guessed, it would require querying the Sonos system to find the currently playing room and then default acting on it.  I opened #27 to track.


[ISSUE #3963] Method invocation 'getBytes' may produce 'NullPointerException'.

Fixes #3963
Method invocation 'getBytes' may produce 'NullPointerException'.
In the modified code, Objects.requireNonNull() is applied to the return value of JsonUtils.toJSONString(result). If the result is null, a NullPointerException will be thrown immediately, preventing the subsequent invocation of getBytes().
By using Objects.requireNonNull(), it explicitly check for nullness and handle it in a more controlled manner, ensuring that it don't encounter unexpected null values during the execution of the code.

@ZahaanMahajan you can merge the latest master branch code to fix the ci check error.

How to do that?

I didn't get you. How to do that?

pull apache:master to your local branch and push it to your remote master branch.


OrderBookDB may clear recent updates (Version: [1.5.0])

Issue Description
There is a rare race condition within OrderBookDB, that results in missing data, between OrderBookDB::update() and OrderBookDB::addOrderBook().  OrderBookDB::update() processes an entire ledger and then overwrites the underlying datastructures via swap: https://github.com/ripple/rippled/blob/97712107b71a8e2089d2e3fcef9ebf5362951110/src/ripple/app/ledger/OrderBookDB.cpp#L148. OrderBookDB::update() only holds a lock when performing the swap.
OrderBookDB::addOrderBook() writes to these same underlying datastructures. update() could take a long time, as it processes the entire ledger, whereas OrderBookDB::addOrderBook() can execute much more quickly.
The race condition is as follows:
OrderBookDB::update() is called, with a ledger with sequence i
OrderBookDB::addOrderBook() is called, while processing a transaction from ledger with ledger sequence j, such that j > i
OrderBookDB::addOrderBook() returns
OrderBookDB::update() executes the swap operation and returns
This above sequence leads to the OrderBook added by OrderBookDB::addOrderBook() being discarded.
It should be noted that OrderBookDB::update() is called when the software starts, and is only called subsequently if there is a gap of 100 or more ledgers in the published ledger stream: https://github.com/ripple/rippled/blob/97712107b71a8e2089d2e3fcef9ebf5362951110/src/ripple/app/ledger/impl/LedgerMaster.cpp#L1235
Therefore, this situation is very rare, and if it does occur, only results in some missing data. It has no ability to cause a crash and is not a security vulnerability.
Possible Solution
OrderBookDB::update() needs someway to know about the Books that were added by OrderBookDB::addOrderBook() while OrderBookDB::update() was running. OrderBookDB::update() could then merge these Books after performing the swap.

A couple of other ideas that occur to me:

update() holds the lock during the entire update process. This could have performance implications, obviously, so it remains to be determined if any trade off is worth it. It seems a very safe option for data integrity, though.
addOrderBook() signals to update() that it made changes while update() was working (via a flag or something). update() then aborts and starts over with the latest ledger, thus including the new data. This risks update() never really finishing on a heavily used node, but it remains to be determined how likely that is to happen.

I make no claim that either of these are good ideas. Just throwing them out for consideration.

* `update()` holds the lock during the entire update process.

This would be great for data integrity, but on my machine, if I start the software with --load (meaning I load a full ledger from the database, instead of starting from genesis until network sync), update takes upwards of 5 minutes, which is much too long to hold the lock for.

@gregtatcam could you take a look at this?


Get ghz compliant with errcheck

This PR does a few things:

Get ghz compliant with errcheck by propagating all errors, and documenting those that are purposefully ignored.
Remove all panics.
Remove all calls to the log package. Having a library log with the global stdlib logger makes it not possible for users to do logging in their own way.
Remove all calls outside of cmd/ to fmt.Print* for the same reason.
Add TODOS in the Requester where errors should be handled.
Do some locking in the Requester. Probably better to do some more extensive connection management in the future, but this is a start.
Other minor cleanups.
My editor removes whitespace, so a lot of that done.

Note this adds calls to https://godoc.org/go.uber.org/multierr#Append, which is a super simple library to handle having multiple errors.

Hello again, and thanks for the PR!
One thing that I didn't catch in the review but noticed quickly with some manual smoke tests was the errors from actual RPC calls were being returned which affected the end result of running the tests. The errors were reported rather than the report. The RPC errors are actually also detected in stats handler and sent to the reporter so we shouldn't return them in our actual flow inside requester. I made a small change to swallow those up in makeRequest() and added a comment about this.
I do have one question, which may be embarrassingly dumb, but why are we doing the deferred closes of connection and db inside a goroutine. I do not believe this is a pattern I have seen before.
Published as 0.31.0.


Documentation for LocalizationConnector vs LocalizationWrapper

I think the documentation for LocalizationConnector vs LocalizationWrapper has some typos.
Under LocalizationConnector you refer to it as wrapper and then the sample code references a LocalizationWrapper as well.
Not actually sure which is which or I'd send a PR

Sorry I didn't see this earlier. I'm planning on updating these a little bit for the next release but I'll try to get the documentation updated as well for this current release.


Add Timeout Value Option to Wait()

This is a great idea, thanks for the patch.

This is a great idea, thanks for the patch.


How can I use deepdive to perform unsupervised verification?

Hi, I have a unlabeled data set in which each sample has an initial confidence. I want to do some verification by writing rules about commonsense to reduce the confidences of samples which violate commonsense. How can I do this?

How can I use the initial  confidence?
How can I assign weights to the commonsense rules?

Thank you very much!

When I want to define a unsupervised model by using "p(x,y)=NULL: ....", the program report:
column "label" is of type boolean but expression is of type text
It seems that the error is about a bug in sql generation module.

You may want to check out http://hazyresearch.github.io/snorkel/, which is
more directly about weak supervision.
DeepDive can do all this (and much more!). This flexibility means that not
all representation decisions are obvious.
Chris
On Mon, Dec 19, 2016 at 8:24 PM 孙明明<EMAIL_ADDRESS>wrote:

When I want to define a unsupervised model by using "p(x,y)=NULL: ....",
the program report:
column "label" is of type boolean but expression is of type text
It seems that the error is about a bug in sql generation module.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/HazyResearch/deepdive/issues/608#issuecomment-268151386,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHPtuEFo-aPa3pwU-pGppjuv12ulh3Pfks5rJ1iIgaJpZM4LRaM2
.

@chrismre  Thank you for mentioning another amazing projects. I'll look at it.
However, currently I'd rather like to know how to do this by deepdive.

DeepDive can express essentially any factor graph. You'll need to write the rules that create the required factor graph. A default factor graph for this process is described in the Snorkel/data programming paper.
Hope that helps! Chris

@rudaoshi Here is an example rule with manually assigned weights (adapted from the "census" example):
@weight(1.2)
rich(id) :- adult(id, _, workclass, _, _, _, _, _, _, _, _, _, _, _, _, income_bracket).

You can change the rule body (the adult... part) to encode the "common sense" cases.

@rudaoshi Regarding the NULL issue alone, you should try to use casting within ddlog. Writing = NULL::boolean instead of = NULL will take care of the label casting issue.

Thank you very much to every one. I'll try your recommended solutions.

@thodrek I got following error using Deepdive 0.8:
2016-12-21 11:43:00.679659 [error] app.ddlog[176.44] failure: :-' expected but :' found
2016-12-21 11:43:00.679741
2016-12-21 11:43:00.679756 has_relation(p1_id, p2_id, relation) = NULL:boolean
2016-12-21 11:43:00.679767
2016-12-21 11:43:00.679947                                                                       ^
Does this feature need newer version cloned from github?

@rudaoshi note that there are two colons: NULL::boolean. If that doesn't work, try cast(null as boolean).


How to use hooks?

Maybe I'm just ignorant or not keeping up on changes that I should be, but I haven't seen any information on how to use hooks.  As the initial "definer" of an early issue, I'm curious how they work so I can try them out.  That particular issue was closed and locked before I could ask why the hook solution was better than folding the function in as a choco.exe switch, but regardless, I don't know how to use a hook.
Are there examples or documentation for how to use/call hooks?  All I can find is this documentation page advocating for extending Chocolatey with hooks, but nothing for "regular users".
Thanks.

Let me see if I can help to explain...
A hook is a specifically named PowerShell script, that lives (by default) within the C:\ProgramData\chocolatey\hooks folder.  Depending on the name of this file, either before (pre) or after (post) the install/beforemodify/uninstall phase of a Chocolatey operation, this PowerShell script (hook) will be executed, for either a specifically named package, or for all packages.
These PowerShell scripts (hooks) can be placed in the above mentioned location either manually, or by installing a hook package.
A hook package is a specifically named Chocolatey package (specifically a package that ends with .hook).  This type of package includes nothing but a collection of hook scripts, and when installed, Chocolatey will do the work of extracting the hook scripts to the above mentioned location.  An example of a hook package can be found on the Chocolatey Community Repository.  Hook packages are similar in function to a .template or .extension package, where Chocolatey has specific handling for where these packages are installed to.
If we take a closer look at the contents of this hook package, you will see that it contains the following two files:

pre-install-all.ps1
post-install-all.ps1

Based on what we said before, these scripts will be executed before (pre) and after (post) the normal Chocolatey install operation, for all Chocolatey packages.  If you look inside the contents of these scripts, you will see that they are attempting to prevent the creation/placement of a shortcut on the users desktop by taking a snapshot of the shortcuts that exist before the installation of a Chocolatey package, and then comparing this to the contents of the desktop after the installation has completed.
So, once this package is installed, any subsequent Chocolatey operations (say for example the installation of another package), will make use of the newly installed (or available), hook scripts.
Why are we suggesting that this work is done this way?  Simply put, there are lots of ways during the installation of a  Chocolatey package that a desktop shortcut might find its way to your desktop.  For example:

via one of the over 20 different installer formats that exist for Windows
via a custom mechanism in one of the above installer formats for Windows
via one of the PowerShell helper functions that exist for Chocolatey
via direct use of PowerShell
and others that I can't think of right now

So, bottom line, there is no sure fire way that we will catch all of the possible edge cases for how/when a shortcut might appear on your desktop.
As a compromise, we have implemented the hook functionality within Chocolatey that provides the end-user of Chocolatey with the power to write their own scripts (or consume hook packages that are contributed by the community) to do the work that they are looking for.
Trying to put this type of work directly into choco.exe would never cover all the potential ways that folks would want this type of functionality to work, so we opted to extend Chocolatey in a way that would mean that an end-user can do whatever is required for their specific needs.
Just to be clear here, hook scripts (and therefore hook packages) are not constrained to only caring about desktop shortcuts.  You can create a hook script that literally does anything that is required for you, or your team, or the wider Chocolatey community, during the normal operation of Chocolatey commands.
So, to recap, to take advantage of hooks, and specifically, a hook that aims to prevent the creation of desktop shortcuts, simply do one of the following:

write one or more PowerShell scripts (with a specific name) and place them in the above mentioned folder

or
choco install cleanup-desktop-shortcuts.hook --pre

Then, when you install another package, say for example, windirstat, the desktop shortcut that would normally have been created after the package was installed, will no longer be present.
We recently did a Chocolatey Product Spotlight live stream where we show hook scripts in action.  If you wanted to check this out, you can watch it back here:
https://chocolatey.org/events/chocolatey-spotlight-2022-november
If the above helps you to understand hook scripts and hook package better, and you think that there are improvements that can be made to our documentation, I would encourage you to create an issue here:
https://github.com/chocolatey/docs/issues
and ideally, if possible, create a pull request to suggest the required modifications.  All of our open source documentation for Chocolatey CLI is on GitHub, and anyone can contribute to it.
If you are unclear about how to contribute to GitHub, and specifically the Chocolatey documentation on GitHub, I did a short series of videos that walk you through exactly how to do this.  You can find this playlist here:
https://www.youtube.com/watch?v=MEYjsdZZ5_M&list=PL84yg23i9GBjLbj44FipAIChz98_drwij

I've pinned this issue as I think it's useful for everybody to see. It also highlights a hole in our docs.

@gep13 Thank you for that write-up; It helps a lot.
One key piece of it that I was unclear on and, unfortunately, is not the answer I was hoping for is that hooks are generic and indiscriminate.  They run before or after every package install with no ability for the user to selectively apply them (short of installing and uninstalling pre/post package install).  That is not necessarily a bad thing, and I see a logical reason and can imagine the potential usefulness to offer the hook feature in Chocolatey.  I will be interested in seeing what hook packages get developed.
However, for the purposes of the desktop shortcut question, I feel like it swings too far too the other side.  Installing users still do not have individual control over this one (rather annoying) aspect of application installation.

A hook package can be created that targets a specific package, you use the packageId in the name of the file.  Also, you can pass the --skip-hooks option during the operation that is being performed.

@teknowledgist said...
However, for the purposes of the desktop shortcut question, I feel like it swings too far too the other side. Installing users still do not have individual control over this one (rather annoying) aspect of application installation.

Can you perhaps elaborate on this point a little bit?

@gep13 Actually, I think you resolved it.
There are two types of application packages:

Packages for which users can control desktop icons either because the installer has the option or because the maintainer has graciously coded the option.
Packages for which the user cannot control the creation of desktop icons.  These are the packages of concern here.

My issue was that with the desktop shortcuts hooks, users still didn't have control over 2 but in the other direction.  The hooks do prevent desktop icons from appearing after an install or update, but the user doesn't have granular control.  All desktop icons created on installation are removed (or so I thought).
The --skip-hooks option -- that isn't in the documentation on hooks or your explanation above -- does give per-package granular control, so I think you/Chocolatey are "off the hook".  😄 That said, I can imagine someone in the future asking how to skip some hooks and not others.
I'm not convinced that desktop icon prevention/removal couldn't have been adequately managed from within Chocolatey, but I can accept the hooks solution.
Thanks.

All desktop icons created on installation are removed (or so I thought).

For the cleanup-desktop-shortcuts.hook hook package, that is correct. All desktop icons/shortcuts created during while running all chocolateyinstall.ps1 scripts are removed. Desktop icons/shortcuts that are preexisting or are added outside Chocolatey package installations should be kept.

The hooks do prevent desktop icons from appearing after an install or update, but the user doesn't have granular control.

So you are saying that you would like a way to configure the hook script via a whitelist/blacklist to keep/remove desktop icons from specific package IDs, instead of removing icons from all package IDs?
This makes sense to me, and is something I would like to add.
Once https://github.com/chocolatey/choco/issues/2854 is released (or another mechanism for configuring hooks is decided on), then support for configuring which package IDs have icons removed could be added.
For example, this could be a semicolon separated list of package IDs, and the cleanup-desktop-shortcuts.hook would no-op on those package IDs, and so not remove any icons created.
Would you be able to create an issue for this, with any wants/needs/suggestions for what this configuration would look like?

The hook solution with the --skip-hooks switch is granular at the package level.
If you look at my description of the original issue, I wanted to be able to have sub-package granular control.  Some packages deposit multiple icons, and my dream as a user/administrator was to be able to remove (or leave) only the icons I wanted.
A whitelist/blacklist to "lock" the hook on/off for packages isn't a bad idea, but does require some planning and keeping track of another file somewhere whereas the --skip-hooks (and my imagined, ideal switch) would be on-the-moment solutions.  For the purposes of desktop icons, it would allow for a Choco upgrade all to be more granular with regard to which applications get to place/keep icons on the desktop.  Otherwise, I imagine upgrades would have to be split into the --skip-hooks set and the "all hooks" set.
The brings me back to my assumption that someone at some point is going to want to apply hook "A" to some packages but not hook "B", and unless the hooks are written to affect only specific packages, it wont' be possible (unless --skip-hooks allows for listing the hooks to skip -- idea?).
I still haven't had a chance to play with the hooks, so I don't know when I will be able to create an intelligent issue or documentation, but I will keep both in mind.

@teknowledgist said...
A whitelist/blacklist to "lock" the hook on/off for packages isn't a bad idea, but does require some planning and keeping track of another file somewhere whereas the --skip-hooks (and my imagined, ideal switch) would be on-the-moment solutions.

The file that @TheCakeIsNaOH was referring to here is the chocolatey.config file, which would be updated via the normal choco config command, i.e. you would do something like choco config set --name="hook-alldesktopshortcutfrom" --value-"windirstat;vlc". This configuration value would then be read with the hook script, and shortcuts would be allowed from those packages.  So it wouldn't be a separate file that would have to be maintained, it would be within the normal Chocolatey configuration.

@teknowledgist said...
If you look at my description of the original issue, I wanted to be able to have sub-package granular control. Some packages deposit multiple icons, and my dream as a user/administrator was to be able to remove (or leave) only the icons I wanted.

This is perhaps where the benefit of moving this work into a hook package starts to come into play.  For example, it would be possible to add an almost "interactive" mode to desktop shortcut cleanup to the hook package.  In the final part of the hook execution, rather than simply deleting all the shortcuts that were created, it could prompt and say "these are the shortcuts that were just created, what do you want to do, keep or remove".  Having that sort of functionality directly in choco.exe would likely not make sense, but confined within a hook package, which it tailored to the specific action that is being executed, it is more than possible.  This interactive mode could again be enabled/disabled via a configuration value in the chocolatey.config file, so that folks can get the sort of functionality that they want.

@gep13 Very good.  Thanks.
I can see why you might want to push that interactive stuff onto optional hooks, but have you considered allowing Chocolatey to pass parameters to specific hooks?  Imagine, for example:
choco install mypackage --hookset @{cleanup-desktop-shortcuts=@{regex='^sidekick'}; aNoisyHook=@{sound='quack'}; dummyHook=$false}

If the cleanup-desktop-shortcuts hook had an optional regex parameter to match shortcut names to remove (defaulting to .*), and a fictional "aNoisyHook" had an optional sound parameter, and a "dummyHook" shouldn't be run for this package this time.  That would give a lot of power to hook developers who might want to offer granularity without interaction and users who might want in-the-moment options.
Just my $.02.


Some issue in LINE

    # self.h_e = tf.nn.l2_normalize(tf.nn.embedding_lookup(self.embeddings, self.h), 1)
    # self.t_e = tf.nn.l2_normalize(tf.nn.embedding_lookup(self.embeddings, self.t), 1)
    # self.t_e_context = tf.nn.l2_normalize(tf.nn.embedding_lookup(self.context_embeddings, self.t), 1)
    self.h_e = tf.nn.embedding_lookup(self.embeddings, self.h)
    self.t_e = tf.nn.embedding_lookup(self.embeddings, self.t)
    self.t_e_context = tf.nn.embedding_lookup(self.context_embeddings, self.t)
    self.second_loss = -tf.reduce_mean(tf.log_sigmoid(self.sign*tf.reduce_sum(tf.multiply(self.h_e, self.t_e_context), axis=1)))

I see those code in method LINE, and I am concerned about the same question why we cannot use l2_normalize in this step. Could you please give me some brief explanation about this point?
By the way, this is a really nice repository. Thank you for sharing your work.

Thanks.
Basically, you can use l2_normalize in this step. According to the author's implement, I comment these code here.

Thank you for your response so quickly.
However, I have try it right now and it give me an output like 'micro': 0.21136173767752714, 'macro': 0.048879544686962276.  When I comment the l2_normalize, it can shows 50% accuracy in Wiki dataset with 50% data for training.
My input like: python src/main.py--method line --label-file data/wiki/Wiki_category.txt --input data/wiki/Wiki_edgelist.txt --graph-format edgelist --output vec_all.txt --order 2 --epoch 50 --clf-ratio 0.5
I am really confused about this point, and it seem like that if I choose l2_normalize before training the model the gradient disappear.

Interesting result. Actually, the norms of embeddings are not the same among different nodes. If you check out the output of LINE or Deepwalk, you will find that the norms of some nodes with high degrees are very small. It means that they connect to so many neighbors that the model cannot find significant representations for them.
When it comes to normalization during training, I think it will influence the model somehow. So, just forget it :)

You are right.
Actually, I have tried add L2_normalize in two place in training process, which one is before calculating loss just like code you comment and another is after it computing the sum of the dot product. Both of them shows a result like above.
However, by your response, I realise that in this process I only do a L2_normalize for the node selected by every batch. It lead to that those nodes with high number links  will be normalised for many times.
It confused me for a week and really thank you for sharing=)


Make the logic block the only mandatory block

Old design made sense for umbrella interview--one mandatory block is much cleaner default.

#153


Still maintaining this?

If not I'd love to have the pub package name (discord). No biggie though.

Just out of curiosity, what kind of project would you use it for? :eyes:

It has been a while because I have been working on some other projects. I would love to do some updates soon. Since you are a collaborator you can also do some work if you want.

Is this dead?

It's been over a year since the last release.

I'm also wondering if this project is still active. ^_^

What is the best way to contribute to this project?

If I may, Nyxx is a Nyx fork which is currently maintained.


How to create a compass

I'd like to make a separate compass. But no corresponding interface was found.

You need to do it manually, look here https://github.com/NativeScript/NativeScript/issues/5889#issuecomment-507413711


High CPU usage

I've noticed that in my (very small) VM instance the CPU usage sometimes stays at 100% for a long time. Using pprof added in 7acd50c3201c2652d3adb0c57cb80d7549c52256 the CPU profile looks like:

See: https://gitlab.com/snippets/1974746


favicon missing again

@arghadipmanna101 sir assign me this issue someone have removed the favicon that i added , I am working under gssoc24

Hey if possible please asssign this task to me i can work on it

hey please assign this issue to me

i would like to work upon this issue..please assign me this issue.

@arghadipmanna101 Please assign me the task.

Go ahead @Meetjain1

@arghadipmanna101 sure sir

@arghadipmanna101 just created a pr #246, related to this Thankyou

Have a look on PR #246 related to this issue sir , Thankyou

Could you please assign this issue to me i can solve this bug quicky please assign this to me...


Can't bind to 'svgStyle' since it isn't a known property of 'svg-icon'.

Getting this error.

If 'svg-icon' is an Angular component and it has 'svgStyle' input, then verify that it is part of this module.
If 'svg-icon' is a Web Component then add 'CUSTOM_ELEMENTS_SCHEMA' to the<EMAIL_ADDRESS>of this component to suppress this message.

I am using Angular 7.2.0 version.

That just looks like you either haven't installed via npm, or haven't included the AngularSvgIconModule import in the relevant module.

@Epsilon-Rashmi I also faced the same issue and included in the relavent module and it's working fine.
@Supratentorial thank you :)


Does not seem to compile in VS2019

I can't get wise_enum to work in Visual Studio 2019 16.1.4..I'm using version 3.0.0.
My code:
WISE_ENUM(MyEnum, MyEnumCaseOne, MyEnumCaseTwo)

Error output:
1>------ Build started: Project: WiseEnumTest_App, Configuration: Debug x64 ------
1>Main.cpp
1>E:\WiseEnumTest\Source\Main.cpp(3,1): warning C4003:  not enough arguments for function-like macro invocation 'WISE_ENUM_IMPL_ARG_N'
1>E:\WiseEnumTest\Source\Main.cpp(3,1): error C2143:  syntax error: missing '}' before '('
1>E:\WiseEnumTest\Source\Main.cpp(3,1): error C2143:  syntax error: missing ')' before ','
1>E:\WiseEnumTest\Source\Main.cpp(3,1): error C2059:  syntax error: ')'
1>E:\WiseEnumTest\Source\Main.cpp(3,1): error C2143:  syntax error: missing ';' before '}'
1>E:\WiseEnumTest\Source\Main.cpp(3,1): error C2059:  syntax error: '}'
1>E:\WiseEnumTest\Source\Main.cpp(3,1): error C2059:  syntax error: '>'
1>E:\WiseEnumTest\Source\Main.cpp(3,1): error C2062:  type 'unknown-type' unexpected
1>E:\WiseEnumTest\Source\Main.cpp(6,34): error C2059:  syntax error: 'public'
1>E:\WiseEnumTest\Source\Main.cpp(7,1): error C2143:  syntax error: missing ';' before '{'
1>E:\WiseEnumTest\Source\Main.cpp(7,1): error C2447:  '{': missing function header (old-style formal list?)
1>E:\WiseEnumTest\Source\Main.cpp(97,1): error C2061:  syntax error: identifier 'WiseEnumTestApplication'
1>Done building project "WiseEnumTest_App.vcxproj" -- FAILED.
========== Build: 0 succeeded, 1 failed, 0 up-to-date, 0 skipped ==========

I've tried adding the /permissive- compiler flag, but I still get the same errors. I've also tried running create_generated.py to generate just 125 cases, but still get the same error.
Thank you again for the great work, and any help would be appreciated 😊

Sorry the response has been a long time coming; I had a kid and bought a house so the last 3 months were hectic. The windows situation for wise enum has always had issues, windows pre-processor and compiler are the least compliant (IME) and I can't really easily debug it myself. If you are interested in working together a bit on the windows issues, I'll be able to be more responsive and we can work through figuring out what's causing the issues.

This is fixed in merge request #45

I'm going to close this for now since it seems to now compile according to the CI (thanks to @samangh 's amazing work). However, I would encourage @martinfinke or anyone else to reopen this issue if it still does not work.


Impossibility to delete the first character of the list in an empty document

Is there an existing issue for this?

[X] I have searched the existing issues

Flutter Quill version
9.2.10
Steps to reproduce

Add list without text
Try deleting by attempting to erase

Expected results
The first item in the list will be deleted
Actual results
The first element of the list remains without the possibility of deletion
Code sample
Code sample
 QuillEditorConfigurations(
        controller: controller,
        scrollable: scrollable,
        padding: padding,
        autoFocus: autofocus,
        expands: expands,
        readOnly: readOnly,
        showCursor: showCursor,
        elementOptions: QuillEditorElementOptions(
          orderedList: QuillEditorOrderedListElementOptions(
            customWidget: Center(child: MyCustomWidget()),
          ),
        ),
       ...
       );

Screenshots or Video

Screenshots

cannot be deleted with the "delete" button

Logs
No response

If you read the Delta spec there is no empty document. It has at least a '\n'

I assume by erasing you are using the backspace key.
This does not work because there is no text to erase - list is an attribute.
I agree that this is not expected if you create the list by entering the keys '1' then '.' then space which auto-converts to list. Others have commented that this auto-conversion should not occur.
You can erase the list by pressing the enter key or using the toolbar button.

@AtlasAutocode , is it possible to delete using a system button, without using a custom toolbar? My application does not provide such functionality and it looks pretentious

Are there any questions?
Can I close this issue?


[WFCORE-3155] Upgrade JBoss Remoting to 5.0.0.Final

https://issues.jboss.org/browse/WFCORE-3155
Also includes the JBoss Invocation upgrade commit from #2732 in order to let the CI run test the combination.

Full integration - Windows Build 4120 outcome was FAILURE using a merge of 8e642d942c9bffccf736889238a5ee86db0552a6
Summary: Execution timeout (new); tests passed: 4116, ignored: 112 Build time: 02:31:08


Fix WebView crash

please push this

Can we get this merged into master please.

please merge this fix.

anyone still maintaining this???

any news?


Fix for config load

Somethings not right, query strings and config have no effect...

Somethings not right, query strings and config have no effect...

I've added 2 commits, did you pull those in your test?

I've added 2 commits, did you pull those in your test?

That's better haha, sorry for the introduced bug :)

That's better haha, sorry for the introduced bug :)

No worries, appreciate the hand!

No worries, appreciate the hand!