[Feature] Custom Formulas

Takes advantage of FoundryVTT's Roll to evaluate a string formula with actor data.
  static withActorData(formula, actor) {
    const r = new Roll(formula, actor.data.data);
    r.evaluate();
    return r.total;
  }

Features
This has been used to

Allow each individual Spell Point cost to have their own formula
Allow the hp cost multiplier to have a formula
Added a new field for a custom max spell points formula.
Added a new field for a custom formula to multiply the total cost of spell points. The result is added on to the custom max spell points formula.

Breaking Changes
The following settings have been removed as they would overwrite the custom formulas:

spFormula
spellPointsByLevel

Usage
DMG
To reproduce what is in the DMG, which is the default setting,  set the following:
Base Formula
0

Spell Point Multiplier
1

I propose that this is better than the previous hardcoded DMG values were for maximum spell points. Now it calculates the total spell point costs of slots for a character even if

The character's class is unknown
The character overwrites how many spell slots they have, such as an epic feat that grants an additional 9th level slot
The spell point costs differ from what the DMG suggests

Advanced Magic
To reproduce what is in Advanced Magic, the bellow code can be used. To understand this formula please see Advanced Magic.
Base Formula
ceil(
  (
    1 * @spells.spell1.max +
    2 * @spells.spell2.max +
    3 * @spells.spell3.max +
    4 * @spells.spell4.max +
    5 * @spells.spell5.max +
    6 * @spells.spell6.max +
    7 * @spells.spell7.max +
    8 * @spells.spell8.max +
    9 * @spells.spell9.max
  ) / 2
)
+
@attributes.spelldc - 8 - @attributes.prof

Spell Point Multiplier
0

A note on the above formula. There is no @attributes.spellcasting.mod. There is @attributes.spelldc which can be used to reverse engineer a spell casting modifier by subtracting 8 and subtracting the character's proficiency.
As Unusual As You Want It
You do not need to reproduce a known formula. You can make your own. Consider
Spell Points cost for Level 1 slots
<EMAIL_ADDRESS>, 1}kh

Spell Points cost for Level 2 slots
<EMAIL_ADDRESS>1}kh

Spell Points cost for Level 3 slots
<EMAIL_ADDRESS>1}kh

Spell Points cost for Level 4 slots
<EMAIL_ADDRESS>1}kh

Spell Points cost for Level 5 slots
<EMAIL_ADDRESS>1}kh

Spell Points cost for Level 6 slots
<EMAIL_ADDRESS>1}kh

etc
Making Your Own Formula
Create a macro with the following script:
console.log(actor.data.data)

While selecting a token, execute the macro. Then, look at the console. This will tell you all the data that is available to you when writing a formula.
You can test out a formula by rolling it in chat while selecting a token. The following formula would determine the largest modifier which are typically used for spell casting, and it would roll it in chat.
/roll { @abilities.wis.mod,  @abilities.int.mod,  @abilities.cha.mod}kh

It is also possible to add a module or create your own module that edits or adds information to actor.data.data. This new information will be available to the formulas with no concerns over compatibility.
Edits needed
The following should be looked at

I cannot read or write Spanish or Italian. I used a free translator tool. Please look at those changes and apply something more sane.

thank you for your commit, looks like a lot of work, I'm testing it.

Ok I've tested it and the formulas appear to work.
I have some concerns anyway.
In the current state this feature cannot be released as an update since there is no upgrade path.
Current users of this module will need to setup the module again. If they simply update it, their saved settings are there but their spell point system will chease to work since new settings are not fully compatible with old settings.
Also this feature make the module setup a lot harder regular users.
Finally the Advanced Magic system has been updated and now includes warlocks, but pact magic uses a different formula and is not possible to have that extra calculation right now.
I think that an upgrade path should be planned, where as a minimum the following should happen:

Old setting should keep working.
There should be a selector or a button to auto fill the Standard DMG formula or the advanced magic formula (maybe in the future more common formulas could be added)

What do you think?

That path sounds reasonable to me. I will also think on Warlocks.

This is a pretty cool feature.

Thank you for reviewing my work so quickly!


Notify organization admins when funds are added

Resolves https://github.com/opencollective/opencollective/issues/2705

We should probably mention the host that added the funds, because added funds are linked to a specific host an can only be used within this host.


feat(carousel): exposing next and prev on the controller.

Interesting note that we don't currently have functional tests in place for testing that next() acutally navigates to the next slide.  Same for prev().  @Foxandxss, is this something you can add to your test sweep?

@icfantv actually we do. https://github.com/angular-ui/bootstrap/blob/master/src/carousel/test/carousel.spec.js#L175-L195
Those click are calling the appropriate methods.
About this PR. What is the use case? and I like better tests (more than checking that there is a function, those are poor).

About this PR. What is the use case?

I have a directive that used uib-carousel in it's template.  I wrote another attribute directive that requires my element directive and uib-carousel.  The attribute directive decorates .select on uib-carousel and notifies my element directive so it can hide/show the left|right controls accordingly.  My logic would be simpler if I could decorate next|prev instead.

... and I like better tests (more than checking that there is a function, those are poor).

I couldn't agree more.  The only reason I did this was because $scope.next and $scope.prev were already being tested.  Perhaps I should update the tests to verify that ctrl.next === $scope.next?
Thanks for the great work on this lib!

Perhaps I should update the tests to verify that ctrl.next === $scope.next?

Just pushed this to the PR, so it's now verifying that the methods are one and the same between ctrl and scope.

About this PR. What is the use case?

Also, I've had edge cases where the markup in uib-carousel doesn't meet my needs, so I ended up grabbing .next|.prev off of scope to make my own custom controls, but this was dirty.  Easier to require the controller and avoid $timeout to wait for the control elements to be present.

Ahhhh, my bad.  I was looking for direct invocations and not, you know, actually clicking the thing that matters...the navigational elements.

I'm ok with this PR, but I'm not sure tests really make any sense, or at least these anyhow.

I'm ok with this PR, but I'm not sure tests really make any sense, or at least these anyhow.

I'm open to suggestions.  $scope.prev and $scope.next are tested extensively in other parts of the spec file.  To avoid duplication, the tests as they currently stand are just asserting that the controller methods reference the $scope methods.

I was thinking that this is ok with no extra tests - I'll wait for @Foxandxss's opinion here though.

Gotcha.

Thanks for merging this!


heap-buffer overflow at line 1807 in SBaseFileTable.cpp

A heap-buffer overflow occurs in the below program when provided with malformed input.


[Question] How to Bundle a Plugin

How can I compile bitbar in a way that would bundle a plugin with it? I imagine it would need to place my .py file in the plugins directory once it is installed/run. Do you know how or where in the code to put that logic?

Closing issue since the answer is here:
https://github.com/matryer/bitbar/blob/master/Docs/DistributingBitBar.md


threadpool: send signal only when queue is empty v3

@saghul  now bool got implicitly convert to int.

Landed in 0f1bdb6, thanks! For future PRs, please use a single PR and add commits to it, instead of opening a new one, that way we can keep the conversation in a single place :-)


Union Types

Feature Request 🛍

Proposed syntax
concept A {
  o Integer | String value
}

This would allow new types to be created by composition rather than extension. Sometimes, this a more natural way to model business concepts. See the first example here
https://flow.org/en/docs/types/unions/#toc-disjoint-unions

We could also consider disjointed unions, https://jsontypedef.com/docs/jtd-in-5-minutes/#discriminator-schemas

This is a majorly different type system (i.e., switching from OO-style to ensemblist). This will notably make a strongly typed language like Ergo on top of Concerto close to impossible (or very hard).
I would vote against supporting that feature.
A possible alternative type system with a form of union could leverage enums. E.g., see in Swift:https://docs.swift.org/swift-book/LanguageGuide/Enumerations.html
Syntax:
concept RGB {
  o Integer R
  o Integer G
  o Integer B
}

enum Color {
  o red
  o green
  o blue
  o String name
  o RGB rgb

With the source example in mind:
enum IntOrString {
  o Integer int
  o String string
}
concept A {
  o IntOrString value
}

Hello everyone . First timer this side . I want to contribute to accordproject , so is the discussion currently going or may I can get this issue to work ?

hi @Rishabhraghwendra18 welcome. This is very much on-going. I don't think we have decided we should do this.

Welcome @Rishabhraghwendra18 !
This issue doesn't have strong support right now, and should probably be closed. However, there are several issues tagged with "Good First Issue" across the organization. In Concerto, see this list, https://github.com/accordproject/concerto/issues?q=is%3Aopen+is%3Aissue+label%3A"Good+First+Issue+%3Aoctocat%3A"

Reopening this, as it has been raised recently by the community - and support has been added for it to the Typescript generator:
https://github.com/accordproject/concerto/issues/429


Module 'CocoaAsyncSocket' not found

Xcode 10.1, MacOS Mojave, Swift 4.2
After installing the pods and calling it from my project, I get this error :
Module 'CocoaAsyncSocket' not found

for those who do not want to use_frameworks! just add this into your Podfile
pod 'CocoaAsyncSocket', :modular_headers => true, :configurations => ['Debug']
pod 'Bagel', :configurations => ['Debug']
:configurations => ['Debug'] is optional for sure. that's was just in my case

for those who do not want to use_frameworks! just add this into your Podfile
pod 'CocoaAsyncSocket', :modular_headers => true, :configurations => ['Debug']
pod 'Bagel', :configurations => ['Debug']
:configurations => ['Debug'] is optional for sure. that's was just in my case

This works for me! Thx!


Django 3.0 compatibility

Django will remove the vendored django.utils.six in the upcoming 3.0 release. six is in use in django-formtools for normalization of string types and iteration.
Since I see that the next version of django-formtools will drop support for Django versions 1.10 and below, and will add support for 2.1, the smoothest experience for everybody concerned would probably be to depend on six directly for a bit?

Any updates on this?  I'm trying to upgrade my project to use Django 3.0 and it seems that formtools/utils.py is importing django.utils.six which is now deprecated.

I see the source installed from pypi is still using six, but the latest code on github doesn't.
I guess there hasn't been a new pypi release with these latest changes?

I just added this to my requirements.txt and my project is back to normal, using django 3.0
-e git+git://github.com/jazzband/django-formtools@db1f4f4049d913a3774a0edcf5107203b8a15e91#egg=formtools
Maybe there can be a new PyPi release, so we don't have to have this in requirements.txt ?

It would be awesome to have a new release in order to use it in django 3

Soon we can have new PyPi releases compatible with Django 3.0.
The new version is created already.

What is keeping us from releasing it to pypi?

A releaser!

django-formtools 2.2 (with Django 3.0 compatibility) is now on PyPI.


Initial commit of Jenkins pack

This PR moves Jenkins over from st2incubator.
Configuration

url - FQDN to Jenkins API endpoint
username - Jenkins Username (if auth is enabled)
password - Jenkins Password (if auth is enabled)

Actions

build_job - Kick off CI build based on project name
list_running_jobs - List all currently running jobs

Sweet. :+1:


Host on IPFS

I will collect some experience with my own blog. Your site is more complex but maybe I can help a bit.

Is using ipns acceptable? The drawback is that during the propagation links may be not working or assets are not present.
The other method would be updating your DNS on every publish step.

@Jorropo is working on a PoC -> https://jorropo.ovh/ipns/k2k4r8pdq8w3382yrqib8vpt60v0282nzsmv17qp4rtxg1h8lca76tlv/
There are various issues:

font-awesome is configured to run on certain hostname only -> is there a way to fallback nicely?
some images are loaded with a bad path

some content are hardcoded to be served by berty.tech

probably other things I missed
@cdeleeuwe can you tell us what you think?

@Jorropo above PR fixed some relative urls. Hopefully it will now work without errors :)

@cdeleeuwe seems, to work fine, only font awesome is left to tackle (I'll take a look later).

@Jorropo awesome :). From what I see, font awesome is loading OK..

Just made a PR (#153) to fix the homepage Lottie animation relative url. After that I think there might still be some assets missing that are used in the style.scss file. These are are a bit tricky to make relative paths, since Hugo only renders the SASS once, so is not aware of the current path. A possible workaround would be to move it the partial/head.html file (similar as what is done with the fonts).

@Jorropo awesome :). From what I see, font awesome is loading OK..

Just made a PR (#153) to fix the homepage Lottie animation relative url. After that I think there might still be some assets missing that are used in the style.scss file. These are are a bit tricky to make relative paths, since Hugo only renders the SASS once, so is not aware of the current path. A possible workaround would be to move it the partial/head.html file (similar as what is done with the fonts).

I can't reproduce it either.

Now working on higher priority items. This could be a good project for someone who wants to experiment with IPFS. Backlog for now.


Handling different names of output libraries

Context: This was encountered recently with boost (#278).
Depending on the configuration of library linkage and static linkage (and others, see link) the output boost libs are named differently. For example, for dynamic libs we get:
boost_atomic-vc140-mt-1_62.lib (release)
boost_atomic-vc140-mt-gd-1_62.lib (debug)
whereas for static libs we get:
libboost_atomic-vc140-mt-s-1_62.lib (release)
libboost_atomic-vc140-mt-sgd-1_62.lib (debug)
[diff = the "lib" in the beginning and the -s- before the version]
FindBoost.cmake (which ships with CMake, so it will be in C:\Program Files\CMake\share\cmake-3.7\Modules or equivalent) looks for the dynamic lib names by default and you can specify Boost_USE_STATIC_LIBS to have it look for the static lib names instead.
However, in vcpkg only one flavor of the library may be present in the same triplet. Therefore, the intention is that a library's functionality is either present or not present; names should not matter. We have therefore opted to rename the output static libs to the corresponding names of the dynamic libs. This will make them automatically discoverable by FindBoost without the need to specify the Boost_USE_STATIC_LIBS flag.
This is not isolated to boost; the same thing is being done for zlib.lib. In the static flavors the output is zlibstatic.lib and we rename it to zlib.lib. In fact, unlike boost, FindZLIB.cmake only looks for zlib.lib and there does not seem to be a way for it to look for zlibstatic.lib.
We would like to ask you about your experience regarding the matter and whether you think this is an appropriate way to handle differently named lib files between dynamic and static. Do you think it is a good idea to adopt this strategy in general (i.e. not just boost, zlib)?

most libraries seem to have different named libs on windows (probably due to the fact that import libraries and static libs share the same format/extension). That said since cmake really does not have great facilities for saying "static link all the things" renaming may be correct.
I'm not sure how the linter can verify things are named correctly without looking at the dynamic version (which may not be around).
Also maybe we should change boost to a system install (it's an option in boost's build system that gets rid of all that version and linkage cruft in the filenames).

A thought: If a package depends on another package and has a non-cmake build system then it probably does not support linking against different export libraries depending on debug or release configuration. HOWEVER renaming things solves this issue because the symbols are almost by definition the same, and the debug and release versions should be drop-in replacements for each other.
It does mean more mucking about with the package config.cmake files though.

I was not aware of the "system install". It looks interesting, I will investigate.

3588f534dbc97f74b90ca8c7d1e9713281ae8b81 adds the flag --layout=system.

The current layout is as follows:

All dynamic libs are built with name: boost_xyz.lib.
All static libs are built with name: libboost_xyz.lib. They are renamed to boost_xyz.lib
In the dynamic build, there are a few libs that are build statically (possibly because they can only be built statically), for example libboost_exception.lib. Those are renamed to boost_xyz.lib, unless that name already exists, in which case libboost_xyz.lib is simply deleted.

This is because FindBoost.cmake is by default only looking for boost_xyz.lib names.

IMO support for building libs on Windows is still a mess.. not vcpkg's fault
Ideally it would be automated (and standardized). Boost is special, let's focus on a simple example like png or z. The lib names shouldn't be chosen manually, but this requires support from the build system.

This issue hasn’t been updated in two year, if it is still an issue, please reopen.


Freeze in partition_by_union with non-overlapping colors

@syamajala recently showed me a case where we hang when we take the union of partitions A | B if A.colors and B.colors is non-overlapping (i.e. each of A.colors and B.colors contain points not in the other). It probably also occurs in other cases too, this is just the reproducer we have right now.
What's the expected behavior in this case? Clearly, it shouldn't freeze, but is this expected to work?
import "regent"

task main()
  var mat = region(ispace(int2d, {10, 10}), double)

  var colors = ispace(int2d, {1, 1}, {-1, -1})
  var part = partition(equal, mat, colors)
  regentlib.c.printf("vol: %d\n", part[{-1, -1}].ispace.volume)

  var colors2 = ispace(int2d, {1, 1})
  var part2 = partition(equal, mat, colors2)
  regentlib.c.printf("vol2: %d\n", part2[{0, 0}].ispace.volume)

  var up = part | part2
  regentlib.c.printf("vol3: %d\n", up[{0, 0}].ispace.volume)
end
regentlib.start(main)

To be clear, when Regent detects that the color spaces are not equal, it internally unions them before passing this on to the runtime: https://github.com/StanfordLegion/legion/blob/482e188d06bf9050b5d5a85fd0050c22374cf0ee/language/src/regent/codegen.t#L6952-L6957

@elliottslaughter How do you compute this parameter to create_partition_by_union:
https://gitlab.com/StanfordLegion/legion/blob/master/runtime/legion/legion_c.cc#L1187

It's in the highlighted code up above. I'm taking the two argument color spaces and passing them to legion_index_space_union to get the union color space.

What happens if you run the code with:
var up = part & part2

That also freezes.

Actually I think this is a compiler bug. I think the compiler is unioning the color spaces when it should be intersecting them. @elliottslaughter will have to say for sure. But at least the runtime is expecting all these kinds dependent partition operations to happen on intersected color spaces which you can't control from user code in Regent.

@magnatelee can you look at this?

In my opinion the Regent interpretation is defensible; why would you ask
for a union operator but then expect the color spaces to be intersected?
But either way, it should be trivial to fix in Regent. The change to get it
to use intersections will be one line. The main thing to keep in mind is
that Seshu's code will need to change, in that all the regions on either
side of the union operator will have to be created up front, and the ones
that currently don't exist in one side or the other will just be made empty.
The one thing I don't think we should do is allow the Regent and Legion
behaviors to diverge. That's just asking for confusion.
On Wed, Dec 19, 2018, 2:39 PM Seshu Yamajala<EMAIL_ADDRESS>wrote:

@magnatelee https://github.com/magnatelee can you look at this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/StanfordLegion/legion/issues/452#issuecomment-448769358,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAMOQbdfxzY2gc-k63nrArENo_Whikcks5u6sAwgaJpZM4Y4bhR
.

I found that even if one color space contains the other you can't assign a partition to itself. I'll try to see if I can come up with a small example.

Assigning this to @elliottslaughter and @lightsighter to agree on the right way for both Legion and Regent to deal with this.

I can make Regent conform to Legion's interpretation. It wouldn't solve @syamajala's problem but I think I've convinced myself we need a different approach there anyway at this point.

@lightsighter Did we decide to go with Legion or Regent's interpretation on this one?

I still don't think Regent's approach is defensible. Consider the case of subtracting one partition from another: what do you do if the colors spaces of the two partitions don't match? You can't subtract a subregion from nothing. In my opinion Legion's approach is the only one that is well defined: regardless of the set operation being performed, we only perform it on the points that exist in the color space of both partition operands. It was even documented like this in legion.h before Regent implemented the operators. :)

I'm bringing another piece into this because I think they're coupled:  If I have a partition in which many of the subregions are empty (e.g. a 2-D tiled partition of a sparse matrix that has few off-diagonal entries), what is the space-efficient way to represent that partition in Legion?  Is it:
a) a full (i.e. dense 2-D rectangle) color space for which Legion (efficiently) knows which colors actually have (non-empty) index spaces allocated to track the subspace, or
b) a sparse color space, allowing Legion to always pay for an IndexSpace for each point in the color space
If the answer is (a), then I'm fine with Legion's semantics.  Set operations will use two partitions with the same dense 2-D color space even though their sparsity patterns may be different, and the right thing will happen.
But if the answer is (b), then I think Regent's semantics are needed.  Otherwise, any set operation on two partitions with the same "conceptual color space" (i.e. the colors of possible subregions before empty-ness is considered) will require the application to do the corresponding set operations on the two partitions' color spaces first to get the desired behavior.

Except for (b) in the common case which is Legion will recognize that the names for the color spaces of the two partitions are the same and then we don't need to do the intersection explicitly.

My point was that with (b), it'd be the application's problem to use a sparse color space for each of the two partitions, and then they wouldn't be the same from Legion's perspective.  The application being able to (space-efficiently) use the same color space for both partitions, regardless of which subregions are empty in each partition, is choice (a).

Legion always requires a color space to be an index space (literally a realm index space). Whether that color space is space efficient or not depends on realm's implementation of index spaces, not anything that Legion does.
The Legion interface for all these operations currently requires that the application always provide the color space for the new partition, so we could get rid of the "intersection" color space semantics, but if we do I want to know what the runtime should do when it iterates over the color space provided by the user and then can't find the corresponding sub-region for one of the colors in one of the input partitions.

The scalability concern applies at several levels.  Yes, individual Realm index spaces need storage and computation costs that are no larger than linear in the number of points in the set, but there are other places where super-linear costs can come in.  For example, the cross-product intersection of two O(N) partitions conceptually yields an O(N^2) color space but will have only O(N) non-empty intersection subregions.  If we create a Realm index space for every possible subregion, that's a scalability issue.  If we don't, then there are a few different ways to prune out the empty subregions, and they can be classified by whether they are the application's problem to pre-compute the sparse color space that covers just the non-empty intersections (which becomes case (b) above), or whether it's Legion's problem to take advantage of the sparsity in the intersection and present the original O(N^2) (but dense, so cheap) color space to the application.
It may make sense to table this discussion until @lightsighter and I can do a round in front of a white board.

Ok, after several more rounds of discussion, I believe the "regent interpretation" is the one we should go with.  Since the Legion operations demand that the application pre-compute the desired color space for the result of a partitioning operation (e.g. https://gitlab.com/StanfordLegion/legion/blob/master/runtime/legion.h#L3673), Legion should respect that request.  In particular, rather than ignoring requested colors that do not appear in one or the other of the two input partitions (i.e.

The zip operation is only applied
to the points contained in the intersection of the two color
spaces.

), I think we should change the semantics to not ignore colors and assume that missing colors in either of the input partitions correspond to empty subregions.  This would apply to all Legion partitioning operations that accept a color space from the application.
@elliottslaughter , @lightsighter , thoughts?

So the rule is that even empty sub-regions in the output partition are always represented? For example, in a partition difference, if there is no color for the lhs sub-region then that gets converted to empty and the resulting partition still has an explicitly empty sub-region for that point in the color space.

So the rule is that even empty sub-regions in the output partition are always represented?
Yes, subregions (empty or not) exist for exactly the colors requested by the application.


This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
Originally posted by @stale in https://github.com/z-song/laravel-admin/issues/2060#issuecomment-485735730

See https://github.com/z-song/laravel-admin/issues/2060#issuecomment-487808483


WIP merge hardcoded bits content into content items

Work in progress mechanism for extending content items with hardcoded content in collections.
This approach is being considered for a project that's currently [OFFICIAL], so I can't give full details of the use case. If you know, then you know. If you don't know, then don't worry. If you worry, you just suffer twice.
Opening this early as a draft to facilitate discussion of the general approach. There's a bit of not-very-good explanation in the READMEs I've added.

Before
After

I think that rather than hard coding the content into a rendering app, it should be put into a publishing app.
The presentation logic could be updated to pick up content from the YAML file to be passed to publishing-api. It would mean having to update the content schema, but that's something that will be required anyway.
Publishing the content via a publishing tool means that it will also be pushed to search without any extra effort.
The work in the rendering app could then be focused on rendering the extra fields from the content item as usual. This is similar to the approach we took for sign in pages. Though in that case, there wasn't an existing content item that needed to be extended.
There are things to work out, such as what the publishing workflow will be for any content changes, but it means that we'd be adding far fewer hacks to the system for content item changes that are expected to be permanent.

Sorry about the unfinished sentences in my comment yesterday 😅  - Rory was barking in my face.
I've added some screenshots which show that this approach "works" (although obviously it's not a particularly good implementation of a hero image component, even if the image itself is very good).

To be honest, I think this is all a bit premature as we don't have any designs yet and it's likely that we're going to have to render multiple linked pages.
Saying that, it's very easy to make things extremely flexible by hard-coding directly into the frontend, and then struggling later to apply that flexibility into the publishing app. We struggled with this on the COVID landing page, when we had to reduce flexibility of the sections to move it into collections publisher. I think timeline section was never moved for this reason.
I do think we should be using the content schemas to restrict how the data is presented so that we don't have to "take away" flexibility later.

Closing this as it's just us scratching ideas together.


build failure

[ npm install output.....]

><EMAIL_ADDRESS>test /home/whyrusleeping/code/orbit-core
> LOG=none ./node_modules/.bin/mocha

  Orbit with js-ipfs-api
    1) "before all" hook

  0 passing (3s)
  1 failing

  1) Orbit with js-ipfs-api "before all" hook:
     Error: non-zero exit code 1
  while running: /home/whyrusleeping/code/orbit-core/node_modules/subcomandante/subcom 22100 /home/whyrusleeping/code/orbit-core/node_modules/@haad/go-ipfs-dep/go-ipfs/ipfs daemon --enable-pubsub-experiment

  Error: serveHTTPApi: manet.Listen(/ip4/<IP_ADDRESS>/tcp/5001) failed: listen tcp4 <IP_ADDRESS>:5001: bind: address already in use

      at ChildProcess.<anonymous> (node_modules/subcomandante/fork.js:21:23)
      at maybeClose (internal/child_process.js:885:16)
      at Socket.<anonymous> (internal/child_process.js:334:11)
      at Pipe._handle.close [as _onclose] (net.js:501:12)

npm ERR! Linux 4.8.7-1-ARCH
npm ERR! argv "/usr/bin/node" "/usr/bin/npm" "run" "test"
npm ERR! node v7.1.0
npm ERR! npm  v3.10.9
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>test: `LOG=none ./node_modules/.bin/mocha`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>test script 'LOG=none ./node_modules/.bin/mocha'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the orbit_ package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     LOG=none ./node_modules/.bin/mocha
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs orbit_
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls orbit_
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /home/whyrusleeping/code/orbit-core/npm-debug.log
make: *** [Makefile:8: test] Error 1

Fixed in master


feat: editor awareness of default schemas

Video Demo/Explanation of Feature: https://youtu.be/w0z4pU_E0bM
Issue
QueryBook QueryEditor is unaware of QueryEngine Connection URI's Default/Named database/schema
Resolution (see video for walkthrough):
Add simple get_default_schema method to QueryEngine python model to parse any provided default schema on the connection URI
Make this defaultSchema known to the front-end for Linting, Analysis, and AutoCompletion
Retain support for use statements and explicit schemas when needed, and no-op to implicit "default" schema name when not provided

@jczhong84 thanks again for taking a look!

@alexgann sorry for not looking it sooner. thanks for the demo and PR! this will be quite useful. left some comments


Support one time tokens for file download

Add support for one time tokens so that the access_token is not used in the URL for file downloads

comment from Alistair:
We approached this in a different way:
– The user clicked on a link to a document. A request is sent to the API checking the user has permission to view the file.
– If the user has permissions a one time token is created, stored in the database and returned.to the client.
– The client then makes another request with the token and filename in the query string to a different end point.
– The API checks to make sure the token exists, has not expired and is for that file. The file is then downloaded. The token is then deleted.
All this was done in an Angular directive so all the user did was click on the link. This is a slightly longer process but gets round the problem of the token appearing in the URL and being logged. If someone tries to use the same token it doesn’t matter as it no longer exists.

Implemented using one time access ids


antennas binary won't run

I downloaded and extracted the linux-x64 binary (3.1.0), and edited config.yml with my settings.
antennas fails to run with:
-bash: ./antennas: cannot execute binary file: Exec format error
I tried as sudo and got:
./antennas: 1: ./antennas: Syntax error: "(" unexpected
Running on Ubuntu 18.04.2 LTS
My config.yml:
tvheadend_url: http://:@:9981
antennas_url: http://:5004
tuner_count: 6 # numbers of tuners in tvheadend
device_uuid: 2f70c0d7-90a3-4429-8275-cbeeee9cd605
TVHeadend is on the same server and running fine.
Plex is running as a warden jail plugin on FreeNAS 11.1U7 and also runs fine.
I just need an easy way to use the PlexPass DVR to get IPTV from TVHeadend.
Thanks,
Steve

I have since tried installing it via node. This works partially. Plex can now see the device in Live TV & DVR setup, and correctly lists the channels. It also loaded the EPG but this took a long time - about an hour and not all programs have thumbnails loaded. Unfortunately it won't play or record anything. When I try to play something from 'on right now' I get: Playback Error - Could not tune channel.
I'm going to put it aside for a month while away for work, then probably try upgrading Plex to 11.2.

I meant upgrade FreeNas to 11.2, then make a new VM to install the latest Plex. I cannot upgrade Plex at the moment because it is running as a FreeNas plug-in. Also VMs are broken in the current FreeNas version I'm running (11.1U7).

Hm, that's interesting, it's possible the Linux binaries are bork.
I've got an Elementary OS install I could try to give it a whirl on (or inside a VM)
Alternatively, you might have more luck with another project that takes Tvheadend out of the equation entirely, https://github.com/tellytv/telly


Serializer breaks the right part of URLs containing multiple ? characters

Context
What's your version of nuqs?
2.2.3

What framework are you using?

✅ Next.js (app router)
❌ Next.js (pages router)
❌ React SPA (no router)
❌ Remix
❌ React Router
❌ Other (please specify)

Which version of your framework are you using?

Relevant Packages:
  next: 14.2.20
  react: 18.3.1
  typescript: 5.5.4

Description
When a serialize is executed with a string base parameter that have query string parameter with a string with "?", some of the string disappear

Reproduction
https://stackblitz.com/edit/vitejs-vite-wcnxdakb?file=src%2Fmain.ts
In this example you can see that we call serialize with a base parameter that have a query string with value1 that have a ? in the middle, and the result is http://localhost:3000/?value1=value&prompt=value3 that deletes part of value1 and value 2.
import { parseAsString, createSerializer } from 'nuqs';

const serialize = createSerializer({
  prompt: parseAsString,
});

const url = serialize('http://localhost:3000/?value1=value?With&value2=35435', {
  prompt: 'value3',
});

Reading you code I found the issue in line https://github.com/47ng/nuqs/blob/54067b39e91c42f30f45f38cb6b410960032d78b/packages/nuqs/src/serializer.ts#L84, where base is split but only use the first two elments of the returning array and get rid of the remaining

Thanks for the report, sounds like it should be an easy fix, would you like to open a PR?


Blacklisting does not actually prevent module from being loaded

When you blacklist a module it won't actually prevent you from doing a modprobe . It was certainly my expectation that the blacklisting would not only prevent other modules from loading the blacklisted module but also prevent people from loading the module from the command line.
My testing has been done on Red Hat Enterprise 7.1
To achieve the expected behaviour you need the following entry:
install  /bin/false
in the blacklist.conf file.
I am happy to submit a pull request but wanted to discuss the existing behaviour first.

@gjngeldenhuis if this is true on every distro then +1 for a PR

@mcanevet I don't know about other distros unfortunately.

@mcanevet so a bit of an update. I have read up some more and my understanding of what blacklisting does were wrong. Blacklisting will block specified kernel module aliases. Quoting from the Red Hat man page:
"the blacklist keyword indicates that all of that particular module's internal aliases are to be ignored."
The correct way to stop a module from loading is to do the following:
kmod::install { 'dccp': command => '/bin/false', }
So no bug, just a misunderstanding. The only small issue is that the file that gets created have an unnessarry newline entry at the top. I will submit a PR for the documentation to clarify usage and prevent others from making the same assumption as I did.

@gjngeldenhuis great! thanks!

https://github.com/camptocamp/puppet-kmod/pull/32

merged


Update regula version to 2.5.0

Description
Update the regula version.
Safety checklist

[ ] This change is backwards compatible and safe to apply by existing users
[ ] This change will NOT lead to data loss
[ ] This change will NOT lead to downtime who already has an env/service setup

How has this change been tested, beside unit tests?
Tested Regula manually on the System.

The new version causes some tests to fail; you may want to (1) either fix the RC on the failed tests - but this means testing that the change in TF is backward compatible and doesnt cause infra to be torn down and spun up OR (2) Add more exceptions to the config/config.rego.

@bigbitbus for this, I am not sure which one would be the better way to go forward with, it depends upon the errors and how we want to get through these. Shall we have a discussion this monday, to figure out which ones do we need to waive and which ones do we need to add and make those backward compatible.

@nsarupr you may close this since regula has been removed from the repo and github.

Superseded by #837


react-step-2

react-step-2 pull request.  Thanks!

I made the fix and re-pushed.  Would you please check?
Thanks!

👍 Looks good - 🎉 You are one step closer to completion of the Carbon React tutorial and earning the digital credential.


Apostrophe in Email Address Throws Error

Apostrophes are valid in email addresses before the @ symbol (ie, mikeo'flanahan@someplaceirish.com). However, Postal is consistently throwing the following error when sending to such an email:
An invalid character was found in the mail header: ';'.

Try escaping the email address. For example, @Html.Raw(Model.ToEmailAddress)

Thanks. That does, in fact, work.

I'm not sure this should be closed. This seems like a workaround for a bug still present.


Add file header to editorconfig

Not sure if this is supported widely - but at least Roslyn seems to support it.

 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.


Update issue templates

@ethanaobrien Does this issue template look good? If so please approve and merge.

Looks good!


is the start function needed?

Currently the start function defines the inital services a plugin provides. This same functionality can be duplicated within the consume function. The question I ask is does it make sense to have the start function when we could reduce plugins to be a single saga function (and an optional installation plugin). I'm leaning towards yes since it would reduce complexity however the original thought behind the start function was that there were things that needed to be do in order for a plugin to be considered "on" and the existence of the "requires" property to depend on specific plugin handles waiting for required plugins to start before starting itself.  (if requires is needed we would either keep start or add a "start" callback to call when the plugin is considered started)

i decided to get rid of it, not enough compelling reasons to keep the additional complexity. Also removed the "requires" option so plugins can't require other plugins. this kind of functionality should be achieved with consuming things.


docs(ngMessages): ng-messages-include defined on ng-messages element

I was having issues implementing ngMessages together with ngMessagesInclude where the template I was trying to include simply wouldn't show up. This was until I realized that, contrary to how the docs describe it, Angular expected ng-messages-include to be defined on the same element as ng-messages (which I later discovered was how eg. this tutorial described the implementation of ngMessages).

Signed

You are obviously seeing the docs for the latest build of Angular (master), while using an older (stable) version. ngMessages has been recently revamped (landed in v1.4.0-beta.5), so the latest docs are reflecting the latest changes.
For more info, take a look at this breaking change notice in the changelog.

Ah, thanks. My bad!


Update whisper.cpp

This will make it possible to add an OpenBLAS feature. The only new commit is https://github.com/ggerganov/whisper.cpp/commit/80bf931668d00e0bf88b7bd4e1ea996c2ec71915
EDIT: let's also wait for https://github.com/ggerganov/whisper.cpp/pull/1350

This is ready to be merged now.


[Reboot] State Manager state Reset

Add Power policy check as part of the valid state..
VALID_STATES = {
    'reboot':
    {
         # (Power Policy, BMC state, Chassis State, Host State)
         ('LEAVE_OFF','Ready','Off','Off'),
         ('ALWAYS_POWER_ON','Ready','On','Running'),
         ('RESTORE_LAST_STATE','Ready','On','Running'),
         ('RESTORE_LAST_STATE','Ready','Off','Off'),
    },
}

Code pushed for review


Improved pointer handling on UWP and Hover events

Improved UWP Pointer handling
By correctly handling the PointerEntered and PointerExited events and capturing the pointer (Container.CapturePointer(e.Pointer);) in TouchEffect.UWP/PlatformTouchEff.cs, Touch effects created with this library will now behave in the exact same way as native UWP ones, which is especially noticeable when pressing the pointer down, and dragging the cursor outside the bounds of the control.
Hover events for UWP and Android
As a "side-effect" of the above feature, I have started adding basic support for hover events. This is manifested as the new TouchStatus.HoverEnter and TouchStatus.HoverLeave statuses, which can be handled by users through the StatusChanged event. This feature can be previewed with the new HoverSample in the sample project.
Currently, HoverEnter and HoverLeave are only triggered on UWP and Android (although I have not tested it on Android, as I do not own a mouse to test it with).
I have looked into adding support for Xamarin.Mac, however as far as I can tell this would require creating a subclass of NSView in order to override the MouseEntered and MouseExited events. Not being familiar with Cocoa, I don't really know how to approach this issue.

Hi) This is a very cool enhancement! But I believe it should be achieved in a little bit different way.

separate enum HoverStatus { Started, Ended }
I don't think that Container_PointerExited or Container_PointerEnetered should raise TOUCH Canceled or Started events, but they will raise HoverStatus updates

Also, I think it would make sense to add HandleHover() method with own transformations stuff (usually it's a shadow/color changing)
Look how it works on mac on the web (Github). I like this behavior.

[Regular state] if no hover and no press
[Hovered state] if there is hover, but no press
[Pressed state] if there is hover and there is press (if the user ends press, we should call command)
[Pressed state] if no hover, BUT there is press ((if the user ends press, we should NOT call command. Canceled status)

So if the user pressed a button and shifted mouse outside of button boundaries we still show the button as pressed, but DON'T raise "Success" once he/she ends press
Here is the GIF.

For UWP, I implemented the behaviour which is seen in native UWP apps:

It's the same as what you described, except that moving the pointer outside of the control causes it to lose it's pressed state, however it regains this state if you move the cursor back in while the pointer is pressed. Does this not make it clearer that releasing the pointer outside the bounds of the control does NOT trigger the command?
I don't know what the native behavior is on other platforms though, maybe it would be best to imitate the native behavior of each platform?

For UWP, I implemented the behavior which is seen in native UWP apps:

It's the same as what you described, except that moving the pointer outside of the control causes it to lose it's pressed state, however it regains this state if you move the cursor back in while the pointer is pressed. Does this not make it clearer that releasing the pointer outside the bounds of the control does NOT trigger the command?
I don't know what the native behavior is on other platforms though, maybe it would be best to imitate the native behavior of each platform?

Totally agree)
Just checked on native mac app (now browser)

[Regular state] if no hover and no press
[Hovered state] if there is hover, but no press
[Pressed state] if there is hover and there is press (if the user ends press, we should call command)
[Regular state] if no hover, BUT there is press ((if the user ends press, we should NOT call command. Canceled status)

but anyway we should keep hower and touch separately
Do you agree?

but anyway we should keep hower and touch separately
Do you agree?

Yep that sounds good. If you want I can work on separating the hover and touch logic, however I can't work on adding the hover functionality to Mac, seeing as I'm not at all familiar with developing for it, and don't even own a macOS machine.

@piersdeseilligny let's split work for several steps
1st step:

Create new enum HoverStatus { Exited, Entered }
Handle UWP events OnPointerExited and OnPointerEntered with passing corresponding statuses.

Then decide what's next :)
And please don't update XF and all other files. Only necessary things should be included

btw, i am not sure if it makes sense to keep _pressed in UWP. It was added by a contributor. But is it needed indeed?

Yeah _pressed is necessary to get the right behaviour when pressing the pointer, and exiting and re-entering the control. Although if all the inrange checks are moved to the crossplatform code, then it might not be necessary. However are we sure this is the best thing to do? Because all platforms handle the hover events slightly different (for example windows fires them even when using a touchscreen, while android doesn't) so I think it might be easier to get consistent behaviour by implementing hover individually for each platform? Although I'm not entirely sure yet; i'il look at it more closely later today.

Yeah _pressed is necessary to get the right behaviour when pressing the pointer, and exiting and re-entering the control. Although if all the inrange checks are moved to the crossplatform code, then it might not be necessary. However are we sure this is the best thing to do? Because all platforms handle the hover events slightly different (for example windows fires them even when using a touchscreen, while android doesn't) so I think it might be easier to get consistent behaviour by implementing hover individually for each platform? Although I'm not entirely sure yet; i'il look at it more closely later today.

I think we can start with crossplatform solution, then provide possibility to override it per platform if needed. For now, I like UWP approach

So I've looked over it some more, and I think it makes sense to have the _inrange stuff on the platform-specific code. I think the code will be less messy, because some platforms (ios) don't even trigger hover, so there would be no way of knowing if _inrange is true, unless we add platform conditional code in the crossplatform part, at which point might as well make it platform-specific to start with.
Additionally, I think the codebase is more readable if the HandleTouch TouchStatus events triggered in the platform specific code are the same ones which make it to the TouchStatusChanged event on the other side.
If we adhere to the following guidelines, we should get consistent behavior on each platform:

Pressing the pointer down and dragging it outside the bounds of the control triggers TouchStatus.Canceled

If it is then released outside the bounds of the control, nothing happens
If it reenters the bounds of the control, before being released, TouchStatus.Started is triggered again.

If the platform supports it, HoverStatus is set to Hovering every time a pointer enters the bounds of the control. However it must always occur **before(( any change to TouchStatus triggered by the same pointer action.

If the platform supports it, HoverStatus is set to Regular every time a pointer exits the bounds of the control. However it must always occur after any change to TouchStatus triggered by the same pointer action.

So I've added everything HoverState and HoverStatus related. Additionally, I have added an _inrange check to Android in order to get the same behavior as on UWP when dragging the pointer outside the bounds of the control while it is pressed.
What I'm guessing the next steps are is:

UWP

Nothing, it's pretty much finished

Android

Test hover with a mouse connected to an android device

iOS

Nothing, it doesn't support hovering. Maybe an _inrange check would be useful though? I don't have a mac, so can't really develop anything for iOS though.

Mac

Figure out how to use NSTrackingArea without creating a subclass of NSView (if possible?). Then, implement all hover-related events.
Like for iOS, I can't really work on this, because I don't actually own a mac :/

Cross-Platform

Add HoverBackgroundColor, HoverScale, HoverRotation, HoverRotationX, HoverRotationY, HoverTranslationX, HoverTranslationY, HoverAnimationDuration

@piersdeseilligny can you please clean the PR?
I mean remove all files which are not related to this PR
(Resource files, packages updates etc.)

@piersdeseilligny overall: good job! 👍

I've reverted my initial Update Xamarin.Forms commit, but now I have all the build errors which caused me to update everything in the first place. Although I'm not sure if it's just issues with my specific environment or not.
Would you mind handling this on your side of things?

Do we really need both (HoverStatus and HoverState)? They both have 2 "states"

I added both of them in for consistency with the TouchStatus and TouchState, one of them being a state, while the other is more of an event.
In a multi-touch situation, HoverStatus is updated any time a pointer enters the bounds of the control, while HoverState is only updated if the overall "hover state" changes. Also, HoverStatus might be useful in the future, if we want to add additional hover-related events?

So I'm pretty sure everything is fine now: Only the relevant files are included in this pull request, it builds fine, and seems free of bugs.

well done
I will take a look at the weekend

Hover states on UWP will be very useful, thanks for your efforts!


Support override vcpkg installed dir

This might be more generic than #42 . As manifest mode is a sub case of overriding an existing vcpkg installed directory, So this pr is made.

Thank you for your submission! This looks reasonable to me in isolation, although I don't feel like I understand the broader context of when this mode is useful. (Which is just a matter of my ignorance, not a problem with the PR.)

In a hybrid C++/Rust project, one can use a vcpkg.json manifest file to manage vcpkg installations. The path for vcpkg installations can be altered to any desired location using CMake. 'vcpkg-rs' thus can be a bridge between the existing vcpkg installation and Rust, even in manifest mode.

@waych Any feedback on this one?

Hmm. My initial thought is that this is fine and can go in as is.
Thinking longer term, I'm not particularly fond of the way this extends the interface on the Config directly, because this can cause problems with dependent crates that also want to pull in vcpkg dependencies.  This makes me think that perhaps we should be propagating this information down through the crate dependencies somehow so that dependent crates' vcpkg dependencies can be fulfilled by a single top level vcpkg.json manifest. Given that there isn't any good way for the top-level crate to pass this information to dependent crates (?) from build.rs, it seems perhaps it would be clearer, although maybe more inconvenient to be passing this installed path via an environment variable where all vcpkg-rs::Config's in the build tree see the same thing.
That said I think this is okay for now given the problem already exists in the config interface with the existing Config::vcpkg_root() setter that has the exact same problem. Any library crate using these will have similar challenges. This kiss approach is good enough for now and can easily be further extended with an environment variable when the time comes.

Hmmm. I can see how environment variables can address the issue you raise, although I guess it's less "passing information from the top-level crate to dependents" and more "user specifies global-level information".
Maybe there would be some way to specify this configuration, or where to find this configuration, in the Cargo.toml metadata? I am wondering if build.rs scripts can use some magic to get information about the toplevel Cargo.toml in their workspace, not just their local file.

Thank you for those good reviews. I think perhaps adding environment variable like VCPKG_INSTALLED_ROOT should workaround the problem. As we already have VCPKG_ROOT playing the same role.  One thing need to concern is that VCPKG_ROOT is also a env used by vcpkg itself, while VCPKG_INSTALLED_ROOT is a env defined by ourselves.
Considering a real example, ffmpeg-next-sys tries to using vcpkg in their build.rs.
Defining environment VCPKG_INSTALLED_ROOT from the top crate's config.toml is the only way to pass vcpkg installed location information to ffmpeg-next-sys. So I guess it is valuable to extend environment variable at this moment.

OK; more to do on this topic, but I've merged this piece.


The process has been signaled with signal "11". for Dates filters

<?php

namespace Modules\Branches\Scopes;

use Baro\PipelineQueryCollection\BooleanFilter;
use Baro\PipelineQueryCollection\DateFromFilter;
use Baro\PipelineQueryCollection\DateToFilter;
use Baro\PipelineQueryCollection\RelationFilter;
use Baro\PipelineQueryCollection\ScopeFilter;
use Illuminate\Database\Eloquent\Builder;
use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\Scope;

class BranchScope implements Scope
{
    /**
     * Apply the scope to a given Eloquent query builder.
     *
     * @param \Illuminate\Database\Eloquent\Builder $builder
     * @param \Illuminate\Database\Eloquent\Model $model
     * @return void
     */
    public function apply(Builder $builder, Model $model)
    {
        $builder->filter([
            new ScopeFilter('search'),
            new RelationFilter('manager', 'id'),
            new BooleanFilter('status'),
            new DateFromFilter('created_at'),
            new DateToFilter('created_at'),
        ]);
    }

}

an error
Symfony\Component\Process\Exception\ProcessSignaledException
The process has been signaled with signal "11".
at vendor/symfony/process/Process.php:434
430▕             usleep(1000);
431▕         }
432▕
433▕         if ($this->processInformation['signaled'] && $this->processInformation['termsig'] !== $this->latestSignal) {
➜ 434▕             throw new ProcessSignaledException($this);
435▕         }
436▕
437▕         return $this->exitcode;
438▕     }
  +15 vendor frames

16  artisan:37

Can you provide more context when this error occurred, like request query or the db schema

<?php
# this is the test
namespace Modules\Branches\Tests\Unit;

use Tests\TenantTestCase;
use Illuminate\Foundation\Testing\WithFaker;
use Illuminate\Support\Facades\DB;
use Modules\Branches\Entities\Branch;

class BranchTest extends TenantTestCase
{
    public function test_to_retrieve_branches_filter_with_created_at_date() : void
    {
            $user = User::factory()->create();
            $user->givePermissionTo('list-branches');
            Branch::factory(3)->create();
            $branch = Branch::factory()->create([
                 'created_at' => '2019-01-01',
            ]);
            Branch::factory(3)->create();
            $url = $this->url . '/api/v1/admin/branches';
            $response = $this->actingAs($user)->get($url . '?created_at_from=2018-01-01&created_at_to=2020-01-01');
            dd($response->json());
    }
}

# my Branch Model

namespace Modules\Branches\Entities;

use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\Factories\HasFactory;
use Astrotomic\Translatable\Contracts\Translatable as TranslatableContract;
use Astrotomic\Translatable\Translatable;
use Baro\PipelineQueryCollection\Concerns\Filterable;
use Baro\PipelineQueryCollection\Contracts\CanFilterContract;
use Modules\Branches\Scopes\BranchScope;

class Branch extends Model implements TranslatableContract, CanFilterContract
{
    use HasFactory, Translatable, Filterable;
    protected static function booted()
    {
        static::addGlobalScope(new BranchScope);
    }
}

there seems to be infinite loop with global scope and local scope, investigating

I checked a lot about this problem and did not find a solution. Do you have any updates for this? the issue came with DateFromFilter only

@karimalihussein sr I have quite rough week, will investigate more in weekend.


Error processing append operation on partition

I have a load of ~10k messages/sec. As the load increases, I see a burst of following error in Kafka (before everything works fine again):
Error: ERROR kafka.server.ReplicaManager: [Replica Manager on Broker 22]: Error processing append operation on partition _topic_name_-_partition_
Pastebin: http://pastebin.com/QbkwHJ3W
Strangely, when I retry the same message again it works. I'm using Snappy Compression while publishing.
Does this issue sound familiar to anyone?

This doesn't feel like a sarama issue, but more likely a Kafka one. I'd recommend asking on the user's mailing list: http://kafka.apache.org/contact.html

Yes, AFAICT this is an upstream issue.


Uncaught Error: Cannot find module './repl'

Trying to install atom for the first time to use the atom-supercollider package on windows 10 I get this error when I try to install the package. Don't know how to proceed, is this a bug or something I do wrong?
Uncaught Error: Cannot find module './repl'
module.js:341
Error: Cannot find module './repl'
at Module._resolveFilename (module.js:339:15)
at Function.Module._resolveFilename (C:\Users\ahorse\AppData\Local\atom\app-1.10.1\resources\app.asar\src\module-cache.js:383:52)
at Function.Module._load (module.js:290:25)
at Module.require (module.js:367:17)
at require (C:\Users\ahorse\AppData\Local\atom\app-1.10.1\resources\app.asar\src\native-compile-cache.js:50:27)
at Object. (C:\Users\ahorse.atom\packages\supercollider\lib\controller.coffee:7:10)
at Object. (C:\Users\ahorse.atom\packages\supercollider\lib\controller.coffee:449:4)
at Module._compile (C:\Users\ahorse\AppData\Local\atom\app-1.10.1\resources\app.asar\src\native-compile-cache.js:103:30)
at Object.defineProperty.value [as .coffee] (C:\Users\ahorse\AppData\Local\atom\app-1.10.1\resources\app.asar\src\compile-cache.js:208:21)
at Module.load (module.js:357:32)
at Function.Module._load (module.js:314:12)
at Module.require (module.js:367:17)
at require (C:\Users\ahorse\AppData\Local\atom\app-1.10.1\resources\app.asar\src\native-compile-cache.js:50:27)
at Object. (C:\Users\ahorse.atom\packages\supercollider\lib\supercollider.coffee:4:16)
at Object. (C:\Users\ahorse.atom\packages\supercollider\lib\supercollider.coffee:53:4)
at Module._compile (C:\Users\ahorse\AppData\Local\atom\app-1.10.1\resources\app.asar\src\native-compile-cache.js:103:30)
at Object.defineProperty.value [as .coffee] (C:\Users\ahorse\AppData\Local\atom\app-1.10.1\resources\app.asar\src\compile-cache.js:208:21)
at Module.load (module.js:357:32)
at Function.Module._load (module.js:314:12)
at Module.require (module.js:367:17)
at require (C:\Users\ahorse\AppData\Local\atom\app-1.10.1\resources\app.asar\src\native-compile-cache.js:50:27)
at Package.module.exports.Package.requireMainModule (C:\Users\ahorse\AppData\Local\atom\app-1.10.1\resources\app.asar\src\package.js:718:27)
at Package.module.exports.Package.activateConfig (C:\Users\ahorse\AppData\Local\atom\app-1.10.1\resources\app.asar\src\package.js:225:12)
at PackageManager.module.exports.PackageManager.packageHasSettings (C:\Users\ahorse\AppData\Local\atom\app-1.10.1\resources\app.asar\node_modules\settings-view\lib\package-manager.js:55:14)
at PackageCard.module.exports.PackageCard.hasSettings (C:\Users\ahorse\AppData\Local\atom\app-1.10.1\resources\app.asar\node_modules\settings-view\lib\package-card.js:699:34)
at PackageCard.module.exports.PackageCard.updateSettingsState (C:\Users\ahorse\AppData\Local\atom\app-1.10.1\resources\app.asar\node_modules\settings-view\lib\package-card.js:397:16)
at PackageCard.module.exports.PackageCard.updateInterfaceState (C:\Users\ahorse\AppData\Local\atom\app-1.10.1\resources\app.asar\node_modules\settings-view\lib\package-card.js:390:12)
at C:\Users\ahorse\AppData\Local\atom\app-1.10.1\resources\app.asar\node_modules\settings-view\lib\package-card.js:643:24
at C:\Users\ahorse\AppData\Local\atom\app-1.10.1\resources\app.asar\node_modules\settings-view\lib\package-card.js:712:20
at Function.module.exports.Emitter.simpleDispatch (C:\Users\ahorse\AppData\Local\atom\app-1.10.1\resources\app.asar\node_modules\event-kit\lib\emitter.js:25:14)
at Emitter.module.exports.Emitter.emit (C:\Users\ahorse\AppData\Local\atom\app-1.10.1\resources\app.asar\node_modules\event-kit\lib\emitter.js:125:28)
at PackageManager.module.exports.PackageManager.emitPackageEvent (C:\Users\ahorse\AppData\Local\atom\app-1.10.1\resources\app.asar\node_modules\settings-view\lib\package-manager.js:680:27)
at C:\Users\ahorse\AppData\Local\atom\app-1.10.1\resources\app.asar\node_modules\settings-view\lib\package-manager.js:468:26
at exit (C:\Users\ahorse\AppData\Local\atom\app-1.10.1\resources\app.asar\node_modules\settings-view\lib\package-manager.js:73:16)
at triggerExitCallback (C:\Users\ahorse\AppData\Local\atom\app-1.10.1\resources\app.asar\src\buffered-process.js:215:47)
at ChildProcess. (C:\Users\ahorse\AppData\Local\atom\app-1.10.1\resources\app.asar\src\buffered-process.js:237:18)
at emitTwo (events.js:100:13)
at ChildProcess.emit (events.js:185:7)
at Process.ChildProcess._handle.onexit (internal/child_proce

Restarted atom and it seems to work anyways...

That's strange. Maybe atom failed to compile all of the coffee files during the installation and didn't properly load it.
Please let me know of any issues you encounter, windows related or otherwise.

It appears to work quite well. Occasionally I have to restart Atom but I never get this particular error.
FYI I had to change default locations of sclang.exe and sclang_config.yaml in config in order to get things running here on Windows 10:
The defaults appears to be (I got this from a "Executable not found" error prior to changing locations):
sclang:	C:\Program Files (x86)\SuperCollider\sclang.exe
sclang_conf:	C:\Program Files (x86)\SuperCollider\sclang_conf.yaml
I run Atom with SuperCollider 3.7.2 successfully with these settings:
sclang:	C:\Program Files\SuperCollider-3.7.2\sclang.exe
sclang_conf:	C:\Users\ahorse\AppData\Local\SuperCollider\sclang_conf.yaml
One thing I really miss from SCIDE is Ctrl-Enter "Evaluate Selection, Line or Region". Have you considered introducing such a command? Or should I rely on common Atom functionality for this? I'm totally new to Atom. I find myself spending a lot of keystrokes selecting stuff compared to SCIDE where a Ctrl-Enter typically is enough to evaluate code sections inside parenthesis.
Anyways, the work on this is much appreciated. At last I have SuperCollider editor support in Windows 10 for an editor with a nice dark background and - most importantly - with ok vim bindings (vim-mode). I'm a vim guy.

Yes, there is an issue for "evaluate inside paragraphs".  I want that too, just haven't gotten around to it.

If you use vim-mode you might like this tip<EMAIL_ADDRESS>I'm still trying to get a good navigate-by-folds plugin. There is one, but not quite suited to me.


return contact even in case the epa algorithm gets stuck.

Fixes https://github.com/dimforge/parry/issues/157
Still investigating if it doesn't break anything else.

Will come back if I see it working, still investigating. Seems to work, but am not 100% sure.


HTML lint is reporting weird errors

If you know what this is or how to fix it, please let us know!
I couldn't find bannedAttrs or for
[htmllint error in paste-templates/admin-created-account-welcome.html ] TypeError: bannedAttrs.forEach is not a function
[htmllint error in paste-templates/base.html ] TypeError: bannedAttrs.forEach is not a function
[htmllint error in paste-templates/digest.html ] TypeError: bannedAttrs.forEach is not a function
[htmllint error in paste-templates/email-confirmation.html ] TypeError: bannedAttrs.forEach is not a function
[htmllint error in paste-templates/password-reset_design_editor.html ] TypeError: bannedAttrs.forEach is not a function
[htmllint error in paste-templates/password-reset.html ] TypeError: bannedAttrs.forEach is not a function
[htmllint error in paste-templates/receipt.html ] TypeError: bannedAttrs.forEach is not a function
[htmllint error in paste-templates/welcome.html ] TypeError: bannedAttrs.forEach is not a function

bannedAttrs.forEach is a function in htmllint's attr-bans.js ... not sure why it's throwing an error right now though...

Okay, I think I've figured this out.
Consulting the htmllint Options page, it seems like "attr-bans" isn't one of the options that accepts a format specifier (such as "none"). What do you intend by setting "attr-bans": "none"? Do you want no attributes at all to be banned (i.e. all attributes are allowed)? If that's the case, just set "attr-bans": false.


No Renderer for file type application/vnd.openxmlformats-officedocument.wordprocessingml.document

Hello!
I am getting this message when I try to use this library:

This is my code:
Am I doing anything wrong? Thanks in advance for your answer!
import React from 'react'
import DocViewer from 'react-doc-viewer'
function Viewer() {
// const path = './Sample1.docx'
const docs = [
// { uri: path }, // Local File
{ uri: require('./Interview.docx') }, // Local File
]
return (

)
}
export default Viewer

Apologies for the late reply.
Even though I created this repo, I didn't get notifications. I'm not watching it.
You must make sure you supply <DocViewer> with pluginRenderers.
https://github.com/Alcumus/react-doc-viewer#included-renderers
import DocViewer, { DocViewerRenderers } from "react-doc-viewer";

<DocViewer
  pluginRenderers={DocViewerRenderers}
  {/* ... */}
/>;

DocViewerRenderers includes the MSDocRenderer.
Or if you don't need the other renderers you can use it like this...
import DocViewer, { MSDocRenderer } from "react-doc-viewer";

<DocViewer
  pluginRenderers={[ MSDocRenderer ]}
  {/* ... */}
/>;

@Gusyavo If you are still having issues, respond here and I will gladly help out.

@Gusyavo If you are still having issues, respond here and I will gladly help out.

Hey Matt!
Thank you very much for taking the time to answer, I really apreciate it.
I did as you adviced and added the pluginRenderers, this is the code:
the url is a url to an s3 bucket that contains a .docx file. I verified it and it works.

But I get this error:

I believe when fetching the file extension aws returns xml instead of the actual filetype extension.
Do you know the fileType extension in advance? If so could you provide it to the docs array object?
const { url, extension } = props
const docs = [{uri: url, fileType: extension}]

You were right!!! It now looks like this:

It renders the doc, but it shows the urlson top of it too. Am I doing anything wrong?
And, to adjust the height, I guess I have to create a custom renderer and add use css in it, right?

1. The Header
Is separate from the custom renderers for file types.
You have a few options for the header, I will add more in the future.
https://github.com/Alcumus/react-doc-viewer#config
a) disableHeader: true - Then Create your own header/file name display wherever you want. But this will remove file navigation if you plan on using multiple files.
b) disableFileName: true - Will keep the header, and the multi-file navigation, but hide the file name.
c) https://github.com/Alcumus/react-doc-viewer#overriding-header-component - You can override the whole header with whatever you want, as long as it is a valid React Component.
2. The Height
Without seeing your code exactly I can only guess. But I'm almost certain you just need to add a height to the main component.
So you can try one of the following from https://github.com/Alcumus/react-doc-viewer#styling -
a) Adding a className to the <DocViewer> component like you would any other React component then having a css class with height as a property in a css file.
b) In a css file override the main component id styles with #react-doc-viewer { height: 900px }
c) Inline style - style={{width: 500, height: 500}}
d) Styled Component

@Gusyavo  I would also recommend hiding the access key etc. in your screenshot just in-case.

1. The Header
EDIT
If you update to 0.1.1 - the header will be fixed for you.
~Is separate from the custom renderers for file types. You have a few options for the header, I will add more in the future. https://github.com/Alcumus/react-doc-viewer#config a) disableHeader: true - Then Create your own header/file name display wherever you want. But this will remove file navigation if you plan on using multiple files. b) disableFileName: true - Will keep the header, and the multi-file navigation, but hide the file name. c) https://github.com/Alcumus/react-doc-viewer#overriding-header-component - You can override the whole header with whatever you want, as long as it is a valid React Component.~
2. The Height
Without seeing your code exactly I can only guess. But I'm almost certain you just need to add a height to the main component.
So you can try one of the following from https://github.com/Alcumus/react-doc-viewer#styling -
a) Adding a className to the <DocViewer> component like you would any other React component then having a css class with height as a property in a css file.
b) In a css file override the main component id styles with #react-doc-viewer { height: 900px }
c) Inline style - style={{width: 500, height: 500}}
d) Styled Component

Matt, I can't thank you enough for your help! It was really kind of you telling me what to do. And this library works beautifully!!!

One last thing: everything works perfectly but I am getting these warnings, I do not know if they are normal:

One last thing: everything works perfectly but I am getting these warnings, I do not know if they are normal:

This is an old but common issue with JQuery. I guess you are using JQuery? Or a library that is using jquery.
This is outside of the scope of react-doc-viewer, but I have read that this can solve your issue
jQuery.event.special.touchstart = {
  setup: function( _, ns, handle ){
    if ( ns.includes("noPreventDefault") ) {
      this.addEventListener("touchstart", handle, { passive: false });
    } else {
      this.addEventListener("touchstart", handle, { passive: true });
    }
  }
};
jQuery.event.special.touchmove = {
  setup: function( _, ns, handle ){
    if ( ns.includes("noPreventDefault") ) {
      this.addEventListener("touchmove", handle, { passive: false });
    } else {
      this.addEventListener("touchmove", handle, { passive: true });
    }
  }
};

Although I'm not sure where that code needs to go.
You may need to clone more of those blocks of code to get rid of mousewheel warnings too, and any others that are still happening.


Add bookmark options to load username/password from env vars

We keep our configuration for pgweb in a private Git repository. Currently we still have to input the username and password every time, but it would be nice to refer to an environment variable in the bookmarks. That'll allow us to have the 'click-and-go' functionality without the security implications of keeping passwords stored in the repository itself. Using SSH keys is unfortunately not an option.

Is https://github.com/AGWA/git-crypt not an option? You can securely store the encrypted data right in your repo

Thanks for the quick reply. git-crypt isn't suitable for our situation I'm afraid; we'd like to use existing Kubernetes secrets to specify username/password. I opened this PR in case this might be of value to you or other users of pgweb, but I also understand this is a very niche requirement 😅

Thank you!

Likewise, and thanks for the great tool!

Would you mind dropping me an email with some info on how you run pgweb in your org? This helps me plan for new features/improve things etc.


Adding Travis CI tests for Go version 1.6

Coverage remained the same at 41.121% when pulling 579d4e54195882ba90fa1ef8e41a171612a55e44 on 1.6 into 11c9c4620901fc6d826fbe7a5f3bd8e0871c6f21 on master.

\o/


Develop and Run SAP Fiori Application With SAP Business Application Studio

Tutorials: https://developers.sap.com/tutorials/abap-environment-deploy-cf-production.html
Having an issue with Step 8: Create IAM App and business catalog
When creating the IAM app I cannot see the Service Type of 'OData IWSG' - this does not show in my list. I can only see the following.

Is there a setting I need to make to allow this option to appear?
thanks
Phil Cooley

Hi @pjcools ,
please try out OData V2, there was a name change. I'll update this.
Thank you.
Best regards,
Merve

This did not work

But I chose the deprecated item and then it worked. If you can please update this tutorial to show the right steps that would be great.
This screenshot shows it has worked when I selected the deprecated entry.

Kind Regards
Phil Cooley

Hi @pjcools ,
in my last message I mentioned deprecated in brackets. I also ment the deprecated version of odata v2. I'll update this as soon as possible.
Best regards,
Merve


[HUDI-1478] Introduce HoodieBloomIndex to hudi-java-client

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contributing.html before opening a pull request.

What is the purpose of the pull request
(For example: This pull request adds quick-start document.)
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Codecov Report

Merging #2608 (302400e) into master (06dc7c7) will increase coverage by 18.22%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master    #2608       +/-   ##
=============================================
+ Coverage     51.22%   69.44%   +18.22%
+ Complexity     3230      363     -2867
=============================================
  Files           438       53      -385
  Lines         20093     1944    -18149
  Branches       2069      235     -1834
=============================================
- Hits          10292     1350     -8942
+ Misses         8954      460     -8494
+ Partials        847      134      -713

Flag
Coverage Δ
Complexity Δ

hudicli
?
?

hudiclient
?
?

hudicommon
?
?

hudiflink
?
?

hudihadoopmr
?
?

hudisparkdatasource
?
?

hudisync
?
?

huditimelineservice
?
?

hudiutilities
69.44% <ø> (-0.08%)
0.00 <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ
Complexity Δ

...hudi/utilities/sources/helpers/KafkaOffsetGen.java
85.84% <0.00%> (-2.94%)
20.00% <0.00%> (+4.00%)
:arrow_down:

...in/java/org/apache/hudi/utilities/UtilHelpers.java
64.53% <0.00%> (-1.17%)
33.00% <0.00%> (+1.00%)
:arrow_down:

...rg/apache/hudi/metadata/HoodieMetadataPayload.java

...apache/hudi/common/util/ParquetReaderIterator.java

...a/org/apache/hudi/common/util/ClusteringUtils.java

...sioning/clean/CleanMetadataV2MigrationHandler.java

...tSpotMemoryLayoutSpecification64bitCompressed.java

.../org/apache/hudi/operator/StreamWriteOperator.java

...che/hudi/common/util/BufferedRandomAccessFile.java

...e/hudi/common/table/log/HoodieLogFormatReader.java

... and 378 more


Check size is not greater than 8 for RV32A and RV63A

Description

Tests

Additional context

Metadata

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

Project coverage is 61.89%. Comparing base (8ff9318) to head (36b7895).

Files with missing lines
Patch %
Lines

rvgo/fast/vm.go
0.00%
0 Missing and 1 partial :warning:

rvgo/slow/vm.go
0.00%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #103   +/-   ##
=======================================
  Coverage   61.89%   61.89%
=======================================
  Files          27       27
  Lines        4091     4091
=======================================
  Hits         2532     2532
  Misses       1427     1427
  Partials      132      132

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


@conda-forge-admin, please add bot automerge

@conda-forge-admin, please add bot automerge

@conda-forge-admin, please add bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/pytest-bandit-feedstock#3.


metadata: Add provider for compilers

Signed-off-by: Kemal Akkoyun<EMAIL_ADDRESS>

This is cool. LGTM :100:

Very nice!


enhancement to Input to allow for a node prefix/suffix

 (1) React:ions - allow for Input Prefix/Suffix to receive a node


Removal of Build Targets

This may or may not be an issue, but should/would it be possible to remove build targets on deletion of all files matching the isEligible() check from a project?
For example, with the build-coffeescript package, if all .coffee and .cson files are deleted from a project then I would expect the build targets in the above package to disappear from those available to me within that project.
Is this something that is currently possible that I need to implement or is it currently not within the capability of the atom-build system?

Anytime a build provider thinks that the build targets must be updated, it can emit the refresh event (see how it's done in build-gulp). This will remove all the build targets and replace them with a new invocation of settings().


pkg/allocator: do not fail to allocate identity in CRD mode

In case 2 allocations with the same key are being allocated at the
same time one of them might retrieve the ID while the other will
try to allocate the ID in the CRD store. The issue occurs when the
localKeys map will ignore the 2nd key allocation since it won't accept 2
IDs for the same key. Having this failure might cause endpoints from
being restored in CRD mode.
Signed-off-by: André Martins<EMAIL_ADDRESS>Fix https://github.com/cilium/cilium/issues/10774
Fix possible endpoint restore failure in CRD mode.

test-me-please

Coverage decreased (-0.03%) to 45.478% when pulling bc448232e665444892126350ba9746e5fd62e70d on pr/fix-issue-10774 into 60bd7f50d84db54d33e552ec0ccebc3f31628e03 on master.


Player flags

Hello,
thank you for the pull requests you have prepared.
I am sorry I couldn't check them earlier. I wasn't active on GitHub for a while and haven't been notified about these.
I will review your work and include it in the main track when I catch up.


use commit hash instead of doc-specific tag for 3.6 docs ref

Confirmed I am able to build docs locally with this change, and that the 3.6 docs look reasonable (attaching screenshot)


Fix CR reference in the docs

In the minikube section of the docs, the final step of creating a CR
references a file that no longer exists. Point it to example-host.yaml
instead.

D'oh, thanks!


Adjust gitignore with virtualenv files and folders

Can I suggest we use (gi)[https://www.toptal.com/developers/gitignore] to auto-generate our .gitignore files instead of manually adding what we happen to need ?
There's a quick setup that installs it in your shell as a convenient gi function
for this I'd run:
gi venv > .gitignore

this results in
# Created by https://www.toptal.com/developers/gitignore/api/venv
# Edit at https://www.toptal.com/developers/gitignore?templates=venv

### venv ###
# Virtualenv
# http://iamzed.com/2009/05/07/a-primer-on-virtualenv/
.Python
[Bb]in
[Ii]nclude
[Ll]ib
[Ll]ib64
[Ll]ocal
[Ss]cripts
pyvenv.cfg
.venv
pip-selfcheck.json

# End of https://www.toptal.com/developers/gitignore/api/venv

@skateman  please rebase after #22  is merged, it should fix why this PR's CI pipeline is failing


Point experiment

As in raster tiles, our point has the capability to have border (marker-line-*), this PR add this feature starting from a basic option.
At the moment, tangram options for a point are:

size: 10px || [10, 20] (here we only support a point, not a rectangle)
color: rgba/rgb/hex/name color

With this PR we've added this two new features:

border_size: 2px
border_color: rgba/rgb/hex/name color

After adding this two new options we have points with strokes.
I've followed the actual trigonometry of the point used in mapnik.

cc/ @rochoa & @javisantana

This feature is already present in Tangram.


Documentation - website integrations service

Complete the documentation on the websites integrations service, including all methods etc.

Completed in 00a7ee53c93b94e83318f428f3b02d31d9945575


Gateway: e2e custom image tests

What is the expected enhancement?
Add e2e custom image function tests

Test was added in #1406


sync-repo.ps1 should pull first

Shouldn't sync-repo.ps1 execute git pull before git push? Otherwise, you will often get
 ! [rejected]            develop -> develop (fetch first) error: failed to push some refs to 'https://bitbucket.materna.net/scm/ipslhonce/ui.git' hint: Updates were rejected because the remote contains work that you do hint: not have locally. This is usually caused by another repository pushing hint: to the same ref. You may want to first integrate the remote changes hint: (e.g., 'git pull ...') before pushing again. hint: See the 'Note about fast-forwards' in 'git push --help' for details. ⚠ Error in line 31: 'git push' failed
My git-sync command executes first git pull --all and then git push. That worked well for years!

Hi Klaus, yes, I think you're right. This has been changed now.


As a developer I would like documentation related to performance and best practices

Type of Change

[x] New Feature Request
[x] Documentation
[ ] Future Spec
[ ] Improvement / Suggestion
[ ] Bug
[ ] Other (please clarify below)

Summary

It should include docs around

App shell model
Bundling JavaScript / CSS
Polyfills
images / fonts

Use Case

This would be useful for documenting technical decisions made within the project (e.g. webpack configuration choices) and for sharing with other developers.
Code Sample

N / A, aside from general configuration snippets, like for webpack
Pros / Cons

None

related to https://github.com/ProjectEvergreen/create-evergreen-app/issues/25


[Do Not Merge] Added support for UserAssignedIdentities

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[y ] I have reviewed the documentation for the workflow.
[ y] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ y] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

Can one of the admins verify this patch?

@shail2208 Any update on this PR?

Ping again @shail2208

@shail2208 Please open this PR again when you are ready.


No response for date selection

Hi @f33chobits , I met a problem here.
I used your control in a xib IB file , the setting was showing in the below screenshots:

And I implemented the protocol:

When I was running the simulator, there was no print information if I selected a date. Do you have any ideas?

Have you set the Custum Class of your Files Owner ?

If you have, can you provide a sample project to reproduce this?

Yes, I set the custom class as my controller.  I created a git of my sample project.  You can check to see if I made anything wrong --- > https://github.com/pcpLiu/fscalendarDebug
Thanks for your help.

You should always add the viewController as the child view controller after you add it's view as a subview.

The delegate should response then.
But it seems today has not been selected in this case. It's a wierd usage by the way. I'll figure out a solusion.

Can you try this?
https://github.com/f33chobits/FSCalendar/tree/0.9.2

Hi @f33chobits ,
It works now!
Thanks for being so nice! I'm fresh in iOS and it is very helpful.
Have a good one.

Hi @f33chobits ,
It works now!
Thanks for being so nice! I'm fresh in iOS and it is very helpful.
Have a good one.

The same is happening here, i can scroll the page but a i cant select any date, can you help me?


Allow X notifications every Y min. Or wait Z min after answering to send the next notification.

Awesome app - thanks a lot for this!  I just have a suggestion for a feature - at the screen to select a deck, add an option to only send (1-some) notifications every X min.  Either that, or the option to add a delay in min after answering a notification.

Hey hey, thanks for the kind words! I'm glad it's useful to you.
It's an interesting use case. So correct me if I understand wrong, you don't want to be constantly exposed to notifications. You'd prefer the app to send the next notification after an interval, starting from the moment you reply your latest notification.

the option to add a delay in min after answering a notification

In my opinion this one is a lot easier to do than the first one you've proposed, and as you said it can be behind a config that is disabled by default. I'm planning another feature close to this one, so I will take a look at this when I do that!

Yep, I find it hard to get a chunk of time to focus on a whole block of cards, so spreading them out would be better.  I tried to do this with the app Buzzkill, but it has trouble with Companion for some reason.
the option to add a delay in min after answering a notification
Whatever route seems easier to pull off.  Could be a pull down menu at the Companion start screen (0 = immediately, 5 min, 10 min, 15, 30....) or a text field for custom time.
Oddly I've found few flashcard apps with support for notifications, and only one with timed notifications (but no SRS), so I think you'd have something novel.


OS X install

When following your install instructions I ran into some issues specific to OS X I believe.
After running sudo pip install -r requirements.txt and attempting to run ./nosqlframework.py -ip localhost -scan I received the following error.
File "/Library/Python/2.7/site-packages/cassandra/connection.py", line 35, in
from six.moves.queue import Queue, Empty  # noqa
ImportError: No module named queue
There is a conflict with the python six module that is installed on OS X by default.
A quick sudo pip install six resolves that conflict.
I was then met with the next error when attempting to run the framework.
File "/Library/Python/2.7/site-packages/scapy/arch/pcapdnet.py", line 30, in
import pcapy as pcap
ImportError: No module named pcapy
A quick sudo pip install pcapy fixes this error.
I was then met with the following error when attempting to run the framework again.
File "/Library/Python/2.7/site-packages/scapy/arch/pcapdnet.py", line 173, in
import dnet
ImportError: No module named dnet
The installation of dnet was a little bit more involved but the following worked for me.
wget http://libdnet.googlecode.com/files/libdnet-1.12.tgz
tar -xzvf libdnet-1.12.tgz
cd libdnet-1.12
./configure
make
sudo make install
cd python/
sudo python setup.py install
Once these additional steps were completed the framework runs fine on OS X :)

Hey @7h3kid thanks for the pointer , ill definitely update with mac instructions


Copy as SQL IN (...)

When copying some values from a column, i would like to have an advanced copy of the following type:
Copy as SQL IN statement.
For example if i select the values 1,2,3 from the id column i would like to have a copy type that creates the text:
in (1, 2, 3)
or if it's a text type:
in ('1', '2', '3')
and so on. It could be an option to add the column name at the beginning, but usually this is useful for queries on other tables, so the column name is probably different.

Hello @Mizar01
As workaround you can use "Filter by column value"

Also this is duplicate of https://github.com/dbeaver/dbeaver/issues/8760

FilterByColumn name is good but it cannot be considered a practical workaround for most of the situations, as said in the other similar thread (that I couldn't find before). Thanks.


[CS2103T-F11-2] RecruitIn

RecruitIn is a desktop app for recruiters in Singapore to keep track of the plethora of clients with different skill sets, availability and experience. It is optimized for quick text-based inputs via a Command Line Interface (CLI) while still having the ease of use of a Graphical User Interface (GUI). This product will make recruiters’ lives easier through categorisation and filter features to easily access candidates they have in mind.

Codecov Report

Merging #31 (b22de56) into master (ba53b8c) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #31   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

Continue to review full report at Codecov.

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


Cache compilation artefacts

Currently, all the policies in a compilation unit are recompiled even if only one of them has changed. This is not a problem because compilation is fast enough. However, in extreme cases (millions of policies) or in resource-constrained environments, it might be more efficient to avoid recompiling unchanged policies.
One way to achieve this is to store (cache) the compilation artefact for a policy using a hash of the policy. If the policy is unchanged, the hash won't change and we can simply read back the cached artefact from wherever it is stored.
As policies are protobuf objects, https://github.com/deepmind/objecthash-proto is a good candidate for a hash library. However, it hasn't been updated for a while and needs to be refreshed to support the Go protobuf API v2 first.

Objecthash is overkill for this purpose as it is designed for hashing messages with confidentiality guarantees (e.g. redactions). A simple, non-cryptographic hash of protobuf messages should be more than enough for our use case. It's tempting to simply hash the encoded bytes of the protobuf message but the spec does not guarantee that the serialized form will be stable.
We should be able to generate a stable hash by traversing the protobuf message in a specific order. The obvious order is the field number order. Since the encoded representation only deals with field numbers, this should shield us from cosmetic field renames as well.
For map fields, the map needs to be sorted by key before being hashed. For lists, we can simply use the encoded item order. Sorting the list is an option as well but that becomes quite complicated when the list contains complex objects. This means that re-ordering the items in a list would create a different hash even if everything else is unchanged. For our purposes of caching and fingerprinting protobufs, that should not be a problem.

Traversing protobuf message via reflection is very slow. Generated code for the same task is over 500% faster.
name                old time/op    new time/op    delta
Hash/nesting=1-8      30.9µs ± 1%     5.0µs ± 2%   -83.92%  (p=0.000 n=9+9)
Hash/nesting=5-8       159µs ± 2%      25µs ± 2%   -84.25%  (p=0.000 n=10+10)
Hash/nesting=10-8      312µs ± 0%      50µs ± 1%   -83.92%  (p=0.000 n=7+10)
Hash/nesting=50-8     1.56ms ± 1%    0.26ms ± 1%   -83.53%  (p=0.000 n=10+10)
Hash/nesting=100-8    3.16ms ± 1%    0.52ms ± 1%   -83.49%  (p=0.000 n=9+10)

name                old speed      new speed      delta
Hash/nesting=1-8    15.3MB/s ± 1%  95.1MB/s ± 2%  +521.86%  (p=0.000 n=9+9)
Hash/nesting=5-8    15.0MB/s ± 2%  94.9MB/s ± 2%  +534.67%  (p=0.000 n=10+10)
Hash/nesting=10-8   15.3MB/s ± 0%  94.9MB/s ± 1%  +521.77%  (p=0.000 n=7+10)
Hash/nesting=50-8   15.2MB/s ± 1%  92.5MB/s ± 1%  +507.30%  (p=0.000 n=10+10)
Hash/nesting=100-8  15.1MB/s ± 1%  91.5MB/s ± 1%  +505.93%  (p=0.000 n=9+10)

name                old alloc/op   new alloc/op   delta
Hash/nesting=1-8      6.31kB ± 0%    1.18kB ± 0%   -81.24%  (p=0.000 n=10+10)
Hash/nesting=5-8      31.3kB ± 0%     5.5kB ± 0%   -82.33%  (p=0.000 n=10+10)
Hash/nesting=10-8     62.6kB ± 0%    11.0kB ± 0%   -82.47%  (p=0.000 n=10+10)
Hash/nesting=50-8      313kB ± 0%      54kB ± 0%   -82.58%  (p=0.000 n=10+10)
Hash/nesting=100-8     626kB ± 0%     109kB ± 0%   -82.59%  (p=0.000 n=8+10)

name                old allocs/op  new allocs/op  delta
Hash/nesting=1-8         230 ± 0%       105 ± 0%   -54.35%  (p=0.000 n=10+10)
Hash/nesting=5-8       1.15k ± 0%     0.52k ± 0%   -54.89%  (p=0.000 n=10+10)
Hash/nesting=10-8      2.29k ± 0%     1.03k ± 0%   -54.95%  (p=0.000 n=10+10)
Hash/nesting=50-8      11.5k ± 0%      5.2k ± 0%   -55.01%  (p=0.000 n=10+10)
Hash/nesting=100-8     22.9k ± 0%     10.3k ± 0%   -55.02%  (p=0.000 n=10+10)

Code generator implementation: https://github.com/cerbos/protoc-gen-go-hashpb
Reflection based implementation (no longer used): https://github.com/cerbos/protoc-gen-go-hashpb/blob/618cf7d357c785e76958c78337a469dc4f53b8f1/hashpb.go


chore: release master

:robot: I have created a release beep boop
ton-grpc: 0.6.2
Dependencies

tonlibjson-client: 0.11.0
0.11.0 (2023-11-15)
Features

Graceful handling for next block request (#530) (a045855)

Bug Fixes

deps: bump itertools from 0.11.0 to 0.12.0 (fe22337)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/getgems-io/ton-grpc/releases/tag/ton-grpc-v0.6.2 :sunflower:

:robot: Release is at https://github.com/getgems-io/ton-grpc/releases/tag/tonlibjson-client-v0.11.0 :sunflower:


Error while configuring nodes (azure ubuntu and aws redhat)

hi,
i'm facing the below error while i try to bring online the jenkins nodes on azure ubuntu and aws redhat. Previously node on azure ubuntu worked fine and i was able to run the builds.
After i restarted the jenkins node(azure ubuntu) and jenkins master (aws ubuntu) the next day, i'm getting below error.

please help.

got the issue..