OWF 7.16 Custom Banners Not Working

Per https://github.com/ozoneplatform/owf-framework/wiki/OWF-7-Configuring-Custom-Banners, setting custom banners is a matter of setting (in OwfConfig.groovy) something like:
customHeaderFooter {
header = 'location/within/web/server/root/example.html'
headerHeight = 14
footer = 'location/within/web/server/root/example.html'
footerHeight = 14
jsImports = ['location/within/web/server/root/exampleImport1.js',
' location/within/web/server/root/exampleImport2.js']
cssImports = [' location/within/web/server/root/    exampleImport1.css',
' location/within/web/server/root/exampleImport2.css']
}
And in previous versions of OWF 7, that worked just fine.
Looking at to https://github.com/ozoneplatform/owf-framework/blob/v7.16.1/src/groovy/ozone/owf/enums/OwfApplicationSetting.groovy,
[and also ./grails-app/services/ozone/owf/grails/services/CustomHeaderFooterService.groovy which uses the same], it seems like the format changed to something more like:
custom.footer.url = 'OWF_footer.html'
custom.footer.height = 14
custom.header.url = 'OWF_header.html'
custom.header.height = 14
I've tried both; no headers, and looking at the config.js, I see:
"custom":{"footer":{"url":"OWF_footer.html","height":14},"header":{"url":"OWF_header.html","height":14}},"customHeaderFooter":{"header":"","headerHeight":0,"footer":"","footerHeight":0,"jsImports":[],"cssImports":[]},
Did a change get half made or am I missing something else?

Sorry, it appears the wiki is out of date.  If you view the OWF Administrator's Guide found at https://github.com/ozoneplatform/owf-omp-docs/blob/master/OWF Administrator's Guide.doc in section 2.3.3 it will explain how to configure custom headers and footers.  The ability to configure them was moved to the front end and removed from the groovy file.

Thanks.  Going to mark this closed


fix: show bacalhau id error

Review Type Requested (choose one):

[ ] Glance - superficial check (from domain experts)
[ ] Logic - thorough check (from everybody doing review)

show error in "bacalhau id" command.
Summary
Provide a one line summary and link to any relevant references
Task/Issue reference
Closes: add_link_here
Details (optional)
Add any additional details that might help Code Reviewers digest this PR
How to test this code? (optional)
Anything else? (optional)

bacalhau id have error

and when runing rp, lilypad not print pow worker message

this pr return this error, and tell user you got sometheing error

@AquiGorka

add a new commit to resolve this problem,  cmd output some odd error log. just parse the last line

In discussing with @taoshengshi - if we can get this merged, he can do testing in the morning (china time) with RPs to confirm this unblocks the issue.


Cloudflare DNS Plugin won't install

Describe the bug

Error: Command failed: . /opt/certbot/bin/activate && pip install --no-cache-dir --user certbot-dns-cloudflare==$(certbot --version | grep -Eo '[0-9](\.[0-9]+)+') cloudflare && deactivate
ERROR: Can not perform a '--user' install. User site-packages are not visible in this virtualenv.

    at ChildProcess.exithandler (node:child_process:402:12)
    at ChildProcess.emit (node:events:513:28)
    at maybeClose (node:internal/child_process:1100:16)
    at Process.ChildProcess._handle.onexit (node:internal/child_process:304:5)

Nginx Proxy Manager Version

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior

Screenshots

Operating System

Additional context

I activated the virtual env and did a sudo install and it worked:
sudo python -m pip install requests --user

@jeeftor
Can you paste a step-by-step solution?
I have exactly the same problem.

So I'm on vacation but at home I have notes written down.
I believe you need to activate the virtual env in /opt/???/bin/activate
Then run that pop command

I just retested everything I think all you need is to log in to the box/vm/contaienr as root and run:
sudo pip install certbot-dns-cloudflare

Ok here is the step by step ... (oh baby)
I'm running Proxmox so I use one of the scripts to make a new container for proxymanager:
bash -c "$(wget -qLO - https://github.com/tteck/Proxmox/raw/main/ct/nginxproxymanager.sh)"

Once that's done I open a shell into that container and run:
sudo pip install certbot-dns-cloudflare

Login with default creds:

Go make a cert:

And then everything should work...
👏🏼 👏🏼 👏🏼 🚀

@jeeftor Thanks! This solved the issue for me! :)

Thanks @jeeftor for good and clear instruction.
I also have proxmox and install NPM with tteck script.
I follow you steps, but still have same error.
certbot-dns-cloudflare was already installed on my host


Multiple injections of script tag

Just added LinkedIn InsightTag 2.0 to my GTM and for some reason, it's injecting multiple script tags in my header for https://snap.licdn.com/li.lms-analytics/insight.min.js

Do you know the website this is happening? This could be a GTM configuration issue since it works the same for all GTM installations. LMK, so I can look into it.

I am seeing the same problem.  My tag only fires once but the snap url is injected 12 times.

OK so what's happening is that
injectScript('https://snap.licdn.com/li.lms-analytics/insight.min.js', didInjectInsightTag, didFailInsightTag);
```

Loads `insight.min.js` and then calls didInjectInsightTag.

BUT `insight.min.js` was a script that dynamically injects `insight.min.beta.js`.  or `insight.min.old.js`

So by the time it finished "load" - for me it was still async loading `insight.min.beta.js` -so `window.lintrk` wasn't available.

It seems like how you roll out the beta should be included all in the insight.min.js file OR you should fix this tag to handle the fact that insight.min.js can be async.   🤔

For reference, the old insight.min.js code (last night) was this - and that's what caused the infinite script injection because after this block of code loads window.lintrk is still not defined.

```!function(){"use strict";var s=function(n){return/^\d+$/.test(n)},_={1549081:!0},n=function(n,t){for(var a=0,i=n;a<i.length;a++){var r=i[a],e=parseInt(r,10)%100<t,r=_.hasOwnProperty(r);if(e||r)return!0}return!1}(function(n){var t={},a=[];if(n._bizo_data_partner_id&&(t[n._bizo_data_partner_id]=!0,a.push(n._bizo_data_partner_id)),n._bizo_data_partner_ids)for(var i=0,r=n._bizo_data_partner_ids;i<r.length;i++)!t[e=r[i]]&&s(e)&&(t[e]=!0,a.push(e));if(n._linkedin_data_partner_id&&!t[n._linkedin_data_partner_id]&&(t[n._linkedin_data_partner_id]=!0,a.push(n._linkedin_data_partner_id)),n._linkedin_data_partner_ids)for(var e,_=0,d=n._linkedin_data_partner_ids;_<d.length;_++)!t[e=d[_]]&&s(e)&&(t[e]=!0,a.push(e));return a}(window),50)?"https://snap.licdn.com/li.lms-analytics/insight.beta.min.js":"https://snap.licdn.com/li.lms-analytics/insight.old.min.js",t=document.createElement("script"),a=document.getElementsByTagName("script")[0];t["async"]=!0,t.src=n,a.parentNode&&a.parentNode.insertBefore(t,a)}();```

This code appears to have changed this morning.

This is correct and is being fixed now. This is also weird since the code is very old, and this issue has only started happening recently. Probably GTM updated the underlying mechanism. Acknowledged and being worked upon. Expect a fix in a day.

This is matching to our analysis.

insight.min.js becomes a transient script when ramping changes and doesn’t define window.lintrk
due to unavailability of this API on 1st script install, we get this error.

Also, didInjectInsightTag should be called only when the script has been downloaded and executed. So we are covered on that part. The issue is only the unavailability of lintrk API from the transient script. Fixing now.

Update that this work is now done, and I am testing it before updating the template.

Update: the fix is code complete and being tested before linking to the community template again. Stay tuned. Expecting by tomorrow.

https://github.com/linkedin/linkedin-gtm-community-template/issues/4#issue-1423249197 [](https://github.com/linkedin/linkedin-gtm-community-template/issues/4#issue-1423249197)

เพิ่งเพิ่ม LinkedIn InsightTag 2.0 ลงใน GTM ของฉัน และด้วยเหตุผลบางอย่าง มันแทรกหลายscriptแท็กในส่วนหัวของฉันสำหรับhttps://snap.licdn.com/li.lms-analytics/insight.min.js

It's been eight months and this bug is still happening.  Will this be fixed, please?
@OmShiv ?
I can donate a PR for the fix, if you want.  This is affecting production for us.

Hello Duane
This bug fix hasn't made it since we were unsure of injecting another temporary script. Since the script was temporarily injected for 2 weeks, none of our users complained about any problems post those initial weeks. I will get this re-prioritized and update by end of next week.
Om

From: Duane Hutchins @.>
Sent: Wednesday, July 5, 2023 10:23 AM
To: linkedin/linkedin-gtm-community-template @.>
Cc: Om Shankar @.>; Mention @.>
Subject: Re: [linkedin/linkedin-gtm-community-template] Multiple injections of script tag (Issue #4)
It's been eight months and this bug is still happening. Will this be fixed, please?
@OmShivhttps://github.com/OmShiv ?
I can donate a PR for the fix, if you want. This is affecting production for us.
—
Reply to this email directly, view it on GitHubhttps://github.com/linkedin/linkedin-gtm-community-template/issues/4#issuecomment-1622180698, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAL4YXWQDRORRIRHT7PSZCTXOWPJLANCNFSM6AAAAAAROOXUJ4.
You are receiving this because you were mentioned.Message ID: @.***>

Any updates or workarounds here? This is affecting production for us as well

We have made changes to the code and saw it fixed. Can you verify if you still see this issue? Please reopen if persists!


Login with url

This PR adds the possibility to log in just from a link.
Closes #3464

@SchrodingersGat this is intended for easier opening the demo (without putting in the credentials). I will do a PR to the docs ASAP


BE AS CLEAR AND CONCISE AS POSSIBLE.
describe the bug
When hiding the discussion/comments on any notion page it is also making the dividers invisible.
Also if I remove the discussion hide styling from custom.css the dividers are showing again.
expected behaviour
The dividers should be visible even if the discussion is hidden.
screenshots

The blue highlighted area is where the divider should be.
platform
Windows 10

@Crimson723 which dividers are missing?

Yes. The divider that is added with three - or by /divider.

use this:
[style*='env(safe-area-inset-left)']:not(.notion-page-content)
  [style*='width: 100%; height: 1px;'],
.notion-page-view-discussion {
  display: none !important;
}


Implementation and tests for IntervalView.

Implementation and tests for Interval MutableView, using IntervalBuckets.
Overall idea:
For each IntervalView, we always keep a queue of N + 1 buckets (by default N is 4). Each bucket has a duration which is interval duration / N. Ideally:

the buckets should always be up-to-date,
current time should always be within the latest bucket, currently recorded stats should fall into the latest bucket,
there are always N buckets before the current one, which holds the stats in the past interval duration.

When getView() is called, we will extract and combine the stats from the current and past buckets (part of the stats from the oldest bucket could have expired).
However, in reality, we couldn't track the status of buckets all the time (keep monitoring and updating the bucket queue will be expensive). When we call record() or getView(), some or all of the buckets might be outdated, and we will need to "pad" new buckets to the queue and remove outdated ones. After "padding" new buckets, the bucket queue will able to maintain the three invariants in the ideal situation.
For example:

We have an IntervalView which has a duration of 8 seconds, we register this view at 10s.
Initially there will be 5 buckets: [2.0, 4.0), [4.0, 6.0), ..., [10.0, 12.0).
If users don't call record() or getView(), bucket queue will remain as it is, and some buckets could expire.
Suppose record() is called at 15s, now we need to refresh the bucket queue. We need to add two new buckets [12.0, 14.0) and [14.0, 16.0), and remove two expired buckets [2.0, 4.0) and [4.0, 6.0)
Suppose record() is called again at 30s, all the current buckets should have expired. We add 5 new buckets [22.0, 24.0) ... [30.0, 32.0) and remove all the previous buckets.
Suppose users call getView() at 35s, again we need to add two new buckets and remove two expired one, so that bucket queue is up-to-date. Now we combine stats from all buckets and return the combined IntervalViewData.

Codecov Report

Merging #545 into master will increase coverage by 0.31%.
The diff coverage is 94.44%.

@@             Coverage Diff              @@
##             master     #545      +/-   ##
============================================
+ Coverage        90%   90.32%   +0.31%
- Complexity      566      573       +7
============================================
  Files           110      111       +1
  Lines          2361     2500     +139
  Branches        211      228      +17
============================================
+ Hits           2125     2258     +133
- Misses          180      182       +2
- Partials         56       60       +4

Impacted Files
Coverage Δ
Complexity Δ

...ava/io/opencensus/implcore/stats/StatsManager.java
100% <100%> (ø)
5 <0> (-1)
:arrow_down:

...a/io/opencensus/implcore/stats/IntervalBucket.java
100% <100%> (ø)
8 <8> (?)

...o/opencensus/implcore/stats/StatsRecorderImpl.java
100% <100%> (ø)
2 <0> (ø)
:arrow_down:

...io/opencensus/implcore/stats/MeasureToViewMap.java
91.48% <83.33%> (+0.37%)
12 <0> (ø)
:arrow_down:

.../opencensus/implcore/stats/MutableAggregation.java
96.8% <87.5%> (-3.2%)
1 <0> (ø)

.../io/opencensus/implcore/stats/MutableViewData.java
90.68% <95.08%> (+4.72%)
17 <8> (ø)
:arrow_down:

Continue to review full report at Codecov.

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

I've reverted a bad commit and added a verbose description on how I implement interval stats with buckets. The algorithm could be a little difficult to understand, so feel free to leave comments if you feel unclear about the algorithm.

Coverage increased (+0.5%) to 80.909% when pulling f6c3ed18d6ec12673fd68499096cad4fdd2b38bf on songy23:interval-stats into 821b15f2114e4bea3a78dd69a8b2ab63eb8a399b on census-instrumentation:master.

Coverage increased (+0.5%) to 80.909% when pulling a8ae58f817403322e45716e6ecb79fbdffcb22f2 on songy23:interval-stats into 821b15f2114e4bea3a78dd69a8b2ab63eb8a399b on census-instrumentation:master.

Coverage decreased (-0.9%) to 79.535% when pulling dc8a773b669f37721a7993ce56b0d0968cb95f40 on songy23:interval-stats into 821b15f2114e4bea3a78dd69a8b2ab63eb8a399b on census-instrumentation:master.

Thanks!

Coverage decreased (-2.6%) to 77.823% when pulling f2f65583715985aaeadd6cf81b6d82b5623d2012 on songy23:interval-stats into 821b15f2114e4bea3a78dd69a8b2ab63eb8a399b on census-instrumentation:master.


[readosm]update to 1.1.0a

update to 1.1.0a
support linux and osx build

The detection failure is caused by spatialite-tools. I fixed this port in PR #15039. I don’t know if I need to block the linux and osx compilation of spatialite-tools in this PR.

@longhuan2018, after PR #15039 merged, I will rerun the CI testing, so you don't need to block them in this PR.

Do I need to add spatialite-tools to ci.baseline.txt first to exclude its compilation in linux and osx? Because the current version of the spatialite-tools port only supports windows. @PhoebeHui

@longhuan2018, since you have fixed it by PR #15039, so this is not required, I will rerun the CI testing after the PR merged.

But spatialite-tools is dependent on readosm. If this PR fails, then it cannot be compiled.

@longhuan2018, I think you should update the 2 ports togather, then the CI would test the changes and catch the regressions.

This is what I split from PR #14951, CI has been verified

@longhuan2018, that's fine, you can enable the spatialite-tools in PR https://github.com/microsoft/vcpkg/pull/15039 after this PR merged.

Thanks!


added my name Anita Idemudia on line 77

Please I am sorry for all the trouble we have caused you but count it all joy. Your PR is about to be merged and you will be able to see your name on this url.
To enable me to do this, READ the README again carefully, update your code with the latest and push the update. I am here for you for the next 6 hours before I close the PR so you can initiate a fresh PR which will be easier to merge.

Hi Anita!
Please open a new PR! Remember to make a branch as you work on your changes!
We look forward to merging your next PR! :partying_face:


Add/remove entities modal doesnt allow adding to a 2D space view

The add/remove entities modal doesn't let adding anything to a 2D space view with he ARKit example. This is possibly a heuristics bug?

Repro

Open ARKit example
Remove everything from the blueprint
Add an empty 2D space view
Use the add/remove entities modal

Expected: stuff can be added (at least camera_lowres etc.)
Actual: nothing is enabled

The reason for this is that that 2D space view has a space origin that is not compatible with any 2D content. It's space origin should be set to the entity that has the pinhole transform (or below) to be able to accept content.
For 0.12, the workaround is to create a 2d space view with a suitable space origin from the + button in the blueprint panel.
In 0.13, it will be possible to change the space origin of a space view.

The behavior hasn't changed but:

it is now possible to change the origin
internally the situation is much better defined: None of the entities are visualizable from the space since they'd require a 3D->2D projection starting with a 2D eye which is an invalid transformation

So what's to do here now is to allow adding entities anyways (blueprint could do this!) and give the right hints why something doesn't execute a given visualizer.


Build docker image for ethql

This PR adds a Dockerfile for dockerising ethql, as well as sigterm and sigint shutdown hooks.

[ch40]


Service Graph YAML isn't in the location specified.

Related to page: https://github.com/istio/istio.github.io/blob/master/content/docs/tasks/telemetry/servicegraph/index.md
The linked file "https://raw.githubusercontent.com/istio/istio/release-0.8/install/kubernetes/addons/servicegraph.yaml" is not found.

I don't find it in 0.8 still.


Problem with JDK 1.7 when executing integrationtests (HTTPS) in local development environment.

Original issue 292 created by soi-toolkit on 2012-09-05T12:10:37.000Z:
SSL "Handshake" problem: javax.net.ssl.SSLHandshakeException: Received fatal alert...

2.0.0-M5 doesn't have this problem.
Closing this ticket.


Update README.md

Added example usage

Sorry, but I think the readme is big enough, and that usage of ag is pretty straightforward. If someone really has trouble, they can always run man ag or ag --help.


Update Client method signatures

When calling a streaming method, an enum of request objects is sent in a single method argument. But when calling a non-streaming method the individual fields of the request object are used as the method arguments. We would like to revise how client methods are defined.
We have a couple ideas. One is to change the request object's individual fields to all be named arguments instead of positional arguments. We would also like to allow passing in a request object instead of the individual field values, either as a proto resource object or a Ruby Hash.

We decided this was resolved by #82.


Panel overflow icon no longer shows

Compare 1.40 stable:

To 1.39:

I am not seeing the "..." action anymore when panels are narrow. The element is in the dom though. Maybe a icon regression?

Oh, on hover it appears to the top:

Already logged as #84387

Duplicate of #84387


#264 - Improve minos.common.model performance

Thanks you so much for your review @vladyslav-fenchak!


Generic class interface for an abstract class in TS1.6?

I use the following definition in some generic code to capture the type of a class argument:
interface IClass<T> extends Function {
    new (...args: any[]): T;
}

However, this does not work if the class in question is abstract. Is there any alternative for this? If not, can some syntax be added to support this?

@mhegazy and I had a recent conversation about this and the question we were wrestling with was whether it is the class constructor that is aware of abstractness or if it is the prototype that is abstract because it doesn't implement expected methods.

@billccn can you shed some light on how this is used, and how you want to use abstract classes with them?

Hi,
I use the IClass<T> interface in two places: a checked cast function (if (x instanceof cls) then return <T>x else throw) and a manual sub-class function (mainly used to augment instance creation at run time).
--
What I really want is a typing syntax that expresses the relationship between an expression that evaluates to a Function and objects with the prototype of that Function.
In TS1.6, the expression does not have to be new-able. For example, classes with private constructors, abstract classes or everything that is allowed on the RHS of extends. So we can say the {new (): T} syntax is no longer fit for purpose regardless of how we interpret abstractness in abstract classes.
Personally, I see three possible solutions:

Make typeof X sub-typeable. Currently, interface Clazz<T> extends typeof T does not work, but this is artificial as manually enumerating typeof T in a differnet type would work so why can't the type checker do it for us? (I think this would also add a nice symmetry with extending class expressions in ES6.)

A new keyword like new in {new (): T} to state the instance type when sub-classed. This is more generic than 1 and carries all the benefits of the original new syntax. Especially, this allows protected constructors to be typed and maybe even allow overloaded constructors to return different types (e.g. to model that different overloads set different fields).

Treat the prototype property as first class citizen in the type system. So instead of new ():T, people can say interface X<T> {prototype: T}. I think this already works, but maybe it should be advertised more in the documentations instead of the new() syntax.

@mhegazy, on https://github.com/Microsoft/TypeScript/issues/4709#issuecomment-138983087, @OlegDokuka was interested in specifying a catch-all type for classes to decorate, but not being able to specify that the type could be abstract limited the approach.

In my opinion, the best thing that would be is create in native code. It should provide two main things:
Base idea

It can be generic
it can be empty

The first one should be something like this:
type Class = <T>{
        new (...args: any[]): T;
 };

it can be used something like this
function newInstance<T>(clazz:Class<T>):<T>{
}

The second one should looks like this:
type Class = {
        new (...args: any[]): any;
 };

it can be used like that
function newInstance(clazz:Class):any{
}

Both cases should be in the TS system. But in generic case The type of Class is static and equals T.
In the second case type of Class is hided and  equals any.
Using with Abstract class
In this case if we support code above the TSs compiler can check in compile time what type is passed - for example:
abstract class TestWithAbstractKW{
}

var clazz :Class<TestWithAbstractKW> = TestWithAbstractKW;

var a = new clazz() //here we get compiler error that abstract class cannot be directly instantiated

function testInGeneric<T>(clazz:Class<T>){
     var a = new clazz();
}

testInGeneric(clazz); // everything is okay, but in runtime we get exception that abstract class can not be directly instantiated.

The last one idea
I heard that TypeScript 2.0 (AtScript) will give us menu feature like type manipulation at runtime. It  would be cool if Class would be something like in Java or C# and give us more reflection than we have ever imagined about JS language.
It would be cool if we would do something like
var clazz:Class<Something>;

var abstract:boolean = clazz.isAbstract()
var implementedInsterfaces:Class[] = clazz.listInterfaces();
var fields: PropertyDescriptor[] = abstractclazz.fields();
var methods: PropertyDescriptor[] = clazz.methods();

That is all.
What do you think about it?

@billccn, just to make sure i understand the issue..
function checkedCast<T>(x: any, cls: IClass<T>): T {
    if (x instanceof cls)
        return <T>x;
    else
        throw new Error("not cls");
}

class C {
}

abstract class A {
}

var c = checkedCast({}, C);  // OK
var a = checkedCast({}, A);  // Error, A is abstract

are there other use cases?

@mhegazy (Sorry, lost track of this thread.)
I have two uses for it, one as you showed above and another one which creats a sub-class of the given argument at run time. (The latter is a replacement of the __extend function TS inserts.)

so why not just use Function. this to me seems to be the accurate representation of what you want to do. if you also want to get the type inference in the above example, define cls as IClass<T>|Function.

We choke if you actually try to drop something of that type into an extends block:
interface IClass<T> extends Function {
    new (...args: any[]): T;
}

type Extendsable<T> = IClass<T>|Function;

function makeClass<T>(a: Extendsable<T>) {
    return class extends a {}
}

When it's a union we say it's not a constructor function type, and when it's just IClass<T> we get that the return type of the extends clause is not a class or interface type.
So I guess the simple question would be to ask what is the broadest way to type a?

but this is not the scenario @billccn is describing. we are talking about generic helper functions that can reason about classes. extends clause is a bit of a special case.

@billccn does https://github.com/Microsoft/TypeScript/issues/4693#issuecomment-140157887 addresses the issue?

please reopen if there is more information to consider.


Eliminate implicit property ordering for cache key

There is an implicit ordering of setting properties and the invocation
of base class methods in order for GithubLoader to properly initialize
itself with the appropriate cache key.  This change removes that
implicit ordering and passes, explicitly, the cache key string value to
the GithubLoader::_initGithubLoader method.
Note that I am working towards taking care of #186, but in order to do so, I need to make some incremental changes along the way.

The build failure by AppVeyor and Travis CI is reporting exists already on the master branch ad does not appear to due to my commit in this PR.

@ericvw I'm solving this test. This is not with your implementation.

merged!


Vickel: Blacklist \w{0,30}assign(?:ments?)?[\W_]+helps?.[\w.]+

Vickel requests the blacklist of the keyword \w{0,30}assign(?:ments?)?[\W_]*+helps?\.[\w\.]*+. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

\w{0,30}assign(?:ments?)?[\W_]*+helps?\.[\w\.]*+ has been seen in 360 true positives, 6 false positives, and 0 NAAs.

check chat: https://chat.stackexchange.com/transcript/message/66086733#66086733

Approved by tripleee in Charcoal HQ


[WinForms] deprecated controls removed

Removal of deprecated controls
While preparing the support for Windows Forms applications in .NET Core, we've decided to remove a set of controls that are no longer supported and have had replacements since .NET Framework 2.0.  These controls will no longer exist in the .NET Core code base and Windows Forms developers should refer to below for the suitable replacements.
The following types are no longer available:

StatusBar
StatusBarDrawItemEventArgs
StatusBarDrawItemEventHandler
StatusBarPanel
StatusBarPanelAutoSize
StatusBarPanelBorderStyle
StatusBarPanelClickEventArgs
StatusBarPanelClickEventHandler
StatusBarPanelStyle

Version introduced
.NET 5.0
Old behavior
The above controls and types were available.
New behavior
The controls and types are no longer available.
Reason for change
Since after .NET Framework 2, these set of controls have had replacements with updated design and support. This list of controls has been removed from designer toolboxes for some time in favor of their replacements and we have observed customers using these newer controls. In examining what areas of Windows Forms we will bring forward to .NET Core, it was decided that we should not carry forward deprecated APIs like this set in order to maintain support for areas like high DPI, accessibility, and reliability.
Recommended action
Move to the replacement APIs for these controls and their scenarios:

Old Control (API)
Recommended Replacement
Other associated APIs removed

StatusBar
StatusStrip

StatusBarPanel
ToolStripStatusLabel

Category

Windows Forms

Affected APIs

https://docs.microsoft.com/en-us/dotnet/api/system.windows.forms.statusbar?view=netcore-3.1
https://docs.microsoft.com/en-us/dotnet/api/system.windows.forms.statusbarpanel?view=netcore-3.1
https://docs.microsoft.com/en-us/dotnet/api/system.windows.forms.statusbardrawitemeventargs?view=netcore-3.1
https://docs.microsoft.com/en-us/dotnet/api/system.windows.forms.statusbardrawitemeventhandler?view=netcore-3.1
https://docs.microsoft.com/en-us/dotnet/api/system.windows.forms.statusbarpanelautosize?view=netcore-3.1
https://docs.microsoft.com/en-us/dotnet/api/system.windows.forms.statusbarpanelborderstyle?view=netcore-3.1
https://docs.microsoft.com/en-us/dotnet/api/system.windows.forms.statusbarpanelclickeventargs?view=netcore-3.1
https://docs.microsoft.com/en-us/dotnet/api/system.windows.forms.statusbarpanelclickeventhandler?view=netcore-3.1
https://docs.microsoft.com/en-us/dotnet/api/system.windows.forms.statusbarpanelstyle?view=netcore-3.1

Issue metadata

Issue type: breaking-change

@RussKie Is this for Preview 1?

Yes, .NET 5 Preview1
Thank you


Question about function OnAcknowledgementPacket in apps/transfer/keeper/relay.go

I find that in ibc-go, when the acknowledgement error, the tokens will be refunded.
func (k Keeper) OnAcknowledgementPacket(ctx sdk.Context, packet channeltypes.Packet, data types.FungibleTokenPacketData, ack channeltypes.Acknowledgement) error {
	switch ack.Response.(type) {
	case *channeltypes.Acknowledgement_Error:
		return k.refundPacketToken(ctx, packet, data)
	default:
		// the acknowledgement succeeded on the receiving chain so nothing
		// needs to be executed and no error needs to be returned
		return nil
	}
}

But in IBC Spec ICS-20, when the acknowledgement is not successful, the tokens will be refunded.
function onAcknowledgePacket(
  packet: Packet,
  acknowledgement: bytes) {
  // if the transfer failed, refund the tokens
  if (!acknowledgement.success)
    refundTokens(packet)
}

Is this a special design? Is there any difference?

Thank you for bringing your question, @michwqy. As Aditya replied to your question in the spec repository, the spec should check that the acknowledgement is not of the error type. There was recently a situation where an IBC implementation checked explicitly for the string AQ== in the acknowledgement and that caused some successful acknowledgements to be misinterpreted as error acknowledgements because they didn't match that format (see the details here). I will close the issue now, I hope your question is answered.
I see that you're working on TLA+ models for IBC. Do you mind me asking what your goal is?

Thanks for you reply, @crodriguezvega . I am a student learning how to apply model checking to blockchain. I will analyze some parts of the IBC protocol based on my time and ability.

Thanks for you reply, @crodriguezvega . I am a student learning how to apply model checking to blockchain. I will analyze some parts of the IBC protocol based on my time and ability.

Ah, that's great. You've been asking very good questions about the protocol, so keep them coming. :)


'TileJSON' is not a valid value for URLTemplateTypeResourceType.

GeoServer 2.20.0
MapsUI 3.0.2
code: var tileSources = WmtsParser.Parse(response);
wmts.txt

Thanks, this is a very minimal bug report, but it contains all information needed :)

I would like to report that, i have been using your MapsUI in our implementation of using WMTS geoserver by the example of https://github.com/Mapsui/Mapsui/blob/00914ae139abf0926dfdb0eff317d40b359f258a/Samples/Mapsui.Samples.Common/Maps/Data/WmtsMichellinSample.cs

This should definitely be fixed but unfortunately I have other priorities right now. I am focusing on a v4 beta release.
btw, this is probably a bug in BruTile.

BruTile 178
Any news for this bug? It occurs when server running on GeoServer 2.20+, I had to downgrade to 2.19.x..
const string url = "http://URL/geoserver/gwc/service/wmts?request=GetCapabilities";
try
{
    var httpClient = new HttpClient();
    var response = httpClient.GetStreamAsync(url).Result;
    {
        var tileSources = WmtsParser.Parse(response);

        var tileSource = tileSources.First(t => t.Name == "LayoutName");
        TileLayer layer = new TileLayer(tileSource) { Name = tileSource.Name };
        return layer;
    }
}

I'll have a look into it, but can't promise anything.


Simple native button for change dataset

screenshot?

I don't think i'm satisfied yet, but given that reduce the size of the codebase - I'm merging.


FlaUI unable to identify the change in Automation ID

Hi,
Thanks for the Support !!!
I am facing issue with the FlaUI with the latest version, I have my application opened already and attaching it to FlaUI with ID.
Getting an error that,
FlaUiError: Element from XPath '//*[@AutomationId="inst1-base-Sales-ringing"]' could not be found, but still it can be found with the FlaUI inspect and the element is visible in the screen. The ID of this button changes dynamically when some actions are triggered.
PFA attached screen shot - In the FlaUI inspect in details changed ID is reflecting , but in the summary it is old ID.

@Muthuvenaktesh
Did you refresh the flaui inspector? Because it's probably an old cache from flaui from element if you don't refresh you will not see any changes here on the inspector tool.
On Robotframework side you can check it on multiple ways.
By robotframework 5 IF Version
IF  Element Should Exist <XPATH_A> ${FALSE}   Log  Found Element A ELSE IF    Element Should Exist <XPATH_B> ${FALSE}   Log  Found Element B END
The second argument avoids an element not found exception and returns true if element could be found otherwise false.
So you can write your logic code what to do here aftewards your id was changed.

@Nepitwin , I have tried it, the framework is still not able to find the ID.
Is there any way we can refresh the cache from framework also.

@Muthuvenaktesh
All keywords from robotframework flaui will receive all gui element from user interface.
So if you use some of these keywords if the id was changed the element should be accessible.
Can you pls use the keyword
${RESULT}  Get Childs From Element  /Pane/Document
Log  ${RESULT}

Here you can verify if you Button element exists from child element and which automation properties they got. It's a keyword for debugging purpose to identify xpath from childs.
Only the inspector tool contains a cache implemented. To refresh by inspector tool simple use F5 the element should be updated if not the user automation id is not changed.
This could be verify by Get Childs From Element keywords.

Hi @Nepitwin ,
Tried using the command, with the tool I can see the change in ID, but the framework is still showing the old ID.
Refer the attached screen shot,

Thanks for the information i look into FlaUI C# Code and found that code.
public override AutomationElement GetDesktop()
{
return Com.Call(() =>
{
var desktop = CacheRequest.IsCachingActive
? NativeAutomation.GetRootElementBuildCache(CacheRequest.Current.ToNative(this))
: NativeAutomation.GetRootElement();
return WrapNativeElement(desktop);
});
}
My first idea is that the Flag IsCachingActive is True so your first request stores all data but does not refresh, But in your use case you want to update the change. I think i can easily reproduce it now and fix it by an adittional new keyword
Disable Cache
Enable Cache
So afterwards your test can disable the caching feature and reenable it because it will be a heavy intensive operation. But for your use case you want to recognize the change.

@Muthuvenaktesh
If you want to pre test changes you can install one of these wheel files from zip.
robotframework_flaui-1.0.549rc549.zip
This has now the ability to disable caching feature by Library include or by Keywords
Enable Caching
Disable Caching
${RESULT} Is Cache Enabled
Refresh Cache
Library FlaUILibrary use_cache=True/False
Currently it's missing test by automation id change behavior like your mentioned. If it's implemented then this will be merged to main branch and released.

@Muthuvenaktesh
After i write your use case that an automation id is changed by an interaction i can not reproduce your issue.
I extend my wpf test application by an button Change Automation Id.

If i click on it i will update the automation id property by C# Code.
private void ChangeAutomationId(object sender, RoutedEventArgs e)
        {
            if (sender is Button button)
            {
                button.SetValue(AutomationProperties.AutomationIdProperty, "ChangeAutomationIdUpdate");
            }
        }

Afterwards my test is quite simple
Element Should Be Updated If Property Is Changed
   [Setup]       Init Main Application
   [Teardown]    Stop Application  ${MAIN_PID}
   Element Should Exist      ${XPATH_BUTTON_ELEMENT_ORIGIN}
   Element Should Not Exist  ${XPATH_BUTTON_ELEMENT_CHANGED}
   ${CHILDS}  Get Childs From Element  ${MAIN_WINDOW_SIMPLE_CONTROLS}
   Log  ${CHILDS}
   Click                     ${XPATH_BUTTON_ELEMENT_ORIGIN}
   ${CHILDS}  Get Childs From Element  ${MAIN_WINDOW_SIMPLE_CONTROLS}
   Log  ${CHILDS}
   Element Should Not Exist  ${XPATH_BUTTON_ELEMENT_ORIGIN}
   Element Should Exist      ${XPATH_BUTTON_ELEMENT_CHANGED}

After the execution id was changed.

This is tested by robotframeowrk-flaui version 2.0.1.
Could it be that your test execution is too early that the automation id property does not change on your robot test?
Did you use an Wait Until Keyword Succeeds?  Because i think your test is that you wait for an call an service instance is changing the id if the telephone is ringing. So your test has to wait an amount of time if this signal is coming to your application.

Hi @Nepitwin,
Thanks for the support !!!
My requirement is as you said it was telephony app, so if the call was answered by someone, it will reflect in my app that particular contact is ringing, active and hold.
Yes I have waited few seconds also for the ID to reflect .

Hi @Nepitwin ,
Is there any way we can get the button attributes like background colour and border colour. If its there, I can validate the colours instead of ID's.

@Muthuvenaktesh
I analyze the code from flaui and found some properties --> https://github.com/FlaUI/FlaUI/blob/master/src/FlaUI.Core/FrameworkAutomationElementBase.Properties.cs
I would test it by my own if this values are the correct one and implement this feature to your response from #72
But one thing is clear if your solution will work by your changing of color codes then automation id should asweel work because these are all properties which are updated automatically by each request. Why this not working on your side i currently don't know sadly.

@Muthuvenaktesh
Intresting now i know which gui tool your use case is. It's not an desktop user interface element. This is quite easy to answer for you. This html5 elements are not implements with UIA3 or UIA2 interface from microsoft.
https://learn.microsoft.com/en-us/windows/win32/winauto/windows-automation-api-overview
I thought your application is based on a desktop application like wpf but it seems it's a hybrid application as html5 as dom tree usage. Yeah you probable see xpaths from this element on gui side but probably it's quite not supported to interact with them like your use case.
Did you think to use web application based testing libraries like
https://applitools.com/tutorials/quickstart/web/robot-framework
https://robotframework.org/SeleniumLibrary/SeleniumLibrary.html
Which you can handle xpaths like robotframework-flaui by your dom tree.
FlaUI can be used for browser related testing but it's not recommended for my opinion because it handles windows automation based elements. If you want to test it you can try UIA2 interface maybe it's supported their if not you don't have access to an automation interface implementation from windows.

Hello,
I am having a similar problem to the original poster where robotframework-flaui is not picking up a dynamically changed property on an element. It is a WPF application, but unfortunately I don't have access to the source code, so I don't know exactly how this update is done in the code. I have tried using the Wait Until Element Exist, but it does not work in this scenario. When I log the elements using Get Childs From Element, I can see that the element still has the old property (Name in this case), but the screenshot taken at the same time shows that the UI has been updated. I think a "Refresh Cache" keyword would help in this scenario to force a new scan of the UI elements. Would it be possible for you to reimplement the "Refresh Cache" keyword for the newest version of robotframework-flaui for a test?
Thank you!


Stacked electrical parts are not detected in build recipes.

Description
When electrical components like capacitors are stacked the build menu is not detecting the multiple instance of them.
Reproduction
Try make a medsec hud with stack capacitors, they must be split to 1 stacks only.
Screenshots
NA
Additional context
Assuming it affects anything with multiple elec parts but can be replicated with that medsec hud

Interesting, if you try to merge them into one entry and give it amount: 2 then the character can make the medsec hud while they are stacked, but the entire stack will be removed, no matter what number is required on the prototype


build script for enabling eager builds with package.json one-liner

There should be a rust-bindings binary that can be used to run eager builds with a single line addition to package.json. It should accept as command-line arguments all the configuration options accepted by the API.

Way out of date. Now you can re-build with npm install any time.


Fixed documentation

Hi, this is shika358.
I fixed README.md.
(length validator sample: required -> length)
Would you check this?
thanks.

Fixed, thank you!!


Option for colored items for the calendar module and the weatherforecast module

Calendar Module

If set colored to 'true' there can be a color definded for each calendar

example config:
`config: {
colored: true,
calendars: [
{
url: 'http://www.calendarlabs.com/templates/ical/US-Holidays.ics',
symbol: 'calendar',
color: '#8cd7ac'
},
{
url: 'http://www.calendarlabs.com/templates/ical/US-Holidays.ics',
symbol: 'calendar',
color: '#da7e7e'
},
			],
		}`

Weather Forecast Module

If set colored to 'true' min-temp get's a blue tone and max-temp a red-tone

example config:
config: { 			// See 'Configuration options' for more information. 			location: 'Amsterdam,Netherlands', 			locationID: '', //Location ID from http://openweathermap.org/help/city_list.txt 			appid: 'abcde12345abcde12345abcde12345ab', //openweathermap.org API key.             colored: true 		}

Really nice feature @artifactdev. There a problem, the Pull Request is sended to develop branch not to master. You can close this and send a new. I don't know if possible change the destination branch.

Sorry, that was my fault. I added a new Pull Request


Implement url_component escaping/unescaping

As requested by @ptoomey3:

during unescape_url you should substitute + for space before you do any percent decoding. Doing it after makes it such that a percent encoded plus (%2b) is first decoded to a +, but then that is translated to a space. In the end, the current code makes it so that a + can never end up in unescaped output.
add a new function escape_uri_component. This works exactly the same as escape_url, but encodes space as %20 instead of +. I guess technically we also need an unescape_uri_component, though I think the implementation will be exactly the same as unescape_uri.

I think this has the functionality you wanted.

Thanks @vmg. I'll take a closer look at this when I get in. One quick note..the name of the new functions should be houdini_(un)escape_uri_component. Thanks again!

Are you sure it should be called uri_component? I think url_component makes more sense because it behaves like escape_url (except for spaces).

I think so. The only real difference between them is whether a space is encoded as a + or a %20. Rack has similar methods (called escape and escape_path) that do this in https://github.com/rack/rack/blob/d1d380c53a1199670e4a640882d95c4a1bfbf94e/lib/rack/utils.rb#L35-L44. We are modeling the naming here after the JavaScript equivalents encodeURI and encodeURIComponent.

:+1:

Just to be sure I can double check the lookup tables to make sure it matches up with what we are looking to do.

Ok, I think we should be good here. The only values that are not encoded are alphanumeric, ., _, and -. These are all fine in URI paths so we should be good.

All yours!


panic: file not found

Hi,
I ran into a panic while the i18n library tried to open a file that diddn't exist on my filesystem.
Some sort of graceful error handling, replacing the call to panic (https://github.com/go-macaron/i18n/blob/master/i18n.go#L57), would be nice.
If I'm not wrong this would break the API.

Hi, panic meant for errors too critical to make package work, so I don't think remove panic is a good idea. You shouldn't add those files to the module in the first place if they don't exist.


MongoDB: WriteConcern not used when updating an entity.

Hi,
I was debugging my code in order to validate that the WriteConcern I set was actually being used.
I've set WriteConcern using the following method:
EntityManager em = emf.createEntityManager();
em.setProperty(MongoDBClientProperties.WRITE_CONCERN, WriteConcern.ACKNOWLEDGED);

Which works just fine and WriteConcern.ACKNOWLEDGED is used for inserting records.
However, on update I noticed this was not the case.
User user = new User();
user.setUserId(key);
user.setField0("Some value");

em.merge(user);

I noticed when debugging that Kundera does not pass on the WriteConcern when saving the updated object. I traced this back to the following line of code.
A simple fix involves calling the other save method and supplying the write concern.

@vreniers
Thanks for pointing out. We'll verify it at our end and fix it if required.
Chhavi

@vreniers
This is a bug. We will fix it in next release.
By the way,  default writeconcern property for the Kundera-MongoDB is ACKNOWLEDGED.
-Dev

@vreniers
I added a fix in the trunk. This fix will be available from next release (3.4). Meanwhile, you can build Kundera from trunk use 3.4-SNAPSHOT version if needed.
-Dev

Thank you.


Question about live_grep with rg's --fixed-strings option

Hi again!
I've read documentation and wiki after my yesterday's PR but I still have a question.
I want to perform all my searches using live_grep with no regex processing, only literal strings. So I use --fixed-strings option for rg and disable escaping with no_esc = true. But I still can't search option-like strings (-info, --help, etc.). This fails because whole query is passed as argument for nvim --headless ... command for mt version of live_grep.
This can be fixed by using -- (shell's options/arguments separator) here: https://github.com/ibhagwan/fzf-lua/blob/267098387dd7a3c53f8a4cfb17285ba05d841353/lua/fzf-lua/providers/grep.lua#L301-L302
Like this:
command = command:gsub(core.fzf_query_placeholder, "{argvz}")
        .. " -- " .. core.fzf_query_placeholder

Am I overcomplicating things again and there is some existing option for my kind of case?

Am I overcomplicating things again and there is some existing option for my kind of case?

You're not overcomplicating your use case but it does overcomplicate everything else :-)
The purposed solution solves your use case alone and only for live_grep with glob support enabled, this doesn't support using --fixed-strings with grep, for example try running :FzfLua grep debug=true rg_glob=false no_esc=true search=--no, this will result in:

A more generic solution would be to add the -- separator to the grep command builder at:
https://github.com/ibhagwan/fzf-lua/blob/267098387dd7a3c53f8a4cfb17285ba05d841353/lua/fzf-lua/providers/grep.lua#L103
By changing the code to:
command = ("%s -- %s %s"):format(command, search_query, search_path)

Which will no result in:

But as you can guess from the above code, this would break the command when a specific file is specified which would break many more providers such as grep_curbuf, lgrep_curbuf and the tags_xxx variants (which reuses the grep code to perform searchs only within the tags file), will probably also break many other things I can't think of right now (needs to be tested with all other greppers, fish shell, ctrl-g switches and what not...) - needless to say this is not a rabbit hole I'd like to descened into :-)
Which brings me to a purposed solution, if you're using rg_glob=true you can solve your use case (and perhaps even enhance it) by using a custom rg_glob_fn function as discussed in https://github.com/ibhagwan/fzf-lua/issues/373#issuecomment-1079786343, you can modify the query, add -- or even decided on parsing new types of flags, craft your own rg command and return it to fzf-lua for execution.
Let me know what you think?

But as you can guess from the above code, this would break the command when a specific file ...

This is why I used -e for rg's pattern instead of usual first positional argument in my previous PR :)
rg and grep support this flag (ag isn't) and with it search_path can be first positional argument and everything works like usual, but as you said before this will reduce the flexibility of the plugin, so probably this isn't the best deal.
I'll try to play with rg_glob_fn. Thanks for the link to #373!

Unfortunately rg_glob_fn can't help in my case. As far as I understand fzf runs nvim --headless ... <query> before rg_glob_fn, which will be called inside nvim, so if <query> is something like -hello -- *.txt then it breaks and print Usage: ... for nvim, which means nvim thinks that it was called with -h option.
I use this function as rg_glob_fn, but it doesn't matter what is inside this function actually:
function M.glob_parse(query, opts)
  local function strsplit(inputstr, sep)
    local t = {}
    for str in string.gmatch(inputstr, "([^" .. sep .. "]+)") do
      table.insert(t, str)
    end
    return t
  end

  if not query or not query:find(opts.glob_separator) then
    return query, nil
  end
  local glob_args = ""
  local search_query, glob_str = query:match("(.*)" .. opts.glob_separator .. "(.*)")
  for _, s in ipairs(strsplit(glob_str, "%s")) do
    glob_args = glob_args .. ("%s %s -- ")
        :format(opts.glob_flag, vim.fn.shellescape(s))
  end
  io.write(("q: %s -> flags: %s, query: %s\n"):format(
    query, glob_args, search_query))
  return search_query, glob_args
end

@deponian, I’m not sure what’s the solution here then, unless we find a something that works for all use cases I can’t sacrifice other uses cases for the edge case of --fixed-strings (which isn’t that common).
Perhaps you create your own live grep’er using the fzf_live API?

Yeah, I don't see any easy ways to do it :(
Thanks for the idea with a shell command! I forgot about it.
I think we can close this issue :)

Np will close, I’d be interested to see your final solution with fzf_live once you finalized it :)

I'll share it if it any good :upside_down_face:

Hi @ibhagwan!
I guess I can share my results now.
After my unsuccessful experiments with shell command mode for fzf_live I've decided to go the other way. What if I don't build one complicated command, but switch from one to the other instead? Like when you switch between live_grep and usual grep with ctrl-g. With that thought in mind I built this configuration:
function M.setup_grep()
  local rg_opts_tbl = {
    ["--color"]         = "always",
    ["--max-columns"]   = "4096",
    ["--glob"]          = "'!.git/'",
    ["--column"]        = "",
    ["--line-number"]   = "",
    ["--no-heading"]    = "",
    ["--no-ignore"]     = "",
    ["--smart-case"]    = "",
    ["--hidden"]        = "",
    ["--fixed-strings"] = "",
  }

  -- merge these options without resetting to defaults
  require("fzf-lua").setup({
    grep = {
      fzf_opts = {
        ["--delimiter"] = ":",
        ["--nth"] = "4..",
      },
      -- rg_opts will be constructed from this table
      -- if you run one of the wrapper functions above
      -- (e.g. live_grep_native)
      rg_opts_tbl = rg_opts_tbl,
      -- these are defaults that will be used if you run
      -- functions like live_grep directly from require("fzf-lua")
      rg_opts = "--color=always --max-columns=4096 --glob='!.git/' " ..
                "--column --line-number --no-heading --no-ignore " ..
                "--smart-case --hidden --fixed-strings",
      actions = {
        ["ctrl-g"] = { switch_to_grep_project },
        ["ctrl-n"] = { switch_to_live_grep_native },
        ["ctrl-o"] = { switch_to_live_grep_glob },
        ["ctrl-h"] = { rg_toggle_option("--hidden") },
        ["ctrl-r"] = { rg_toggle_option("--fixed-strings") }
      },
      exec_empty_query = true,
    },
  }, true)
end

I created switch_to_*() functions to switch between different grep modes and rg_toggle_option() to change rg's options during the search. Now I can activate one of the commands and then change everything with my ctrl-* mappings.
To differentiate modes and see current active rg options I change prompt to Literal search: , when it's live_grep_native with --fixed-strings, Regex search with globs if it's live_grep_glob without --fixed-strings and so on. You can see full configuration here: https://github.com/deponian/vim.config/blob/9160c06ef3990f321bdb4ba9abcf1f88bff7d2ac/lua/deponian/fzf-lua/init.lua

Thanks again for your awesome plugin! The level of customization options is just enormous!

Thanks again for your awesome plugin! The level of customization options is just enormous!

Looks brilliant @deponian!
So am I getting this right that you’re using fixed strings only without globs or were you able to solve that as well and perhaps it wasn’t clear to me from the code?

No, I wasn't able to solve it :(
Now when I see error during live_grep_glob this means it's time to hit ctrl-r and disable --fixed-strings option. It's an ideal solution, but nothing is :upside_down_face: This is a good compromise for me, since I don't often use globes.

No, I wasn't able to solve it :(

I’ll take another stab at it and see if I can solve it without impacting other use cases.

I’ll take another stab at it and see if I can solve it without impacting other use cases.

This would be great, thank you!

https://github.com/ibhagwan/fzf-lua/commit/5b5e6359f9fbd0156b31e3046767c8dee3987cb7
@deponian, a bit of fine tuning to #777 and I was able to achieve the usage of  the literal separator -- without adversly affecting anything else, namely tags and grep'ing current buffer, also tested on fish shell and seems to be working as expected, hopefully I didn't miss anything.
Try this commit with --fixed-strings and lmk if this solves your issue?

@ibhagwan, it works flawlessly! Now I can remove -e hack from my configuration for live_grep_native and get both live_grep and live_grep_glob working with --fixed-strings! Thank you so much!

@ibhagwan, it works flawlessly! Now I can remove -e hack from my configuration for live_grep_native and get both live_grep and live_grep_glob working with --fixed-strings! Thank you so much!

That’s great :)

Hey! I was looking for a similar feature at one time, but unfortunately, I couldn't find a way to configure it by hotkey.
Now I have the following config
vim.keymap.set('n', '<leader>/', function () fzf_lua.grep({ no_esc = 2 }) end, {noremap = true, desc = 'FZF grep'})
This allowed me to search for both fixed strings (Grep For> -F 'only.fixed') and use other flags (Grep For> -uu 'smth in git-ignored'). But after these changes, it doesn't work anymore :(

Hey! I was looking for a similar feature at one time, but unfortunately, I couldn't find a way to configure it by hotkey. Now I have the following config vim.keymap.set('n', '<leader>/', function () fzf_lua.grep({ no_esc = 2 }) end, {noremap = true, desc = 'FZF grep'}) This allowed me to search for both fixed strings (Grep For> -F 'only.fixed') and use other flags (Grep For> -uu 'smth in git-ignored'). But after these changes, it doesn't work anymore :(

Unfortuntely, doing it the way you describe  is not possible anymore due to the separation by -- literals, but allow me suggest a different solution using rg_glob_fn, by hijacking the glob parsing funciton we can simiarly pass raw arguments (or have more complex customizations):
require("fzf-lua").setup({
  grep = {
    debug = true, -- remove this line once you're done testing
    rg_glob = true,
    rg_glob_fn = function(q, o)
      local search_query, args = q:match("(.-)" .. o.glob_separator .. "(.*)")
      return search_query, args
    end
  },
})

With the above customiztaion anything after o.glob_separator (by default space followed by two hyphens  --, customizable) will be appended as raw arguments to the rg command , so to craft a query with -uu you'd need to first type the query, then the glob separator followed by the raw arguments:

note the full command in the debug output:

The customization also affects grep and the ctrl-g switch to fuzzy, so you can also use it in the Grep For> prompt or directly using the search param:
:lua require("fzf-lua").grep({ search ="foo -- -uu" })

Will result in:

Same for -F|--fixed-strings :

added no_esc to avoid regex escape the hyphens

:lua require("fzf-lua").grep({ search ="--no -- -F", no_esc=true })

Oh, thank you very much! I missed that rg_glob_fn. I use that flag-passing and narrowing with "--iglob" quite often, so unfortunately not a win-win for me. The "rg_glob_fn" function you suggest requires typing "ctrl-g > -- --iglob 'some/dir/'" instead of just "ctrl-g > -- some/dir/". However, I appreciate the advice and maybe tinker with regex to make it fit for me.

Oh, thank you very much! I missed that rg_glob_fn. I use that flag-passing and narrowing with "--iglob" quite often, so unfortunately not a win-win for me. The "rg_glob_fn" function you suggest requires typing ctrl-g > -- --iglob 'some/dir/**' instead of just ctrl-g > -- some/dir/**. However, I appreciate the advice and maybe tinker with regex to make it fit for me.

You’re correct, using this customization requires escaping the globs as they’re being passed as raw args directly to rg, maybe you can customize it further and get to a happy medium? You can split all the args (after the glob separator) and if the literal doesn’t start with - you can shell escape it, for reference here’s the original function:
https://github.com/ibhagwan/fzf-lua/blob/b58799714513083bd646f0b716e29f224ebba48e/lua/fzf-lua/make_entry.lua#L236-L242
Note that in the customization you cannot use references to other modules (e.g. utils.strsplit) as the function is passed as bytecode to the external neovim process (multiprocess=true) so if you wanted to copy the above you’d have to create a local copy of strsplit.

@k33nice, I have another suggestion, first, update to the latest commit https://github.com/ibhagwan/fzf-lua/commit/cb0943a41df4bb59475a13c4dcc7256cc230a2e3 - the commit adds support for rg_glob_fn that is part of the opts (rather than a global setting), you can then keep using live_grep_glob normally and have a specialized leader-/ mapping that's using a custom rg_glob_fn:
require("fzf-lua").grep({
  rg_glob = true,
  rg_glob_fn = function(q, o)
    local search_query, args = q:match("(.-)%s%-%-(.*)")
    return search_query, args
  end
})

If you wish to have the exact setup as before you can modify the glob_separator to be a single space, that way rg_glob_fn will get called as soon as a space is detected in the query/search string. The full setup should look as follows and should replicate your existing no_esc=2 "hack":
vim.keymap.set('n', '<leader>/', function () require("fzf-lua").grep({
  debug = true,
  no_esc = true,
  glob_separator = "%s",
  rg_glob = true,
  rg_glob_fn = function(q, _)
    local args, search_query = q:match("(.*)%s(.*)")
    return search_query, args
  end
}) end, {noremap = true, desc = 'FZF grep'})

With the above customization, using the search query -uu foo results in:

Oh, that’s a nice one! Works like a charm. Thanks again man!

Oh, that’s a nice one! Works like a charm. Thanks again man!

Perfect :)
Ty for the update @k33nice!

@deponian, due to #794 I mady a small change so we can support both -e and --.
After you update to https://github.com/ibhagwan/fzf-lua/commit/903dc4d3bc60bdb47dacbc13b8d70ac6713be30e, in order to use --fixed-strings you nee to add -- to the end of your rg_opts or cmd.
require("fzf-lua").setup({
  grep = {
    rg_opts = "--color=always --max-columns=4096 --glob='!.git/' " ..
              "--column --line-number --no-heading --no-ignore " ..
              "--smart-case --hidden --fixed-strings --",
  },
})

@deponian, please use the latest commit https://github.com/ibhagwan/fzf-lua/commit/672fe2781fcfd527cc82125c5267113f544075d2, refactored the glob args code and smoothed out potential issues.

@ibhagwan Hi! Checked latest commit 672fe27, added  -e to my rg_opts builder and it's work like a charm, thanks!

@ibhagwan Hi! Checked latest commit 672fe27, added  -e to my rg_opts builder and it's work like a charm, thanks!

Out of curiosity, you prefer -e to -F --? I was under the impression you preferred fixed strings.

I use -F -e pattern. When -F is active pattern is literal, so -e is used for the same reason as --, which is searching for patterns that start with a dash.
man rg gives two examples with -e and --:

Makes sense, ty @deponian!


distorted paving mesh

Hello,
I have tried using the example 2 to generate unstructured quad mesh but unfortunately I am not getting the same mesh (comparing the following screenshot of the mesh I get and the screenshot from the README of the project):
Obtained mesh:

Mesh from the README file:

Note: I have tried both the main and develop branches but I am still getting the same issue
What I am doing wrong?

Hey there,
thank you so much for pointing out this error. I just reverted back to tag v1.2.1 (e37d966) and then I get the mesh that is shown in the README file.
It seems that the changes of commit 1cac9f9 have a negative effect on the smoothing of all-quad-meshes.
I will fix this in the coming days!
Greetings
Flo

Many thanks for the reply.

I fixed that bug in the main branch - the mesh of example 2 should now be ok.
Greetings
Flo

Thank you for the fix. I confirm that it works as expected.


Add com.snowplowanalytics.oss/oss_context/jsonschema/1-0-2

https://github.com/snowplow/iglu-central/issues/1344

Hi @jbeemster, do we have any incoming Iglu Central release ? If we have, would it possible to add this schema to that release ? If we don't, I can prepare one.

Not as far as I am aware @spenes - please do start preparing a new release if you would like a new schema published!

@jbeemster Opened the release PR. Could you give it a look, please ? Closing this one.


update minimist manually

Coverage remained the same at 33.681% when pulling a8525a6d5c2e0f1e19709a1feddba946b790f932 on suiguoxin:minimist into 2434a05e406be58ba16ff981ec3b06b0a390a40f on microsoft:master.

refer to https://github.com/microsoft/openpaisdk/pull/84/files


Remove az_ulib_ulog

Logging will be handled by az_log from now on

Let's regenerate the entire documentation at the end of the cleanup. For now, let's just make sure that the documentation in the code is correct.

Refers to: docs/index.html:1 in 701e680. [](commit_id = 701e680ce416189ae749969e97870bb319e7f41a, deletion_comment = False)

uLog: a small logging system with constant strings for reduced memory footprint.

Please remove.

Refers to: README.md:12 in 701e680. [](commit_id = 701e680ce416189ae749969e97870bb319e7f41a, deletion_comment = False)


volume 2 should be one folder

@msmock Volume 2 should not be multiple folders. All the content in volume 2 is independent numbered or alpha content, so it should not conflict.

done


Add accounts list

Add @fluentui/react-components
Add AccountsList component
FetchAccounts from /accounts api

I hadn't moved the loading piece into it's own component, because unless we wrap the state and such, it doesn't really dry up the code any more? As we're just using the spinner from fluentui. Thoughts?


microphone plugin: remove deprecated MediaStream.stop call

fixes #2168

Coverage remained the same at 82.454% when pulling fb9de3fad216427a1f5cc9af03e2453896fc6fd5 on mic-stop-2168 into d2005178dbd2deeb243b5362af1cafd1f3b29d66 on master.

Coverage remained the same at 82.454% when pulling fb9de3fad216427a1f5cc9af03e2453896fc6fd5 on mic-stop-2168 into d2005178dbd2deeb243b5362af1cafd1f3b29d66 on master.


Switch to Decimal.js and retire BigNumber.js

Blocked by #431

For future reference. We want decimal.js because

We use Decimal.js in the SDK
We can specifiy precision on Decimal.js
BigNumber have precision limitations

As a second step we could start working with BigInt, but it's wider PR as it involves changing the SDK too.


With Quark Bugs

Describe the bug
I found 2 bugs:

Right click harvest from Quark affects reeds and cacti. You can place reed or cacti, press right click and reed or cacti will return, but it will leave seeds and you can gather them. So this way you can produce seeds endlessly.

Mushroom Stew required only 2 mushrooms instead of 4 for a full cauldron if you exactly drop them into cauldron. Right clicking works well and consumes 4. It works fine without Quark. Idk if some cauldron recipies are bugged too. I tried it with Beetroot and Potato but it seems works fine.

Tested with clear client with only forge and these mods.
Versions:

Minecraft: 1.12.2
Mantle: <IP_ADDRESS>
Inspirations: 0.2.9

Other mods required

Quark: r1.6-179
AutoRegLib: 1.3-32

Closing as I have moved on from 1.12, was reported a bit too late to get fixed.


fix error: "bytes-like object is required not 'str'" when use compile with v2 functions

Thank you @Ianneee for the fix


RFC : Operator behaviour when a Topic is manually deleted from Kafka ?

With Kafka Operator 0.6.x there is support for Topic Management.
When we create a kafkatopic CR, the corresponding topic is created in Kafka.
When you delete the topic from Kafka by hand, the operator loops through an error :
kafka-operator-7bc9c8cd8f-l4qxk manager 2019-10-01T15:26:51.139Z	INFO	controllers.KafkaTopic.alerting/example-topic_sync	Syncing topic status
kafka-operator-7bc9c8cd8f-l4qxk manager 2019-10-01T15:26:51.233Z	ERROR	controllers.KafkaTopic.alerting/example-topic_sync	Failed to describe topic to update its status	{"error": "kafka server: Request was for a topic or partition that does not exist on this broker."}
kafka-operator-7bc9c8cd8f-l4qxk manager github.com/go-logr/zapr.(*zapLogger).Error
kafka-operator-7bc9c8cd8f-l4qxk manager 	/go/pkg/mod/github.com/go-logr/zapr@v0.1.1/zapr.go:128
kafka-operator-7bc9c8cd8f-l4qxk manager github.com/banzaicloud/kafka-operator/controllers.(*KafkaTopicReconciler).doTopicStatusSync
kafka-operator-7bc9c8cd8f-l4qxk manager 	/workspace/controllers/kafkatopic_controller.go:269
kafka-operator-7bc9c8cd8f-l4qxk manager github.com/banzaicloud/kafka-operator/controllers.(*KafkaTopicReconciler).syncTopicStatus
kafka-operator-7bc9c8cd8f-l4qxk manager 	/workspace/controllers/kafkatopic_controller.go:234

The kafkatopic CR is not enforced and the topic is not re-created.
Should the Kafka Operator enforce presence of topics in kafkatopic and re-create it ?
Note that even once the topic does not exist in Kafka, deleting the kafkatopic CR is well handled by the Operator :
Topic has been deleted, stopping sync routine

Just to add to the discussion - what should be the desired behavior when a cluster has topic deletion disabled and a KafkaCluster or KafkaTopic gets deleted? In the current state on master, if I am understanding sarama's documentation correctly, is that no error will be raised but the topic will remain in Kafka/Zookeeper. So an attempt to recreate the topic on a new or existing cluster using the same zookeeper will fail.

From Kubernetes perspective Kafka Topics and Users are external resources. In release 0.7.0 we removed the loop which periodically checked Kafka Topics. Since it is an external resource the operator will not receive any event in case of modification. I think this is the way to go.
It has some benefits like we don't need to implement an inner watcher logic for the Kafka resources.
Of course it has some drawbacks as well since if the user deletes a topic created through CR the operator will recreate it once an update happens on the related CR.

That's a fair move. Now we know how to handle such a situation. Thanks @baluchicken
Is this clearly stated in the docs ? (will check ASAP)
I think this can be closed then.
Thanks again for your great work


TypeError: e.isPersistent is not a function.

Description
I collected a crash log in my app
Content:
Exception Reason:Unhandled JS Exception:
TypeError: e.isPersistent is not a function. (In 'e.isPersistent()', 'e.isPersistent' is undefined),
...
scrollResponderHandleScrollBeginDrag@335:2652
scrollResponderHandleScrollBeginDrag@(null):(null)
...
Component: Animated.FlatList
React Native version:
"react-native": "0.63.3"

This tracker is intended to track RN issues, but not developer support. It looks like an issue in your code or third-party module you use, because RN doesn't have isPersistent method.

This tracker is intended to track RN issues, but not developer support. It looks like an issue in your code or third-party module you use, because RN doesn't have isPersistent method.

This tracker is intended to track RN issues, but not developer support. It looks like an issue in your code or third-party module you use, because RN doesn't have isPersistent method.

https://github.com/facebook/react-native/blob/v0.64.2/Libraries/Renderer/implementations/ReactNativeRenderer-prod.js#L597


Move coveralls into project_plugins

This PR moves coveralls to project_plugins so that users of pot do not download coveralls and it's dependencies (i.e., when adding pot as dependency from hex)

Pull Request Test Coverage Report for Build 63

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 61:
0.0%

Covered Lines:
140

Relevant Lines:
140

💛  - Coveralls

Great catch! I heartily endorse this. Thanks.


Document on developers.cardano.org

Add this service to developers.cardano.org

Looking at the requirements for listing, I don't think this app qualifies. It's more of an "end user" application than something used to build upon.


uncaught error with invalid model

Hi
If I put an invalid value in my form, I get a nice error message in my template and the submit function isn't called. But I also get an error message in the developer console:
bs-form.js:28 Uncaught (in promise) Error
    at BsFormWithChangesetValidationsSupport.validate (bs-form.js:28:1)
validate                            @ bs-form.js:28
Promise.then (async)
iterations                          @ backburner.js:18
[...stack trace...]

How can I catch this error? And why do I even have to catch this error? Or do I miss something else?
Thanks for your help and all your work

I've found the cause: I want a form, that is automatically submitted, when the input field loses focus.
For this I've build the form this way:
<BsForm @model={{changeset this.item this.validations}} @onSubmit={{this.saveItem}} as |form|>
    <form.element @property=... as |el|>
        <el.control {{on "focusout" form.submit}}/>
    </form.element>
</BsForm>

If I use a regular submit button, I don't get the error message in the console.
I could call another action in the focusout event handler, and check myself if the model is valid:
        <el.control {{on "focusout" (fn this.otherAction ...)}}/>

... but how do I get the changeset?
If I'm interpreting the documentation for the submit action correctly, my approach should (or could?) work. Ember-Changeset would be pretty ideal for this "no-save-button-style".

If I'm interpreting the documentation for the submit action correctly, my approach should (or could?) work.

Yes. It should work. Do you have time to investigate why it is not working? A reproduction would be very helpful to understand what is happening. Also would be interesting to know if it is specific to Ember Bootstrap Changeset Validations. Or if it's a general issue with Ember Bootstrap's validation API. And if it's happening always if using form.submit or specific to timing of modifiers / runloop. To be honest I have no clue yet, what might happen.

I have created a simple test repository: Auto Submit
In my actual project I'am still using Bootstrap 4, but as you can see Bootstrap 5 shows the same error.


Adds TextSExpWriter_1_0 implementation

Adds a text s-expression writer to the LazyTextWriter_1_0 implementation of LazyEncoder.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Closed in favor of #684, which is a superset of these changes.


onkeypress does not work in the following browsers: Chrome and Edge

what is working in mozilla,
Backspace
Backspace
Backspace
ArrowLeft
ArrowUp
ArrowDown
ArrowRight
ArrowLeft

@using Microsoft.JSInterop
Type here:

    @foreach (var key in keysPressed)
    {
        @key
    }

@functions {
List keysPressed = new List();
void OnKeyPressed(UIKeyboardEventArgs eventArgs)
{
Console.WriteLine(Json.Serialize(eventArgs));
keysPressed.Add(eventArgs.Key);
}
}

onkeypress does not work in the following browsers: Chrome and Edge
Backspace
Backspace
Backspace
ArrowLeft
ArrowUp
ArrowDown
ArrowRight
ArrowLeft

You are using the incorrect event:
"The keypress event is fired when a key that produces a character value is pressed down. Examples of keys that produce a character value are alphabetic, numeric, and punctuation keys."
You should use onkeyup or onkeydown for non printable characters. This is a browser thing, and not related to blazor.


murmur3 hashing code

Added the murmur3 hashing code, 0x22 as in the go-ipfs constant.

LGTM are you also adding it to js-multihashing-async, so we can actually generate these?

@dignifiedquire yes, doing that right now :)


OV2640 support

OV2640 support (thanks @Oitzu)
better configurability (options moved to sdkconfig, thanks @Oitzu)
20MHz XCLK support
automatic selection of image format (grayscale or JPEG) depending on camera
PGM output for grayscale mode

There are some occasional glitches when running in JPEG mode at 20MHz, which i think are related to signal integrity. I have a small board with OV5640 here, will see if shorter traces will make a difference. Otherwise i think it is enough of an improvement to merge, even with this caveat.


ci: add codecov

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@ebfd128). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #11   +/-   ##
=========================================
  Coverage          ?   98.36%
=========================================
  Files             ?        1
  Lines             ?       61
  Branches          ?       17
=========================================
  Hits              ?       60
  Misses            ?        1
  Partials          ?        0

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 ebfd128...57b1441. Read the comment docs.


Bump to latest kuisp

For Reasons(TM) grafana's container has been crashing on http access, not sure when that started.  It's likely something on our underlying kubernetes/coreos versions.  But hey, bumping the kuisp version seems to have fixed it, so, Progress!

LGTM


Bot Driver

Sub task of #17
Write the code necessary to control the bots.

[ ] Keyboard control to debug.
[ ] Update bot code.
[ ] Write the code to send raw messages to the bots.

Closed for archive.

Closed for archive.


Merge objects GameObject with state RuleList

Changes to consider to GameObject:

[ ] Fix method naming (this class also does not follow the conventions that it should)
[ ] Add method to set entire array of Rules at once

Changes to consider to RuleList:

[ ] This class should not have an RNG attribute called Score. Remove this and instead include the ability to include RNG for different attributes where it makes sense.

Never mind we're keeping score.
Add score to constructors.
Add getScore() method.


make Preview more human-readable

Previewing a resource, add option to view in more human-readable format like JSON-LD.
QUESTION:
what other formats would be easier for catalogers to read / review?

For starters, please implement the format supported by N3.js (https://github.com/rdfjs/N3.js), except N-Quads.

Initial implementation is a drop-down which allows the user to select the format.


Play two videos at the same time

Is it possible?

Hmm right now it’s not possible but I guess it’s doable. So then it would be:

Plot
Video (primary - controller)
Video (Secondary)
Right?

Yes, I think that would work for me.


y-axis labels are cut off

The longer y-axis labels are cut off, United States, United Kingdom in chart. I tried to set a margin, but nothing changed.

@shoukewei Can you provide us with some code so we can reproduce this problem?
The thing is: Plotly, the front-end rendering library, is in charge of all those layout details and leaves the user in charge of fine-tuning the final output. Margins should be the way to go... but you're saying it has no impact...
Thanks!

@FabienLelaquais Can you add an example in the chart documentation to show how to solve this issue?
Thank you.

I don't know if I'll do that.
That would be a setting in Plotly, and there is no reason to focus on this setting and not others.
I'll try to come up with a rationale for a go or a no go rapidly.

@shoukewei does this issue still persist?

Still an issue.
from taipy import Gui

data = {
    "Temperatures": [
        [17.2, 27.4, 28.6, 21.5],
        [5.6, 15.1, 20.2, 8.1],
        [26.6, 22.8, 21.8, 24.0],
        [22.3, 15.5, 13.4, 19.6],
    ],
    "Countries": ["United Kingdom", "United States", "Brazil", "Germany"],
    "Seasons": ["Winter", "Spring", "Summer", "Autumn"],
}

md = "<|{data}|chart|type=heatmap|z=Temperatures|x=Seasons|y=Countries|>"

Gui(md).run()

@jrobinAV  I have solved it can you assign to me

I can certainly do that! Thanks @mahanteshimath (and sorry @shoukewei...).

Please assign this issue to me @jrobinAV. (If  !stale or !closed)

It seems that Streamlit creates the correct width or height for the chart in every case according to a user

To solve this problem, you can set yaxis and xaxis in the layout with the parameter automargin=True for both axes. Then, in the layout, set autosize=True. This will make the height of the chart quite large. To adjust it, pass a class name to the chart function in Taipy and adjust the size to your preference, like this:
plotly:
any_figure = px.bar(df, x='any', y='any', orientation='v', title='any')
any_figure.update_layout(yaxis=dict(automargin=True), xaxis=dict(automargin=True), autosize=True)
in taipy:
tgb.chart(figure='{any_figure}', class_name='any_name')
Then style it in your CSS as follows:
.any_name {
display: flex;
position: relative;
height: vh; /* Replace with any desired height using vh units */
}
For example, you could use:
.any_name {
display: flex;
position: relative;
height: 35vh;
}

It is a layout issue that Taipy cannot solve.


[FEATURE REQUEST] Using arrow keys when viewing

Is your feature request related to a problem? Please describe.
I tried to use arrow keys just now when viewing album pics, but had to instead click off and then click a separate picture to view it.
Describe the solution you'd like
I'd like to be able to use arrow keys to browse an album, it'd be much easier to view things :)
Additional context
Add any other context or screenshots about the feature request here.

Implemented in https://github.com/chibisafe/chibisafe/pull/617


Use only selected sentences in source document

Thanks for sharing interesting works & source code.
In section 2.2, greedily selected sentences from a document highly correlated with reference summary. While other sentences are expected to have a low correlation with reference summary. Selected sentences exist for both training & inference.
I wonder what is the expected pros/cons when using 'keywords + selected sentences' as input of the BART encoder instead of 'keywords + all sentences'.
Do you have any ablation study results on this?

Hi,
This is a good point. To clarify, the input of the BART encoder is 'keywords + source article', where the keywords are from selected sentences -- the selected sentences are not used as direct input to the encoder.
Unfortunately, we don't have ablation results on not using selected sentences. I guess that removing the step of selecting sentences is probably fine at training time, while at inference time directly tag keywords from a long document may be too noisy.


error creating project on Windows

Hi,
There is some problem when I create some project.
I get:
npm ERR! Windows_NT 6.3.9600
npm ERR! argv "C:\Program Files\nodejs\\node.exe" "C:\Program Files\node
\node_modules\npm\bin\npm-cli.js" "install"
npm ERR! node v0.12.7
npm ERR! npm  v2.11.3
npm ERR! shasum check failed for C:\Users\nicolas\AppData\Local\Temp\npm-6056-
ee852a\registry.npmjs.org\browser-sync-ui-\browser-sync-ui-0.5.15.tgz
npm ERR! Expected: c3a684313aaa379783231d826a5c6d8e3b5da128
npm ERR! Actual:   4dafd1face4763f2e7127aefa8a5f056360a97de
npm ERR! From:     https://registry.npmjs.org/browser-sync-ui/-/browser-sync-u
0.5.15.tgz
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     https://github.com/npm/npm/issues
npm ERR! Please include the following file with any support request:
npm ERR!     C:\sites\dir2\npm-debug.log
Everything looks ready! Get started by running "gulp serve".

Hi @carlillos,
This is most likely due to some missing dependencies. Check out http://www.browsersync.io/docs/#windows-users for info on how to solve this issue.
Closing as this is specific to BrowserSync and now Yeogurt.


Construct the graph model from geometry files 👍

construct the graph with node and edges attributes representing the physical features of the network.

[x] #21
[x] #24
[ ] #25

There might be need to update hydromt-delft3dfm to get necessary physical parameters, e.g. diameters, slope and friction of the pipes, size and depths of the manholes and area and slope of the subcatchments.
There might be need to refactor previous scripts.
See also https://github.com/xldeltares/hydrolib-nowcasting/tree/master

--- Network configurations --------------------------------------------------------------------------
which data set to use [delft3dfm, infoworks]
source: delft3dfm
Path to data set root
source_dir: ../../network/Eindhoven
network component [edges, nodes]
can be either a list or a single
edges:

pipe
pump

nodes:

manholes
weirs

which columns to compute
target_variables:

Q(mm/d)
depth(m)
T(/yr)

clip negative values to zero for all variables listed below. Should be a list, even for single variables.
clip_targets_to_zero:

Q

Currently using hydrolib-nowcasting. So far it works. but requires refactoring. try to remove the dependencies on different workflows and call from hydromt-delft3dfm directly.
maybe a todo for later: maintain of network module to latest hydromt --> maybe good to do that in global vector roads work.


chore(core): upgrade otaScript

Checklist:

README.md includes:

[ ] The details of signing data and transaction data, this includes parameters.
[ ] Official docs or white paper.
[ ] Website or api can query assets and broadcast transaction.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

Task linked: CW-19858 Implement signMessage method


Construct HashableNamespace Parameter

Replace usage of DictParameter to pass around parameters with your HashableNamespace implementation. That way is cleaner. You need to write a way to serialize HashableNamespaces to JSON and then decode them.

Replaced need for this by refactoring via 144d358


ci: Remove nightly clippy

This seems to be broken, it was added in response to this issue  but it probably needs to be removed if it is going to occasionally break CI.

yup


Add np padding support

JetStream are jax or pytorch frame independent inference stack. It's binding to jax array in current code base.
Please support np padding.

Finished.


Add two new columns to model outputs: safety_prediction, manual_override_active

This will require modifying a lot.
Basically, the idea here is that the model outputs should also include the manual overrides.

The home page's function index() should not do the job of joining the manual overrides.
safety_prediction and manual_override_active should be in detailed_outputs(), but neither of these columns should be red/blue even though they will say "true" and "false." They should not be colored because they are not representations of flags.

The model outputs table itself should not include it, just the functions that return the model outputs. (This requires some namespace finesse because "model outputs" will sometimes refer to a table that has the manual overrides in it and sometimes not, so we should distinguish these concepts somehow).

I'll do this one.

Not sure about this one as a thing we should do. It becomes really complicated when you mix it in with the manual overrides by boathouse instead of by reach. So I'm gonna make this low priority.

If we do something like this, there are better ways to approach it. Locking.


langchain integration

Hello, I'm trying to use the langchin integration but I cannot figure out how to use it, I'm following some examples in langchain:
import { LLM } from "llama-node";
import { LLamaRS } from "llama-node/dist/llm/llama-rs.js";
import readline from "readline";
import fs from "fs";
import path from "path";
import { SerpAPI } from 'langchain/tools';
import {initializeAgentExecutorWithOptions} from 'langchain/agents';
import { Calculator } from 'langchain/tools/calculator';
import { LLamaEmbeddings } from "llama-node/dist/extensions/langchain.js";

const SERPAPI_KEY = '';

const model = path.resolve(process.cwd(), "./ggml-vic7b-q4_1.bin");
const llama = new LLM(LLamaRS);
llama.load({ path: model });

const tools =[
    new SerpAPI(SERPAPI_KEY,{
        hl:'en',
        gl:'us'
    }),
    new Calculator(),
]

const executor = await initializeAgentExecutorWithOptions(tools, llama, {
    agentType: 'chat-zero-shot-react-description'
});
console.log('initialized')
const ret = await executor.call({
    input: "Who is Olivia Wilde's boyfrient? What is his age raised to the 0.23 power?"
});

console.log('ret:', ret.output);

but I got:
TypeError: this.llm.generatePrompt is not a function
    at LLMChain._call (file:///Users/lvx/dalai/node_modules/langchain/dist/chains/llm_chain.js:80:48)
    at async LLMChain.call (file:///Users/lvx/dalai/node_modules/langchain/dist/chains/base.js:65:28)
    at async LLMChain.predict (file:///Users/lvx/dalai/node_modules/langchain/dist/chains/llm_chain.js:98:24)
    at async ChatAgent._plan (file:///Users/lvx/dalai/node_modules/langchain/dist/agents/agent.js:197:24)
    at async AgentExecutor._call (file:///Users/lvx/dalai/node_modules/langchain/dist/agents/executor.js:82:28)
    at async AgentExecutor.call (file:///Users/lvx/dalai/node_modules/langchain/dist/chains/base.js:65:28)
    at async file:///Users/lvx/dalai/agent.js:35:13

i understand that is because the LLM model does not have this function is there any method to call it or do I have to create a translation class?

by now you have to manually adapt generate function to langchain.

I'm not as familiar with gpt4all, but noticed they are adding langchain support, maybe there is some overlap: https://github.com/hwchase17/langchainjs/pull/1204


calamari: Move check out of the ifdefs

These SOL_NULL_CHECK should have been moved out of the ifdefs
when new_options and free_options were assigned to type.
Signed-off-by: Anselmo L. S. Melo<EMAIL_ADDRESS>

+2

Applied, thanks


[Suggestion] Dither blocks obstructing camera from viewing the player

It would be very cool, useful, and less annoying if there could be something to make the blocks between the camera and the player:
Become transparent or dithered
or completely stop rendering
The mod is unusable in forests, caves, etc because of this, so it would be life - changing, since no other mod does it

This is similar to my earlier suggestion
https://github.com/DimasKama/OrthoCamera/issues/16 !


ABP with allow property runes is no longer working

when the PF2E Variant Rule
Automatic Bonus Progression set to ABP: Allow Property Rune
Items that can have property runes no longer have an interface for them (was working sometime before 5.10.5, is not working in 5.11.2)
remove of the variant rule brings back the interface, but then again requires manual control of Fundamental Runes

Duplicate of https://github.com/foundryvtt/pf2e/issues/12279. Will be ok next hotfix


Support for Firebase Cloud Messaging

I'm trying to get Firebase Cloud Messaging to use the service worker that is built by sw-precache, but it doesn't seempossible to do so if importScripts is the only option available to extend the worker. A glance at the dev tools shows that the script I'm importing that contains the Firebase initialization is running as its own service worker. Through ImportScripts, I can't seem to pass my sw-precache ServiceWorker to the messaging.useServiceWorker() function, which leaves the Firebase Messaging looking for a nonexistant firebase-messaging-sw.js file.

Actually, I've probably just got it all set up improperly. I see now that I can even edit the lo-dash template itself to add the code, and the same issue arises.


[pyopensci-review] Pip installation issue

Currently the pip installation is raising the follow error:
21:09 $ pip install movingpandas
ERROR: Could not find a version that satisfies the requirement movingpandas (from versions: none)
ERROR: No matching distribution found for movingpandas

I'm leaning towards removing pip install instructions.
The error above appears if pip install movingpandas is called in a Python <= 3.6 environment. So this issue is related to #57
Pip install on Ubuntu with Python 3.7 fails on cartopy with Proj 4.9.0 must be installed in Conda environment "movingpandas-dev".. This seems to be a known issue and conda install is recommended instead of pip: https://stackoverflow.com/questions/53697814/using-pip-install-to-install-cartopy-but-missing-proj-version-at-least-4-9-0

I was able to install the movingpandas package from PyPI (on Linux, and not using conda or anaconda).  Some packages needed custom installations.  The instructions that I used are here: https://devel.ilikecarrots.com/howto-movingpandas.html

Thank you for the insights @wendellwt! The dependency situation seems very dynamic, particularly with respect to proj versions right now. This makes it hard to keep up.

I spoke too soon.  Proj-5 compiles and links, but doesn't run ("no such column: publication_date").  It looks like projpy needs Proj-6 or later.  Revised instructions are at: https://cherry.ilikecarrots.com/howto-movingpandas.html
(A while back I tried Anaconda, but it seemed like it took over my entire machine.  Also, I don't have admin privs on several of the machines I use.  That's why I often have to compile everything from sources and install to a user-writable directory.)

I spoke too soon ^2.  GeoPandas 0.7 is incompatible with MovingPandas.  GeoPandas 0.6 must be used.

MovingPandas master should already support GeoPandas 0.7. (Changes include https://github.com/anitagraser/movingpandas/pull/50.) I'm working on a new release.

Yes.  The current git repo works fine with GeoPandas 0.7.  Sorry about that.

v0.3rc1 has landed on conda-forge https://anaconda.org/conda-forge/movingpandas


Make this project better findable

Currently this project is hard to find with a search engine. When I search the following queries in Google, this project does not show up on the first page:
c# check API changes
c# api backwards compatibility tool
c# verify compatibility of a public APIs
It would probably be good to update the project documentation to have some of these keywords in. It would also be nice to add some referrals to existing stackoverflow topics;
https://stackoverflow.com/questions/3090792/tool-to-verify-compatibility-of-a-public-apis
https://stackoverflow.com/questions/2377855/tool-for-backwards-compatibility-for-the-c-net-api
https://stackoverflow.com/questions/1456785/a-definitive-guide-to-api-breaking-changes-in-net
https://stackoverflow.com/questions/8279299/how-can-i-test-the-backward-compatibility-of-api-between-net-assemblies/8279436

Good ideas! Feel free to do so. Unfortunately, my time is very limited to work on this project at the moment.


Add translation process automation

Refers to: #194.
Please, review.

Thank you for this awesome contribution -- I'll test it out within a day or two and likely merge as is 👍

@whyboris, I'm using Manjaro Linux. I'll start to test your OS X solution for sed. For sake of portability, it could be handy to add a function to get system's label (bash's uname* command). Thanks for reviewing!

I already tried my solution on a Mac -- it worked :)
I think it's just worth seeing if including the "" still works on Linus -- if it does, we may as well merge it in 👌
I always feel uneasy whenever I request any edits to a PR -- a person (you in this case) have already done some awesome work, and to request more feels like in poor taste 😅
I'm happy to merge the feature as is too -- no pressure to do more work 🙇 🙇 🙇

Hey, @whyboris, I just added a function to check whether it is running on a Mac or Linux (and could be added more OS cases). If it's a Linux, use Linux's sed otherwise it will use Mac's one

I'm unsure what the problem with Mac is -- but the script generates new files on my machine:
        "../components/common/app-state.ts\"\""
        "../components/home/home.component.html\"\""
        "../components/home/home.component.ts\"\""

Note the \"\"" at the end. The files look like what we want, but the stuff at the end is likely some other OS X bug 😞
ps - the OS detection code is in the first block, but the other sed commands would need the same treatment. Perhaps this would be easier? Use your code somewhere above to define a new variable:
get_os
if [ $result = 'linux' ]; then
  SED_HACK=''
elif [ $result = 'osx' ]; then
  SED_HACK='""'
else
  echo 'System not supported'
  exit 1
fi

Now insert the SED_HACK into the command, e.g.
sed -i $SED_HACK "/export type SupportedLanguage/ s/;$/ | '$NEW_LANG_CODE';/" ../components/common/app-state.ts

So on Linux this hack would insert an empty string, not causing trouble, while on Mac we'll have the desired "" 🤔

Automation++ 😄
I wonder if this would be easier if we had a separate file for the LANG_CODES, and dynamically populated them at runtime in each location, so we only need to edit one file?
Also, this won't run on Windows by default, which is why I used node for the original script. not sure if that's an issue or not? 🤔
Great work @Lakshamana 👍

get_os
if [ $result = 'linux' ]; then
  SED_HACK=''
elif [ $result = 'osx' ]; then
  SED_HACK='""'
else
  echo 'System not supported'
  exit 1
fi

Now insert the SED_HACK into the command, e.g.
sed -i $SED_HACK "/export type SupportedLanguage/ s/;$/ | '$NEW_LANG_CODE';/" ../components/common/app-state.ts

@whyboris, yeah, that approach would be easier. I was thinking in such a way to do this but didn't figure out a way to do this properly.

Thanks, @cal2195. If you prefer I could migrate my code to node. However, Windows 10 can run bash, though it is a beta feature (and it's not a native windows application). Since I don't use windows at all, I don't know if all bash commands are available for it to use.

@Lakshamana If this is working, I'd say merge as is for now! Just something we could do in the future, especially if we wanted to add more automation to this/other tasks! 😄
WSL is certainly good, and should basically appear as a Linux system, but if you're doing VHA dev on Windows, you already have node, so it's one less thing to get installed! 😅

@cal2195, totally agree on dev dependencies for improving dev experience. This works in my machine. So it's ready to merge (for now, I'm just waiting for tests in OS X).

This worked beautifully on my Windows machine ✅
On Mac it still created 3 new files rather than update the old ones (see https://github.com/whyboris/Video-Hub-App/pull/195#issuecomment-509895401).
I'll merge as is -- it's probably just a few characters away from perfect and I'll get it to work and patch it in the near future 🤝
Thank you again for your work @Lakshamana and thank you for joining in @cal2195 🙇