Allowing Access to Java Method in GraalVM Context

I'm trying to grant access to a Java method jMethod(x,y) to a GraalVM Context like so:
Context SCRIPT_CONTEXT = Context.newBuilder("js").allowHostAccess(HostAccess.ALL).build();
SCRIPT_CONTEXT.getBindings("js").putMember("jMethod", (Function<Integer, Integer>)this::jMethod);

This seems to work just fine, but only if I include allowHostAccess(HostAccess.ALL).  This seems a little overbroad (and is explicitly recommended against in the documentation), since all I want the script to be able to access is that one particular method.  However it seems like the recommended solution per #152 may be ever broader, suggesting that allowAllAccess(true) be used.
Is there any way to give JavaScript access to just a single method without also giving it unrestricted access to other parts of my Java code?

Hi @Sammy1Am
you can either use HostAccess.EXPLICIT and annotate the relevant method with @Export (https://www.graalvm.org/sdk/javadoc/org/graalvm/polyglot/HostAccess.html#EXPLICIT), or define your own HostAccess using HostAccess.newBuilder() (https://www.graalvm.org/sdk/javadoc/org/graalvm/polyglot/HostAccess.html#newBuilder--).
Best,
Christian

I did try HostAccess.EXPLICIT, but got the same error as when I didn't add
any additional access.
I played around with the HostAccess builder a bit, but couldn't find
anything that seemed to have any effect.  Are there any other items I need
to grant access to (the containing class, etc.?) or should a single @Export
on the method be enough?
On Thu, Aug 13, 2020, 02:34 Christian Wirth<EMAIL_ADDRESS>wrote:

Hi @Sammy1Am https://github.com/Sammy1Am
you can either use HostAccess.EXPLICIT and annotate the relevant method
with @Export (
https://www.graalvm.org/sdk/javadoc/org/graalvm/polyglot/HostAccess.html#EXPLICIT),
or define your own HostAccess using HostAccess.newBuilder() (
https://www.graalvm.org/sdk/javadoc/org/graalvm/polyglot/HostAccess.html#newBuilder--
).
Best,
Christian
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/graalvm/graaljs/issues/334#issuecomment-673372989,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADSF6EHVCEQS633VOGGUTLSAOXQXANCNFSM4P5TRQ3Q
.

Hi @Sammy1Am
yes, I just realized while trying out myself. Maybe the underlying issue of #152 is broader and still exists. My assumption is that should work without HostAccess.ALL. I am investigating.
-- Christian

Thanks; let me know if there's anything else I can provide / test that
would be helpful.
For what it's worth, the actual source (method and calls to eval) is in
this file here, in case that provides any needed context:
https://github.com/Sammy1Am/Moppy2/blob/master/Java/MoppyLib/src/main/java/com/moppy/core/events/mapper/MIDIScriptMapper.java
On Thu, Aug 13, 2020, 02:45 Christian Wirth<EMAIL_ADDRESS>wrote:

Hi @Sammy1Am https://github.com/Sammy1Am
yes, I just realized while trying out myself. Maybe the underlying issue
of #152 https://github.com/graalvm/graaljs/issues/152 is broader and
still exists. My assumption is that should work without HostAccess.ALL. I
am investigating.
-- Christian
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/graalvm/graaljs/issues/334#issuecomment-673378178,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADSF6D72C3MSEAZ4OVUEGDSAOY3DANCNFSM4P5TRQ3Q
.

Hi,
so, the problem is that the Lambda generated interferes here. You have put @Export on the function, but not on the lambda created by (Function<Integer, Integer>)this::jMethod syntax.
It will work if you do it like this:
ctx.getBindings("js").putMember("jMethod", new MyFunction());
ctx.eval("js", "print(jMethod(43));");

public static class MyFunction implements Function<Object,Object> {
    @Override
    @Export
    public Object apply(Object arg) {
        System.out.println("called MyFunction.jMethod with argument "+arg);
        return null;
    }
}

Best,
Christian

Ahhh, okay, that makes perfect sense.  Thank you!
Just out of curiosity, since it seems like @Export is intended to work directly on methods, I could also get it working like this, right (maybe with also an @Export on the enclosing class?
ctx.getBindings("js").putMember("jClass", this);
ctx.eval("js", "print(jClass.jMethod(43));");

@Export
public void jMethod(int x) {
    ... implementation....
}

I'd be exposing the whole class to JavaScript instead though and would need to call it as jClass.jMethod() so it doesn't exactly fit my use case this time, but just want to better understand what the traditional use case looks like.
Thanks again for your help!

Hi @wirthi
I'm facing a similar issue but with a different setup. Instead of the Function Interface I'm implementing my own custom interface since the return value of the JS function I'm calling returns more than one argument. The implementation is more or less the same as you posted about 2 weeks ago. However, I'm still getting an
execute on JavaObject[graal.GraalScriptExecution$EmitReturn@1fc32e4f (graal.GraalScriptExecution$EmitReturn)] failed due to: Message not supported.

error when executing the JS function.
My setup is as below:
public void executeScript() {
        Context jsContext = Context.newBuilder("js")
                .allowHostAccess(HostAccess.EXPLICIT)
                .build();
        Source source = Source.newBuilder("js", Utils.script, "script.js").buildLiteral();
        jsContext.eval(source);
        jsContext.getBindings("js").putMember("emit", new EmitReturn());
         ...
    }

    public static class EmitReturn implements utils.EmitReturn<String, Map<String, Object>> {
        @Override
        @HostAccess.Export
        public void emit(String arg1, Map<String, Object> arg2) {
            log.info("{}, {}", arg1, arg2);
        }
    }
}

I have a working version which is passing a utils class to the script and then calling utils.emit() instead of the Java Method directly, but the above solution would be way cleaner since I don't have to eval a wrapper function before actually calling the function.
Cheers and thanks,
David

Hi @Biglr
I've tried to replicate your setup, and the following example is working for me. But of course my version is a bit simplified; can you maybe extend it to the point where it is not working any longer?
import org.graalvm.polyglot.Context;
import org.graalvm.polyglot.HostAccess;
import java.util.Map;
import java.util.HashMap;

public class Example2 {
    public static void main(String[] argv) throws Exception {
        Context jsContext = Context.newBuilder("js")
                .allowHostAccess(HostAccess.EXPLICIT)
                .build();
        Map<String,Object> myMap = new HashMap<String,Object>();
        myMap.put("foo", "bar");
        jsContext.getBindings("js").putMember("myMap", myMap);
        jsContext.getBindings("js").putMember("emit", new EmitReturn());
        jsContext.eval("js", "emit.emit('test', myMap);");
    }

    public static class EmitReturn
    {
        @HostAccess.Export
        public void emit(String arg1, Map<String, Object> arg2) {
            System.out.println("emit called: arg1="+ arg1 +", arg2="+ arg2);
        }
    }
}

One thing I can think of (because it happened to my while trying): you need to call emit.emit(...), not just emit(...) (which would trigger exactly the error message you included).
Best,
Christian

Hi @wirthi
Thank you very much for your fast reply!
The setup you described is exactly what I have in place as working version :wink: however our requirements are that only emit(...) should have to be called from within the scripts. So I have added an eval call which adds this "wrapper" function to the original script on runtime.
I have good news though! I was able to get my described setup working by investigating the Function Interface and adding the @FunctionalInterface annotation to my custom Interface as well. This seems to have done the trick and everything now works like a charm!
Cheers and thanks again,
David


Preferred Language "de-DE" (in Openstreetmap.org/My Settings/Preferred Languages) is not recognised as German while "de" is

User Interface is german in the first line und Englisch in der zweiten Zeile.

.
Chromium Version 37.0.2062.120 Built on Debian 7.6, running on Debian 7.8 (281580) (64-bit)
Iceweasel Version 31.4.0, running on Debian 7.8 (281580) (64-bit)
Safari on iOS 8.1.3

Same for Me.
Chrome  40.0.2214.111 (64-bit) on Mac OS-X 10.9.5.
Safari Version 7.1.3 (95<IP_ADDRESS>) on Mac OS-X too.

I have the same problem with Firefox 35.0.1 on Windows 7.

If   I set    Openstreetmap.org .  "My Settings" / "Einstellungen" .

"Preferred Languages" / "Bevorzugte Sprachen:"   to   "de-DE"

THEN   I get  similar result.
But with "Preferred Languages" = "de"  is OK.
Can you provide more information?

Your openstreetmap user settings  : "Preferred Languages"  = ?
Your browser settings :
"Language:" = ? ,
"System Language:" =?
"User Language:" =?

My browser language is "DE".
My OpenStreetMap preferred Languages is "de-DE,de,en-US,en" (don't ask my why :-))

switching to the preferred language "de" will work for ID.
I do not know, if they are some implications of this change to other OSM App. (I do not know the difference between DE-DE and DE, either)

My OSM-Language-Settings were "de-DE,de,en-US,en", After changing to "de,de-DE,en-US,en" iD shows the German menu.

It looks like the last time I ran make translations it pulled down an empty
de-DE.json file (and others) from Transifex.
The quick fix is to just remove the empty files.. The better fix is maybe: if detecting a culture code (e.g. de-DE), also load in the language code (de), and use it as a fallback in the t() function.
@jfirebaugh thoughts?

Ideally we would follow the user's preferred fallback order, e.g. de-DE,de,en-US,en, but we might have to balance that with not making too many requests for individual files.

The rails site is only sending us the first one from the list...
(though we could change that)

It is probably better for you to be consistent with what we do for the web site code though - we select the primary locale by matching against the user's list but once we have done that we don't do any fallback along that list. We do however fallback from de-DE to de.
Falling back along the user's language list could give really weird results if they have listed a number of unrelated languages that they happen to speak.
So basically we choose a language, and set it in I18n.locale and then relay on I18n::Backend::Fallbacks to do the fallbacks, which just falls back to parent locales basically (one exception is that we add a mapping from no to nb to cope with Norwegian weirdness) and then eventually to english.

For now I have removed all the empty locales from iD in the web site repository, namely:

vendor/assets/iD/iD/locales/de-DE.json
vendor/assets/iD/iD/locales/en-DE.json
vendor/assets/iD/iD/locales/gan.json
vendor/assets/iD/iD/locales/hi-IN.json
vendor/assets/iD/iD/locales/km.json
vendor/assets/iD/iD/locales/km-KH.json
vendor/assets/iD/iD/locales/nn.json
vendor/assets/iD/iD/locales/ru-RU.json
vendor/assets/iD/iD/locales/sq-AL.json
vendor/assets/iD/iD/locales/sq.json
vendor/assets/iD/iD/locales/sr-RS.json
vendor/assets/iD/iD/locales/zh-CN.GB2312.json

I've removed de-DE, en-DE, km-KH, ru-RU, sq-AL, sr-RS, and zh-CN.GB2312 on Transifex -- those were probably a misunderstanding. Keeping the empty language code ones, someone might actually want to start translating those. We might want to drop empty translations in the processing script on the iD side though.

Now the behavior is as expected on www.openstreetmap.org. Just tested for de-DE. Who does close this issue?

Now the behavior is as expected on www.openstreetmap.org. Just tested for de-DE. Who does close this issue?

The issue can stay open, because removing the empty translation files is not a permanent fix.  (It may pop up again next time we release).

I have created a simple statisctics from changesets for iD Editor - locale :
https://gist.github.com/ImreSamu/aebfa8316b32941e562a


fix: make EventHandler better handle mouseenter/mouseleave events

This PR makes the EventHandler handle mouseenter and mouseleave more like their native counterparts.
Background
Currently the EventHandler is implemented to simply map mouseenter to mouseover and mouseleave to mouseout events - see this comment https://github.com/twbs/bootstrap/issues/31646#issuecomment-761166484
The problem is that mouseenter and mouseleave behave different to some extent but this wasn't respected yet. As such there are situations e.g. within the tooltip plugin where one does expect nested elements not to trigger the bound event handlers.
With the PR https://github.com/twbs/bootstrap/pull/33289 by @RyanBerliner this is addressed although in my opinion (and I assume Ryan agrees to some degree) some general fix to the EventHandler module would be more appropriate.
TBD
With this implementation the targets are not rewritten. In some situations e.g. when the window loses focus and some nested element is hovered when it receives back focus, the targets differ between mouseenter and mouseover.
Also like asked here https://github.com/twbs/bootstrap/discussions/33220 - are there any reasons as to why mouseover/mouseout was used in the first place?

tl;dr I think that the custom event mapping can be removed altogether. Otherwise if there is a reason it exists, it would be beneficial to add documentation inline as to why.

I think it should be understood why these 2 custom events were replaced in the first place. I've done some digging and found the origin of this event swapping to be this commit, which is converting the carousel from jquery to vanilla js. I'm assuming here that the reason these events were treated as "custom" were because of jquerys claims about implementing these events themselves. This is an assumption, so there's a good chance I'm wrong, though reading this on jquery's website is the only thing I can find that would explain to me why its treated as "custom" in bs5.

The mouseenter JavaScript event is proprietary to Internet Explorer. Because of the event's general utility, jQuery simulates this event so that it can be used regardless of browser.

Again, I'm assuming that this is what caused the reaction to treat these as custom events. I'd bet that the carousel was the first component with these mouseenter/mouseleave events to be converted to vanilla js, so the event swapping was set up at this point. Then, the fact that the actual behavior was never implemented correctly wasn't caught because in the case of the carousel, it doesn't actually matter! It still works no matter what mouse event is used.
From what I can tell jquery's claim about these events are no longer correct (at least for the browsers that bs5 supports), so I don't think any custom event mapping is necessary, and this PR can simply be removing the mapping altogether.

@RyanBerliner nice investigation there! And your conclution about the why seems reasonable. :+1:
Initially, my naive me had the same thoughts.😊
But there is a problem though by simply removing the event mapping. With capturing listeners, that we use for our delegated handlers, the mouseenter and mouseleave events behave a bit different - like a mixture of mouseover/mouseout and mouseenter/mouseleave. (Some codepen to check for yourself.)
So they need some sort of special treatment I guess. I can try to optimize it a bit, so to use the native events and only fix the behaviour in the delegated version.
WDYT?

Gotcha - I hadn't considered delegated mouseenter/mouseleave events (in any event phase really, capturing or not). I agree that if you want delegated mouseenter/mouseleave handlers to act in a meaningfully different way than mouseover/mouseout, you can't rely on the events native implementation alone. Disregard my previous notes! And if my understanding of the goal here is correct, I think adding a short message like this near the customEvents = {} assignment would provide a lot of clarity.

If we want delegated mouseenter/mouseleave handlers to act in a meaningful way, we can't rely on their native implementation alone.


KAFKA-8713: JsonConverter NULL Values are replaced by default values even in NULLABLE fields

https://issues.apache.org/jira/browse/KAFKA-8713
This should not be merged without an approved KIP.
Update Mar 18 2020
Create KIP-581: Value of optional null field which has default value
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@rhauch Review please

@guozhangwang Review please

@guozhangwang @rhauch
Any concerns about this PR? Please let me know.

There is some discus about it: https://issues.jboss.org/browse/DBZ-1064

@omkreddy @rhauch @guozhangwang  Could you please kindly review this PR and confirm it's a bug or actual by design? This PR was stalled few months with none response. As mentioned above, the discussion in project debezium is here: https://issues.jboss.org/browse/DBZ-1064 we think it's a bug. If you think the behavior now is really by design, I just close this PR.

cc @rhauch @kkonstantine

Thanks, @pan3793. This PR looks pretty good, but as I just commented on KAFKA-8713 this will need a KIP because of the potential backward compatibility concerns.

@rhauch thanks for your reply. I'd like to follow the process to come out a KIP to push on this issue.

I do a new implement at https://github.com/apache/kafka/pull/8575 follow the KIP-581, this PR is deprecated, will close it soon.


Update http4s-client, http4s-dsl, ... to 0.21.18

Updates

org.http4s:http4s-client
org.http4s:http4s-dsl
org.http4s:http4s-okhttp-client

from 0.21.7 to 0.21.18.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

labels: library-update, semver-patch

Superseded by #191.


[Bug]: css not loading properly

Contact Details
<EMAIL_ADDRESS>Bug Description

It seems that the CSS is not working properly, and all images are too big. I managed to fix some by adding CSS code directly to the JSX file, but it looks ugly. I also manually fixed the icons that were too big. As you can see in the picture, it appears that the CSS might be missing or something else might need to be installed. Thank you for your help.
P.S. I'm still learning.
Environment
Linux
Reproduction Steps
No response
Browsers
No response
Relevant Log Output
No response

Hi everyone,  can somebody help please,  thank you.

Can you share the screenshot of network tab after loading the website

Dear Hassan, thank you for your help,


Bug on search button for NHS.UK live site

Bug Report

What is the issue?
Search button returns an error when selected on the search results page for mobile

What steps are required to reproduce the issue?
Go to NHS.UK ---> Select search icon ----> Type any thing in search -----> Once on results page -----> select search icon at the top again ------> ERROR

What was the environment where this issue occurred?

Device: iPhone XR
Operating System: iOS 13.6
Browser: Safari
Browser version: N/A
NHS.UK frontend package version: N/A
Node version: N/A
npm version: N/A

Is there anything else you think would be useful in recreating the issue?

Hey @Tosin-88 thanks for raising this, it looks like this was an issue with the Search page implementation rather than the frontend library component. It looks like it has been fixed on the website now.


ORecordLazyList.get(index) started to return ORecordId instead of ODocument

OrientDB Version: 3.1.4
Java Version: 8
OS: Windows, Linux
Some of our tests started to fail after switching from 3.1.3 to 3.1.4 due to ClassCastException.
After review we have figured out that problem is in ORecordLazyList: first item is ODocument, but all subsequent are just ORecordId (but a corresponding  ODocument does exist)
Expected behavior
ORecordLazyList should return ODocument
Actual behavior
All items after first one are ORecordId
Steps to reproduce
Not fully sure how to reproduce from scratch. The following our testcase stopped working: https://github.com/OrienteerBAP/Orienteer/blob/future/orienteer-notification/src/test/java/org/orienteer/notifications/test/TestNotificationLifecycle.java

@tglman git bisect between 3.1.3 and 3.1.4 show that the issue was introduced by the following commit: https://github.com/orientechnologies/orientdb/commit/6265642427def0a8c474c66e1619fe7b658c8815

@tglman, @laa, @luigidellaquila, OrientDB team, do you have ideas how to fix that? 3.1.4 is unusable due to this issue...

Apparently, the reason is in fact that mentioned above commit doesn't affect logic of iterator(). Defect is pretty visible in the following testcase:
@Test
public void testLinkListAndODocumentConversion()
{
	ODatabaseSession db = wicket.getTester().getDatabaseSession();
	OSchemaHelper helper = OSchemaHelper.bind(db);
	helper.oClass("TestODocumentConverstion")
			.oProperty("name", OType.STRING)
			.oProperty("items", OType.LINKLIST);
	ODocument mainDoc = helper.oDocument("name", "document").saveDocument().getODocument();
	helper.oClass("TestODocumentConverstionItems")
		  	.oProperty("name", OType.STRING);
	ODocument doc1 = helper.oDocument("name", "item1").saveDocument().getODocument();
	ODocument doc2 = helper.oDocument("name", "item2").saveDocument().getODocument();
	ODocument doc3 = helper.oDocument("name", "item3").saveDocument().getODocument();
	List<ODocument> items = Arrays.asList(doc1, doc2, doc3);
	mainDoc.field("items", items);
	mainDoc.save();
	db.commit();
	db.begin();
	mainDoc = (ODocument)mainDoc.reload();
	items = mainDoc.field("items", List.class);
	items.iterator().next(); // IMPORTANT LINE
	assertTrue(items.get(0) instanceof ODocument);
	assertEquals(doc1, items.get(0));
	assertTrue(items.get(1) instanceof ODocument);
	assertEquals(doc2, items.get(1));
	assertTrue(items.get(2) instanceof ODocument);
	assertEquals(doc3, items.get(2));
	db.commit();
}

This test-case will fail, but if you comment out line marked as IMPORTANT LINE - test-case will be OK

Investigation shown that HelperClasses.readLinkCollection() should be able to adjust ORecordLazyList.contentType. Otherwise: during iterating method OLazyRecordIterator.next() invokes update() which switch ORecordLazyList.contentType from EMPTY to ALL_RECORDS

Issue was fixed by PR #9403
Please merge

Issue can be closed, because corresponding PR was merged. Thank you, @tglman !


[Lesson04] Raspberry Pi 4 interrupts not working after first call to timer_tick()

I followed the advice given in issue #237 and was able to get my project behaving properly on a pi4, with the timer ticking nicely.
However, I've run into an odd problem with the scheduler.  Using most of the code from Lesson04 I have built a primitive scheduler in my project, and things are now behaving strangely.  Here is some output with helpful debugging printf()s:
Current Exception Level is: 1
Enabling interrupt 97
EnableRegister: ff84110c
About to call enable_irq()...
Calling switch_to()...
1234512345123451234512In handle_irq()...
In timer_tick()....
About to call enable_irq()...
Calling switch_to()...
abcdeabcdeabcdeabcdeabcdeabcd[...]

As you can see, everything is  just peachy until timer_tick() is called by the interrupt handler.  Thereafter, it seems that even though interrupts are re-enabled, the interrupt handler never gets called again.  My handle_irq() function is using the fix described by @maxstreitberger in issue #237, and my timer_tick() function and enable_irq() assembler function are identical to those in Lesson04.
I'm very confused!

Ah, the wisdom of the rubber duck.
In handle_interrupt(), I had:
  switch (irq) {
    case (SYSTEM_TIMER_IRQ_1):
      handle_timer_irq();
      put32(GICC_EOIR, irq_ack_reg);
      break;

and what I actually needed was:
  switch (irq) {
    case (SYSTEM_TIMER_IRQ_1):
      put32(GICC_EOIR, irq_ack_reg);
      handle_timer_irq();
      break;

Order matters!


Not working on specific games that block input from external softwares

I notice that in some games this software is not working.
In recent updates (May be on January 6th? Because it is still working while i was playing on January 5th ), Street Fighter 6 seems to have blocked input from external softwares (or maybe other softwares just cannot listen to the keyboard? ) , And i find hotkeys of other software (my scrennshot software for example) is not working as well .  I remember that in other games like KOF02UM the software is not working either.
I would like to know if there is a solution to this, or maybe I can only seek other solutions like hitbox ?

Hey, thanks for the report! Yeah, unfortunately, games can detect "injected" software-level inputs and straight-up rejecting them. It's their way of banning macros and stuff like that. The only way to get around it would be to inject these inputs on a driver level, but on windows only signed drivers can be installed by default, and microsoft demands $$$ for that :(
So for now idk if there's a way to fix this, maybe there are some signed programmable drivers, I might look into that

I see.
Now im going to try some keyboard marcro programming things, like  USB adapter that can transfer a normal keyboard to a keyboard supports marcro programming things .
For this issue can hardly be fixed by normal ways , i think it is done , thank you.


False Table Metadata Displayed

Description
The Database Navigator shows me wrong metadata of tables.
For example, I have a table with NVARCHAR columns having a length 10 letters, but the Navigator states that the length were 5.
Even if I copy the table, the data are not cut down to, the length of the data is still 10, but even the copy table states length = 5.
DBeaver (false):

SSMS (correct):

I am working on a Microsoft SQL Database.
This problem occurred since today since I have created the tables yesterday without any issues.
An update to the latest version didn't help.
DBeaver Version
Version <IP_ADDRESS>208051447
Operating System
Windows 10
Database and driver

Database: MS SQL Server / SQL Server, Windows Server 2016 Datacenter (10.0), version 14.0.3445.2
Driver: Microsoft JDBC Driver for SQL Server (standard)

Steps to reproduce

I open the metadata information of the table within the Database Navigator
I open the metadata information of the table's columns within the Database Navigator

Additional context
No response

@chiraPaki could you provide table ddl to reproduce an issue?

Sure. Here it is:
CREATE TABLE DataLake.snow.hlptbl_tickets (
ID bigint IDENTITY(1,1) NOT NULL,
[Issue Type] nvarchar(25) COLLATE Latin1_General_100_CS_AS NULL,
[Number] nvarchar(10) COLLATE Latin1_General_100_CS_AS NULL,
[Short Description] nvarchar(1000) COLLATE Latin1_General_100_CS_AS NULL,
[Assignment group] nvarchar(500) COLLATE Latin1_General_100_CS_AS NULL,
[Assigned to] nvarchar(1000) COLLATE Latin1_General_100_CS_AS NULL,
[Caller Id] nvarchar(1000) COLLATE Latin1_General_100_CS_AS NULL,
Priority nvarchar(2) COLLATE Latin1_General_100_CS_AS NULL,
State nvarchar(1000) COLLATE Latin1_General_100_CS_AS NULL,
[Created date] datetime NULL,
[Updated date] datetime NOT NULL,
[Resolved date] datetime NULL,
[Closed date] datetime NULL,
[Related problem] nvarchar(1000) COLLATE Latin1_General_100_CS_AS NULL,
[Problem Short Description] nvarchar(1000) COLLATE Latin1_General_100_CS_AS NULL,
[Transfer date] datetime NOT NULL,
CONSTRAINT PK_hlptbl_tickets PRIMARY KEY (ID,[Updated date],[Transfer date])
);
I just notice that even the automated DDL generation is gettiung it wrong, as the output is:
CREATE TABLE DataLake.snow.hlptbl_tickets (
ID bigint IDENTITY(1,1) NOT NULL,
[Issue Type] nvarchar(25) COLLATE Latin1_General_100_CS_AS NULL,
[Number] nvarchar(5) COLLATE Latin1_General_100_CS_AS NULL,
[Short Description] nvarchar(1000) COLLATE Latin1_General_100_CS_AS NULL,
[Assignment group] nvarchar(500) COLLATE Latin1_General_100_CS_AS NULL,
[Assigned to] nvarchar(1000) COLLATE Latin1_General_100_CS_AS NULL,
[Caller Id] nvarchar(1000) COLLATE Latin1_General_100_CS_AS NULL,
Priority nvarchar(1) COLLATE Latin1_General_100_CS_AS NULL,
State nvarchar(1000) COLLATE Latin1_General_100_CS_AS NULL,
[Created date] datetime NULL,
[Updated date] datetime NOT NULL,
[Resolved date] datetime NULL,
[Closed date] datetime NULL,
[Related problem] nvarchar(1000) COLLATE Latin1_General_100_CS_AS NULL,
[Problem Short Description] nvarchar(1000) COLLATE Latin1_General_100_CS_AS NULL,
[Transfer date] datetime NOT NULL,
CONSTRAINT PK_hlptbl_tickets PRIMARY KEY (ID,[Updated date],[Transfer date])
);

Hello @chiraPaki
Thanks for the report.
We already have bug ticket this problem: https://github.com/dbeaver/dbeaver/issues/16996
So I'm closing this ticket as duplicate


UMM-G support (ER)

Added UMM-G support for the following checks:

Bounding Coordinate Logic Check
Vertical Spatial Domain Type Check
Depth Unit Controlled Vocabulary Check
Altitude Unit Controlled Vocabulary Check
Temporal Extent Requirement Check
Default Date Check

@jenny-m-wood @spa0002 I changed the Vertical Spatial Domain Type Check, Depth Unit Controlled Vocabulary Check, and Altitude Unit Controlled Vocabulary Check back to their original state for UMM-C and created separate checks for UMM-G (Vertical Spatial Domain Unit Check UMM-G and Vertical Spatial Domain Type Check UMM-G) in the rule_mapping.json file. Also updated the check_messages.json for these checks.


Incompatible with last idea

Fixed it. You can use this build until it's available on the JetBrains plugin repository: https://github.com/DimensionalDevelopment/knit/releases/tag/1.0.5


Reactify dashboard grid

Using react-grid-layout.
Notable changes:

Resizing and dragging are much smoother
the grid is now responsive, the number of columns as well as column width will change based on browser width

Considerations:

this will break custom CSS on dashboards that specifically target ".gridster"; I can switch the class names back to gridster if this is a problem, but I think making people change their CSS would be better for the code
layouts that use less than 12 columns will not be centered anymore
you cannot increase row height of a slice past the screen/browser bounds
some layouts may be messed up by this change, but many of them can be fixed by saving the dashboard and refreshing the page with the new change

Coverage increased (+0.03%) to 82.197% when pulling a2033c46a5ef565581167c5e48b1a672232223ed on georgeke:reactify-grid into 7d276928286687e528318532a9a26ceb597203b5 on airbnb:master.

I think we should leave a gridster class on the div where it used to be for backwards compatibility of CSS templates. I'd just add a comment nearby in the code /* gridster for backwards css compatiblity */.

ahhhh I cannot say how happy this makes me 😁 😭 #tearsofjoy and all of the ES6!

@mistercrunch that's a great idea. There still will be css that needs to be changed since I switched from list elements to divs and some people select with ".gridster li.widget"

Coverage increased (+0.03%) to 82.139% when pulling b1bf018827ac5a511579778c196f9cc6e633a970 on georgeke:reactify-grid into 087c47a37e45bb9c2608bbb2279188b59547e449 on airbnb:master.

Coverage increased (+0.01%) to 82.12% when pulling ed7ab73d66ae27b978e4dc2478565d34a49b9ec4 on georgeke:reactify-grid into 087c47a37e45bb9c2608bbb2279188b59547e449 on airbnb:master.


Modifying Context within a Resolver

Is there a way to access/modify the context object within a resolver. I am trying to load tokens to my context. Currently, I have something like:
func (r *Resolver) Login(ctx context.Context, *args LoginArgs) (*UserResolver, error) {
        // logic to login and get user
        // ....
        accessToken, refreshToken := tokenutil.CreateTokensForUser(user)
        accessTokenString, refreshTokenString, err := tokenutil.SignTokens(accessToken, refreshToken)
	if err != nil {
		return nil, err
	}
	// replace existing tokens
	err = loadTokensToContext(&ctx, *accessTokenString, *refreshTokenString)
	if err != nil {
		return nil, err
	}
	//return user
	return &result, nil
}

Obviously I know that it is not really modifying the context object from the handler, but is there a way to do so? If not, does anyone have another solution to get the tokens in the response?

hi @sjakati98
In my practice ，in native http.request do it, get and set user token to request context
router.Handle("/graphql", middleware.AuthRequired(http.HandlerFunc(graphql.GraphQL)))

add have token's context set to http.request, graphql Resolver get token in context. next start your  resource permission validation logic。
func GraphQL(w http.ResponseWriter, r *http.Request) {
	var params struct {
		Query         string                 `json:"query"`
		OperationName string                 `json:"operationName"`
		Variables     map[string]interface{} `json:"variables"`
	}
	if err := json.NewDecoder(r.Body).Decode(&params); err != nil {
		http.Error(w, err.Error(), http.StatusBadRequest)
		return
	}

	token := r.Context().Value(pkg.Token).(string)
	response, op := mySchema.Exec(r.Context(), params.Query, params.OperationName, params.Variables)
       ....

I don't know you know my meaning or help you with your question.

This has nothing to do with this library. You can add values to the context just like in this example: https://golang.org/pkg/context/#example_WithValue


How to apply UTF16 decoder to backscanner?

Hi,
We would like to read UTF16 file but when we transform into the string, it adds characters between words.
Is there a way to add decoder to backscanner? Like this one:
https://play.golang.org/p/-9t0D_DFTA
Thanks.

For this lib to be able to scan backwards, it requires an io.ReaderAt (instead of the more common io.Reader).
Unfortunately the golang.org/x/text/transform package only provides io.Reader implementation, not io.ReaderAt. So you can't use this lib with golang.org/x/text/transform. If you find another package that does provide io.ReaderAt (or you implement it yourself), then you'll be able to use backscanner with that.

Thanks, due to this requirement I had to move to bufio back :( I will be looking to do this.


Chore/Upgrade to Yarn 3

Context
Proposed Solution
Important Changes Introduced

@rhyslbw Can you rebase and run just yarn?

Yes, sorry @blaggacao, there was a bad rebase merge (package ordering), and I failed to merge and push the the lock file changes. Both issues have been corrected in 716f67830c5eefe7da3965f6eadbf326f4fc67f4 , so you can drop be29a70873fb0e10c0e5352488b1f33382548b43 and rebase.

I'd like to squash f91b6929a5ca2ffa0da03b75feeacf6ef5480eb8 into 84da7bdd37f41eedfb6c180b7bb4dc4f358b4005, but left it separate until it's been reviewed.


myAlert library added to Alert and Action Sheet

Project URL
https://github.com/koust/myAlert
Category
Alert and Action Sheet
Description
MyAlert is an alertview. Shortly , show notifications in the app. It is simple to use and easy to develop.
Why it should be included to awesome-ios (optional)
Checklist

[x] Only one project/change is in this pull request
[ ] Has unit tests, integration tests or UI tests
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 9 / tvOS 10 or later
[x] Supports Swift 4 or later
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

@koust We cannot merge your PR yet until you write some tests for your lib. Feel free to open another PR once it is ready.


Fixed logic of assigning currentPlayerId.

currentPlayerId must be assigned in play() and resume() methods, not in setUrl(). This allows apps to use multiple audio players.
With the previous implementation a player which had the last setUrl() call was interacting with the Notifications Center, even if a totally different player was playing at the moment.
Now a correct player interacts with the Notifications Center.

This is a PR to fix #404 .

@volgin how does it work if you have multiple players playing at the same time though? this will just take the last right?

Notifications Center cannot display multiple recordings playing in parallel. Therefore there is only one currentPlayerId, which is used by the Notifications Center.
In the existing implementation it is set to the player which was the last to get play() or setUrl() calls.
In the proposed implementation, it is set to the player which was the last to get play() or resume() calls.


Mimir 628 more publish calendar features

Thanks! Added a loading-spinner and removed the ability to click multiple times.


Global $GOPATH/src

I wonder if it's possible to use a global $GOPATH/src accross different go version.

This would be nice, gvm's use of custom GOPATH makes development very difficult. I had to fall back to manually managing Go version's due to this.
#update go in path to version ie. 1.3.3
rm -rf $GOPATH/pkg
go test

A similar approach within gvm would preserve GOPATH, by emptying $GOPATH/pkg whenever versions are changed.


Documentation for Developing Locally with Prod Extension Installed

How would one develop locally if they already have the Chrome store extension installed?

Remove TidyTab
Load unpacked extension

I wish it could be better :(


Broken Model & Migration Generation

NEEDS FIX IN 5.1 BRANCH
USING THE FOLLOWING COMMAND

php artisan infyom:api_scaffold Dealer --fieldsFile='resources/model_schemas/Dealer.json'

[ ] OUTPUT MODEL  https://gist.github.com/phillipmadsen/1a83e1d7f29b61d3a7959f5c73729575
[ ] OUTPUT MIGRATION  https://gist.github.com/phillipmadsen/7e5145c907f8c556473c1e01370cca16

The parts that need to be fixed are as follows.

CURRENT
FIXED

$table->string('id');
$table->increments('id');

$table->integer('location_id')->unsigned();
$table->unsignedInteger('location_id')->index();

$table->foreign('location_id')->references('id')->on('locations');
$table->foreign('location_id')->references('id')->on('locations')->onUpdate('cascade')->onDelete('cascade');

NEEDS ENGINE
$table->engine = 'InnoDB';

AFTER CHANGES OUTPUT OF MIGRATION WOULD BE:
    public function up()
    {
        Schema::create('dealers', function (Blueprint $table) {
           $table->increments('id');
            $table->string('dealer');
            $table->string('contact_person')->nullable();
            $table->string('mobile')->nullable();
            $table->string('phone')->nullable();
            $table->string('hours_opening_mf')->nullable();
            $table->string('hours_closing_mf')->nullable();
            $table->string('hours_opening_sat')->nullable();
            $table->string('hours_closing_sat')->nullable();
            $table->string('hours_opening_sun')->nullable();
            $table->string('hours_closing_sun')->nullable();
            $table->string('company_phone')->nullable();
            $table->string('toll_free')->nullable();
            $table->string('public_email')->nullable();
            $table->string('support_email')->nullable();
            $table->unsignedInteger('location_id')->index();
            $table->softDeletes();
            $table->engine = 'InnoDB';
            $table->foreign('location_id')->references('id')->on('locations')->onUpdate('cascade')->onDelete('cascade');
        });
    }

Just got to the fields file and it only generated the foreign key input.  guess there is a problem here also. ill look into it and get back to you.

Ok the table generated has no fields or headers in it.

What file generates the headers and cells for the table and i will look into it?

Working fine with the latest code.


NPM security update breaks slaude

If you update npm due to 3 critical security updates you break slaude.
I understand PandarusAnon is aware, and fix is inbound.
I post because I have time now that slaude is broken. perhaps a fellow traveler will be warned off this path.
To fix
delete *.json in the slaude directory
repull git
npm install

I learnt this the hard way lol, so I leave it untouched. oh and while we're giving advice, never use these scripts on your main accounts, make encrypted emails if possible. yes while the script automatically /reset claude I say it's better to be safe than sorry, if you read the ToS you'll know that claude logs your chat with it unless cleared by user


Mink instance has not been set on Mink context class. Have you enabled the Mink Extension?

Please help!!!
I want to use Behat with Laravel and I have been following these steps here
https://semaphoreci.com/community/tutorials/getting-started-with-bdd-in-laravel
but in my own laravel project not the default "Laravel 5".
When I run

$ behat

I get error :
Feature:
  In order to prove that Behat works as intended
  We want to test the home page for a phrase
  Scenario: Root Test           # features/hometest.feature:5
    When I am on the homepage   # FeatureContext::iAmOnHomepage()

Mink instance has not been set on Mink context class. Have you enabled the Mink Extension? (RuntimeException)
    Then I should see "Welcome" # FeatureContext::assertPageContainsText()
--- Failed scenarios:
    features/hometest.feature:5
1 scenario (1 failed)
2 steps (1 failed, 1 skipped)
0m0.04s (11.91Mb)

Commands that I user in root directory of my Laravel project are :

composer require behat/behat behat/mink behat/mink-extension laracasts/behat-laravel-extension --dev
sudo ln -s /home/vagrant/Code/bdd-setup/vendor/bin/behat /usr/local/bin/behat

behat --init

I have added extensions in my FeatureContext.php

class FeatureContext extends Behat\MinkExtension\Context\MinkContext
implements Context, SnippetAcceptingContext
cp .env .env.behat0
php -S localhost:8888 -t public

behat

Here are my files:

<?php

use Behat\Behat\Context\Context;
use Behat\Behat\Context\SnippetAcceptingContext;
use Behat\Gherkin\Node\PyStringNode;
use Behat\Gherkin\Node\TableNode;

class FeatureContext extends Behat\MinkExtension\Context\MinkContext implements Context, SnippetAcceptingContext
{

    public function __construct()
    {
    }
}

_env.behat_

APP_ENV=local
APP_KEY=base64:I22b/3uOm+B/H793P8aWQ5CucfiCxoMZ/H3JG6aYoLo=
APP_DEBUG=true
APP_LOG_LEVEL=debug
APP_URL=http://localhost

DB_CONNECTION=sqlite

CACHE_DRIVER=file
SESSION_DRIVER=file
QUEUE_DRIVER=sync

REDIS_HOST=<IP_ADDRESS>
REDIS_PASSWORD=null
REDIS_PORT=6379

MAIL_DRIVER=smtp
MAIL_HOST=mailtrap.io
MAIL_PORT=2525
MAIL_USERNAME=null
MAIL_PASSWORD=null
MAIL_ENCRYPTION=null

_behat.yml_

default:
    extensions:
        Laracasts\Behat:
            # env_path: .env.behat
        Behat\MinkExtension:
            default_session: laravel
            base_url: http://localhost:8888
            laravel: ~

And my composer.json requires are :
"name": "laravel/laravel",
    "description": "The Laravel Framework.",
    "keywords": ["framework", "laravel"],
    "license": "MIT",
    "type": "project",
    "require": {
        "php": ">=5.5.9",
        "laravel/framework": "5.2.*"
    },
    "require-dev": {
        "fzaninotto/faker": "~1.4",
        "mockery/mockery": "0.9.*",
        "phpunit/phpunit": "~4.0",
        "symfony/css-selector": "2.8.*|3.0.*",
        "symfony/dom-crawler": "2.8.*|3.0.*",
        "behat/behat": "^3.1",
        "behat/mink": "^1.7",
        "behat/mink-extension": "^2.2",
        "laracasts/behat-laravel-extension": "^1.0"
    },

@zagorkakomljanovic , i face the same issue, could you tell me how to fix the problem?

I ran into this issue and the problem was simply that I put behat.yml in the wrong directory. It's quite dumb but as always, make sure that all your files are in the good directory.


Service DNS2 is DOWN (dns2.casjaydns.com)

Service DNS2 is DOWN (dns2.casjaydns.com)
dns2.casjaydns.com
Alert Details: Port is not listening.

via Uptime Robot - All Monitors https://bit.ly/2Wd5rQ4
August 04, 2020 at 01:42AM

Fixed


PLACEHOLDER - Conservative updates on outdated

This is a branch from #4841 which is still in the final stages of review. At one point that PR contained the conservative update stuff, too, and there were some code review comments pertaining to breaking behavior that I wanted to copy over here, so I"m making this PR to capture that stuff so I can close out the review of the prior PR (and not having done an official 'review' of a new GitHub Review :tm: thing, I dunno if it'll blow it away or not - though it shouldn't).
Once #4841 is merged, I'll rebase this PR.

1.13 has new, un-official options in bundle update to restrict updates to only the latest --patch or --minor version available (see bundler/bundler-features#122). bundle outdated had similarly named options added to it previously, but they don't behave in the same way.
For example, if bundle outdated is going to show:
foo: 1.0.0 => 2.0.0
bar: 1.0.0 => 1.1.0
qux: 1.0.0 => 1.0.1
and you add the --patch flag, then it merely filters the results to only show:
qux: 1.0.0 => 1.0.1
rather than showing what could happen as a result of bundle update --patch:
foo: 1.0.0 => 1.0.3
bar: 1.0.0 => 1.0.9
qux: 1.0.0 => 1.0.1
#4772 is a task I made to update bundle outdated to have matching behavior, and when I saw this PR, I simply linked to it to remind me that this PR would be changing code in a similar place.

This is going to cause a problem though which is a key part of this change: the meaning of the switches --patch / --minor / --major are now changing. Before they filtered the output, now they'll be influencing the resolution process.
There's only two ways here, though - we either CHANGE what they mean and rename the filtering ones to something like --filter-patch (which is breaking), or leave them as-is and introduce new flags (--resolve-patch, etc?) that are translated for GemVersionPromoter, which won't be breaking, but will be inconsistent with the flags for bundle update.
My preference is to the former, those new flags weren't introduced too long ago, but I want @indirect's input on this.

The flags as passed to the GemVersionPromoter change resolution. <=1.13.2 of bundle outdated, those flags merely filter the output, with no influence on resolution.
If the lockfile is set to foo 1.0.0, and all of the following exist: 1.0.1, 1.1.0, 2.0.0, then <=1.13.2 bundle outdated currently will show:
foo (newest 2.0.0, installed 1.0.0)
<=1.13.2 bundle outdated --minor simply filters away that line and won't show it.
With these changes, bundle outdated --minor would be fed to the GemVersionPromoter and actually only resolve foo up to 1.1.0.

from @indirect in this comment:
"IMO, this is what we were trying to do in 1.12, and failed to do because resolving is complicated. :/ I would be okay with this change on the grounds that the previous flags were documented so it sounded like they cause the new resolver aware behavior. 👍"


Dependabot couldn't authenticate with npm.pkg.github.com/

Hi,
I recognized the last release v0.116.0 that causes the authentication issue with npm.pkg.github.com.
Here is my scenario:

Private Repo
Private github package: npm.pkg.github.com/@smisy
@smisy is my organization.

Yarn is not working with npm.pkg.github.com and my repo only works with npm.
dependabot worked well with my github token until the team release v0.116.0. I wonder dependabot only use yarn at v0.116.0?

@ttruongatl there shouldn't be any breaking changes in v0.116.0 and haven't changed anything around npm auth so not sure what's gone wrong. Have you tried authenticating to a package hosted on npm.pkg.github.com/@smisy using your token? Just testing with curl or similar.

@feelepxyz I tested the token and it worked fine on my local and JenkinsCI. As I said, yarn is not working on my project. Do you think it may cause the issue?

I have seen the issue for the last couple of days. Closed the ticket.


fix: fix remove note when board is locked

Description

User can delete own notes although the board is locked
Changelog

In 'api/notes.go': Added check before deleting a note to check whether the board is locked or not
Checklist

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] The light- and dark-theme are both supported and tested
[ ] The design was implemented and is responsive for all devices and screen sizes
[x] The application was tested in the most commonly used browsers (e.g. Chrome, Firefox, Safari)

(Optional) Visual Changes

I would wait until the frontend is finished. Maybe some additional changes need to be made

Fixed issues with https://github.com/inovex/scrumlr.io/pull/4005


Adding configuration for multiple newrelic proxies

Support multiple NewRelic Dashboard account proxies
We have about 15 NR accounts in total and needed to be able to pull dashboards from a few of them.
This adds the ability to configure a prefix for the newrelic-dashboard plugins proxy configuration in the app-config.yaml.
Added annotation to configure the prefix: newrelic.com/dashboard-api-prefix which allows for:
proxy:
  '/newrelic/api':
    target: https://api.newrelic.com
    headers:
      X-Api-Key: ${NEW_RELIC_USER_KEY}
  '/<api-prefix>/newrelic/api':
    target: https://api.newrelic.com
    headers:
      X-Api-Key: ${NEW_RELIC_ACCOUNT2_USER_KEY}

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Hey :wave: thanks for this!
So previously, when this request has come up, we've taken an approach to introduce a backend plugin to encapsulate this logic rather than supporting different proxy paths.
Overall the proxy pattern isn't great, and it's lacking in security, but it's something to get you started with.
I think that we should move this apiKey config into app-config.yaml and then have one endpoint which would be the new relic backend plugin, and that would be responsible for detecting which account to route the request to.
Does this make sense?

Yea, makes total sense. Figured this was quick enough if someone else needed it. I am writing another plugin shortly so that will give me some experience with that and I can look to create that new plugin for this later.


Update Makefile

seems like this PR has modifications included in https://github.com/meaningfy-ws/eds4jinja2/pull/9/files


What permissions are needed for Slack notifications of the report?

When I tried the Slack notification function of Report, the following error was output.
[x] Failed to send report: missing_scope

The command I tried is below.
raven report --severity critical --tag injection slack --slack-token <slack token> --channel-id <channel-id>

I had granted the Slack token the permissions for chat:write and chat:write:public.
What permissions are required for Slack notifications?

The required permissions are: files:read and files:write.
The latest RAVEN version uses file upload API for the report (differently from the older RAVEN version that sent the report as plain text)
We definitely should add this to the documentation.

Make sure to remove the Raven app from the channel -> reinstall the app in the workspace and add the Raven app again. Please ping us if the issue continues

Thank you.
After granting those permissions and trying, I was able to successfully send the report to Slack.

I am keeping this issue open for adding docs about the required permissions.


Pick & Place

Only 1 pick and place location is generated and component requires 2 connectors.

At CM4 Schematic sheet at Property > General > Type > Standard (NO BOM)
2.Add J1,J2 for DF40 connector with dummy footprint place at center of actual connector location.


⚖ Register Validator:

Network Choice
nemeton-1
Who are you?
KJINC.io, a validator mainly on Cosmos. We are glad to be here.
Currently, we are the validator of Bitcanna, BitSong, Cerberus, Desmos, Ki Chain, Vidulum, Lum Network, Meme Network, Stride, Jackal, welcome to delegate with us.
gentx
{"body":{"messages":[{"@type":"/cosmos.staking.v1beta1.MsgCreateValidator","description":{"moniker":"KJINC.io","identity":"E294DFEB5B016EE2","website":"https://kjinc.io","security_contact":"info@kjinc.io","details":"BLOCKCHAIN INFRASTRUCTURE AND STAKING PARTNER BRING YOUR IDEA TO HIGHER LEVELS WITH KJINC.io"},"commission":{"rate":"0.050000000000000000","max_rate":"0.200000000000000000","max_change_rate":"0.050000000000000000"},"min_self_delegation":"1","delegator_address":"okp41hslxk7km5z6srvmr993hsgcdxnezg8a5jc9gzh","validator_address":"okp4valoper1hslxk7km5z6srvmr993hsgcdxnezg8a58l4puk","pubkey":{"@type":"/cosmos.crypto.ed25519.PubKey","key":"LDShuoqja49vsOlLp1ozs2/pXT8oAhjQsebPfIgMwN4="},"value":{"denom":"uknow","amount":"10000000000"}}],"memo":"dd0c2f0bdd3cf47c69151a072f88c4543bcc5187@<IP_ADDRESS>:26656","timeout_height":"0","extension_options":[],"non_critical_extension_options":[]},"auth_info":{"signer_infos":[{"public_key":{"@type":"/cosmos.crypto.secp256k1.PubKey","key":"A0x9TV651K3bpkXWrxLpUf9VzzibzDTzkr7g7nO0CV6c"},"mode_info":{"single":{"mode":"SIGN_MODE_DIRECT"}},"sequence":"0"}],"fee":{"amount":[],"gas_limit":"200000","payer":"","granter":""},"tip":null},"signatures":["YvYe4zAnvZ+4EyJwEFAEX1V47oop/lOxNIi6qvhXVb4Ag5CISchMvVz2HNAgp4AEPwXM+CV0DQQaipPxScY7Tg=="]}

😉 Here is the corresponding PR: https://github.com/okp4/networks/pull/291


HDDS-2397. Fix calling cleanup for few missing tables in OM.

What changes were proposed in this pull request?
Fix calling clean up of few tables which is missing in OzoneManagerDoubleBuffer cleanupcache.
For few tables cleanup of cache is missed:
PrefixTable
S3SecretTable
DelegationTable
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-2397
How was this patch tested?
Ran a few integration tests.

LGTM +1


Fix .env handling when using "deploy function"

I don't know when it was changed, but the proper hooks to create and delete the .env file when deploying an individual function are now before:deploy:function:packageFunction and after:deploy:function:packageFunction.
I'm also including hauboldj's patch to create/remove .env file when invoking locally (PR #7 ), as I used the online editor only and didn't know how to make a PR with just my commits.

Coverage remained the same at 100.0% when pulling 21a775ee23889f1b2b6efad2b71dd4a44f92db59 on patheticcockroach:master into 2046429d4742b46cff25ea37ce726271c2fcd156 on silvermine:master.

Coverage remained the same at 100.0% when pulling 31e1a27c680f555c323d776d6f208b7a126d3ccc on patheticcockroach:master into 2046429d4742b46cff25ea37ce726271c2fcd156 on silvermine:master.

Coverage remained the same at 100.0% when pulling 31e1a27c680f555c323d776d6f208b7a126d3ccc on patheticcockroach:master into 2046429d4742b46cff25ea37ce726271c2fcd156 on silvermine:master.

About the new package/deploy plugin lifecycle: https://gist.github.com/HyperBrain/bba5c9698e92ac693bb461c99d6cfeec

@patheticcockroach I'm sorry for the extreme delay on this PR. I had written off this plugin as no longer useful since Lambda introduced support for environment variables and Serverless supported it. I'm curious if you're still using this plugin, and if so, what is the usecase? If it seems like it's still a necessary plugin, I'll look at merging the open PRs and shipping a release. Thanks!

Hi, yes I'm still using the plugin. TBH we never thought about using Lambda's environment variables for that purpose, nor about checking if Serverless had support for them, so we have quite a few projects in my companies where we use this plugin + dotenv rather that Lambda's env variables.
I guess we'll migrate sooner than later (assuming Serverless's support for Lambda's env variables also covers "invoke local"), still having the plugin updated would be more convenient than patching it manually every time, as we currently do.

Thanks for the quick feedback. I rebased it, fixed it, merged it, and pushed it. Unfortunately because you'd used "master" as your branch name, that messed up the PR history a bit, but it's merged. I'll release 1.0.2 shortly and it will contain your fixes.


integration: make onewatch work on containerd

Hello @jazzdan,
Please review the following commits I made in branch nicks/kind:
fa0f6f27c6ae84618998ce1484e669f96f50d52a (2019-06-25 17:30:17 -0400)
integration: make onewatch work on containerd

I'm not super happy with this solution, but it works
Maybe we could do something clever where, if you specify restart_container(), we inject our own entrypoint that does something like this to allow restarts


Update version to v2.0.0.

Release notes:
We've reworked mainframer to make it much more universal than v1.
Now you can integrate it with almost any kind of project.
Major changes:

v2 requires config files, see OPTIONS.md.
We've added bunch of samples and recipes to ease integration for you.
Text output produced by mainframer now shows duration of major steps and in general more user-friendly.
mainframer covered with good set of tests, all PRs go through CI that includes steps like ShellCheck and others.

Migration from v1:

All options were moved from local.properties to .mainframer/config, see OPTIONS.md.
v1 was implemented to work with Android Gradle projects, please check samples/gradle-android to migrate your project to v2.


[BUG] REPO_INDEXER won't index mirrored repository after sync

Gitea version (or commit ref): 1.13.1
Git version: Not relevant
Operating system:docker image

Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[x] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[x] No

Log gist: Not relevant

Description

REPO_INDEXER won't index mirrored repository after sync
Related: #12380  #11200 #11479 #5329

func (r *indexerNotifier) NotifySyncPushCommits(pusher *models.User, repo *models.Repository, opts *repository.PushUpdateOptions, commits *repository.PushCommits) {
	if setting.Indexer.RepoIndexerEnabled && opts.RefFullName == git.BranchPrefix+repo.DefaultBranch {
		code_indexer.UpdateRepoIndexer(repo)
	}
	if err := stats_indexer.UpdateRepoIndexer(repo); err != nil {
		log.Error("stats_indexer.UpdateRepoIndexer(%d) failed: %v", repo.ID, err)
	}
}

It might be that you don't have a default branch set?
Go to the Repository Settings and change/re-set the default branch.

I tired resetting the default branch but it doesn't work.
Also I didn't see any error in the logs. I am sure that I have RepoIndexerEnabled, as the code search works for plain repos instead of the mirrored ones.

I tired on 1.13.7 with many different mirrored repos, but the problem persists.
Also I tried deleting repos.bleve to rebuild the index. The REPO_INDEXER  will properly index the current default branch of the mirrored repo, but it's not going to index new commits after sync

@zeripath Same on v1.14.1

Same with v1.14.2

Just to chime in that I am also noticing this issue on v1.18.5. Default branch is set as well.


promote to prod

Description

#531
#536
#547
#548
#546

How should this be tested?
Please provide instructions so we can test. Please also list any relevant details for your test configuration.

[x] Gyro2V2 pools
[x] KassandraManaged pools
[x] Attribute block added to JoinExit and Swap entities
[x] Setting poolType via EventEmitter work as expected: Etherscan, Subgraph

dev -> master

[x] I have checked that all beta deployments have synced
[x] I have checked that the earliest block in the polygon pruned deployment is 52775215, Jan-26-2024 02:53:58 PM

[x] The earliest block is more than 24 hours old

[x] I have checked that core metrics are the same in the beta and production deployments*

* Arbitrum has 157M in TVL on production but 133M on staging -- for comparison, Dune reports 115M and DefiLlama 137M. rETH has been added to pricing assets (#548) which should make estimates more accurate.

I've just verified that setPoolType EventEmitter's handler work as expected -- see Etherscan and Subgraph


change the vc setting for ACK

change the clusterrolebinding's name of the vc-manager's service account to avoid the privilege problem on ACK.
increase the polling timeout to 180 seconds (i.e. pulling image from docker on ack may take longer).

/lgtm
/approve


Pin golangci-lint version

Pending https://github.com/golangci/golangci-lint-action/issues/1091
Will avoid future breakages without code change or go version changes like we had with gosec int conversions

damn, didn't check the failure... my bad


Added commit command.

@asmeurer Made a pull request update to add the "git add" command as well.

And another small fix.

I recommend not using -m as it discourages descriptivity, but I'll merge and make some modifications.


Separate current health of stage+freight from freight's promotability to next stage(s)

In #673, @jessesuen wrote:

we have been equating promotability with argo cd app health. But at the same time, we've been continuously mirroring the currentFreight health with Argo CD app health changes. And we have been presenting these changes in UI as living health.
It seems to me that we need to separate the concept of Argo CD health from Freight being "promotable." This is what I propose:
Health status should be a living assessment of the underlying Argo CD app(s) associated with the Stage. Health can flap, and would have all of the Argo CD health statuses (Healhty, Degraded, Progressing, Unknown). Health is not applicable on Stages that have no Argo CD app association. Health is also not applicable to Freight. It is simply a mirror of Argo CD app health.
We introduce a new binary status of a Freight: Qualified vs. Unqualified. A qualified freight has gone through the necessary checks to be promotable to the next stage. If a Stage has an Argo CD app(s), it simply means the apps at one point in time became healthy with that freight. If there is no associated Argo CD app, then Freight is immediately qualified for downstream stages. Once a Freight has been qualified in a Stage, it will never become unqualified.

I would like to work on this issue if that's ok.

I would like to work on this issue if that's ok.

SGTM!


Treat build-docs warnings as errors on Travis

Fixes: https://github.com/quantumlib/Cirq/issues/1965

@Strilanc @dabacon I'd like to get this in before anything else makes it into master that will ruin these "perfect" docs. Thanks!

@Strilanc  Pinging on this before any merges break the docs.


Add TeaCall typedef

just added typedef teacall as dynamic
many people have compiling issues by this, and, a tea call can return dynamic..?

well, i dont know about macros so, idk why it happend

well, i dont know about macros so, idk why it happend

Latest Haxe version bug. Use 4.3.2 instead. It will fix the macro compiling

There have been multiple issues that already fix this, and this is the worst method of doing it.
Just replace TeaCall with Tea, there's 0 reason to use a dynamic type for this


[Live Preview MultiBrowser] Doesn't work with CodeMirror pages

Clone CodeMirror (or download a .zip and extract) and open it in Brackets.
Open index.html in Brackets.
Make sure MultiBrowser Live Preview is enabled.
Start Live Preview.

Result: The Live Preview symbol is only ever half-filled.
The console logs look fine, though:
NodeSocketTransport - start
NodeSocketTransport - send,,{"method":"Runtime.evaluate","params":{"expression":"_LD.hideHighlight()"},"id":1}
NodeSocketTransport - send,,{"method":"Runtime.evaluate","params":{"expression":"_LD.hideHighlight()"},"id":2}
NodeSocketTransport - send,,{"method":"Runtime.evaluate","params":{"expression":"_LD.hideHighlight()"},"id":3}

cc @busykai @sebaslv

From my tests, it looks like the missing html/head/body structure causes these problems.

@MarcelGerber, does it work with the default version?

I haven't tested Live Editing or Highlighting, but the icon is completely filled at least.

Thanks! I will take a look at this. I'm back and have a couple of holidays ahead of me. :)

Btw, you don't need to clone CodeMirror -- a simple page without the aforementioned structure should suffice.
Probably, even a text-only HTML page works.

So, I've just taken another look on this:

I was able to reproduce the issue with a .html file containing nothing than abc
The browser opens just fine and displays the page correctly
As soon as you edit the document, the icon changes from a yellow half-filled one to a purple filled one, indicating a syntax error
With this very simple file, I got a couldn't find the dom for ... error message (I didn't get that message with CodeMirror, though)

Just tested with normal (non-MultiBrowser) Live Preview:
With that simple abc webpage, I get symptoms 3 and 4 from above, too.
But a simple
abc
 works just fine, including Live Editing and Highlighting.
And now as I tested
abc
 with MultiBrowser Live Preview, I got different findings... again:

There's no couldn't find the dom for ... error message
As soon as I edit my document, the half-filled icon turns into a filled icon
The changes are not pushed to the browser, though
Highlighting doesn't work, either


lsscsi: 0.28 -> 0.29

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/lsscsi/versions.
These checks were done:

built on NixOS
/nix/store/2z22y23xx308psxsxn605hsizfhll3jd-lsscsi-0.29/bin/lsscsi passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 0.29 with grep in /nix/store/2z22y23xx308psxsxn605hsizfhll3jd-lsscsi-0.29
directory tree listing: https://gist.github.com/8a27d93aa9459ec96b068a78e73f6f1c
du listing: https://gist.github.com/1c65dda6f5dc9d0ef0fa880d016dc260

meta.description for lsscsi is: '""'.

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: lsscsi
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: lsscsi
Partial log (click to expand)
make[1]: Leaving directory '/build/lsscsi-0.29'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/2z22y23xx308psxsxn605hsizfhll3jd-lsscsi-0.29
shrinking /nix/store/2z22y23xx308psxsxn605hsizfhll3jd-lsscsi-0.29/bin/lsscsi
gzipping man pages under /nix/store/2z22y23xx308psxsxn605hsizfhll3jd-lsscsi-0.29/share/man/
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/2z22y23xx308psxsxn605hsizfhll3jd-lsscsi-0.29/bin
patching script interpreter paths in /nix/store/2z22y23xx308psxsxn605hsizfhll3jd-lsscsi-0.29
checking for references to /build in /nix/store/2z22y23xx308psxsxn605hsizfhll3jd-lsscsi-0.29...
/nix/store/2z22y23xx308psxsxn605hsizfhll3jd-lsscsi-0.29

Success on aarch64-linux (full log)
Attempted: lsscsi
Partial log (click to expand)
make[1]: Leaving directory '/build/lsscsi-0.29'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/b7zs0iscdlxi5yv4nfs11pl0bn0m4k1v-lsscsi-0.29
shrinking /nix/store/b7zs0iscdlxi5yv4nfs11pl0bn0m4k1v-lsscsi-0.29/bin/lsscsi
gzipping man pages under /nix/store/b7zs0iscdlxi5yv4nfs11pl0bn0m4k1v-lsscsi-0.29/share/man/
strip is /nix/store/jk6j4lh9v5mvjdbdc35sj0zffhhf6s56-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/b7zs0iscdlxi5yv4nfs11pl0bn0m4k1v-lsscsi-0.29/bin
patching script interpreter paths in /nix/store/b7zs0iscdlxi5yv4nfs11pl0bn0m4k1v-lsscsi-0.29
checking for references to /build in /nix/store/b7zs0iscdlxi5yv4nfs11pl0bn0m4k1v-lsscsi-0.29...
/nix/store/b7zs0iscdlxi5yv4nfs11pl0bn0m4k1v-lsscsi-0.29


rewrite MultiException to actually contain all causes (FINERACT-858)

FINERACT-858, useful for debugging FINERACT-922

@ptuomola how do you like this?

@ptuomola how do you like this?

@ptuomola and now this actually even builds green :smiley_cat:
@maektwain @awasum @xurror anyone else review this - LGTY?

It sounds like nobody seems to have any strong objections to this one, so I'd like to "self merge" it tomorrow.

Looks good to me - much cleaner than the previous approach


Unable to use Altair 4.0.5 on Mac OS version 10.14.6

Describe the bug
I am unable to use the Altair 4.0.5 client on MAC OS version 10.14.6
To Reproduce
Steps to reproduce the behavior:

Downloaded Altair 4.0.5 client for MAC OS
Installed it by double clicking on the dmg file
It's installed
However, I am unable to open it. When I try to open it, it says you can't use this version of Altair with this version of mac

Expected behavior
My mac version is already later than 10.10 as mentioned in the error
Screenshots

Desktop (please complete the following information):

Mac OS version 10.14.6

Can you verify that you're using using the right version of the mac app? There are two versions now: one for intel arch (x64) and the other one for M1 silicon chip (arm64)

Closing the issue. I was using a wrong version. My bad.


Why my request is so slow?

Hi. I send this request -
query getTxs($address: String) {
    transactions (
        limit: 20,
        order_by: {
            includedAt: desc
        },
        offset: 0,
        where: {
            outputs: {
                address: {
                    _like: $address
                }
            }
        }) {
        fee
        hash
        inputs {
            address
            sourceTxHash
            sourceTxIndex
            txHash
            value
        }
        outputs {
            address
            index
            txHash
            value
        }
        totalOutput
        size
        inputs_aggregate {
            aggregate {
                sum {
                    value
                }
            }
        }
        outputs_aggregate {
            aggregate {
                sum {
                    value
                }
            }
        }
        blockIndex
        block {
            number
            forgedAt
            slotNo
        }
        includedAt
    }
}

{
   "query":"$query",
   "operationName":"getTxs",
   "variables":{
      "address":"DdzFFzCqrhsyjbBTjyyQnELyfbGbu7kfmN3RTwSYmynr3z9sbPn3Xqs6oYfUiD9oYA7uq5TyUgfgVaDJMGWD6hVaiEftfqdZnB5NLRz6"
   }
}

It goes reaaaaly slow (about 8-9 secs). My cardano-graphql instance is running on bare-metal server with this specs:
CPU - Intel(R) Xeon(R) CPU E3-1275 v5 @ 3.60GHz
RAM - 62GB
SSD - 480 NVMe.
I think this resources more than enough for this query. I also tried to tune my PostgreSQL with PGTune. It didn't helped at all.

@alexeirbv I've reformatted your content to present the query as graphql.
I'm getting a 37ms response in my dev env on the master branch. Try profiling with TRACING=true:

Feel free to re-open if you can reproduce this on master


@optional services are risky

I think we recently introduced a lot more @optional services and none of them declare the service as  | undefined which essentially breaks our strict null benefits:

Assigning a few people based on usages and @jrieken to comment whether we should maybe get rid of this concept altogether because it is easy to introduce runtime errors with it.
Imho no service should be optional, rather if one does not exist in a certain environment it should be stubbed as a no-op implementation.

none of them declare the service as | undefined

Many of them do FYI, note that you cannot declare this | undefined in the ctor as you end up with a compile error:
@optional(ILocalTerminalService) localTerminalService: ILocalTerminalService | undefined,

Results in:
[04:14:10] Error: E:/GitHub/microsoft/vscode/src/vs/workbench/contrib/terminal/browser/terminalInstance.ts(933,26): No overload matches this call.
  Overload 1 of 10, '(descriptor: SyncDescriptor2<number, TerminalConfigHelper, ITerminalProcessManager>, a1: number, a2: TerminalConfigHelper): ITerminalProcessManager', gave the following error.
    Argument of type 'typeof TerminalProcessManager' is not assignable to parameter of type 'SyncDescriptor2<number, TerminalConfigHelper, ITerminalProcessManager>'.
      Property 'ctor' is missing in type 'typeof TerminalProcessManager' but required in type 'SyncDescriptor2<number, TerminalConfigHelper, ITerminalProcessManager>'.
  Overload 2 of 10, '(t: typeof TerminalProcessManager, ...args: never): TerminalProcessManager', gave the following error.
    Argument of type 'number' is not assignable to parameter of type 'never'.
[04:14:10] Error: E:/GitHub/microsoft/vscode/src/vs/workbench/contrib/terminal/test/browser/terminalProcessManager.test.ts(37,13): No overload matches this call.
  Overload 1 of 10, '(descriptor: SyncDescriptor2<number, TerminalConfigHelper, TerminalProcessManager>, a1: number, a2: TerminalConfigHelper): TerminalProcessManager', gave the following error.
    Argument of type 'typeof TerminalProcessManager' is not assignable to parameter of type 'SyncDescriptor2<number, TerminalConfigHelper, TerminalProcessManager>'.
      Property 'ctor' is missing in type 'typeof TerminalProcessManager' but required in type 'SyncDescriptor2<number, TerminalConfigHelper, TerminalProcessManager>'.
  Overload 2 of 10, '(t: typeof TerminalProcessManager, ...args: never): TerminalProcessManager', gave the following error.
    Argument of type 'number' is not assignable to parameter of type 'never'.

The current way I handle this is to set the service to another nullable property that has the correct type. It looks like these are the actual problem cases:

Also I much prefer this setting to a private member than stubbing out no-op services. I'd expect a lint rule to ban this regex would prevent this from happening in the future:
@optional.+(private|readonly|public)

to comment whether we should maybe get rid of this concept altogether because it is easy to introduce runtime errors with it.

Yes, optional service dependencies should be used super rarely, likely never. The original intend for optional service dependencies is Monaco vs VS Code, e.g a contribution that's shared between both "products" might work better with an additional, non-essential service. This usually only applies to things in 'editor/contrib' but can also be interpreted as sign of bad/tight coupling.
One sample is @sbatten's ITASExperimentService which is marked optional everywhere. However, the service is registered in this file (which is imported in src/vs/workbench/workbench.common.main.ts) and only used inside the workbench-layer.

removed via https://github.com/microsoft/vscode/commit/60e3fd6a01f11fbec83c6db8b2b3420d49b7eb18

ping @sandy081

Remaining usages

[ ] src/vs/platform/extensionManagement/common/extensionGalleryService.ts @sandy081
[ ] src/vs/platform/extensionManagement/node/extensionManagementService.ts @sandy081
[ ] src/vs/workbench/contrib/externalTerminal/browser/externalTerminal.contribution.ts
[ ] src/vs/workbench/contrib/terminal/browser/terminalEditorService.ts
[ ] src/vs/workbench/contrib/terminal/browser/terminalInstanceService.ts
[ ] src/vs/workbench/contrib/terminal/browser/terminalProcessManager.ts
[ ] src/vs/workbench/contrib/terminal/browser/terminalService.ts

@jrieken My usage of optional services is because they (IStorageService & IDownloadService) are not available in CLI environment.
Added DownloadService to CLI but we do not have support for StorageService in CLI. @bpasero any recommendations?

@sandy081 then we add it? since workspace storage moved to electron-main years ago, we have full access to all storage from the main process.

Seems its not trivial to add based on the comment from here https://github.com/microsoft/vscode/issues/63903#issuecomment-470836331

Yes, CLI needs to join the running instance for such a call to storage.
Since it was optional so far in the CLI, I would just throw in an in-memory storage service that does nothing to get away with it. Looks like we never needed it in the first place...

I felt using in-memory storage service is error-prone especially there is no any hint in the code about it and going forward we might wrongly develop features in cli assuming storage service is available.
Hence I had two versions of my service one with storage service and the other without and instantiating the right one in the given environment.

@Tyriar this is now just terminal land

Terminal part is done 🎉 moving back to @jrieken to remove the decorator and any other related code

Done. Thanks everyone for helping to clean this up 👯


Add support for sending transactions via lightwalletd

Motivation
The sendrawtransaction RPC method in lightwalletd needs to be able to create and send transactions on the network.
We might need to do a full sync to make these tests work.
We might need a developer wallet to make these tests work on mainnet.
Tasks

Add a test harness that syncs to the tip from cached state
Create a new transaction
Add a lightwalletd integration test for sendrawtransaction

Hey team! Please add your planning poker estimate with ZenHub @conradoplg @dconnolly @oxarbitrage @upbqdn

An idea on how to test this without having to set up an account and use real funds (which would limit CI execution and potentially make it more complicated due to the need to protect keys):

Spawn a zebrad instance
Pick a (recent?) block height
Get some transactions from blocks after that block height
Stop the zebrad instance
Spawn a new zebrad instance for lightwalletd to interface with
Synthetically limit this new zebrad instance to the chosen block height
Try to send the transactions that were obtained from future blocks

I guess the hardest part would be step 6. One idea might be to have an optionally compiled in service layer that intercepts state service calls and discards responses that would indicate data from heigher block heights.
Any opinions on this idea?

I guess the hardest part would be step 6. One idea might be to have an optionally compiled in service layer that intercepts state service calls and discards responses that would indicate data from heigher block heights.

Configure the zebrad instance with no seed peers, then it can't possibly get any new blocks.
(We already have a debug_stop_at_height config for step 4, and a restart_stop_at_height acceptance test for steps 1, 4 & 6.)


[Feature] Enable setting a manual seed & seed behavior for next generation

Feature request: allow to control for the seed used.

@deklesen I had it on my list it is one of the necessary features that will be added soon (I added select seed behavior for next generation to the title)


Feature/unscaled gw cost

TODO:

[ ] fix the expected value in test
[ ] test in FGW

Codecov Report

Merging #348 (70ac80c) into main (1feb317) will decrease coverage by 0.09%.
The diff coverage is 73.33%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files

@@            Coverage Diff             @@
##             main     #348      +/-   ##
==========================================
- Coverage   88.30%   88.22%   -0.09%
==========================================
  Files          51       51
  Lines        5464     5484      +20
  Branches      824      565     -259
==========================================
+ Hits         4825     4838      +13
- Misses        522      527       +5
- Partials      117      119       +2

Impacted Files
Coverage Δ

src/ott/initializers/linear/initializers_lr.py
92.30% <ø> (ø)

src/ott/problems/quadratic/quadratic_problem.py
86.45% <69.44%> (-2.25%)
:arrow_down:

src/ott/solvers/quadratic/gromov_wasserstein.py
88.97% <85.71%> (-0.35%)
:arrow_down:

src/ott/geometry/geometry.py
92.66% <100.00%> (ø)

src/ott/initializers/quadratic/initializers.py
86.88% <100.00%> (ø)

LGTM for now!

Thanks, merging this now!


Added PlayerJoinEvent

Obviously the join message wont do anything untill you make it broadcast the join to the server, but you can use it for other things for now.

I havent tested it. Why? Run the server and join the game. Theres so much debug spam its unreal.

Okay it works i disabled debug by editing log4j.xml

This is fired too early. PlayerJoinEvent should really only be fired when the player has been fully spawned into the world.

Hmmm. I checked through Nukkit and its fired after the adventure settings have been sent. It does work however in this PR. I made it send a message to the player on join and it works. I only added so i could start on some plugins

I can update the PR to fix it

Voxelwind does not intend to be a copy of Nukkit and/or PocketMine. That's something I am against.

I know i dont want it to be. But an event fired when a player joins is just basic stuff that it should have. Ive fixed it in latest commit.

Alright - please squash your commits.

Cant you squash it on merge? Im sure i saw an option

Sorry, closing this. I failed at rebasing. It will create a new commit with chat even as well to make up for the spam (And i need chat event then i will stop making PRs)


cc-wrapper: adding a cc-wrapper-hook to the cc-wrapper

Description of changes
Adding a cc-wrapper-hook to the cc-wrapper. The ld-wrapper already has a post-link-hook. My main motivation for adding this is it will allow us to reliably generate compile_commands.json files which are required for LSP servers to function for C/C++ projects. Existing tools for generating compile_commands.json are unreliable on nixos because they struggle to distinguish between the compiler wrappers and the compilers themselves.
See my earlier PR for an example of how this could be used: https://github.com/NixOS/nixpkgs/pull/192694. This PR is just the infrastructure part of my earlier PR. Also see the discussion on discourse: https://discourse.nixos.org/t/using-nix-infrastructure-to-reliably-generate-compile-commands-json/21961/20
I am hoping that this smaller PR will be easier to merge!
Things done

add cc-wrapper-hook to cc-wrapper.sh.
document both cc-wrapper-hook and post-link-hook.

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Looks like it was added here: https://github.com/NixOS/nixpkgs/pull/105026

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/using-nix-infrastructure-to-reliably-generate-compile-commands-json/21961/21


metadata on resumable creation

Please merge following #37 , which fixes the forced order and code 502.

My bad - my repo was not in sync.
Done.

OK, I can merge this, since it's very unobtrusive, but I would like to see a follow up PR with the other parts I mentioned, as well as tests.

Will do - I have a need for setting metadata on existing objects too.
For now I use gsutil setmeta but interestingly it has a batch option to set in bulk. I will check its code to get inspired for the same here.
Thanks @martindurant, my first merge ! (am only a coder by trade)


TRAINING LMDB GENERATING

Hi,Steven. I meet the problem when trying the train mode. I extract the features like the RULSTM, but I only get the data.mdb and lock.mdb. I found the data.lmdb in the eval mode, and the value include global features, hand features ,hand bbox. It seems that the method in LSTM cannot provide us the correct lmdb for training. Could you please tell me how can I get the whole LMDB including hand&object features? Also, I wonder how can I make .lmdb type document(not .mdb).
Thanks!

Hi, Derek, here is the code I used to preprocess, hope that helps.

Hi, Derek, here is the code I used to preprocess, hope that helps.

This will be very helpful to our work! Thank you for sharing the code.

Hi. @Derek0907 . Did you get the right lmdb file?


Map resource

Adds a map_resource() function that takes the URL of an endpoint as the first arg, the resource class as the second, and passes along **kwargs.
Resolves #299.

I think we would be better off implementing a sort of add_resources method that accepts a dict/list of routes and their respective Resource classes (similar to the url routing setup in Django)

@joshfriend Ok. I'll explore that route and open a new PR if I come up with a solution.


[Feature request] element picker & inspector for the embedded view?

You can access the devtools for the embedded view and there should be a picker there. See the screen recording below:
Demo


install dlib for Python3.5.2 in Ubuntu16.0.4

I followed these commands:
git clone git@github.com:davisking/dlib.git
cd dlib/
python setup.py install --yes DLIB_JPEG_SUPPORT
python python_examples/face_detector.py examples/faces/2007_007763.jpg

However I got this error:
   import dlib
  File "/home/mona/.virtualenvs/cv/lib/python3.5/site-packages/dlib-19.0.99-py3.5-linux-x86_64.egg/dlib/__init__.py", line 1, in <module>
    from .dlib import *
ImportError: /usr/lib/x86_64-linux-gnu/libboost_python-py27.so.1.58.0: undefined symbol: PyClass_Type

How can this be fixed?

I just tried these commands on ubuntu 16.04 and they worked as expected.  It looks like you have multiple incompatible versions of python installed.  In particular, boost is apparently compiled against a different version of python than the one you are running.  If you install everything through apt-get it will all be correct.

Although, while those commands you posted work, if you try them with python3 instead of python on ubuntu 16.04 they don't work.  This is because of name changes in boost libraries, which I just fixed in dlib's cmake scripts.


incompatible with Stardew 1.6

I love this mod so much! (Specifically Fishing Info Overlays) and was wondering if you could update it for 1.6! If not that's okay. I know you haven't made a commit to the repo in over 3 years, but I find the mod so helpful!

@barteke22 Are you updating your mods for 1.6? Thank you!

I think I can try converting this as it’s under an MIT license which means it’s fully open source. I have to learn c# first though………

I fixed it. A lot of methods don't work. There may be some minor problems

I fixed it. A lot of methods don't work. There may be some minor problems

@crazy531 Would you be able to upload it here please? Thank you!

FishingInfoOverlays 1.1.5.zip
@ER-95

@crazy531 Thank you! When you say "a lot of methods don't work", do you mean a lot of things were broken and had to be fixed? Or a lot of the mod still doesn't work?

@crazy531 thank you so much! I was trying to patch it myself but couldn't do it 👍

@crazy531 one small problem the lines you're using int.Parse could be replaced by an int.TryParse to avoid this mess of the log

@crazy531 Thank you! When you say "a lot of methods don't work", do you mean a lot of things were broken and had to be fixed? Or a lot of the mod still doesn't work?
@ER-95 The fishing minigame box couldn't override the original BobberBar, couldn't find the right scale, Vector2 calculations were too difficult for me, I removed this feature myself and replaced it with another mod. I like is the top-left prediction function

@benny-dreamly  The main change is the structure of the item, the type int of itemid has been changed to string, and the unique value is prefixed with (O). There are many places in the code that require int.TryParse conversions

@crazy531 one small problem the lines you're using int.Parse could be replaced by an int.TryParse to avoid this mess of the log getting spammed with lots of red error messages like this: [18:27:29 ERROR Fishing Info Overlays] This mod failed in the Display.Rendered event. Technical details: FormatException: Input string was not in a correct format. at System.Number.ThrowOverflowOrFormatException(ParsingStatus status, TypeCode type) at System.Int32.Parse(String s) at StardewMods.Overlay.Rendered(Object sender, RenderedEventArgs e) in C:\Users\xyr\OneDrive\game\Stardew Valley\FishingInfoOverlays\Overlay.cs:line 218 at StardewModdingAPI.Framework.Events.ManagedEvent`1.Raise(TEventArgs args) in SMAPI\Framework\Events\ManagedEvent.cs:line 101

The error should be that the itemid of DeluxeBait is non-numeric. The updated itemid has both numbers and non-numbers, and the experience is very bad

fix DeluxeBait is non-numeric
Uploading FishingInfoOverlays 1.1.6-unofficial.2-cyxyr.zip…

I've ported the mod to 1.6, but won't be making an official release yet.
From a glance at 1.6, it appears the fishing has been fully redone from scratch, so the hardcoded rewards (like Nuts & bosses) might no longer be accurate / lead to save mutations.
I'm also thinking of adding a performance mode that doesn't constantly simulate fishing (will miss out on fish added via weird methods), as 1.6 make it easier to just add fish via CP (so hopefully there'll be less mods hacking them in).
Here's the current 1.6 port (dunno if it works better or worse than @crazy531's:
FishingInfoOverlays.zip

@barteke22 When you say it could lead to save mutations, would you be able to elaborate on that please? Do you mean corrupted saves or something else?

Unfortunately the old fishing (and likely hasn't changed much on that part) had a lot of hardcoded stuff all over the place.
Like Nuts / Furniture editing specific fields directly on Game1.player instead of the passed Farmer object.
Most of the hardcoded stuff could've been bypassed by just passing a 'dummy farmer' to the function, but those specific ones had to be undone after each 'cast' instead.  If any of those flags have changed, the real player might no longer be able to catch some item
All the things I tested so far seemed to work the same, but I'll need to properly examine the code (as fishing used to be, and probably still is) all over the place.

Thank you for coming back to this even after 3 years. I really appreciate it.

Welcome back. The latest fix code is in  repo，Hope it helps you

I want to say that the mod, for some reason, causes an issue when quitting to desktop from in a save. It causes the game to hang and not close for like 4-5 min XD

I'm not experiencing that on my end, does your log say anything?
This might be a working version (don't have time to run splitscreen / multiplayer / thorough tests).  Make sure to delete old config files.
FishingInfoOverlays.zip

I'm experiencing the same thing, where is the log file? I can upload it

Using the version you just linked, Zoom Level affects the size of the overlay. UI Scale doesn't affect it though.
Only really noticeable because I use a mod that lets me quickly change zoom level, which causes this mod's overlay to get smaller and bigger.

Log instructions + link generator: https://smapi.io/log

Smapi Log Shows errors with this mod and the fish/percentages don't show up in the mines fishing levels.


identification of potential masterisation opportunities

as an admin I want to run the masterisation script to identify potential synonyms
Identify and list all localities that cannot break any more on max zoom level

https://github.com/healthsites/healthsites/blob/develop/django_project/localities/management/commands/generate_masterization_by_cluster.py

we don't have masterization anymore


[Suggestion]Add a parameter to start kibana eveloment mode wihout SSL

2 issues found:

run command "npm start" after generate the plugin. It run kibana automatically, no need to  run command  "bin/kibana --dev" as described in README.md. see as following details:
npm start

<EMAIL_ADDRESS>start /test/my-new-plugin
plugin-helpers start

watching for changes  (109 files)
managr    log   [02:32:17.819] [info][listening] basePath Proxy running at https://localhost:5601/vgb
optmzr    log   [02:32:20.793] [info][optimize] Lazy optimization of bundles for test_plugin, kibana and statusPage ready
optmzr    log   [02:32:20.829] [info][optimize] Lazy optimization started
optmzr    log   [02:32:20.856] [info] Plugin initialization disabled.
server    log   [02:32:20.895] [info][optimize] Waiting for optimizer completion
server    log   [02:32:20.929] [info][status][plugin:kibana] Status changed from uninitialized to green - Ready
server    log   [02:32:20.976] [info][status][plugin:elasticsearch] Status changed from uninitialized to yellow - Waiting for Elasticsearch
server    log   [02:32:20.998] [info][status][plugin:test_plugin] Status changed from uninitialized to green - Ready
server    log   [02:32:21.003] [info][status][plugin:devMode] Status changed from uninitialized to green - Ready
server    log   [02:32:21.015] [info][status][plugin:kbn_doc_views] Status changed from uninitialized to green - Ready
server    log   [02:32:21.023] [info][status][plugin:kbn_vislib_vis_types] Status changed from uninitialized to green - Ready
server    log   [02:32:21.027] [info][status][plugin:markdown_vis] Status changed from uninitialized to green - Ready
server    log   [02:32:21.044] [info][status][plugin:metric_vis] Status changed from uninitialized to green - Ready
server    log   [02:32:21.050] [info][status][plugin:spyModes] Status changed from uninitialized to green - Ready
server    log   [02:32:21.055] [info][status][plugin:statusPage] Status changed from uninitialized to green - Ready
server    log   [02:32:21
.062] [info][status][plugin:elasticsearch] Status changed from yellow to green - Kibana index ready
server    log   [02:32:21.064] [info][status][plugin:table_vis] Status changed from uninitialized to green - Ready
server    log   [02:32:21.076] [info][status][plugin:testsBundle] Status changed from uninitialized to green - Ready
server    log   [02:32:21.088] [info][listening] Server running at https://localhost:5603
optmzr    log   [02:33:45.058] [info][optimize] Lazy optimization success in 84.22 seconds

Based on the situation described above, I can not start kibana without enabling ssl. to work around this issue, i just changed the code in<EMAIL_ADDRESS>add a parameter '--no-ssl':
var args = ['--dev', '--no-ssl', '--plugin-path', plugin.root];

reference: https://github.com/elastic/kibana/blob/master/CONTRIBUTING.md
When Kibana runs in development mode it will automatically use bundled SSL certificates.
My suggestion is: add a parameter --no-ssl when run command "npm start" in the my plugin directory to keep consistent with kibana.

You can accomplish this by running npm start -- --no-ssl.

JK, that's not true.

This is now possible, just as lucas said:
npm start -- --no-ssl