CommonSubExpression Pass added

CommonSubExpression Pass added, Along with duplicate constants issue fixed in PrecomputePrune

Common Sub Expression will basically take out common expression and define only once,
for example:
If the Graph has operation like:
a = sqrt(y)
b = sqrt(x)
c = elementwise_add(a, b)
d = sqrt(x)
e = tanh(d)
Then after CommonSubExpression Pass applied the Graph will become as below:
a = sqrt(y)
b = sqrt(x)
c = elementwise_add(a, b)
e = tanh(b)    ## --> The common expression is gone

@ZihengJiang can you do a codereview?

@ZihengJiang/ @Tqchen, request you to provide your valuable comments...or any issue regarding this feature... Thanks!

@ZihengJiang , @tqchen plz conclude on this pull request, thanks!

Thanks for the PR. I will take a look tonight or tomorrow

@ZihengJiang , your review comment is fixed now, would you please conclude on this PR, thanks!

@ZihengJiang , would you please conclude on this PR, thanks!

Hi @PariksheetPinjari909 , in the PR, you find the common nodes by stringified operators, but seems I do not find any place to modify the graph. Correct me if I am wrong.

Hi @ZihengJiang , i have updated clear comments now where the current graph node is replaced with commoned node if macthes, @ Line no: 29, common_sub_expression.cc.
Please let me know in case any other query/concern, Thanks!

@ZihengJiang , would you please conclude on this PR.

Maybe it's a general question for all optimization passes. How many times will this pass be applied to an input graph? Can we expect that there will generate new CSEs after other optimization passes are applied?

c.f. #518 we will redirect further changes to tvm repo, please open a new PR there. Please invite the original set of reviewers when the new PR is opened so they can review and approve the changes


[10.0] Add indexes to those columns used to lookup data in the database

Digging into the code I see that users.stripe_id and subscriptions.stripe_status are both used in the query builder to query data in the database. Additionally, subscriptions.user_id is used to load the relationship between users and subscriptions.
I can't see why you wouldn't want these rows to be indexed for faster lookups, especially with big datasets.

Can we make a single index for user_id,status on the subscriptions table or do we query on status separately by itself?

stripe_status is used in multiple scopes like incomplete, past due, etc. I'd probably leave them separately but in either case, it would be better than having no indexes.


chore: upgrade to storybook 7 & vite 4

Current dependencies on/for this PR:

main

PR #2219   👈

This comment was auto-generated by Graphite.

⏳ This pull request is set to merge as part of a Graphite merge job
Stack job ID: UWDuAwWbBMCT2ACPK8ZX.
See details on graphite.dev


Update Ubuntu to 22.04 from 20.04

What:
Update Ubuntu to 22.04 from 20.04
Why:
We are migrating Ubuntu base images to 22.04.
Testing:
I built the image and it was successful

Yes. I have a setup ready to test this. Will merge it once I am satisfied. Thanks for the review.

Verified this.


Transition Mixin breaks with transform

Transform inside a transistion mixin makes all transition riming set to 0

How did you resolve this?  I'm having issues with transitioning transform, where the time/ease properties are not included in the .css output.
If I transition all, bourbon outputs:
// .scss syntax
@include transition(all 500ms $ease-in-out-expo);
// .css output
transition: all 500ms cubic-bezier(1, 0, 0, 1);

but if I transition transform bourbon doesn't include the time or ease
// .scss syntax
@include transition(transform 500ms $ease-in-out-expo);
// .css output
transition: transform;

I am having this same issue. It used to work in previous versions but I recently updated dependencies and this bit me.

For anyone else, after a little more digging: https://github.com/thoughtbot/bourbon/issues/794


Navbar no longer showing after school list added

When I added the school list component, the navbar is no longer showing, but all the unit tests work somehow

Now a unit test is failing so I have a better idea of where the bug is, guess I forgot to run the unit tests

Here's the error message:

Fixed

Nevermind, unit tests working but still not showing up

Now it works... for now


BWA App Stops Working After Installing Plugin

Hi @vincedarley,
Amazing work on this plugin.  In the readme it seems to indicate that you can use both the Home app and the BWA app to control the spa.  However, after installing the plugin, when I attempt to use the BWA app I get:

Thoughts?

I have the same issue.
When I disable the plug-in on Homebridge, it is working again.
The BWA-App is only working on cloud mode.
Is this an expected behavior?

Pretty sure this used to work, but also, equally doesn't work for me anymore. I assume something has changed on the BWA app side of things to prevent this dual use.

not getting the same error (about another smartphone0, just fails to connect. When I stop the plugin, then it connects from the mobile app again.


Opens in the wrong position

The input is below the date picker while the arrow is pointing the wrong direction and the date picker seems to be a bit off.

My html:
<input
  class="input-box form-control"
  placeholder="Select a date"
  angular-mydatepicker
  name="mydate"
  (click)="dp.toggleCalendar()"
  [(ngModel)]="selectedDate"
  [options]="myDpOptions"
  #dp="angular-mydatepicker"
/>

My typescript:
  // ...
  myDpOptions: IAngularMyDpOptions = {
    minYear: 1960,
    maxYear: 2150,
    dateRange: false,
    dateFormat: 'dd-mm-yyyy',
  };
  myDateInit: boolean = true;
  selectedDate: IMyDateModel = null;
  // ...

Notes:

I'm using boostrap for styling, with some color modifications
The element is inside of a <app-form-group> that is inside of a <form>
if i change the position of the <lib-angular-mydatepicker-calendar> to relative it moves the date picker below my input box but it still it to far a away from the input box, around 45px below the place it is supposed to be.

Did you try to set parent element of input box position: relative style like here: https://github.com/kekeh/angular-mydatepicker/wiki/Own-style-in-input-box-(no-bootstrap)


Enable signing

Fixes #27

:white_check_mark: Build vssetup.powershell 2.1.0+g931f17ada8 completed (commit https://github.com/Microsoft/vssetup.powershell/commit/57c388150a by @heaths)

:white_check_mark: Build vssetup.powershell 1.0.113 completed (commit https://github.com/Microsoft/vssetup.powershell/commit/931f17ada8 by @heaths)


fix(runtime-dom): style patching preserve v-show display property properly

fix #4768

我修改了codepen实例, 您可以直接在示例上测试不同情况

@zhaozhongyu Okay, Your PR looks better. I will close this one.


DataLake Knack conversion

Implements #4931 and #4932.
Tests not included as part of CI as there are some issues with some of them (possibly service flakiness).

View a preview at https://prompt.ws/r/Azure/azure-cli/5100
This is an experimental preview for @microsoft.com users.
(It may take a minute or two for your instance to be ready)
Email feedback to 'azfeedback' with subject 'Prompt Feedback'.


Support Puppet 8 and add automated tests for it

Merge PR #121 first.

yeah \o/

approved; in case you ask ;)


Fixing various typing issues

Description
This pull request addresses two minor typing issues:

MatrixFactoryGate.controlled did not use the same parameter name as the controlled method of the Gate protocol.
One of the decomposition rules tried to access the num_control_qubits attributes of a gate without ensuring that the gate is of type ControlledGate.

Please verify that you have completed the following steps

[x] I have self-reviewed my code.
[ ] I have included test cases validating introduced feature/fix.
[ ] I have updated documentation.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (4af3cdc) 95.08% compared to head (abefda2) 95.08%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #87   +/-   ##
=======================================
  Coverage   95.08%   95.08%
=======================================
  Files          49       49
  Lines        3135     3135
=======================================
  Hits         2981     2981
  Misses        154      154

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


sqelf in Docker doesn't listen on 12201/TCP

Despite the README's claim that it can ingest GELF via UDP or TCP, the Docker image only listens on the UDP socket.
$ k exec -it -n adt-system seq-gelf-translator-64b78f6559-tvxhl -- bash -c "apt update; apt install -y iproute2; ss -nlp | grep sqelf; echo Done"
Hit:1 http://archive.ubuntu.com/ubuntu bionic InRelease
Hit:2 http://archive.ubuntu.com/ubuntu bionic-updates InRelease
Hit:3 http://archive.ubuntu.com/ubuntu bionic-backports InRelease
Hit:4 http://security.ubuntu.com/ubuntu bionic-security InRelease
Reading package lists... Done
Building dependency tree
Reading state information... Done
31 packages can be upgraded. Run 'apt list --upgradable' to see them.
Reading package lists... Done
Building dependency tree
Reading state information... Done
iproute2 is already the newest version (4.15.0-2ubuntu1.3).
0 upgraded, 0 newly installed, 0 to remove and 31 not upgraded.
udp  UNCONN 0      0                                          <IP_ADDRESS>:12201                     <IP_ADDRESS>:*              users:(("sqelf",pid=7,fd=6))
Done

Hi @rjhenry :wave:
What do you get from docker inspect for that container image? Are you setting the GELF_ADDRESS environment variable with a tcp:// prefix? The server will listen on UDP by default.

I'm running this on top of containerd in a Kubernetes cluster; there's
no docker installed! Inspecting with containerd's info tool gives
the following:
# microk8s.ctr c info
e382338439ef0352901abe97a04c5714fc6ff3aeec46f2f65f189bb5b3656b14 | grep
-i gelf
        "io.kubernetes.container.name": "seq-input-gelf",
        "io.kubernetes.pod.name": "seq-gelf-translator-647b8ff5d7-m92pw",
    "Image": "docker.io/datalust/seq-input-gelf:2.0",
                "HOSTNAME=seq-gelf-translator-647b8ff5d7-m92pw",
                "GELF_ADDRESS=",
                "SEQ_GELF_TRANSLATOR_PORT_12201_TCP_PROTO=tcp",
                "SEQ_GELF_TRANSLATOR_PORT_12201_TCP_ADDR=<IP_ADDRESS>",
                "SEQ_GELF_TRANSLATOR_PORT_12201_TCP_PORT=12201",
                "SEQ_GELF_TRANSLATOR_SERVICE_HOST=<IP_ADDRESS>",
                "SEQ_GELF_TRANSLATOR_PORT=tcp://<IP_ADDRESS>:12201",
                "LOGSTASH_SERVICE_PORT_GELF_TCP=8081",
                "SEQ_GELF_TRANSLATOR_SERVICE_PORT_GELF_TCP=12201",
                "SEQ_GELF_TRANSLATOR_SERVICE_PORT=12201",

"SEQ_GELF_TRANSLATOR_PORT_12201_TCP=tcp://<IP_ADDRESS>:12201",
                "source":
"/var/snap/microk8s/common/var/lib/kubelet/pods/ca94cfaf-a628-4448-be8c-174e8ed45294/containers/seq-input-gelf/7df8e178",

I'll give it a go specifying a tcp:// prefix. I've just re-read the
README and it does specify that is should be set to either tcp:// or
udp://, whereas when I first read the documentation I read that as it
supports either!
I think this is solidly user error, but will test and let you know
before closing the issue.

I'm still seeing issues, but I think that they may be at the orchestration level and not the container level. Many thanks!

No problem! If you are running into more issues I’d love to hear about them and anything we can do to improve the experience 🙂 It can be tricky for us to run in all the different shaped environments ourselves so user reports are really valuable.


Comment script fails, unexpected exception

VSCode Version: Code 1.15.1 (41abd21afdf7424c89319ee7cb0445cc6f376959, 2017-08-16T18:07:25.676Z)
OS Version: Windows_NT x64 6.1.7601
Extensions:

Extension
Author (truncated)
Version

project-manager
ale
0.20.0

gitignore
cod
0.5.0

git-extension-pack
don
0.1.3

githistory
don
0.2.3

gitlens
eam
4.4.3

vscode-great-icons
emm
2.1.15

json-tools
eri
1.0.2

vscode-git-tags
how
1.1.0

coldfusion
ili
0.0.11

vscode-cflint
Kam
0.2.0

autotrim
Nat
1.0.6

vscode-docker
Pet
0.0.16

startanyshell
rem
0.3.1

cfml-comment-tags
trs
0.2.2

shell-launcher
Tyr
0.1.3

sort-lines
Tyr
1.3.0

terminal-tabs
Tyr
0.1.1

gitflow
vec
1.1.1

vscode-todo-highlight
way
0.5.9

vscode-open-in-github
ziy
1.2.7

Steps to Reproduce:

Open scripted file.
press CTRL + /

Reproduces without extensions: No
It seems to be a feature of ( coldfusion|ili|0.0.11 ), but that code has not been updated in a long time. Happened after last update of VS Code. (as far as we both noticed).

@sosensible When you say "scripted file", what are you referring to? JavaScript? TypeScript? CoffeeScript? Something else?
You mention that you suspect this may be related to the coldfusion extension. So you cannot reproduce the problem if you start VS Code with code --disable-extensions, correct?

@sosensible More details please


Replace deprecated data template_file with templatefile function

what

Replace deprecated data template_file with templatefile function

why

Users of arm architecture computers cannot use deprecated template provider

references

closes #87

Please see comment and run make pr/readme locally to fix the Auto Format failure.

It seems that the bot may not be able to update the PR. Could you allow the bot to make changes ? Usually personal forks allow this out of the box and only company specific forks prevent this.

I have added the bot as a contributor but its owner must accept the invitation so that I may assign the proper permission.

cc: @aknysh @Gowiem for visibility. Not sure what to do here. Usually we do not need to accept the invite. Perhaps there is another workaround ? Usually open source forks of our repos that are non-company specific orgs work out of the box.
Also, please try pushing a new commit to rerun the auto format check.

We could. I think another way is if you could run make pr/auto-format locally and push up the files. If you have time, could you do this ?


Invalidator

As suggested, both in .meta and in the toplevel config.json

I can do that. I thought i had but clearly not


ENH: integration with NEO

(Not for 0.7)
The NEO project does lots of IO and time series processing
https://github.com/NeuralEnsemble/python-neo
We should explore with the NEO authors how we could work together / make the packages more interoperable if they aren't already / explore where combining functionality might yield interesting perspectives / etc.
cc @samuelgarcia

I think we have decent integration via the *Array classes, feel free to reopen if people have further needs


Game Analyzer

This PR creates a game analyzer which does the following things:

[ ] it replaces dangerfinder
[ ] it estimates the certainty of us having the ball to decide if we should attack, defend or something in between
[ ] it estimates which robots are dangerous (both us and them)
[ ] it tries to detect weaknesses in the positions (both us and them)

The goal vision data is also incorrect, at least in the interface. Although I do suggest we merge this one soon because it's getting big

The goal vision data is also incorrect, at least in the interface. Although I do suggest we merge this one soon because it's getting big

What exactly do you mean with the goal vision data here? The merge blockades?

The goal vision data is also incorrect, at least in the interface. Although I do suggest we merge this one soon because it's getting big

What exactly do you mean with the goal vision data here? The merge blockades?

I'm not sure. the goalvision data in the interface is incorrect.


Promotion of variables after assignment in try

Take the following code:
import 'dart:math';

void main() {
  String? s1;
  String? s2;
  Random r = new Random();
  bool b1 = r.nextBool();
  bool b2 = r.nextBool();
  if (b1) {
    s1 = giveMeString();
    s2 = giveMeString();
  }
  if (s1 == null) {
    print("s1 was null");
    return;
  }
  if (s2 == null) {
    print("s2 was null");
    return;
  }
  // s1 and s2 are now non-null.
  onlyAcceptsString(s1);
  onlyAcceptsString(s2);

  try {
    // s1 = giveMeMaybeString(b2);
    s1 = giveMeStringOrCrash(b2);
  } catch (e) {
    print("Crashed!");
  }
  if (!b2) {
    s2 = giveMeStringOrCrash(b2);
  }

  // Now s1 is nullable.
  onlyAcceptsString(s1);
  onlyAcceptsString(s2);
}

void onlyAcceptsString(String s) {
  print("Happy!");
}

String giveMeString() {
  return "42!";
}

String? giveMeMaybeString(bool giveString) {
  if (giveString) return "42!";
  return null;
}

String giveMeStringOrCrash(bool crash) {
  if (crash) {
    throw "booh!";
  }
  return "42!!";
}

I'm puzzled by s1 being nullable after the try. Either it's assigned a String and thus still a String or it isn't and it's also still a String. I understand that if I had uncommented the first line in the try and it had been
    s1 = giveMeMaybeString(b2);
    s1 = giveMeStringOrCrash(b2);

it could be null, but without it it can't.
Digging around in the flow analysis I see that in tryCatchStatement_bodyEnd it does a conservativeJoin, basically removing all promotion for written variables:
https://github.com/dart-lang/sdk/blob/c7625fc68dfb829d7ab9a3a6cbf50f069ad2cc84/pkg/_fe_analyzer_shared/lib/src/flow_analysis/flow_analysis.dart#L4084-L4098
I've hacked around and noticed there's a nonPromotionHistory available. Utilizing that I've applied the following hack:
    // Hack: Remove written variables that wasn't ever demoted.
    Set<Variable> written = {};
    for (Variable v in info._written) {
      VariableModel<Variable, Type> vModel =
          afterBody.infoFor(info._written.first);
      if (vModel.nonPromotionHistory != null) {
        written.add(v);
      }
    }

    FlowModel<Variable, Type> beforeCatch =
        beforeBody.conservativeJoin(written, info._captured);

which seems to do what I'd want: Accept
    // s1 = giveMeMaybeString(b2);
    s1 = giveMeStringOrCrash(b2);

as non-nullable and mark
    s1 = giveMeMaybeString(b2);
    s1 = giveMeStringOrCrash(b2);

as nullable.
It also still demotes for instance
  Object? foo;
  foo = "Hello";
  if (foo is String) {
    try {
      foo = 42;
    } catch (e) {
      print("Crashed!");
    }
    onlyAcceptsString(foo);
  }

but is happy about
  Object? foo;
  foo = "Hello";
  if (foo is String) {
    try {
      foo = "still string";
    } catch (e) {
      print("Crashed!");
    }
    onlyAcceptsString(foo);
  }

Can we have something like this? Or is there some reason why this wouldn't work?
/cc @stereotype441 @lrhn @johnniwinther

This is an interesting idea.  My intuition is that using nonPromotionHistory won't do exactly what we want, because it tracks failed promotions, not demotions.  But there's probably an approach we could take that would track demotions directly.
One of my goals for the quarter is to investigate possible improvements to flow analysis and type inference.  I'll add this to my list and look at it as soon as I can.


download.sh fails

Thanks for developing this and sharing it with us! 🎉  I was now remembering Racer (nostalgia hehe) and searching about it in GitHub I came across your repo. Discovering this is the best that's happened today! :)
It seems like Active Theory has published an unminified version of the scripts and changed several things (for instance the racer.js script was removed), so the download.sh script fails now. I'll try to adapt it to the new version of the scripts. (btw, could this be considered a wink to your project?)
Cheers :-)

Thank you for your kind words! Indeed, that same nostalgia is what drove me to make this :D
Is there an un-minified version now? On the same page? If so, that's amazing news, it'd be much easier to patch if needed. By all means, feel free to adapt the script :D
I honestly felt that page was stale by this point, while I don't expect my project to be the reason, it would be pretty surreal if my project was why they un-minified it xD

The client that's running on their servers atm doesn't run OotB so scraping the client and patching it will definitely be necessary. If need be, I can adjust the download script and prepare patches :)

I honestly felt that page was stale by this point, while I don't expect my project to be the reason, it would be pretty surreal if my project was why they un-minified it xD

Yep, I don't really know why they did it 🤔  (the only change I would have expected is the removal of the project hehe)
I spent some time creating a patch for the unminified scripts based in the previous one you created, but it didn't work so I had to add more fixes to the code.[1] Trying to fix those things I've changed too many things and it's a mess now, so I have to polish it yet :P
BTW, I discovered that by tweaking the WS server we can get the music to work in all devices while in the lobby or a race (FX sounds worked out of the box, but in order for the music to work the server should save and send whether each player supports playing audio via the webaudio property in the player object).
I'll polish what I have and send it to you so we can build from there :)
[1] Someone commented the piece of code which adds the "Start a new race", "Join a race" buttons, webkitAudioContext is deprecated so I had to include a polyfill which emulates it via audioContext, etc.

No probs, take your time! Thanks for helping out :D

Hey @avm99963, just wanted to know if you're still working on updating the scraper. If not, I could go ahead and update it myself if needed :)

I'm now a little bit busy since I'm in the middle of final exams at uni, but I'll send you what I have right now so you can take a look and continue building from there! I'll send it as a draft PR later today :) (UTC time)

Sure, no problem!

Sorry for the delay! I've uploaded it at #3.


fixed generic lib/* wildcards that globs subfolders

Would cause libs/leaflet-0.7.3 not to be found during installation

Wow, thank you!


ansible: Add CentOS Altra from equinix

Adding just one for now in order to run some ansible deployments on it to verify whether the problems seen in https://github.com/AdoptOpenJDK/openjdk-infrastructure/issues/2119 are specific to docker containers. Not currently added to nagios as this may be a temporary installation.
Signed-off-by: Stewart X Addison<EMAIL_ADDRESS>
Checklist

[x] commit message has one of the standard prefixes
[ ] FAQ.md updated if appropriate
[ ] other documentation is changed or added (if applicable)
[ ] playbook changes run through VPC or QPC (if you have access)
[x] for inventory.yml changes, bastillion/jenkins updated accordingly

The yamlint error isn't to do with the PR - It looks like a 503 error when trying to build a container

Re-ran the checks and it passed 👍🏻


Crashes on startup, presumably because of something in main (Fabric)

No other mods installed, crash log provides all other information.
crash.txt

That looks like you have some kind of corrupted/broken jar? The stacktrace at least doesn't seem to match the codebase. Try redownloading version 2.0.1 or 2.0.2 from curse.

Trust me, I've tried redownloading and the older version. I honestly don't know what the issue could possibly be, because Minecraft works, fabric works, and all of the dependencies work.

https://github.com/tr7zw/FirstPersonModel/blob/1.16/Core/src/main/java/dev/tr7zw/firstperson/FirstPersonModelCore.java#L51 the crash above has a null pointer in that line. That is impossible since it's assigned one line above. And the lines above and below can't be null either. Maybe try to run the game in Java 16 so it can produce a more usable crashreport. Or try to reinstall fabric to check that the Fabric version is correct. But other than that, honestly, no clue how that crash could happen. I mean an alternative would be to just give the 1.17 version a try since you are using Fabric and 1.16 is just in maintenance mode.

I don't think using 1.17 would be a possibility (I'm trying to use a modpack) but I'll certainly try reinstalling fabric and updating Java whenever I can get back to my computer.

This is fabric, the modpack should have a 1.17 version, or alternative 1.17 modpacks with the same mods^^ I don't think any fabric mod that is active in any way is stuck on 1.16.

Here's after updating both fabric and Java to 1
crash-2021-11-18_19.24.05-client.txt
6

Ok that is in fact the old version from Feb 17th. This issue that I guess depends on something like the system lang? has been fixed on May 11th, and should be in the 2.0.2 build. On Curse that build is marked as an alpha cause it has some issue with Forge, but with Fabric it fixed this issue that only seems to be happening for a few people.

That's seems to have fixed it, thanks! Had to look for that build in the version specific spots though (on the right bar on Curse at the bottom), couldn't find it in the files tab.

It's also in the files view ;)

Gonna mark it as Fabric only (because it somehow broke on Forge, but fixes bugs that can be important for fabric).


Hey, any updates on this project?

It was a side project while i was studding. Don't really plan on continue with due to time and switching engines to Unreal.


Fix(core): Openshift securityContext on global operator

In global mode the integration pod securityContext needs to be generated from the integration namespace.
Test done on global/non-global operator deployment on OCP 4.12.13
Release Note
fix(core): Openshift securityContext on global operator

Is this ready to merge?

Is this ready to merge?

Yes


[doc]fix error example

const total = wrapper.find(Bar).reduce((amount, n) => amount + n.prop('amount'));

In this example, total is an Object, because of there is no initial reduce value. we must set the initial value with 0 to get the right result.
And, the right result is 14.

Sorry, the mount docs also need fix. I start a new pull request. #2066


Update brew extension

Description

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

@alexander-cato can you mark the PR as non-draft so I can merge? Thanks.

Thanks for the quick review @nhojb!
Apologies for the initially confusing git history; I was unfamiliar with the operation of the Raycast publish command and surprised when it automatically opened a PR before I was quite finished.
I have since rebased the PR branch with a more granular commit history for easier review and future reference.
The code remains largely unchanged compared initial review.


rtl_nic formware load error during ipxe boot

During boot RancherOS (latest stable and also 0.4.0-rc3 tested) I get the following error.
INFO[0008] [11/17] [network]: Starting
[   34.730113] r8169 0000:01:00.0: Direct firmware load for rtl_nic/rtl8168e-2.fw failed with error -2
[   34.739268] r8169 0000:01:00.0 eth0: unable to load firmware patch rtl_nic/rtl8168e-2.fw (-2)
[   34.760587] r8169 0000:01:00.0 eth0: link down

But nic is used for ipxe boot before without problems.
[   32.358441] device vethaf897de entered promiscuous mode
[   32.364372] IPv6: ADDRCONF(NETDEV_UP): veth2de7183: link is not ready
[   32.371120] IPv6: ADDRCONF(NETDEV_CHANGE): veth2de7183: link becomes ready
[   32.474582] eth0: renamed from veth2de7183
INFO[0007] [6/17] [syslog]: Started
[   32.732714] veth2de7183: renamed from eth0
[   32.777691] docker-sys: port 1(vethaf897de) entered disabled state
[   32.785760] device vethaf897de left promiscuous mode
[   32.790801] docker-sys: port 1(vethaf897de) entered disabled state

Thanks for reporting this.  I too have seen some firmware loading issues.  Let me investigate if firmware is not being detected properly or maybe that firmware is wrong in the updated kernel.

@pwFoo Did iPXE ultimately fail?  I just looked in v0.3.3 we only packaged the following firmware
lib/firmware/qlogic/sd7220.fw
lib/firmware/bnx2x/bnx2x-e1-<IP_ADDRESS>.fw
lib/firmware/bnx2x/bnx2x-e1h-<IP_ADDRESS>.fw
lib/firmware/bnx2x/bnx2x-e2-<IP_ADDRESS>.fw
lib/firmware/bnx2/bnx2-rv2p-06-6.0.15.fw
lib/firmware/bnx2/bnx2-rv2p-09-6.0.17.fw
lib/firmware/bnx2/bnx2-mips-09-6.2.1b.fw
lib/firmware/bnx2/bnx2-mips-06-6.2.3.fw
lib/firmware/bnx2/bnx2-rv2p-09ax-6.0.17.fw
lib/firmware/e100/d101s_ucode.bin
lib/firmware/e100/d101m_ucode.bin
lib/firmware/e100/d102e_ucode.bin
lib/firmware/tigon/tg3.bin
lib/firmware/tigon/tg3_tso.bin
lib/firmware/tigon/tg3_tso5.bin

and basically the same for v0.4.0-rcX

It should boot up, but without a configured nic.
So I can't login with SSH.
RancherOS don't supports serial console output. To See it booting up I added "console=ttyS0,115200" as kernel param.
Could it DHCP related? I don't know why, but it fails with eth0 (static dhcp entry) and works fine with eth1 and eth2. After RancherOS is installed eth0 connects without problems.
After tried "ros os upgrade' RancherOS install isn't able to reboot.
So maybe I could test it again in the next few days. Also with deleted static dhcp Entry.

Some more tests done at home... Also with RC5 behind a Fritzbox (dhcp) because it's my home network.
ssh login isn't possible if interfaces eth1 or eth2 is used. Set an ip address within the dhcp range works. RancherOS can't configure a dhcp ip address at eth0 because there is a static lease. Removed the static lease and iPXE boots ip without problems.

It could be an iPXE issue?
Static dhcp lease works fine after RancherOS is installed and to get an ip address at ipxe boot promt.
After loading vmlinuz and initrd RancherOS boots successful, but can't get the nic up.
Removed static lease, tried again and it works fine... ?!

@pwFoo This does not address your issue, but we are package the rtl firmware now in v0.4.  Will be in rc9.

Thanks for information.
I'll try it again with v0.4(rc9) and post results here.
Maybe issue will be solved with some updates and rtl firmware.

The errors appears again, but it's boot up and finally reachable with ping / ssh. So it should work.
Static dhcp lease configured at Fritzbox seems to be an problem for ipxe, but that shouldn't related to RancherOS...
So if nobody experience problems or erros the issue should be closes...

firmware d102e_ucode.bin for driver e100 missing
How I can fix it? Method A. Do steps:
0) My board's driver mount to my laptop on /media/a/rootfs

cd /media/a/rootfs/lib/firmware/
mkdir e100
copy d102e_ucode.bin to your board's drive
sudo cp /home/a/myb/d102e_ucode.bin /media/a/rootfs/lib/firmware/
Now it should be work.

Or use alternative method B. Just set this:


chore: update linters

Secure Coding Practices Checklist GitHub Link

https://github.com/RedHatInsights/secure-coding-checklist

Secure Coding Checklist

[x] Input Validation
[x] Output Encoding
[x] Authentication and Password Management
[x] Session Management
[x] Access Control
[x] Cryptographic Practices
[x] Error Handling and Logging
[x] Data Protection
[x] Communication Security
[x] System Configuration
[x] Database Security
[x] File Management
[x] Memory Management
[x] General Coding Practices

Codecov Report
Patch has no changes to coverable lines.
:loudspeaker: Thoughts on this report? Let us know!.


Cannot parse all comments

Describe the bug
I'm building a comment crawler and found that I can use replace_more(limit=None) to retrieve all comments, however I'm getting this error when trying to run get_comments():
AttributeError: 'MoreComments' object has no attribute 'body'

This means that exist a MoreComments objects on sub.comments, right?
Without the try/catch block, the RequestException is raised.
Script works like a charm on small submission or even with bigger (some thousands), however, I'm trying to parse this sub: https://www.reddit.com/r/wallstreetbets/comments/i5l1m1/weekend_discussion_thread_for_the_weekend_of/
Am I doing something wrong? There's a limit? There's another way to do this? I searched a lot but didn't find same problem.
GitHub repo: https://github.com/Mdsp9070/reddit-bot/tree/master/backend
To Reproduce
Steps to reproduce the behavior:

login to Reddit via praw
try to execute get_comments()

Expected behavior
Retrieve all comments, from at least, maximum layer
Code/Logs

def get_comments(url: str) -> list:
    sub = reddit.submission(url=url)

    while True:
        try:
            sub.comments.replace_more(limit=None)
            break
        except RequestException:
            continue

    comments: list = []
    for comment in sub.comments.list():
        comments.append(comment.body)

    return comments

System Info

OS: Manjaro
Python: 3.8.5
PRAW Version: 7.1.0

Can you please provide a URL which this code reliably fails on?

Oh I think i know what the issue is. replace_more doesn't leave things in a clean-state if it fails, so I think the RequestException, assuming it's raised, is what's causing the issue.

Can you please provide a URL which this code reliably fails on? Thanks for reporting!

But I provided! Sun URL and complete repository, if needed.
but, here's again: https://www.reddit.com/r/wallstreetbets/comments/i5l1m1/weekend_discussion_thread_for_the_weekend_of/

Oh I think i know what the issue is. replace_more doesn't leave things in a clean-state if it fails, so I think the RequestException, assuming it's raised, is what's causing the issue.

How I could handle this?

I decided to use pushshift to get all surface comments and then replace all replies.


Missing \Humbug\Command\Configure class

Current master branch is broken as the console application tries to load \Humbug\Command\Configure which is not included in the project.
Ref https://github.com/padraic/humbug/pull/117/files#diff-b7620e4d0183e0945db8e010aafee3c7R23

As I was working on some other feature I left it there accidently.

Thanks for the fix :)


How to make BeamSearch work together with AttentionWrapper?

I Want to use Attention during training phase(with Basic decoder),but I want to use BeamSearchDecoder During infer phase ,How I can do it,I have got in trouble a long time

@o-github-o Hi, we build separate graphs for inference and training to achieve this in the tutorial.
During the training time, we use tf.contrib.seq2seq.TrainingHelper with tf.contrib.seq2seq.BasicDecoder. During the inference time, we use the tf.contrib.seq2seq.BeamSearchDecoder to enable beam search.
You can take a look at the _build_decoder method in model.py.

@oahziur Thank you a lot, your method is work for me , my program work out


added validation for text input

Issue #3

Added text input validation in the form on an alert dialog.
Updated dependencies.

Can you please send a screenshot where we can see the changes you made. I need to check it before merging this code into the main branch.
Thank you!

@ShreyaPillai Sure. The following file was changed:

lib/widgets/addExpense.dart

I've attached screenshots below.

Thank you for your contribution! 👍 🎉


No environment variables created after authenticating with an API (Netlify Graph - API Authentication)

I was following along with the documentation for Netlify Graph here
Under the Getting Started section, after connecting to one of the providers (Spotify) and authenticating with it, I had no new environment variables created (after checking Settings > Build & Deploy > Environment), although I'm certain I've completed all steps directly.
Are there additional steps I need to take not written in the documentation?

Thanks for this feedback. We are archiving this repo now that these docs have moved over to docs.netlify.com. If you have further feedback on Netlify Graph, please fill out our survey.


Feature selection when testing specific packages has no effect

I have a workspace with two crates: foo, and bar. The crate foo has 3 features, a, b, and c. I wrote a shell script that tests that each feature works when individually enabled. The script is run from the crate root. It is:
FEATURES=(a b c)
for feature in "${FEATURES[@]}"; do
  cargo test -v -p foo --no-default-features --features "${feature}"
done

Instead of compiling the crate with each feature individually enabled and running those tests, cargo seems to always compile the crate with the default set of features. Here's what the verbose output emits when feature c is being tested, where a and b are default features:
> cargo test -v -p foo --no-default-features --features c
rustc --crate-name foo lib.rs --emit=dep-info,link -C debuginfo=2 --test --cfg 'feature="default"' --cfg 'feature="a"' --cfg 'feature="b"''

Note that feature="c" isn't enabled at all, and the default features are enabled. Everything works fine if I run cargo test --no-default-features --features "${feature}" from the root of the foo crate.

I believe this is a duplicate of https://github.com/rust-lang/cargo/issues/5849, but thanks for the report!


tour: set page title to the title of the tour step

Context: https://tour.golang.org/moretypes/16
Setting the title of the web page to the title of the tour step would make the back/forwards dropdowns in the browser be useful instead of just a repetition of "A tour of Go".

Like this idea, happy to do this one 😄

@ALTree Can I pick this issue up?


Search command

Can you write error in search if user haven't agreed to build database?
Current message:
(hsf)> search test
[-] An error occurred: 'NoneType' object has no attribute 'keys'!
(hsf)>

Hello, @thecakeisfalse
I am glad you interested in my work and project.
Thanks for reporting this bug, I'll fix it.
Regards,
Ivan Nikolsky (@enty8080)

@thecakeisfalse Bug fixed.


Pause button issue

song is paused still button is not converting from ⏸ to ▶.

over the song card? or in nowplaying?

In now playing.

Can you check if it's working now?

Yes,It is working now.


Routing extension requirement of routes having to exist can conflict with fuzzy route matching

The routing extension converts URL arrays to URL strings, which requires the routes to exist, otherwise a MissingRouteException is being thrown. This can conflict with fuzzy route matching, in case one wants to match on a part of a URL for wich no connected route exists.
If for example the following routes are connected:
$routes->connect('/members/about', ['controller' => 'Members', 'action' => 'about']);
$routes->connect('/members/view', ['controller' => 'Members', 'action' => 'view']);

and one wants to fuzzy match on
['controller' => 'Members']

so that all member URLs are being matched, this wont work, as no matching route for that URL array is connected.
Until this is fixed, one workaround would be to create dummy routes, ideally ones that cannot be reached, for example because they are being catched by other, previously connected routes, like:
$routes->connect('/members/about', ['controller' => 'Members', 'action' => 'about']);
$routes->connect('/members/view', ['controller' => 'Members', 'action' => 'view']);
// ...

// this will never be reached, as the previous /members/about route will
// catch the request, but it satsfies the routing extension requirements
$routes->connect('/members/about', ['controller' => 'Members']);

I would recommend adding a URL routes matching, that would solve quite a few of these issues without workaround, right?
You have definitely more options on those arrays than on string comparison.

I don't really want to drop string URL matching support. The decision to move string URL generation into the routing extension was to avoid lots of unneccesary Router::url() calls in the voters themselves, but later this turned out to be largely a caching problem, and unfortunately I forgot to move string URL generation once the caching problem was fixed.
I was thinking about alternative approaches, but I'll probably go for moving URL generation into the URL voter (which unfortunately isn't backwards compatible), though this would currently cause one unneccesary Router::url() call per item when using the URL voter, ideally I'll find a way to avoid that.
Regardless of that, adding a non-fuzzy URL array based voter might be an option. Not sure though what advantages (apart from probably a small performance gain) comparing URL arrays instead of string URLs would have? I'll have to think about that.

For those who'd like to test the new behavior, check out the 2.0 branch. URL array conversion (for the routes option) has been moved into the URL voter.

This is fixed and will be available in the soon to be released 2.0


Fix Riak race condition in test

TestMessageStoreResource.test_disconnect_kills_server leaves some unfinished Riak operations running, which results in a race condition on cleanup and occasional build failures.

:+1:


updated to 2.014 now no macros working

hello i recently updated to 2.014 the macros i previously imported are working but none of your sample macros will work and if i try to create new ones they will not work.  This is very difficult situation.

Date: 2017-01-31 03:32:34
ID: 2
Error occured in: Global
Count: 1
Message: ..\AddOns\GSE\GUI\Editor.lua line 132:
attempt to concatenate field 'classid' (a nil value)
Debug:
GSE\GUI\Editor.lua:132:
GSE\GUI\Editor.lua:132
(tail call): ?
[C]: ?
[string "safecall Dispatcher[4]"]:9:
[string "safecall Dispatcher[4]"]:5
(tail call): ?
...ace\AddOns\AskMrRobot\Libs\AceGUI-3.0\AceGUI-3.0.lua:314: Fire()
...obot\Libs\AceGUI-3.0\widgets\AceGUIWidget-Button.lua:28:
...obot\Libs\AceGUI-3.0\widgets\AceGUIWidget-Button.lua:25
Locals:
None

I can't replicate this.  What did you update from?

ive been updating everytime you have a new update now when i go to my /macro and type in the #showtooltip
/click R_Prot
then import my macro in that way from my old gs-mymacro folder saved to my desktop not under the addon folder it seems to work now very weird.

You know you can select the macro and choose Create Icon and just drag the ? Icon To the toolbar?  You don't need to go to /macro?
TimothyLuke

On 31 Jan 2017, at 8:46 pm, rpartridge07<EMAIL_ADDRESS>wrote:
ive been updating everytime you have a new update now when i go to my /macro and type in the #showtooltip
/click R_Prot
then import my macro in that way from my old gs-mymacro folder saved to my desktop not under the addon folder it seems to work now very weird.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


Update MD file on new branch 2017-21-16_05-11-27

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Few things outstanding to get to an A+

Hello,
@sairafurqan and I are going through your project and you have met all but two of the criteria for an A+. One of the things is really easy, and the other one is slightly more involved.
Here are the two things missing:

[x] Issues aren't closed (though all the tasks are completed) ; effective use of issues requires all resolved issues to be closed.
[x] Upon closer look, we haven't seen evidence of the "complex method chaining" criteria:

Complex method chaining

At least three advanced operations (for e.g anonymous or lambda functions in assign or apply functions)
At least two method chains

At the moment, we're leaning towards downgrading your project to an A, but if you can fix these things before April 24th, we can award you your contracted grade of an A+.
Of course, you are more than welcome to just accept the A, but please let us know either way what you decide.
Thanks,
Dr. Moosvi

All issues resolved for contracted grade A+.


Revert "This PR should fix #261"

Reverts quokkaproject/quokka#264
There is a bug duplicationg menu items @realli

I'm checking this


Joe/definitions

PR Title/Tagline
Python script to parse definitions and add to DB (only part 1 for now)
Run with python definitions.py -prod
This has already been run, adding to repo for the sake of the script being here
How to Test

Please add or build upon a testing card to the Notion page and link it here

Important Changes

Database change / migration to run
Environment config change
Breaking API change

Related Github Issues

#[Number of Issue]
Or, remember to link to an issue on the sidebar

Related PRs

#PRNUMBER

Checklist

[ ]  Database schema docs have been updated or are not necessary
[ ]  Code follows design and style guidelines
[ ]  Code is commented with doc blocks
[ ]  Tests have been written and executed or are not necessary
[ ]  Latest code has been rebased from base branch (usually develop)
[ ]  Commits follow guidelines (concise, squashed, etc)
[ ]  Github issues have been linked in relevant commits
[ ]  Relevant reviewers (Senior Dev/EM/Designers) have been assigned to this PR

see #108 for effects of this code.

definition contains '='

Thought that was the requirement, now changed.

This pr does not include the changes to display the definitions, only the script to generate them. You can see the definitions in #108 as the database has been updated.


spike_fn, reset_fn and surrogate_grad_fn as layer parameters

For spiking layers the spike and reset behaviors as well as surrogate gradients are currently defined by passing an ActivationFunction object at instantiation.
This has two disadavantages:

First having to instantiate an ActivationFunction is a bit tedious
The default behavior is not clear this way.

Passing the spike_fn, reset_fn and surrogate_grad_fn directly as layer parameters rather than an ActivationFunction would solve these problems. The ActivationFunction could then be instantiated inside the forward functionals.

done in 12754809f54b308f7ae17ad99a2fcfffb297c994


Does Virtual Display Driver reduce hardware life?

I wonder if VDD reduce hardware life xd thx for answer

My intuitive answer is: "Just as much as Microsoft word or a screenhot app that runs in the background 24/7 does.
It's just another piece of software running on a hardwareplatform built for running software.
I have a hard time comming up with a reason for even comming close to such an idea, could you explain your reasoning? I'm truely interested if I missed something in my reasoning.

The Virtual Display Driver (VDD) itself is a software-based solution, so it does not directly affect hardware lifespan. However, its  system resource requirements can have indirect effects on hardware longevity, for example your GPU working harder to display a 4k image over a 1080p image.


add env-file flag to build command

Fixes-###
What
Why
How Tested

The PR look ok. If you want you can also add 'env-file' to the 'profile' command, but that's nice to have (not a blocker for this PR).

The PR look ok. If you want you can also add 'env-file' to the 'profile' command, but that's nice to have (not a blocker for this PR).

will work on this today and open the PR


Can't find "Set-Cookie" section in the ev.Response.Headers in chromedp.ListenTarget()

the code I use is :
chromedp.ListenTarget(
		ctx,
		func(ev interface{}) {
			if ev, ok := ev.(*network.EventResponseReceived); ok {
				fmt.Println("event received:", ev.Response.URL)
				fmt.Println(ev.Type)

				if ev.Type != "XHR" {
				 	return
				 }

				fmt.Println(ev.Response)
				if ev.Response.URL == post_url {
					log.Printf(`cookies:%v`, rbcookie)
					m_ret_cookie = ev.Response.Headers
					return
				}
			}
		},
	)

I could fetch all the response headers except the Set-Cookies section:

so why the Set-Cookies section can't be fetched in chromedp.ListenTarget(), and how can I get the Set-Cookies section? Thanks

Some headers are sent by the Network.responseReceivedExtraInfo event. See the demo below.
package main

import (
	"context"
	"fmt"
	"log"
	"net/http"
	"net/http/httptest"

	"github.com/chromedp/cdproto/network"
	"github.com/chromedp/chromedp"
)

func main() {
	ts := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
		w.Header().Set("Set-Cookie", "session_id=test")
		_, _ = fmt.Fprint(w, `
<html lang="en">
<head>
    <meta charset="UTF-8">
    <title>Title</title>
</head>
<body>
Hello
</body>
</html>
`,
		)
	}))
	defer ts.Close()

	ctx, cancel := chromedp.NewContext(context.Background())
	defer cancel()

	chromedp.ListenTarget(ctx, func(ev interface{}) {
		switch ev := ev.(type) {
		case *network.EventResponseReceived:
			log.Printf("ResponseReceived: %+v\n", ev.Response.Headers)
		case *network.EventResponseReceivedExtraInfo:
			log.Printf("ReceivedExtraInfo: %+v\n", ev.Headers)
		}
	})

	err := chromedp.Run(ctx,
		chromedp.Navigate(ts.URL),
	)
	if err != nil {
		log.Fatal(err)
	}
}

Output:
2022/12/11 12:04:48 ReceivedExtraInfo: map[Content-Length:114 Content-Type:text/html; charset=utf-8 Date:Sun, 11 Dec 2022 04:04:48 GMT Set-Cookie:session_id=test]
2022/12/11 12:04:48 ResponseReceived: map[Content-Length:114 Content-Type:text/html; charset=utf-8 Date:Sun, 11 Dec 2022 04:04:48 GMT]


[BUG] MicrosoftOpenXRPlugin.dll missing for linux editor

Describe the bug
There's no MicrosoftOpenXRPlugin.dll for Unity linux editor - we can't build or run tests in our linux-based CI/CD pipeline.
To reproduce
Just open up a unity project with MRT3 on linux.
Expected behavior
No errors.
Screenshots
N/A
Your setup (please complete the following information)

Unity Version 6000.0.5f1
Using newest version of MicrosoftOpenXRPlugin

Target platform (please complete the following information)

Linux Editor

Additional context
N/A

Are there OpenXR extensions that you require from MicrosoftOpenXRPlugin?  If not, the recommended solution would be to remove MicrosoftOpenXRPlugin, as this is not a hard requirement.
@TheXRMonk what platform are you targeting?   Android, Stand Alone, UWP, ect?

We're targeting Android (Quest), but it seems like just removing the package creates a lot of errors in rest of the MRTK packages.

We're getting this warning:

With this setup:

But the MixedRealityFeatureTool does not contain a package named mixedreality.toolkit.input.openxrhandssubsystem

Just upgrade all packages - still getting the warnings above.
 Sorry for the inconvenience!


net/rpc: Make service prefix optional

Any interest in a register method that allows you to define a "root"-level service? Personally I think multiple per-program is an anti-pattern, but having that choice would be great regardless.
cheers

closing since we can just add some hacky wrapping to assume a prefix


I switched the wrong bte up Contract Address: BTE Token information

sorry about my english
i playing bitserial coin(BTE)
i want switch BTE from myetherwallet to BTE wallet
but switched the wrong bte up Contract Address: BTE Token information
i thinked this address is my BTE wallet address but wrong !
There is no way to get back bte ?

Tx
0x3fa3ae8ea8d920645099f8e997b5d1f40907ba8245b56a6ead5185a5f3c41d98

Date
2017-11-16 06:43:56 (GMT--8)

Block
4563992 (5939 confirmations)

From
0x94D132C78AC484F72E841BCC20f921d634902F83

To
Contract 0x6733D909e10DDedB8d6181b213dE32A30cEac7ed

this address BTE token:0x6733D909e10DDedB8d6181b213dE32A30cEac7ed
i thinked is my address BTE ;((

The BTE is an ERC20 smart contract that don't support the retrival for token sent to contract address. Unfortunately your tokens are loked in the contract address and can not be sent back to you address.

There are so many people mistakenly like me, that is the problem, now my
life of the night is very hard, give it back to me, really thank you,good
luck to the project !!
On Monday, November 20, 2017, Francesco F<EMAIL_ADDRESS>wrote:

The BTE is an ERC20 smart contract that don't support the retrival for
token sent to contract address. Unfortunately your tokens are loked in the
contract address and can not be sent back to you address.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/kvhnuke/etherwallet/issues/1285#issuecomment-345646018,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AgMMEpnxEbbiyHVKVf0o9x15sVAYpnHuks5s4UzKgaJpZM4QiDEE
.

And good luck for you !!!
On Monday, November 20, 2017, Thanh hong<EMAIL_ADDRESS>wrote:

There are so many people mistakenly like me, that is the problem, now my
life of the night is very hard, give it back to me, really thank you,good
luck to the project !!
On Monday, November 20, 2017, Francesco F<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:

The BTE is an ERC20 smart contract that don't support the retrival for
token sent to contract address. Unfortunately your tokens are loked in the
contract address and can not be sent back to you address.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/kvhnuke/etherwallet/issues/1285#issuecomment-345646018,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AgMMEpnxEbbiyHVKVf0o9x15sVAYpnHuks5s4UzKgaJpZM4QiDEE
.


Turn NumPy to static class

I think we should turn the NumPy class to be static.
It will look like below:
using np = NumSharp.Core.NumPy;

@dotChris90 , @fdncred What do you guys think?

Hooray, be more like numpy.

works for me


Blockindex implementation

Implemented blockindex instead of etherscan.
Most of changes are done in corebc-blockindex crate.

Other than the last change request I think it is ready.


📌[Spike] Apple Watch Refactor 2: List Issues (an Epic) for Improving the Apple Watch

Subject
The Litewallet team need to discuss what is needed in improving the Apple Watch / Litewallet experience.
Who?
First line is the Litewallet Team
Second line is Preview to the Litecoin Foundation (more announcement than collaboration)
When
We should have resolved the issues in the epic by v3.13.0 maybe sooner.

Please see #191


bug with IE conditional comments:

Minify version:
PHP version: 2.1.0

What steps will reproduce the problem?
1. insert IE conditional formating in HTML
2. <!--[if !IE]>--> is kept normally
3. but <!--<![endif]--> is emptied breaking contents

Expected output: both should be kept

Actual output: loop end is not present

Did you run the unit tests? Did any FAIL? no

Please provide any additional information below.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 6 Oct 2008 at 1:35

This should be fixed in R251. Can you verify this works for you?

Original comment by<EMAIL_ADDRESS>on 6 Oct 2008 at 2:16

Changed state: Fixed
Added labels: Priority-High
Removed labels: Priority-Medium

Original comment by<EMAIL_ADDRESS>on 7 Oct 2008 at 11:37

Changed state: Verified

Yes seems fixed for the issue i mentioned. thanks

Original comment by<EMAIL_ADDRESS>on 7 Oct 2008 at 10:16

Original comment by<EMAIL_ADDRESS>on 7 Oct 2008 at 3:35

Added labels: Release-2.1.0


Free shipping by product tag?

@christianblais thank you for an excellent app so far! I was wondering if there's an option to apply free shipping / rules to items with a certain tag, or point me in the right direction to implement it if it's even possible?

Unfortunately, Shopify doesn't send any tag information. Only line items, their price, name, weight, vendor and SKU. That'd be impossible for now. Sorry.


GKE tests fail because of a deadlock between Cilium and metrics-server

GKE pipeline (https://jenkins.cilium.io/job/Ginkgo-CI-Tests-Pipeline-GKE/) is failing because Cilium pods are in crashing.
The failure message from Cilium is
level=fatal msg="Unable to initialize Kubernetes subsystem" error="unable to retrieve the complete list of server APIs: metrics.k8s.io/v1beta1: the server is currently unable to handle the request" subsys=daemon

Our Kubernetes initialization code tries to retrieve information from API endpoint which is backed by metrics-server-v0.3.1 deployment. Pods from this deployment are not starting because Cilium has not started providing networking in the cluster yet.
This can be fixed by ignoring (preferably with logging) failures while retrieving non-critical K8s capabilities in Kubernetes subsystem initialization code.

We have the same issue for k8s on premise setup with kubeadm.
It last worked for us for docker images with the following sha:

sha256:73012faf74204832839f01eb1938df614271953cac1acd09a0540d3d0ae83c9d

It does not work for:

sha256:fa61df59725e58bbb9cc24bfb4dd3eb4e88bf5b50054e1eb995c2e5282c8bec1
latest

@jrajahalme and I hit this too.

Seems like https://github.com/cilium/cilium/pull/11253 fixed this


Fatal error: Cannot read property 'split' of undefined

I keep getting the error mentioned above when compiling. If I comment out "require('cssnano')()" the build works fine. Nothing has changed in my grunt fine and was working properly up until an hour ago..Any thoughts?

What does grunt postcss -v print?

Not sure if this helps at all, but I started experiencing errors with cssnano in node v0.10.33, but it was running fine with iojs-v2.3.3 when I switched with nvm locally.

I just updated Node to v0.12.7 and cssnano ran fine.  #ymmv (your mileage may vary)

@nDmitry - Running "postcss:dist" (postcss) task
Verifying property postcss.dist exists in config...OK
Files: css/global.css
Options: processors=[{},null,{"plugins":[null,null,null,null,null,null,null,null,null,null,null,null,null,null,null,null,null,null,null,null,null,null,null,null,null,null],"postcssPlugin":"cssnano","postcssVersion":"4.1.16","version":"4.1.16"}], map, diff=false, safe=false
Reading css/global.css...OK
Fatal error: Cannot read property 'split' of undefined
SDGL1388bb8ec:city-lift jjohnson29$

@jonknapp - I am running the latest.
node -v
v0.12.7

I get the same error with 'map'.
Fatal error: Cannot read property 'map' of undefined
I followed your guide, though. Commenting cssnano out helps getting rid of it.

Looks like cssnano compatibility problems with different versions of node and postcss.


Stop skipping builds for markdown changes

I still love the idea of shedding unnecessary build load and avoiding
unnecessary merge latency, but the implementation I submitted is quite
brittle:

Waiting for tests before merging is much less painful now that we're
using mergequeue, so let's remove the nobuilds to keep merging simple
and robust (though not as fast as might be ideal).

@ArthurBandaryk @CodingCanuck any other repos we need to do this to?

@ArthurBandaryk @CodingCanuck any other repos we need to do this to?

Just Respect, I think: https://github.com/reboot-dev/respect/pull/294

@ArthurBandaryk @CodingCanuck any other repos we need to do this to?

Just Respect, I think: reboot-dev/respect#294

I spy one in stout too, please give a look at the rest of the repos you've worked in @ArthurBandaryk, thank you!

Oops, good catch! I only did a local search in repos I had cloned: let's do a more exhaustive search.
Under reboot-dev, I also missed one under respect-integration: https://github.com/search?q=org%3Areboot-dev+filename%3Anobuild but I guess we don't care about that because we're turning that down.
Under 3rdparty, ditto for eventuals-grpc: https://github.com/search?q=org%3A3rdparty+filename%3Anobuild
So I think we only need to change eventuals, stout, and respect.

@ArthurBandaryk @CodingCanuck any other repos we need to do this to?

Just Respect, I think: reboot-dev/respect#294

I spy one in stout too, please give a look at the rest of the repos you've worked in @ArthurBandaryk, thank you!

Oops, good catch! I only did a local search in repos I had cloned: let's do a more exhaustive search.
Under reboot-dev, I also missed one under respect-integration: github.com/search?q=org%3Areboot-dev+filename%3Anobuild but I guess we don't care about that because we're turning that down.
Under 3rdparty, ditto for eventuals-grpc: github.com/search?q=org%3A3rdparty+filename%3Anobuild
So I think we only need to change eventuals, stout, and respect.

PR to remove this pattern from stout: https://github.com/3rdparty/stout/pull/25


Drag resize

Thought it should be an issue as we're working on it

fixed in dc241f5 ? @simonrharris

double click breaks things 👎

8425b4f


Lcc check

What does this implement/fix? Explain your changes.

Add check_lcc for all embedding methods.

default is True

to_laplace changed so that it works on graph with unconnected nodes.

to_laplace no longer checks if input is unconnected as current implementation no longer cares about unconnectedness.
lcc checking is now done in LaplacianSpectralEmbed.

Semipar is changed to only check lcc for the input graphs, but not for when computing the null via bootstrap. I chose not to put check_lcc arg for semipar. Assume that we want to know if the input pair of graphs are unconnected.
Updated tests accordingly.

Codecov Report

Merging #117 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #117      +/-   ##
==========================================
+ Coverage   96.62%   96.64%   +0.01%
==========================================
  Files          22       22
  Lines        1038     1044       +6
==========================================
+ Hits         1003     1009       +6
  Misses         35       35

Impacted Files
Coverage Δ

graspy/embed/base.py
96.66% <100%> (+0.11%)
:arrow_up:

graspy/utils/utils.py
91.09% <100%> (-0.07%)
:arrow_down:

graspy/embed/omni.py
100% <100%> (ø)
:arrow_up:

graspy/embed/ase.py
100% <100%> (ø)
:arrow_up:

graspy/embed/lse.py
100% <100%> (ø)
:arrow_up:

graspy/inference/semipar.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


[SPARK-5790][GraphX]: VertexRDD's won't zip properly for diff capability (added tests)

Added tests that @maropu created for vertices with differing partition counts. Wanted to make sure his work got captured /merged as its not in the master branch and I don't believe there's a PR out already for it.

/cc @maropu @ankurdave @srowen

Test build #28598 has started for   PR 5023 at commit 83bbd29.

This patch merges cleanly.

OK, is the idea that these are tests that pass, that should be included, or tests that fail and demonstrate a problem? Jenkins will tell us soon.

Test build #28598 has finished for   PR 5023 at commit 83bbd29.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/28598/
Test PASSed.

OK to merge, since tests are good. Does this mean SPARK-5790 is essentially resolved by this? sounds like it was fixed separately, and this is about getting in additional tests created by the parallel effort to address this.

Sorry @srowen I should've been more clear, but yes you're correct. SPARK-5790 ended up duplicating most effort from SPARK-1955, PR #4705, but @maropu had created tests with his that I hadn't included. Wanted to make sure those were captured and, since SPARK-5790 wasn't closed, thought it best to include those here. Does that make sense? Sorry about the confusion!


Added description to use packagecloud.io.

Relevant Issue (if applicable)
No issue.
Details
Added description to use packagecloud.io.

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


Add Code Of Conduct

Issue number : #21
Added file for codeofconduct in the repo

Hey @Akash20x you've pushed the changes in the main branch of your local repo, the correct method is to first make a separate branch then make changes and push.

I close this and please check my other pull request


c++ replay: chunking and concurrent downloads

added CURLlGlobalInitializer

If you did not already call curl_global_init(3), curl_easy_init(3) does it automatically. This may be lethal in multi-threaded cases, since curl_global_init(3) is not thread-safe, and it may result in resource problems because there is no corresponding cleanup.

Video looks strange when running from qcameras:

how about  #22195? I think it can handle switch between qcamera and normal video

Sorry, this comment was not supposed to be in this PR. It's happening on master and also on 22195. Let me move it into the appropriate issue.


task bucket size discrepancy

Buckets seems to be way bigger (10x) of the data it should be containing.

Investigating

Not found more evidences after many trials, I thing we can close it


Enhancement: Replace print() with log()

:scroll: Description
:bulb: Motivation and Context
Fixes https://github.com/getsentry/sentry-dart/issues/451
:green_heart: How did you test it?
:pencil: Checklist

[ ] I reviewed submitted code
[ ] I added tests to verify changes
[ ] I updated the docs if needed
[ ] All tests passing
[ ] No breaking changes

:crystal_ball: Next steps

Codecov Report

Merging #453 (519827b) into main (5039731) will increase coverage by 3.95%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #453      +/-   ##
==========================================
+ Coverage   89.98%   93.93%   +3.95%
==========================================
  Files          54        5      -49
  Lines        1698      165    -1533
==========================================
- Hits         1528      155    -1373
+ Misses        170       10     -160

Impacted Files
Coverage Δ

dart/lib/src/protocol/sentry_request.dart

dart/lib/src/protocol/debug_meta.dart

dart/lib/src/version.dart

dart/lib/src/protocol/sentry_event.dart

dart/lib/src/protocol/sentry_message.dart

dart/lib/src/protocol/sentry_user.dart

dart/lib/src/protocol/sentry_stack_frame.dart

dart/lib/src/scope.dart

dart/lib/src/platform/platform.dart

dart/lib/src/isolate_error_integration.dart

... and 37 more

Continue to review full report at Codecov.

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

Logs look like this now

Well without the double brackets around sentry


Increment the Countdown!

Request from Nick!
Please add a block to increment the Countdown.

https://github.com/shakao/pxt-countup is an extension already.

Probably needs 2 blocks
"Change countdown by  ____ (s)"
and "countdown" getter to get the existing countdown.

Adding a +1 since I'll be needing this pretty immediately for Carnival.


DEHYDRATED_USER + --signcsr sends certificate output to stderr

command_sign_csr first does some IO redirection, then calls init_system, which might call sudo:
https://github.com/dehydrated-io/dehydrated/blob/5c1551e946456f534cf46b6ebabe4353bf0b0530/dehydrated#L1762-L1767
Solution might be as easy as calling init_system before doing the redirects, but I don't know the internals.

Thx. I swapped the lines as suggested.
This will result in some informational messages being output to stdout as well, but that should be less of a problem as there was additional output before anyway and PEM format should ignore anything outside BEGIN/END blocks.


Add a method to get the data from a model

It's sometimes useful to be able to extract the data from model objects returned by ml_* functions. It currently takes a bit of digging to figure out what to do.
Here's my suggestion:
tbl.ml_model <- function(x) {
  sdf_register(x$data)
}

Then you can just get the data in the natural way, using tbl(a_model).

Is a tbl method appropriate or should we write something like ml_model_data?

I think I prefer ml_model_data() over a tbl() method. The first argument to tbl() is typically a data source (e.g., in sparklyr, a spark_connection object); I don't think models fit that mold very well.


Detox testing different home screens

I did some research into how we can reach different states in the app with detox. Here is what I tried:

importing the async storage module into detox tests - this fails because "there is no way detox can communicate with react native thread in runtime and change state"
I started thinking through how we could load state from a json file that detox could write to. We would need to add a flag to the .env file that would tell the storage code to read state from a json file in the source code. But I don't think this would actually work, since at runtime the app is on the device and no longer has access to a file on your computer.
The recommended way to mock in detox is to create a different version of each .js file you want to mock, with an .e2e.js extension, and then that gets used by the e2e tests. So we could create a different version of our app with a GUI way to reach different states, BUT....

We already have that - the test menu. So I think the best way to do this is to create a .env.e2e file that gets used in the e2e build, and then detox can open up the test menu and reach different screens the same way a human would. It also makes sense to have a dedicated .env.e2e file so we can use a build that connects to the staging server. We could probably get detox to do a real key claim this way if we want.
If you can think of another way to do this, please comment!

Android and iOS tests are now working. Sometimes iOS tests failing because of a UIManager warning popup that is covering the next button. Re-running the tests fixed the issue for me. All screenshots are being generated correctly for both android and ios.


Create helloWorld.rs

Hello world in Rust

Hello world in Rust

Hello, I saw you're from India and part of the project's goal is to write the code in your selected programming language AND in your native language as well, so if English is not your native language, can you rewrite "Hello, World" in your first language? Then I can accept your PR. Thank you!

Oh i see, okay i will make another PR after changing.

Hello world in Rust

Hello, I saw you're from India and part of the project's goal is to write the code in your selected programming language AND in your native language as well, so if English is not your native language, can you rewrite "Hello, World" in your first language? Then I can accept your PR. Thank you!

Done, Please review.


nixos/tests: add keepassxc

Test whether KeePassXC launches correctly.

Motivation for this change
Automatise PR reviews
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

/marvin opt-in
/status needs_reviewer

CC @prusnak @iAmMrinal0 @jonafato

@ofborg test keepassxc

Hang on, I forgot to add the test to all tests.

Done.
When are these tests executed? Only if someone with ofborg rights remembers to execute them?

When are these tests executed?

I am not sure tbh.

Only if someone with ofborg rights remembers to execute them?

Everyone can tell ofborg to run them.

@ofborg test keepassxc

There is some discussion here: https://discourse.nixos.org/t/difference-between-ofborg-and-hydra/3235/8 I'm assuming now that Hydra will run these tests when the PR is merged..?

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/459

There is some discussion here: discourse.nixos.org/t/difference-between-ofborg-and-hydra/3235/8 I'm assuming now that Hydra will run these tests when the PR is merged..?

Yes. but you should also add it to the keepassxc package itself: https://github.com/NixOS/nixpkgs/blob/master/pkgs/applications/misc/keepassx/community.nix like this: https://github.com/NixOS/nixpkgs/blob/e0d7cadb9963b97283400335a531d6a30be114ca/pkgs/tools/networking/ferm/default.nix#L30

Than also ofborg will run them.

@ofborg test keepassxc
@ofborg build keepassxc.passthru.tests

@Mic92 like this?

/needs_reviewer

/status needs_reviewer


Client Allocation Request for: aoeu

Client Allocation Questions
Core Information

Name: aoeu
Website / Social Media: oeui, (euid, uidh)
Region: North America
DataCap Requested: 32 GiB
Addresses to be Notarized: f245626
Notary Requested: Deon Erda

Application created via filplus.storage

This user’s identity has been verified through filplus.storage


Clicking and dragging over mass error plot throws errors

doesnt break the website

USE.js:2099 Uncaught ReferenceError: circleDataset is not defined
    at SVGGElement.<anonymous> (USE.js:2099)
    at SVGGElement.t (d3.v3.min.js:1)
    at d3.v3.min.js:1
    at c (d3.v3.min.js:3)
    at SVGGElement.p (d3.v3.min.js:3)
    at SVGGElement.__onwheel.zoom (d3.v3.min.js:1)

probably this one.
Triggered by

not loading data
moving over plots
triggers by line 2099 of USE.js


[Bug]: Missing privacy manifest

Platform
macOS
Plugin
device_info_plus
Version
11.1.0
Flutter SDK
3.24.4
Steps to reproduce

change s.resource_bundles = {'device_info_plus_privacy' => ['PrivacyInfo.xcprivacy']} to s.resource_bundles = {'device_info_plus_privacy' => ['device_info_plus/Sources/device_info_plus/PrivacyInfo.xcprivacy']}

PrivacyInfo.xcprivacy is not included in Package.swift

Code Sample
No response
Logs
ITMS-91061: Missing privacy manifest - Your app includes “Contents/Frameworks/device_info_plus.framework/Versions/A/device_info_plus”, which includes device_info_plus, an SDK that was identified in the documentation as a privacy-impacting third-party SDK. Starting November 12, 2024, if a new app includes a privacy-impacting SDK, or an app update adds a new privacy-impacting SDK, the SDK must include a privacy manifest file. Please contact the provider of the SDK that includes this file to get an updated SDK version with a privacy manifest. For more details about this policy, including a list of SDKs that are required to include signatures and manifests, visit: https://developer.apple.com/support/third-party-SDK-requirements.

ITMS-91061: Missing privacy manifest - Your app includes “Contents/Frameworks/package_info_plus.framework/Versions/A/package_info_plus”, which includes package_info_plus, an SDK that was identified in the documentation as a privacy-impacting third-party SDK. Starting November 12, 2024, if a new app includes a privacy-impacting SDK, or an app update adds a new privacy-impacting SDK, the SDK must include a privacy manifest file. Please contact the provider of the SDK that includes this file to get an updated SDK version with a privacy manifest. For more details about this policy, including a list of SDKs that are required to include signatures and manifests, visit: https://developer.apple.com/support/third-party-SDK-requirements.

Flutter Doctor
[✓] Flutter (Channel stable, 3.24.4, on macOS 15.1 24B83 darwin-x64, locale en-CN)
    • Flutter version 3.24.4 on channel stable at /Users/ying/fvm/versions/3.24.4
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 603104015d (13 days ago), 2024-10-24 08:01:25 -0700
    • Engine revision db49896cf2
    • Dart version 3.5.4
    • DevTools version 2.37.3

[✓] Android toolchain - develop for Android devices (Android SDK version 35.0.0)
    • Android SDK at /Users/ying/Android/sdk
    • Platform android-35, build-tools 35.0.0
    • ANDROID_SDK_ROOT = /Users/ying/Library/Android/sdk
    • Java binary at: /usr/local/opt/openjdk@17/bin/java
    • Java version OpenJDK Runtime Environment Homebrew (build 17.0.12+0)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 16.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 16B40
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2024.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 21.0.3+-79915915-b509.11)

[✓] IntelliJ IDEA Ultimate Edition (version 2024.2.4)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 82.1.3
    • Dart plugin version 242.22855.32

[✓] VS Code (version 1.95.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.100.0

[!] Proxy Configuration
    • HTTP_PROXY is set
    ! NO_PROXY is not set

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 15.1 24B83 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 130.0.6723.92
    ! Error: Browsing on the local area network for Kimmy's iPhone. Ensure the device is unlocked and attached with a cable or associated with the same local area network as this Mac.
      The device must be opted into Developer Mode to connect wirelessly. (code -27)

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

Checklist before submitting a bug

[X] I searched issues in this repository and couldn't find such bug/problem
[X] I Google'd a solution and I couldn't find it
[X] I searched on StackOverflow for a solution and I couldn't find it
[X] I read the README.md file of the plugin
[X] I'm using the latest version of the plugin
[X] All dependencies are up to date with flutter pub upgrade
[X] I did a flutter clean
[X] I tried running the example project

@vbuberen This is a high priority, because the privacy manifest will take effect on November 12, 2024.
There should be more than one package with problems.
https://github.com/flutter/packages/blob/bb5a25815cae4125f2a5903a2b376db58cff2e66/packages/file_selector/file_selector_macos/macos/file_selector_macos.podspec#L16
https://github.com/flutter/packages/blob/bb5a25815cae4125f2a5903a2b376db58cff2e66/packages/file_selector/file_selector_macos/macos/file_selector_macos/Package.swift#L24

Thanks for reporting. I will release updates this week addressing this issue.

network_info_plus has the same problem.

Out of curiosity - why don't you open a PR yourself?

I created a few PRs before, but I don't have my Mac with me now.

I created a few PRs before, but I don't have my Mac with me now.

Yes I remember them. But was asking because this issue doesn't really require a Mac and the change takes less time than it took you to find plugins with this issue.
Anyway, thanks for reply. I will handle the network_info_plus as well

Sorry, I didn't make it clear. Because my computer is not with me.


syntax is cleared when switching colorscheme.

Is there anyway to prevent this when switching between colorschemes?
Another issue is if colorscheme is defined in vimrc and vimrc is sourced, syntax is cleared.

Hi - sorry for the delay!
The reason this happens is because the setting the colorscheme clears the previous highlighting, including links between highlighting groups. Direct links to the "built-in" highlighting groups seem to be unaffected, but indirect links (i.e. element A should be highlighted like element B, and element B should be highlighted as a Statement) seem to be cleared. This isn't specific to this syntax definition - you may have noticed others have this issue as well, particularly more complex ones. Whenever I've encountered this, I usually just run :e which sources the syntax files again.
To automatically reload all syntax files, which will also set up all the highlighting links again, put the following autocommand in your vimrc:
colorscheme namehere

augroup ColourSchemeReload
    au!
    au ColorScheme * call <SID>FixSyntaxHighlightGroups()
augroup END

fun s:FixSyntaxHighlightGroups()
    let b = bufnr()
    let &l:filetype = &l:filetype
    bufdo let &l:filetype = &l:filetype
    exec "buffer" b
endfun

This sets the filetype option on every buffer every time the colorscheme command is run, which causes Vim to source all the necessary syntax files again. This will actually fix this issue for any filetype. It's best to make sure the colorscheme command is run before the autocommand is registered, to avoid needlessly running this every time Vim opens.
Note that I've not tested this extensively, so there may be some odd behaviour - note that bufdo stops on the first error it encounters, which should be unlikely for the simple command it's doing.
I hope this helps :)


Support for building a docker container

Note: This PR is on top of PRs #75 and #78 - review only the last commit and merge only after those PRs

Pull Request Test Coverage Report for Build test-83
Details

5 of 8 (62.5%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.5%) to 24.061%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

zhmc_log_forwarder/version.py
5
8
62.5%

Totals

Change from base Build test-58:
1.5%

Covered Lines:
141

Relevant Lines:
586

💛  - Coveralls


No definition/description during autocomplete with the Deno LSP

The tsserver lsp (https://github.com/neovim/nvim-lspconfig/blob/master/CONFIG.md#tsserver) shows a definition of the function during autocomplete:

but the denols lsp (https://github.com/neovim/nvim-lspconfig/blob/master/CONFIG.md#denols) does not:

but the definition is there as it is shown when K (lua vim.lsp.buf.hover()) is pressed:

Is this a problem with nvim-compe or the denols? Thanks.
NVIM v0.5.0-dev+1089-gdc3ca16a9 (updated today to latest version)
nvim-compe (updated today to latest version)

denols doesn't provide documentation for the completion item so nvim-compe can't display it. Here's what tsserver gave to nvim-compe
{
  commitCharacters = { ".", ",", "(" },
  data = {
    entryNames = { "fetch" },
    file = "/home/elianiva/codes/scratch/caesar.ts",
    line = 27,
    offset = 3
  },
  detail = "function fetch(input: RequestInfo, init?: RequestInit): Promise<Response>",
  insertTextFormat = 2,
  kind = 3,
  label = "fetch",
  sortText = "5"
}

compared to what denols gave
{
  insertTextFormat = 2,
  kind = 3,
  label = "fetch",
  sortText = "5"
}

Pretty sure it's in denols's todo list, we'll have to wait, I guess.

@elianiva Thank you for your work. I'm very saved to your work around issue responsing.

Good news, it now works with the latest version of Deno. Bad news, the formatting of the text looks bad, big empty grey area at the top.

Hm... OK. I will check it. Thank you.

There is a new issue for the bad formatting https://github.com/hrsh7th/nvim-compe/issues/373
The issue with no definition/description during autocomplete with the Deno LSP seems solved so I will close the issue.


Migrate checks to operate on the context object to support OpenAPI 3 (follow-up)

Follow-up issue for #714.
After implementing the initial support for OpenAPI3 we have to adjust the check implementations. Instead of operating on the Swagger object, the checks have to use the Context object. Doing this, the checks become universal - applicable against Swagger2 and OpenAPI3 specifications.
Most important rules have already been refactored in #714.
Acceptance Criteria:

The following checks are refactored to use the Context object
Minimal specifications are used in the tests

List of Rules to be refactored

[ ] AvoidXZallyIgnoreRule
[ ] SecureAllEndpointsWithScopesRule (currently not used in production)
[ ] ExtractBasePathRule
[ ] HyphenateHttpHeadersRule
[ ] NoProtocolInHostRule 🛑 Blocked (converter problems) 🎋 Branch
[ ] NoUnusedDefinitionsRule 🛑 Blocked (converter problems) 🎋 Branch

Rule {{HyphenateHttpHeadersRule}} linked to rule 131 in API Guidelines but there is no such rule. Should we remove this rule?

@vadeg You are right, we should remove the check. Would you take over or should I do this?

@maxim-tschumak I will take over in scope of PR for this ticket.

@vadeg great, thanks!

AvoidXZallyIgnoreRule operates on JsonNode not Swagger so I think can be considered done.

Thanks for all the reviews and merges everyone!
They should mean that we can tick these off now:

ExtractBasePathRule
HyphenateHttpHeadersRule
NoProtocolInHostRule

And hopefully #872 will tick NoUnusedDefinitionsRule and allow us to close the issue.

Thanks for your contributions! And sorry that we cannot always respond in one-two days.
Maybe we should set up another communication channel. Something like Slack.
What do you think? @roxspring @tkrop @ddufourboivin-zalando

You're welcome! Instant responses are not necessary, I'm comfortable with the usual response time of a few working days.
Slack or similar would be helpful for occasionally requesting attention on a PR, or for discussing thoughts before they're ready to be tracked as issues and pull requests.

@roxspring @tkrop @ddufourboivin-zalando please join zally maintainers Slack chat.

Think this is done now :)

@maxim-tschumak : Just updated the List of Rules to be refactored section in the description of this issue with the latest status of all of them.


Add "Related libraries" section in README

Coverage remained the same at 99.666% when pulling 33b03b4b28f7c7816d7e88eb6fa263089979510e on typeetfunc:patch-1 into 341fed4e9607ee960c3f53aed9539fb83a3c4208 on pelotom:master.


Add heatens by twenty one pilots

Song Name
heatens
Artist Name
twenty one pilots
Source
Youtube
Youtube Link
https://youtu.be/UprcpdwuwCg?si=z_ePXlU1YySWo2nX
Direct File Link
No response
Song ID
703291
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 34799345

reject
song is not called heatens


DNS problem with branches with very long names

Describe the bug
Branches with very long names are assigned invalid DNS names, so when you try to access the connected branch, you get the error:
This site can’t be reached
(branch name)’s server IP address could not be found.

This is because the label part for the branch name is longer than 63 characters. According to RFC1035:
The labels must follow the rules for ARPANET host names.  They must
start with a letter, end with a letter or digit, and have as interior
characters only letters, digits, and hyphen.  There are also some
restrictions on the length.  Labels must be 63 characters or less.

To Reproduce
Steps to reproduce the behavior:

Go to https://feature-fpafe-6-when-the-user-logs-in-if-the-password-is-about-to-expire-display-a-countdown-or-if.d1m2p9ror6ta9m.amplifyapp.com/
See error

This branch is properly connected and built in the AWS Amplify Console. The problem occurs with both auto connected and manually connected branches.
Expected behavior
The connected branch should be assigned the following domain name:
https://feature-fpafe-6-when-the-user-logs-in-if-the-password-is-about.d1m2p9ror6ta9m.amplifyapp.com
which is a valid DNS name.

I recommend adding an optional regex to the AutoBranchCreationConfig (maybe SubdomainPattern). This would allow both simple extraction and also extracting a prefix such as a ticket number.

A bit of a pain, but I solved this by using the UpdateBranch command and passing the --display-name option.
I am using gitlab so this is what I am doing in my job:
aws amplify update-branch --app-id ${AMPLIFY_APP_ID} --region ${DEV_REGION} --branch-name ${CI_COMMIT_REF_NAME} --display-name ${CI_COMMIT_REF_SLUG} --output json

This ensures that the display name is 63 chars or less. Really though, Amplify should handle this or at least give you a warning.


Dep ensure Solving Failure: No versions of docker.io/go-docker met constraints

Setup: AWS Ubuntu 16.04, ports open 80,443, 3000.
Go installation: /usr/local/go/bin/go
go version go1.10.1 linux/amd64
dep installation: /home/ubuntu/go/bin/dep
PWD installation: /home/ubuntu/go/src/github.com/play-with-docker/play-with-docker
Modified Gopkg.toml added 1 line:
required = ["github.com/googollee/go-socket.io","github.com/urfave/negroni","github.com/docker/distribution","github.com/zbindenren/negroni-prometheus"]
Go Dep solving failure
~/go/bin/dep ensure
Solving failure: No versions of docker.io/go-docker met constraints:
v1.0.0: Could not introduce<EMAIL_ADDRESS>as it has a dependency on github.com/docker/distribution with constraint edc3ab29cdff8694dd6feb85cfeb4b5f1b38ed9c, which has no overlap with existing constraint ^2.6.0 from (root)
master: Could not introduce docker.io/go-docker@master, as it has a dependency on github.com/docker/distribution with constraint edc3ab29cdff8694dd6feb85cfeb4b5f1b38ed9c, which has no overlap with existing constraint ^2.6.0 from (root)
Error Details when running docker-compose up:
~/go/src/github.com/play-with-docker/play-with-docker$ docker-compose up
WARNING: The Docker Engine you're using is running in swarm mode.
Compose does not use swarm mode to deploy services to multiple nodes in a swarm. All containers will be scheduled on the current node.
To deploy your application across the swarm, use the bundle feature of the Docker experimental build.
Starting haproxy
Starting l2
Starting pwd
Attaching to haproxy, pwd, l2
haproxy    | [WARNING] 108/151956 (1) : config : missing timeouts for frontend 'http-in'.
haproxy    |    | While not properly invalid, you will certainly encounter various problems
haproxy    |    | with such a configuration. To fix this, please ensure that all following
haproxy    |    | timeouts are set to a non-zero value: 'client', 'connect', 'server'.
haproxy    | [WARNING] 108/151956 (1) : config : missing timeouts for backend 'pwd'.
haproxy    |    | While not properly invalid, you will certainly encounter various problems
haproxy    |    | with such a configuration. To fix this, please ensure that all following
haproxy    |    | timeouts are set to a non-zero value: 'client', 'connect', 'server'.
haproxy    | [WARNING] 108/151956 (1) : config : missing timeouts for backend 'l2'.
haproxy    |    | While not properly invalid, you will certainly encounter various problems
haproxy    |    | with such a configuration. To fix this, please ensure that all following
haproxy    |    | timeouts are set to a non-zero value: 'client', 'connect', 'server'.
l2         | 2018/04/19 15:19:57 Connected to network [playwithdocker_default] with ip [<IP_ADDRESS>]
pwd        | 2018/04/19 15:19:59 Updating playgrounds configuration
pwd        | 2018/04/19 15:19:59 EVENT: Playground New 4a24b940-e23f-54f5-8867-137433240dd7
pwd        | 2018/04/19 15:19:59 Updating playgrounds configuration
pwd        | 2018/04/19 15:19:59 Listening on port 3000
pwd        | 2018/04/19 15:20:24 Error retrieving playground 8f6492ad-2254-548a-b584-e92a74f54976. Got: NotFound
pwd        | 2018/04/19 15:20:24 Playground for domain <IP_ADDRESS> was not found!
pwd        | [negroni] Started GET /
pwd        | [negroni] Completed 404 Not Found in 141.916µs

I looked up https://github.com/docker/go-docker/blob/master/Gopkg.toml.
I got rid of the error by changing Gopkg.toml:
[[constraint]]
name = "github.com/docker/distribution"
revision = "edc3ab29cdff8694dd6feb85cfeb4b5f1b38ed9c"
I ran dep ensure
I also did a go build, go install
I'm not sure if this pulls all the right images, but it got rid of the Solving Error.
Now, I can curl localhost:
curl localhost/my/playground
Gives me:
{"id":"4a24b940-e23f-54f5-8867-137433240dd7","domain":"localhost","default_dind_instance_image":"franela/dind","available_dind_instance_images":["franela/dind"],"allow_windows_instances":false,"default_session_duration":14400000000000}
but when I curl or try to reach my AWS instance from a browser:
Error retrieving playground d85b75b8-4590-5b9a-8ecf-22fa485e8b34. Got: NotFound
pwd        | 2018/04/20 01:56:07 Playground for domain xxx.us-west-2.compute.amazonaws.com was not found!

I was able to get this running on localhost, but not on AWS server. closing this issue, will open separate issue with running on server